[FFmpeg-devel] [PATCH] Use tkhd matrix for proper display in mov

[John Schmiederer]

> > [...]
> >
> > @@ -1402,12 +1407,30 @@
> >      get_be16(pb); /* volume */
> >      get_be16(pb); /* reserved */
> >
> > -    url_fskip(pb, 36); /* display matrix */
> > +    //read in the display matrix (outlined in ISO 14496-12, Section
> 6.2.2)
> > +    // they're kept in fixed point format through all calculations
> > +    // ignore u,v,z b/c we don't need the scale factor to calc
> aspect ratio
> > +    for (i=0; i<3; i++) {
> > +      display_matrix[i][0] = get_be32(pb); //16.16 fixed point
> > +      display_matrix[i][1] = get_be32(pb); //16.16 fixed point
> > +      get_be32(pb); // 2.30 fixed point (not used)
> > +    }
> >
>
> Indentation in FFmpeg is 4 spaces.
>
> [...]
>
Doh - well I guess it's obvious this is my first FFmpeg patch.
Indentation fixed, patch attached.

-John
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ff_mov_tkhd_matrix3.diff
Type: application/octet-stream
Size: 1833 bytes
Desc: ff_mov_tkhd_matrix3.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080527/b88a837a/attachment.obj>