Re: [FFmpeg-devel] [PATCH v12 12/14] lavc/mjpegdec: Skip useless APPx marker on bayer images

2019-08-10 Thread Michael Niedermayer
On Sat, Aug 10, 2019 at 04:52:10PM +0300, Nick Renieris wrote: > The data the marker contains is only 2 bytes and they're 0x in the > samples above. I don't know what they're for. ok then please call them unknown and not useless in the comments thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH v12 12/14] lavc/mjpegdec: Skip useless APPx marker on bayer images

2019-08-10 Thread Nick Renieris
The data the marker contains is only 2 bytes and they're 0x in the samples above. I don't know what they're for. Στις Σάβ, 10 Αυγ 2019 στις 3:16 μ.μ., ο/η Michael Niedermayer έγραψε: > > On Fri, Aug 09, 2019 at 07:29:57PM +0300, Nick Renieris wrote: > > From: Nick Renieris > > > > Samples:

Re: [FFmpeg-devel] [PATCH v12 12/14] lavc/mjpegdec: Skip useless APPx marker on bayer images

2019-08-10 Thread Michael Niedermayer
On Fri, Aug 09, 2019 at 07:29:57PM +0300, Nick Renieris wrote: > From: Nick Renieris > > Samples: > - Embedded JPEG images in the DNG images here: > https://www.photographyblog.com/previews/pentax_k1_photos > > Signed-off-by: Nick Renieris > --- > libavcodec/mjpegdec.c | 11 +-- > 1

[FFmpeg-devel] [PATCH v12 12/14] lavc/mjpegdec: Skip useless APPx marker on bayer images

2019-08-09 Thread Nick Renieris
From: Nick Renieris Samples: - Embedded JPEG images in the DNG images here: https://www.photographyblog.com/previews/pentax_k1_photos Signed-off-by: Nick Renieris --- libavcodec/mjpegdec.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavcodec/mjpegdec.c