[FFmpeg-devel] [PATCH] avcodec: use HAVE_THREADS header guards to silence -Wunused-function

[Michael Niedermayer]

On Wed, Sep 16, 2015 at 10:22:27PM -0400, Ganesh Ajjanagadde wrote:
> When compiled with --disable-pthreads, e.g
> http://fate.ffmpeg.org/report.cgi?time=20150917015044&slot=alpha-debian-qemu-gcc-4.7,
> a bunch of -Wunused-functions are reported due to missing header guards
> around threading related functions.
> This patch should silence such warnings.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  libavcodec/alac.c          | 2 ++
>  libavcodec/exr.c           | 2 ++
>  libavcodec/ffv1dec.c       | 4 ++++
>  libavcodec/flacdec.c       | 2 ++
>  libavcodec/h264.c          | 2 ++
>  libavcodec/huffyuvdec.c    | 2 ++
>  libavcodec/mdec.c          | 2 ++
>  libavcodec/mimic.c         | 4 ++++
>  libavcodec/mpeg12dec.c     | 2 ++
>  libavcodec/mpeg4videodec.c | 2 ++
>  libavcodec/pngdec.c        | 2 ++
>  libavcodec/takdec.c        | 2 ++
>  libavcodec/tta.c           | 2 ++
>  libavcodec/vp3.c           | 4 ++++
>  libavcodec/vp8.c           | 2 ++
>  libavcodec/vp9.c           | 2 ++
>  libavcodec/wavpack.c       | 2 ++
>  17 files changed, 40 insertions(+)

av_unused might be simpler to silence these warnings than #if

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150917/86489e73/attachment.sig>