Take into account rotation metadata to define video size

[bobatsar]

• ffmpeg autorotates videos but the video_size is unchanged,
  this results e.g. in stretched output images or even unreadable images or videos.
  This commit exchanges width and height if 90 or 270 degree rotation was detected.

[bobatsar]

See #212 and #529

[coveralls]

Coverage increased (+0.02%) to 68.625% when pulling ac6876a on bobatsar:auto_rotate into c8887f0 on Zulko:master.

[kencochrane]

Thanks for the PR, Is there anyway to add a unit test, or show that it works?

[bobatsar]

I'm planning to do a short test. Sorry for not replying earlier. But it can take some time. It's on my list.

[tburrows13]

Linking #683

[mondeja]

It seems to me that this is the right fix, but needs rewriting for the new ffmpeg output parser and a proper test case, maybe using this video or better creating multiples ones with ffmpeg at runtime.

[mondeja]

I've just merged it with master, added two tests and changed the place where the fix is being applied. I prefer just change the size in the FFMPEG_VideoReader instance so parsing ffmpeg -i output the metadata fits as much ffmpeg output as possible.

[mondeja]

4 years later but this is merged now 👍 In the future this could be improved defining optionally the rotation metadata writing files accordingly, (some preserve_rotation-like argument), but this fix is needed before.

[philihp]

This was merged last year, but it has been almost 2 years since a new version was released. Could you release this?

[melyux]

Please release this

[songge8]

Could you release this?