Re: [FFmpeg-devel] [PATCH] mjpeg: Add support for ICC side data

2017-08-24 Thread Derek Buitenhuis
On 8/24/2017 1:22 AM, Michael Niedermayer wrote: > should be ok i think Rostislav asked me on IRC to try with MJPEG, so I am going to try and synthesize such a file and work on a v2. I don't suppose you, or anyone else has an MJPEG file with ICC profiles embedded? :) - Derek

Re: [FFmpeg-devel] [PATCH] mjpeg: Add support for ICC side data

2017-08-23 Thread Michael Niedermayer
On Wed, Aug 23, 2017 at 05:12:34PM +0100, Derek Buitenhuis wrote: > JPEGs store embedded profiles under the APP2 marker, signified > with a "ICC_PROFILE" null-terminated string header, and can be > split across multiple APP2 markers, out of order. > > This patch currently assumes one single ICC pr