[FFmpeg-devel] [PATCH]Use CONFIG_LIBXVID_ENCODER in ratecontrol.c

[Måns Rullgård]

Diego Biurrun <diego at biurrun.de> writes:

> On Fri, Jul 09, 2010 at 11:07:37AM +0100, M?ns Rullg?rd wrote:
>> Diego Biurrun <diego at biurrun.de> writes:
>> 
>> > On Fri, Jul 09, 2010 at 11:13:10AM +0200, Carl Eugen Hoyos wrote:
>> >> 
>> >> Is this patch (that fixes an MPlayer linking problem) acceptable?
>> >
>> > Looks OK to me.
>> 
>> Not to me.
>
> I just checked that ratecontrol.c is only ever used by encoders, so I do
> not see how the patch can do any harm...

It is wrong, plain and simple.  CONFIG_LIBXVID_ENCODER means the user
requested the -vcodec libxvid wrapper.  The libxvid RC is separate
from that.

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