Re: [FFmpeg-devel] [PATCH] lavf/qtpalette: Ignore greyscale bit in certain cases

2016-01-02 Thread Mats Peterson
On 01/02/2016 11:51 AM, Carl Eugen Hoyos wrote: Mats Peterson ffmpeg.org> writes: Also, QuickTime in Windows or Mac ignores the greyscale bit if the video sample description contains a palette, regardless of bit depth. How did you test this? If what you write is true (I didn't test yet) don'

Re: [FFmpeg-devel] [PATCH] lavf/qtpalette: Ignore greyscale bit in certain cases

2016-01-02 Thread Carl Eugen Hoyos
Mats Peterson ffmpeg.org> writes: > Also, QuickTime in Windows or Mac ignores the > greyscale bit if the video sample description > contains a palette, regardless of bit depth. How did you test this? If what you write is true (I didn't test yet) don't you agree that this is not reflected in t

Re: [FFmpeg-devel] [PATCH] lavf/qtpalette: Ignore greyscale bit in certain cases

2016-01-01 Thread Mats Peterson
On 01/02/2016 06:40 AM, Mats Peterson wrote: The text with the links got lost. And the second file is not Animation, but Graphics (smc). It's irrelevant, though. Sample file 1 (1-bit QuickTime Animation): https://drive.google.com/open?id=0B3_pEBoLs0faTThSek1EeXQ0ZHM Sample file 2 (8-bit Quick

[FFmpeg-devel] [PATCH] lavf/qtpalette: Ignore greyscale bit in certain cases

2016-01-01 Thread Mats Peterson
The QuickTime File Format Specification states the following: "Depth: A 16-bit integer that indicates the pixel depth of the compressed image. Values of 1, 2, 4, 8 ,16, 24, and 32 indicate the depth of color images. The value 32 should be used only if the image contains an alpha channel. Values o

[FFmpeg-devel] PATCH] lavf/qtpalette: Ignore greyscale bit in certain cases

2016-01-01 Thread Mats Peterson
The QuickTime File Format Specification states the following: "Depth: A 16-bit integer that indicates the pixel depth of the compressed image. Values of 1, 2, 4, 8 ,16, 24, and 32 indicate the depth of color images. The value 32 should be used only if the image contains an alpha channel. Values o