[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