[FFmpeg-devel] [PATCH] Optimization of original IFF codec

Ronald S. Bultje rsbultje
Sun Apr 25 23:08:00 CEST 2010


Hi,

On Sun, Apr 25, 2010 at 12:47 PM, Sebastian Vater
<cdgs.basty at googlemail.com> wrote:
> So, here are the two patches seperated.
>
> Now comes the micro-op patch as requested by Mans...
[..]
> +    const unsigned b = (buf_size * 8) + bps - 1; \

Shouldn't be const, I think (?).

> +    for(i = 0; i < b; i++) { \
> +        dst[ i ] |= get_bits1(&gb) << plane; \

No spaces around "i".

Rest seems OK to me.

Ronald



More information about the ffmpeg-devel mailing list