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

Michael Niedermayer michael at niedermayer.cc
Mon Dec 28 14:03:01 CET 2015

On Mon, Dec 28, 2015 at 11:46:21AM +0100, 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).

right, i missed that
that one should have been a seperate patch


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151228/b4e25bbd/attachment.sig>

More information about the ffmpeg-devel mailing list