[MPlayer-dev-eng] [PATCH] Do not override H.264 video dimension with aspect info

Reimar Döffinger Reimar.Doeffinger at gmx.de
Tue Feb 7 23:04:22 CET 2012


Hello,
I don't know what the idea behind that was (possibly some
misunderstanding?) but it causes the aspect ratios to be
printed as video dimensions if (and only if) a custom
aspect was encoded.
So I propose below change:
Index: libmpdemux/mpeg_hdr.c
===================================================================
--- libmpdemux/mpeg_hdr.c	(revision 34650)
+++ libmpdemux/mpeg_hdr.c	(working copy)
@@ -325,10 +325,8 @@
     n += 8;
     if(picture->aspect_ratio_information == 255)
     {
-      picture->display_picture_width = (getbits(buf, n, 8) << 8) | getbits(buf, n + 8, 8);
+      // aspect numerator and denominator
       n += 16;
-
-      picture->display_picture_height = (getbits(buf, n, 8) << 8) | getbits(buf, n + 8, 8);
       n += 16;
     }
   }



More information about the MPlayer-dev-eng mailing list