[FFmpeg-devel] [PATCH v1 2/2] fftools/ffmpeg: add exif orientation support per frame's metadata

[Nicolas George]

Jun Li (12019-05-25):
> I believe it has been enabled by default, the 'autorotate' value is true by
> default.

Yes, but it is currently enabled for constant rotation. Variable
rotation is more tricky, so the question of enabling it by default must
be asked again.

> Do you mean the orientation case 1, 2, 3, 4 have the same resolution and 5,
> 6, 7, 8 are rotated, so the reinit is only necessary when switch between
> these two cases?

I do not know the numeric values by rote, but I think that is what I
mean.

> I think it is doable. That transpose filter can do partially dynamic
> processing based on frame's orientation, but the filter args still need to
> set resolution info.

You can have the filter examine the metadata directly, as an option.

> Addressed in the new version.

Did you send it? If so I missed it.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190527/bf5e3800/attachment.sig>