[FFmpeg-devel] [PATCH] HAM6/HAM8 support for IFF demuxer/decoder

[Reimar Döffinger]

On Wed, May 05, 2010 at 10:02:03PM +1000, Peter Ross wrote:
> On Tue, May 04, 2010 at 09:13:46PM +0200, Sebastian Vater wrote:
> > Sebastian Vater a ?crit :
> 
> > +/**
> > + * IFF extra context. This structure is prepended to actual palette data
> > + */
> > +typedef struct IffExtraContext {
> > +    int      compression;  // Compression method used
> > +    unsigned masking;      // TODO: Masking method used
> > +    unsigned screenmode;   // Amiga Screenmode used (HAM/EHB are of interest here)
> > +    unsigned transparency; // TODO: Transparency
> 
> these could all be chars.

Use char only for strings, use uint8_t or int8_t for anything
else to avoid signedness issues.