[FFmpeg-devel] [PATCH] Extract rotation in MOV metadata
Michael Niedermayer
michaelni at gmx.at
Wed Apr 6 12:19:54 CEST 2011
On Wed, Apr 06, 2011 at 09:51:33AM +0200, Jean-Daniel Dupas wrote:
>
> Le 6 avr. 2011 à 02:50, James Zern a écrit :
>
> > On Tue, Apr 5, 2011 at 16:53, Baptiste Coudurier
> > <baptiste.coudurier at gmail.com> wrote:
> >> Hi Dave,
> >>
> >> On 04/05/2011 04:49 PM, Dave Badia wrote:
> >>> 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);
> >>> + }
> >>
> >> Set the metadata on AVStream, not AVFormatContext.
> >>
> >> Are you sure that the check is 100% correct for the rotation ?
>
> why not extracting the whole affine transform matrix ?
+1
but that doesnt mean i have any objections to just extract the
rotation
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110406/a34d3375/attachment.asc>
More information about the ffmpeg-devel
mailing list