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

Måns Rullgård mru
Wed Jan 17 17:35:55 CET 2007


Alex Beregszaszi <alex at fsn.hu> writes:

> Hi,
>
> 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.
>
> Here is the monster patch. Its only cosmetics.
>
> The second patch introduces the write methods of the same.
>
> --
> Alex Beregszaszi
>
> Index: libavutil/intreadwrite.h
> ===================================================================
> --- libavutil/intreadwrite.h	(revision 7567)
> +++ 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.

-- 
M?ns Rullg?rd
mru at inprovide.com




More information about the ffmpeg-devel mailing list