[FFmpeg-devel] [PATCH v28] lavf/movenc: Add palette to video sample description

[Mats Peterson]

On 02/26/2016 08:28 AM, Mats Peterson wrote:
> On 02/26/2016 05:08 AM, Mats Peterson wrote:
>> Should hopefully fix the big-endian issue.
>>
>>
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
> Forget this one as well. Using AV_RL32() shouldn't be needed, since the
> palette is already stored in host byte order as far as I know. And I
> don't understand whatsoever why it will fail on a big-endian machine,
> Michael.
>
> Mats
>

Snippet from avidec.c:

ast->pal[i] = 0xFFU<<24 | AV_RL32(pal_src+4*i);

And from qtpalette.c (mov and matroska):

palette[i] = (a << 24 ) | (r << 16) | (g << 8) | (b);


Something obviously goes wrong when storing the palette during yuv420p 
to pal8 conversion of that matrix file.

Mats

-- 
Mats Peterson
http://matsp888.no-ip.org/~mats/