[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