[FFmpeg-devel] [PATCH] fix build with --disable-encoders

Alexander Strange astrange
Sat Feb 23 05:16:35 CET 2008


On Feb 22, 2008, at 2:56 PM, Michael Niedermayer wrote:
>>
>
> Why is there an #ifdef CONFIG_ENCODERS there at all?
> Its gccs job to omit unused static functions.
>
> [...]

Well, if you want?

Attached patches:
* remove those uses of CONFIG_ENCODERS from dsputil_mmx
* add -Wno-unused-function to the build

Compile time is exactly the same for me with the patch, but it prints  
a lot more warnings. Looking through the FATE stderr for "defined but  
not used" only finds one that's valid (h264pred.c:201), so that  
warning might as well be off by default.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-noconfig-encoders.diff
Type: application/octet-stream
Size: 1676 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080222/0da81005/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-wno-unused-functions.diff
Type: application/octet-stream
Size: 388 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080222/0da81005/attachment-0001.obj>



More information about the ffmpeg-devel mailing list