[FFmpeg-devel] [PATCH] CDXL demuxer and decoder

[Reimar Döffinger]

On 7 Jan 2012, at 21:06, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:
> 
>> On 7 Jan 2012, at 20:52, Paul B Mahol <onemda <at> gmail.com> wrote:
>>> On 12/30/11, Reimar Doeffinger <Reimar.Doeffinger <at> gmx.de> wrote:
>>>> only partial review...
>>>>> +            AV_WL32(out + (x * 4), (0xFF << 24) | (r | g | b));
> 
>> I don't know which, but there is for example both RGBA and RGB32 (I think
>> RGB32 is it).
> 
> Since the codec does not support transparency (iiuc), RGB24 and 0RGB32 are also
> possible options.

Oh. Yes, without transparency RGB24 and writing the values bytes-wise instead of first combining them into 32 bit should be better.