[FFmpeg-devel] remove int readers
Thu Jun 14 09:21:08 CEST 2007
"Ronald S. Bultje" <rsbultje at gmail.com> writes:
> On 6/14/07, mark cox <melbournemark+ffmpeg at gmail.com> wrote:
>> - seq = (buf << 8) | buf;
>> + 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 also, but I try to
> conform to what's already there.
&buf is IMO far harder to read than buf + 2, and it's ugly to
mans at mansr.com
More information about the ffmpeg-devel