Re: [FFmpeg-devel] [PATCH 2/2] pixfmt: Use enum assignment for aliases

2016-10-05 Thread Timothy Gu
On Tue, Aug 23, 2016 at 12:28 PM Michael Niedermayer wrote: > On Sun, Aug 21, 2016 at 05:09:01PM -0700, Timothy Gu wrote: > > This way, Doxygen is happier as aliases are now grouped together, and > > it never handled #define's in an enum well in the first place. > > > >

Re: [FFmpeg-devel] [PATCH 2/2] pixfmt: Use enum assignment for aliases

2016-08-23 Thread Michael Niedermayer
On Sun, Aug 21, 2016 at 05:09:01PM -0700, Timothy Gu wrote: > This way, Doxygen is happier as aliases are now grouped together, and > it never handled #define's in an enum well in the first place. > > Y400A already exists as an enum assignment. > --- > libavutil/pixfmt.h | 6 ++ > 1 file

Re: [FFmpeg-devel] [PATCH 2/2] pixfmt: Use enum assignment for aliases

2016-08-21 Thread Timothy Gu
On Sun, Aug 21, 2016 at 05:09:01PM -0700, Timothy Gu wrote: > -#define AV_PIX_FMT_XVMC AV_PIX_FMT_XVMC_MPEG2_IDCT > +AV_PIX_FMT_XVMC = AV_PIX_FMT_XVMC_MPEG2_IDCT, I don't think this constitutes an API break, but comments are welcome. Timothy ___

[FFmpeg-devel] [PATCH 2/2] pixfmt: Use enum assignment for aliases

2016-08-21 Thread Timothy Gu
This way, Doxygen is happier as aliases are now grouped together, and it never handled #define's in an enum well in the first place. Y400A already exists as an enum assignment. --- libavutil/pixfmt.h | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavutil/pixfmt.h