From patchwork Fri May 1 21:45:48 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Carl Eugen Hoyos X-Patchwork-Id: 19422 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id EB68544A648 for ; Sat, 2 May 2020 00:46:07 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C7F4668C827; Sat, 2 May 2020 00:46:07 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-il1-f194.google.com (mail-il1-f194.google.com [209.85.166.194]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 83D6668C087 for ; Sat, 2 May 2020 00:46:01 +0300 (EEST) Received: by mail-il1-f194.google.com with SMTP id e8so5684610ilm.7 for ; Fri, 01 May 2020 14:46:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to; bh=reIbUZgeUcofdfNYFWW908RnElxhub+mBVuWawhF52E=; b=ZvLORDNJyzFokj2e9gjx86p3r7b+sUDYLHWt+dmD9Oabr9vEV/YUEMjIUCaxsoDd6A C86RZvhfA6TL+w7LbXdGorSQKUIraEMp80mlmpH6sj/05vT7Ra/W04PeekYyhXkP8wrZ 4xz9vYkUj1L3pYomLSC7F6pof3TEzLUW4l3ak6gw5VYueU/akceBTv+pTV4yuG7nu7Fq dbYN4FSdL1Up50erK8quAMwm7Xkz1hnZ+naXrDEIH1aItETZK3eAf2D5+gSDa4KfGdyq 90ZilQX/Cc79mMrs6ziqgGYFqs9kIu97uUlbyK42bEL+G22ZOhJjGUw4zWNAOMY9x8+S PjlQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=reIbUZgeUcofdfNYFWW908RnElxhub+mBVuWawhF52E=; b=flCMZi7EMlCx0fl//jIu7whd6HAjlC4Z7r1cIVqyDpJf0lHtdW8X6ORJxcy4hMo5Df qxMc1QYBWSN9/udw/IOk9C09YJNbzR8Xew6yihb1CePwt+GvTRe/hrgqy+y7O54aI/q5 Qk6bV3G1FAepBeGHSC5LMprwfbR6Xo3pd6/EwimRt8O58xMwBQtBJzeWwzUIxDeyR6wT oE3d8+6+IPHU9ah0rw3NUw6YYf/wE6SXl3CP+ihJQHt4G7JL3h4Ri+MuFbaEVNCbpDWt EPiAxnNaShyCXqPQPk0LCJmVJ2XWzl0k+2/McWgGoE7DAlPxxjcN+ZYXPZBh8g5cleTA ZiIQ== X-Gm-Message-State: AGi0PuY9wDq9ofZTcVEH3GHxcRaE7dYrvUZent8Wv+L8R8sNNPaEARif u0NSvSXenVDtTn2uDEVGK3oeMVEFhyClHmks9DTTAfPCx/A= X-Google-Smtp-Source: APiQypJVSOLYDKuCF1U5KNH7UMWWp9a/VpGi56P9EHkUypAgReUPXVUyVAwBpbTz3hQYdgM6t49097pMlY8QqTOj8QE= X-Received: by 2002:a92:aa0f:: with SMTP id j15mr5753165ili.211.1588369559690; Fri, 01 May 2020 14:45:59 -0700 (PDT) MIME-Version: 1.0 From: Carl Eugen Hoyos Date: Fri, 1 May 2020 23:45:48 +0200 Message-ID: To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [PATCH]lavu/threadmessage: Properly declare a function pointer X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Hi! Attached patch, inspired by a patch by Andreas, fixes the following warning when -Wpedantic is used: CC libavutil/threadmessage.o libavutil/threadmessage.c: In function ‘av_thread_message_flush’: libavutil/threadmessage.c:222:23: warning: ISO C forbids initialization between function pointer and ‘void *’ [-Wpedantic] 222 | void *free_func = mq->free_func; | ^~ Please comment, Carl Eugen From 3f0b6c654b7473452638c1cc06dfe45eebb59079 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Fri, 1 May 2020 23:42:01 +0200 Subject: [PATCH] lavu/threadmessage: Properly declare a function pointer. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixes a pedantic warning: libavutil/threadmessage.c:222:23: warning: ISO C forbids initialization between function pointer and ‘void *’ --- libavutil/threadmessage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/threadmessage.c b/libavutil/threadmessage.c index 764b7fb813..797ba6c44c 100644 --- a/libavutil/threadmessage.c +++ b/libavutil/threadmessage.c @@ -219,7 +219,7 @@ void av_thread_message_flush(AVThreadMessageQueue *mq) { #if HAVE_THREADS int used, off; - void *free_func = mq->free_func; + void(*free_func)(void *) = mq->free_func; pthread_mutex_lock(&mq->lock); used = av_fifo_size(mq->fifo); -- 2.24.1