[FFmpeg-devel] [PATCH] Common code for Indeo interactive

[Vitor Sessak]

Maxim wrote:
> Hello guys,
> 
> here is the 1st part of my patch making FFmpeg decode Indeo interactive
> (indeo4/indeo5) streams. Both codecs are very similar but use abit
> different bitstream formats, so I decided myself to put all common
> functions and tables together and submit them as a separate patch.
> The 2nd part will be the long-awaited indeo5 decoder...
> Please review!

Just a small comment:

> /**
>  *  Run-value (RLE) table descriptor
>  */
> typedef struct {
>     uint8_t     eob_sym; ///< huffman symbol for EOB (end of block)
>     uint8_t     esc_sym; ///< huffman symbol for ESC (escape)
>     uint8_t     runtab[256];
>     int16_t     valtab[256];
> } RVMapDesc;

Am I missing something or valtab[] fits in int8_t?

-Vitor