[Ffmpeg-devel] [PATCH] [RFC] makefile cleanup

Måns Rullgård mru
Sat Jul 8 17:59:55 CEST 2006


Diego Biurrun <diego at biurrun.de> writes:

> On Sat, Jul 08, 2006 at 01:11:06AM +0100, M?ns Rullg?rd wrote:
>> Here's a patch that slashes a massive 202 lines (of 486) from the lavc
>> makefile.  In my opinion it's more readable this way.  Comments?
>
> [..]
>> +OBJS-$(CONFIG_MMVIDEO_DECODER) += mmvideo.o
>> +OBJS-$(CONFIG_ZMBV_DECODER) += zmbv.o
>> +OBJS-$(CONFIG_FLASHSV_DECODER) += flashsv.o
>
> Well, while you're at it you could prettyprint this and align the +=.
> No, it's not important...
>
>>  # sun mediaLib specific stuff
>> -# currently only works when libavcodec is used in mplayer
>> -ifeq ($(HAVE_MLIB),yes)
>> -OBJS += mlib/dsputil_mlib.o
>> -CFLAGS += $(MLIB_INC)
>> -endif
>> +OBJS-$(HAVE_MLIB) += mlib/dsputil_mlib.o
>>  
>> -# Intel IPP specific stuff
>> -# currently only works when libavcodec is used in mplayer
>> -ifeq ($(HAVE_IPP),yes)
>> -CFLAGS += $(IPP_INC)
>> -endif
>
> This is not an equivalent change.  We can discuss removing this, but it
> should not be part of this cleanup patch.

OK, here's a new patch.

-- 
M?ns Rullg?rd
mru at inprovide.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lavc-makefile.patch
Type: text/x-patch
Size: 19811 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060708/f7305368/attachment.bin>



More information about the ffmpeg-devel mailing list