[FFmpeg-devel] [PATCH] Use tkhd matrix for proper display in mov
John Schmiederer
jschmiederer
Tue May 27 21:46:18 CEST 2008
> > [...]
> >
> > @@ -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>
More information about the ffmpeg-devel
mailing list