gst/matroska/matroska-mux.c: Look for pixel-aspect-ratio in caps, not pixel_width...
authorJan Schmidt <thaytan@mad.scientist.com>
Mon, 28 Nov 2005 15:13:22 +0000 (15:13 +0000)
committerJan Schmidt <thaytan@mad.scientist.com>
Mon, 28 Nov 2005 15:13:22 +0000 (15:13 +0000)
Original commit message from CVS:
* gst/matroska/matroska-mux.c:
(gst_matroska_mux_video_pad_setcaps):
Look for pixel-aspect-ratio in caps, not pixel_width and
pixel_height (Fixes: #322645)

ChangeLog
common
gst/matroska/matroska-mux.c

index cdf55d2..5e31809 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,13 @@
 
        * gst/matroska/matroska-mux.c:
        (gst_matroska_mux_video_pad_setcaps):
+       Look for pixel-aspect-ratio in caps, not pixel_width and
+       pixel_height (Fixes: #322645)
+
+2005-11-28  Jan Schmidt  <thaytan@mad.scientist.com>
+
+       * gst/matroska/matroska-mux.c:
+       (gst_matroska_mux_video_pad_setcaps):
        From Michal Benes:
        frame duration should be GST_SECOND / framerate, not
        GST_SECOND * framerate. (Fixes: #322643)
diff --git a/common b/common
index c04ad83..b0b08bc 160000 (submodule)
--- a/common
+++ b/common
@@ -1 +1 @@
-Subproject commit c04ad83a0c184fcd311de39371570b9e773a60d0
+Subproject commit b0b08bcd66ca385364b6f90663585b09693aec67
index 4de8b1b..709840d 100644 (file)
@@ -426,7 +426,6 @@ gst_matroska_mux_video_pad_setcaps (GstPad * pad, GstCaps * caps)
   gint width, height, pixel_width, pixel_height;
   const GValue *framerate;
   GstStructure *structure;
-  gboolean ret;
 
   /* find context */
   collect_pad = (GstMatroskaPad *) gst_pad_get_element_private (pad);
@@ -454,9 +453,8 @@ gst_matroska_mux_video_pad_setcaps (GstPad * pad, GstCaps * caps)
       gst_value_get_fraction_denominator (framerate),
       gst_value_get_fraction_numerator (framerate));
 
-  ret = gst_structure_get_int (structure, "pixel_width", &pixel_width);
-  ret &= gst_structure_get_int (structure, "pixel_height", &pixel_height);
-  if (ret) {
+  if (gst_structure_get_fraction (structure, "pixel-aspect-ratio",
+          &pixel_width, &pixel_height)) {
     if (pixel_width > pixel_height) {
       videocontext->display_width = width * pixel_width / pixel_height;
       videocontext->display_height = height;