search for: oggplayoverlaydata

.../ > unsigned char *mask; /* may be NULL if RGBA */ > } OggPlayVideoRGBData; That's what I have at the moment - two types and something like the above, except that : typedef struct { unsigned char *rgb; /* may be NULL if alpha */ unsigned char *rgba; /* may be NULL if no alpha */ } OggPlayOverlayData; (I think it's probably faster to keep RGBA in one plane, so memcpy works if needed, etc, though no hard numbers to back it up). If I go this way, I think the question of whether the API is supposed to be not changed is moot, so it's all good. Cheers

> I think adding an RGB/RGBA frame type to liboggplay should *also* > happen. yuv2rgb is tedious at best and hard to get right at worst, so I'd added an RGBA type, but if this is going to also carry video, it might be better to merge this with the existing video type, and have a 'type' enum, or just several pointers, only one set being non NULL (makes client code simpler). Would