[FFmpeg-devel] [PATCH 2/3] lavf/avienc: Write palette after BITMAPINFOHEADER

Mats Peterson matsp888 at yahoo.com
Fri Feb 19 22:05:32 CET 2016


On 02/19/2016 10:00 PM, Michael Niedermayer wrote:
>>>> -
>>>>                   pc_tag = ff_start_tag(pb, tag);
>>>>                   avio_w8(pb, 0);
>
>>>>                   avio_w8(pb, 0);
>
>>>>                   avio_wl16(pb, 0); // reserved
>>>> -                for (i = 0; i<256; i++) {
>>>> +                for (i = 0; i < pal_size; i++) {
>>>
>>> this mismatches the stored size above
>>
>> Mismatches what stored size?
>
> the "avio_w8(pb, 0);" above
>

Right. I'll just set that one to 1 << bits_per_coded_sample then. Funny 
that not even Media Player supports these palette change chunks, by the way.

Mats



More information about the ffmpeg-devel mailing list