[FFmpeg-devel] [PATCH] lavf/avienc: Store palette at keyframes

[Mats Peterson]

Mats Peterson <matsp888-at-yahoo.com at ffmpeg.org> skrev: (5 mars 2016 23:31:40 CET)
>On 03/05/2016 11:23 PM, Mats Peterson wrote:
>> On 03/05/2016 11:20 PM, Mats Peterson wrote:
>>> On 03/05/2016 11:00 PM, Mats Peterson wrote:
>>>> Here is an experimental and much simpler solution that stores the
>last
>>>> used palette at keyframes regardless. Maybe too simple, Michael, I
>don't
>>>> know.
>>>>
>>>
>>> This simplicity leads to storing an xxpc chunk with the same palette
>at
>>> every keyframe when there is only one palette in the original file.
>>> Perhaps not acceptable.
>>>
>>
>> It's at least better than storing an xxpc chunk for every frame.
>>
>> Mats
>>
>
>Ouch. Every packet in toon.avi contains a keyframe...
>
>Mats
>
>_______________________________________________
>ffmpeg-devel mailing list
>ffmpeg-devel at ffmpeg.org
>http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This thing about storing xxpc palette switching chunks at every keyframe is rather much overkill, and more of academic interest than anything else in my book. Take that toon.avi file for example, every frame in it is a keyframe, yet it only contains two xxpc chunks. And Reimar, you would be hard pressed to find a 2gb 8 bpp file with xxpc chunks. This only concerns older and relatively short clips really. But if you want to implement something on top of what's already done, Michael, by all means, go ahead. Just as long as it will work with both stream copy, raw and non-raw video.

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