[FFmpeg-devel] [PATCH] define AVPixelFormat aliases as enumerators instead of macros

[wm4]

On Thu, 14 Jan 2016 13:58:14 -0800
Richard Smith <richard at metafoo.co.uk> wrote:

> libavutil/pixfmt.h defines a collection of endian-specific pixel formats as
> macros. These macro names can cause conflicts with external projects that
> use those identifiers for their own purposes. Here's a patch to define
> these aliases as enumerators instead of macros, please consider merging:
> 
> 
> https://github.com/zygoloid/FFmpeg/commit/c20a0e2e66e52c45b9193bc750165b7ecf7f3ca4
> 
> (Note that AV_PIX_FMT_Y400A was already defined as an enumerator in the
> PixelFormat enumeration, so I deleted its (no-op) macro entirely.)

API users might check for the existence of such pixfmts with #ifdef,
and I don't understand the reasoning behind your patch. Why would
external projects redefine these macros?