[FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

Mats Peterson matsp888 at yahoo.com
Mon Dec 28 11:54:47 CET 2015


On 12/28/2015 11:46 AM, Mats Peterson wrote:
> On 12/28/2015 10:34 AM, Michael Niedermayer wrote:
>> On Mon, Dec 28, 2015 at 03:10:18AM +0000, Carl Eugen Hoyos wrote:
>>> Michael Niedermayer <michael <at> niedermayer.cc> writes:
>>>
>>>> Patch splited in move and matroska part
>>>> i removed this memcpy() for now from what i
>>>> commited as there is clearly no consenus on it
>>>
>>> I would really have appreciated a real review:
>>> Apart from the unrelated audio fix
>>
>> what audio fix ?
>>
>
> He probably means the change from 86 to 36 as the lower limit for
> private data in A_QUICKTIME (36 is the size of a version 0 sound sample
> description).
>
> Mats
>

We have been talking about that before, Michael. Perhaps I should have 
emphasized that I've changed that one as well.

Mats

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


More information about the ffmpeg-devel mailing list