[FFmpeg-devel] [PATCH] ffplay: support rotated video.

Clément Bœsch u at pkh.me
Tue May 20 19:54:05 CEST 2014


On Fri, Apr 18, 2014 at 05:43:35PM -0400, Don Moir wrote:
[...]
> >+    if (autorotate) {
> >+        AVDictionaryEntry *rotate_tag = av_dict_get(is->video_st->metadata, "rotate", NULL, 0);
> >+        if (rotate_tag) {
> >+            if (!strcmp(rotate_tag->value, "90")) {
> >+                INSERT_FILT("transpose", "1");
> >+            } else if (!strcmp(rotate_tag->value, "180")) {
> >+                INSERT_FILT("hflip", NULL);
> >+                INSERT_FILT("vflip", NULL);
> >+            } else if (!strcmp(rotate_tag->value, "270")) {
> >+                INSERT_FILT("transpose", "2");
> >+            } else {
> >+                INSERT_FILT("rotate", rotate_tag->value);
> >+            }
> >+        }
> >+    }
> 
> you might want to do this instead of stcmp. Catches values greater than 360 as well.
> 
> int rotate = atoi(rotate_tag->value) % 360;
> 

No I can't, it can be a floating value.

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140520/73f31f24/attachment.asc>


More information about the ffmpeg-devel mailing list