On 12/13/2015 12:10 PM, Mats Peterson wrote:
On 12/13/2015 12:01 PM, Paul B Mahol wrote:
On 12/13/15, Mats Peterson <matsp888-at-yahoo....@ffmpeg.org> wrote:
On 12/13/2015 11:06 AM, Paul B Mahol wrote:
On 12/13/15, Mats Peterson <matsp888-at-yahoo....@ffmpeg.org> wrote:
On Sat, 12 Dec 2015, Paul B Mahol wrote:
On 12/12/15, Mats Peterson <matsp888-at-yahoo....@ffmpeg.org> wrote:
On Sat, 12 Dec 2015, Michael Niedermayer wrote:
On Sat, Dec 12, 2015 at 11:17:00AM +0000, Mats Peterson wrote:
Obviously that private data is cropped in some way then, since the
minimum size of a video sample description in QuickTime video
is 86
bytes.
FFmpeg tries to support all kind of odd and broken files, so
even if
the file is invalid, continuing to support it would be better
Personally I don't see why it should support broken files, but your
mileage may vary, of course. So just keep the value 21 then.
Have you provided file that doesn't work?
I got a sample Matroska file from Michael Niedermayer with V_QUICKTIME
video and with a private data of only 21 bytes, when it should be at
least
86 bytes, which is the minimum size for a video sample description in
QuickTime.
What about non-broken files?
Mats
--
Mats Peterson
http://matsp888.no-ip.org/~mats/
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
I'm sorry, I don't understand what you're after. Of course I have tried
this patch with several test files.
Have you provided such several test filest to others?
In this directory are three sample Matroska files with QuickTime video
that will display with the wrong palette (none at all, really) without
the patch: http://matsp888.no-ip.org/~mats/qtpalette-test/
Here is a directory at Google Drive in case my local server should be
down: http://bit.ly/1I1Bw2I
--
Mats Peterson
http://matsp888.no-ip.org/~mats/
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
