[FFmpeg-devel] remove int readers

mark cox melbournemark+ffmpeg
Thu Jun 14 08:15:53 CEST 2007


On 14/06/07, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>
> 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.


Thanks for clarifying.
Regards,
- mark

Ronald
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>




More information about the ffmpeg-devel mailing list