[FFmpeg-devel] [PATCH] Extract rotation in MOV metadata

Dave Badia dbadia at gmail.com
Wed Apr 6 01:49:57 CEST 2011


On Wed, Mar 30, 2011 at 11:18 AM, Baptiste Coudurier
<baptiste.coudurier at gmail.com> wrote:
> Hi Dave,
<snip>
>
> --
> Baptiste COUDURIER
> Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
> FFmpeg maintainer                                  http://www.ffmpeg.org
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Thanks for the feedback as I am very new to ffmpeg.
I've revised my patch per you suggestions, hope I understood  properly.


Interrogate MOV metadata for orientation of video so that software can
determine if the video needs to be rotated.
---
 libavformat/mov.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 23ba1d4..b12890f 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -1907,6 +1907,10 @@ static int mov_read_tkhd(MOVContext *c, AVIOContext *pb,
MOVAtom atom)
     sc->width = width >> 16;
     sc->height = height >> 16;

+    if (display_matrix[0][0] == -65536 && display_matrix[1][1] == -65536) {
+         av_metadata_set2(&c->fc->metadata, "rotate", "180", 0);
+    }
+
     // transform the display width/height according to the matrix
     // skip this if the display matrix is the default identity matrix
     // or if it is rotating the picture, ex iPhone 3GS


More information about the ffmpeg-devel mailing list