[FFmpeg-devel] remove int readers

Ronald S. Bultje rsbultje
Thu Jun 14 07:18:31 CEST 2007


Hi,

On 6/14/07, mark cox <melbournemark+ffmpeg at gmail.com> wrote:
>
> -    seq  = (buf[2] << 8) | buf[3];
> +    seq  = AV_RB16(buf + 2);
>
> Why have you changed all the indexes from [] to +. [] are preferred
> because
> they avoid compiler warnings and are easier to read IMHO.


Consistency with the rest of the code. I prefer &buf[2] also, but I try to
conform to what's already there.

Ronald




More information about the ffmpeg-devel mailing list