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

[Reimar Döffinger]

Hello,
On Thu, Jan 18, 2007 at 10:18:47AM +0100, Diego Biurrun wrote:
> 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?

To allow for a consistent naming system for the write functions, too.

Greetings,
Reimar D?ffinger