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

Baptiste Coudurier baptiste.coudurier
Tue May 27 21:07:07 CEST 2008


Hi,

On Tue, May 27, 2008 at 02:49:24PM -0400, John Schmiederer wrote:
> [...]
>
> Index: libavformat/mov.c
> ===================================================================
> --- libavformat/mov.c	(revision 13467)
> +++ libavformat/mov.c	(working copy)
> [...]
>
> @@ -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.

[...]

--
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA




More information about the ffmpeg-devel mailing list