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

Aurelien Jacobs aurel
Mon Feb 25 22:13:00 CET 2008

Michael Niedermayer wrote:

> On Sun, Feb 24, 2008 at 02:44:09PM +0100, Diego Biurrun wrote:
> > 
> > It might be good to split off encoding-related stuff from that file
> > but this question is orthogonal to adding one more #ifdef.
> No, its not orthogonal, ifdef CONFIG_ENCODERS is specific to the case
> of mixed encoder and decoder code, if its split the encoder ifdefs
> become unneeded.

OK. Attached patch splits encoding part of dsputil_mmx in its own file.
It removes all #ifdef CONFIG_ENCODERS without adding any warning,
without needing -wno-whatever.
It should please everyone.
(Note that the patch is compressed because it's big, but basically, it
only moves encoder functions to dsputilenc_mmx.c)

I guess the patch is OK, so I will apply it very soon.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: dspenc.diff.gz
Type: application/x-gzip
Size: 22874 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080225/4e156fbc/attachment.bin>

More information about the ffmpeg-devel mailing list