[FFmpeg-devel] [PATCH] split mpegvideo.c

Benoit Fouet benoit.fouet
Tue Jul 3 08:57:43 CEST 2007


Diego Biurrun wrote:
> On Wed, Jun 27, 2007 at 10:30:27PM +0200, Benoit Fouet wrote:
>   
>> here is a first attempt to $subj
>>
>> --- libavcodec/Makefile	(revision 9447)
>> +++ libavcodec/Makefile	(working copy)
>> @@ -67,6 +67,7 @@
>>  OBJS-$(CONFIG_EIGHTBPS_DECODER)        += 8bps.o
>> +OBJS-$(CONFIG_ENCODERS)                += mpegvideo_enc.o
>>     
>
> This should really be CONFIG_MPEGVIDEO_ENCODER or similar.
>
>   

mpegvideo.o is unconditionnal, i thought mpegvideo_enc.o should be as
generic as possible too...
i'm not sure it would work properly with CONFIG_MPEG1VIDEO_ENCODER or
something like that.

but, as always, i'm here to learn !
if you have such a solution in mind, please let me know
-- 
Ben
Purple Labs S.A.
www.purplelabs.com




More information about the ffmpeg-devel mailing list