[Ffmpeg-devel] [PATCH] AV_R{L,B}{8,16,32}

[Diego Biurrun]

On Thu, Jan 18, 2007 at 09:14:24AM +0000, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Wed, Jan 17, 2007 at 05:02:18PM +0100, Alex Beregszaszi wrote:
> >> 
> >> As discussed in a previous thread (FFmpeg development discussions and
> >> patches <ffmpeg-devel at mplayerhq.hu>) we might change LE_16/BE_16 to
> >> include an AV_ prefix.
> >> 
> >> --- libavutil/intreadwrite.h	(revision 7567)
> >> +++ libavutil/intreadwrite.h	(working copy)
> >> @@ -26,14 +26,14 @@
> >>  
> >> -#define BE_16(x)  ((((uint8_t*)(x))[0] << 8) | ((uint8_t*)(x))[1])
> >> +#define AV_RB16(x)  ((((uint8_t*)(x))[0] << 8) | ((uint8_t*)(x))[1])
> >
> > What is the RB supposed to stand for?
> 
> My guess is read big endian.

Why not keep the name with an AV_ prefix?

Diego