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

Michael Niedermayer michael at niedermayer.cc
Thu Aug 24 03:22:45 EEST 2017


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 profile for the
> whole context, i.e. a single JPEG input file. This likely does
> not work for MJPEG files with embedded ICC profiles, but I could
> not find a real, exisiting file, that had that.
> 
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> ---
>  libavcodec/mjpegdec.c | 90 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  libavcodec/mjpegdec.h |  5 +++
>  2 files changed, 95 insertions(+)

should be ok i think

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- 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/20170824/f763fdc5/attachment.sig>


More information about the ffmpeg-devel mailing list