[FFmpeg-devel] [PATCH 1/2] lavf/avienc: Palette handling code only concerns AV_PIX_FMT_PAL8

Mats Peterson matsp888 at yahoo.com
Mon Mar 7 18:16:48 CET 2016


Mats Peterson <matsp888-at-yahoo.com at ffmpeg.org> skrev: (7 mars 2016 18:04:34 CET)
>Mats Peterson <matsp888-at-yahoo.com at ffmpeg.org> skrev: (7 mars 2016
>17:49:41 CET)
>>Mats Peterson <matsp888-at-yahoo.com at ffmpeg.org> skrev: (7 mars 2016
>>15:00:29 CET)
>>>Forget the separate patch posted before, and my previous experimental
>
>>>variants of your patch, Michael. And if we're going to use keyframe
>>>xxpc 
>>>chunks, we'll have to invent something really good. Here is some
>>useful
>>>
>>>stuff added in the meantime.
>>>
>>>Mats
>>
>>I've come to the conclusion that for any file with a palette change
>>somewhere in it, you have to add xxpc chunks to *every* keyframe in
>the
>>file, not only the ones following the palette change like your patch
>is
>>currently doing, as far as I can tell.
>>
>>Mats
>>-- 
>>Mats Peterson
>>http://matsp888.no-ip.org/~mats/
>>_______________________________________________
>>ffmpeg-devel mailing list
>>ffmpeg-devel at ffmpeg.org
>>http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>One could perhaps arrange to detect the AVISF_VIDEO_PALCHANGES flag in
>the source file (if AVI, of course), and, if set, add xxpc chunks to
>every keyframe in the destination file.
>
>Mats
>-- 
>Mats Peterson
>http://matsp888.no-ip.org/~mats/
>_______________________________________________
>ffmpeg-devel mailing list
>ffmpeg-devel at ffmpeg.org
>http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Or detect if there is any palette change regardless of that flag. For AVI it would be easy, since they come as xxpc chunks as usual, but for nut it would take a different logic. End of monologue.

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


More information about the ffmpeg-devel mailing list