Ronald S. Bultje wrote:

> Hi,
> 
> On Wed, Nov 17, 2010 at 2:37 AM, Justin Ruggles
> <[email protected]> wrote:
>> -            v = 14 - log2_tab(input_samples, AC3_BLOCK_SIZE*2);
>> -            if (v < 0)
>> -                v = 0;
>> +            v = FFMAX(0, 14 - log2_tab(input_samples, AC3_BLOCK_SIZE*2));
> [..]
> $ grep FFMAX libavutil/*.h
> libavutil/common.h:#define FFMAX(a,b) ((a) > (b) ? (a) : (b))
> 
> This will calculate 14 - log2_bla bla twice, no?

Yes. I need to be more careful with FFMAX/FFMIN.

Thanks,
Justin

_______________________________________________
FFmpeg-soc mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc

Reply via email to