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

[Alex Beregszaszi]

Hi,

> > +++ libavutil/intreadwrite.h	(working copy)
> > @@ -26,14 +26,14 @@
> >  #endif /* !__GNUC__ */
> >  
> >  /* endian macros */
> > -#if !defined(BE_16) || !defined(BE_32) || !defined(LE_16) || !defined(LE_32)
> > -#define BE_16(x)  ((((uint8_t*)(x))[0] << 8) | ((uint8_t*)(x))[1])
> > -#define BE_32(x)  ((((uint8_t*)(x))[0] << 24) | \
> > +#if !defined(AV_RB16) || !defined(AV_RB32) || !defined(AV_RL16) || !defined(AV_RL32)
> 
> That #if !defined(foo) should go away.  We want a warning/error if
> something else has also defined those names.

Right, it was left from the old one.

--
Alex