[FFmpeg-devel] [PATCH] lavf/matroskadec: Process QuickTime palette per track

Mats Peterson matsp888 at yahoo.com
Sat Feb 27 04:40:36 CET 2016


On 02/27/2016 04:25 AM, Mats Peterson wrote:
> On 02/27/2016 04:24 AM, Mats Peterson wrote:
>> On 02/27/2016 03:05 AM, Michael Niedermayer wrote:
>>> On Wed, Feb 24, 2016 at 06:16:25PM +0100, Mats Peterson wrote:
>>>> The QuickTime palette was incorrectly stored in the
>>>> MatroskaDemuxContext instead of a MatroskaTrack.
>>>>
>>>> --
>>>> Mats Peterson
>>>> http://matsp888.no-ip.org/~mats/
>>>
>>>>   matroskadec.c |   19 +++++++++++--------
>>>>   1 file changed, 11 insertions(+), 8 deletions(-)
>>>> 48aff2b29969597ce84512c15dbf6ed26468a228
>>>> 0001-lavf-matroskadec-Process-QuickTime-palette-per-track.patch
>>>>  From 079df7d5971a6ff0136597dfe1dbefc1e2c239f9 Mon Sep 17 00:00:00 2001
>>>> From: Mats Peterson <matsp888 at yahoo.com>
>>>> Date: Wed, 24 Feb 2016 18:14:05 +0100
>>>> Subject: [PATCH] lavf/matroskadec: Process QuickTime palette per track
>>>
>>> applied
>>>
>>> btw, maybe you want to add fate tests for all the new palette features
>>>
>>> thanks
>>>
>>
>> Thanks, Michael. Regarding adding FATE test, THAT is something I'm NOT
>> very knowledgable of, Michael!
>>
>
> FATE tests, even.
>

There's also lavf/matroskaenc to take care of palette-wise, by the way, 
but that is hardly an "urgent" issue in my book. The Matroska muxer in 
Ffmpeg is severely broken in other aspects (I always use mkvmerge from 
MKVToolnix to mux Matroska if I can), so I'm not overly keen on touching 
it right now.

Mats



More information about the ffmpeg-devel mailing list