[FFmpeg-devel] [RFC]lavc/mjpegenc_common: Fix aspect ratio

[Michael Niedermayer]

On Mon, Dec 10, 2018 at 12:33:52PM +0100, Michael Niedermayer wrote:
> On Mon, Dec 10, 2018 at 02:54:54AM +0100, Carl Eugen Hoyos wrote:
> > Hi!
> > 
> > Reading the specification and Wikipedia, it appears to me that FFmpeg
> > is writing wrong values as aspect ratio for jfif files.
> > I hope somebody can prove me wrong!
> > 
> > This would need a slightly more sophisticated update to the decoder.
> > 
> > Please comment, Carl Eugen
> >  mjpegenc_common.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 03077e11ffcc0b2faf5a2be46b0cb2ea224eca57  0001-lavc-mjpegenc_common-Fix-aspect-ratio.patch
> > From 9c42114da17c20ef6d81d3989b5521eaefc15819 Mon Sep 17 00:00:00 2001
> > From: Carl Eugen Hoyos <ceffmpeg at gmail.com>
> > Date: Mon, 10 Dec 2018 02:50:39 +0100
> > Subject: [PATCH] lavc/mjpegenc_common: Fix aspect ratio.
> > 
> > Reported-by: Ulf Zibis
> > ---
> >  libavcodec/mjpegenc_common.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> if its correct, this requires a bump to the version of libavcodec so
> the decoder can detect if a file is from before or after the change

also there is related code in libavcodec/vaapi_encode_mjpeg.c

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20181210/1c4151bb/attachment.sig>