[FFmpeg-cvslog] r20950 - in trunk/libavcodec: h263.h mpegvideo_enc.c

Måns Rullgård mans
Fri Jan 8 03:02:24 CET 2010


Diego Biurrun <diego at biurrun.de> writes:

> On Thu, Jan 07, 2010 at 04:19:19AM +0100, Michael Niedermayer wrote:
>
>> You removed code that is a prerequestie to spliting the core h263
>> routines from the rest.  Please put the code related to any_h263
>> back
>
> If you need it for your current work, fine.  But it was not used before
> and the proper place to define it is configure.
>
> enabled_any h263_encoder msmpeg4_encoder ... && enable h263_any_encoder
>
> .. or some such ..

Use _select variables.  As a bonus, that will allow easy disabling of
everything h263-related, should someone wish to do so.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-cvslog mailing list