[FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg Add stream metadata from first frame's metadata

[Michael Niedermayer]

On Sat, May 04, 2019 at 07:13:34PM -0700, Jun Li wrote:
> Fix #6945
> Exif extension has 'Orientaion' field for image flip and rotation.
> This change is to add the first frame's exif into stream so that
> autorotation would use the info to adjust the frames.
> ---
>  fftools/ffmpeg.c | 45 +++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 45 insertions(+)

This breaks converting jpegs

try this 
(file shoudl be here: https://trac.ffmpeg.org/raw-attachment/ticket/3424/bug.jpg)

./ffmpeg -i ~/tickets/3424/bug.jpg new.jpg
./ffmpeg -i ~/tickets/3424/bug.jpg new.bmp

./ffplay ~/tickets/3424/bug.jpg
./ffplay new.jpg
./ffplay new.bmp

All 3 above should produce a image with the same and correct orientation
they do before these 2 patches, they do not after

Thanks

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

Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad
-------------- 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/20190506/82fb7e2e/attachment.sig>