[FFmpeg-devel] [PATCH][RFC] Indeo3 replacement
Maxim
max_pole
Sun Jul 26 15:59:45 CEST 2009
Aurelien Jacobs schrieb:
> On Sun, Jul 26, 2009 at 08:00:10AM +0200, Maxim wrote:
>
>> Vitor Sessak schrieb:
>>
>>> [...]
>>>
>>>
>>> A few comments:
>>>
>>>
>>>> //@{
>>>> //! vq table selector codes
>>>> #define DELTA_DYAD 0
>>>> #define DELTA_QUAD 1
>>>> #define RLE_ESC_F9 2
>>>> #define RLE_ESC_FA 3
>>>> #define RLE_ESC_FB 4
>>>> #define RLE_ESC_FC 5
>>>> #define RLE_ESC_FD 6
>>>> #define RLE_ESC_FE 7
>>>> #define RLE_ESC_FF 8
>>>> #define RLE_FORBIDDEN 9
>>>> //@}
>>>>
>>> Does this group Doxy formatting works for you?
>>>
>> Ummh, I just looked into the generated doc and seen that these #defines
>> are documented separately! Therefore I'd say it doesn't do exactly what
>> I want to get - I just want to have in particular these defines grouped
>> and documented together like an enum. Is it possible to do it that way
>> or should I remove the group brackets?
>>
>
> What about actually using an enum here ?
>
>
Changed to use enums, many thanks! It works actually better and looks
better as well...
Will be posted with the next patch...
Regards
Maxim
More information about the ffmpeg-devel
mailing list