[FFmpeg-devel] [PATCH v28] lavf/movenc: Add palette to video sample description
Mats Peterson
matsp888 at yahoo.com
Fri Feb 26 08:34:07 CET 2016
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/
More information about the ffmpeg-devel
mailing list