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

[Sebastian Vater]

Peter Ross a ?crit :
> 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.
>
> otherwise it looks good.
>   

I just wanted the appended palette data just be aligned at least to a 16
byte boundary, so I did choose 4 32-bit values for storing them.

Of course, they can be 64-bit as well, then they are aligned on a 32
byte boundary.

-- 

Best regards,
                   :-) Basty/CDGS (-: