gst-plugins-bad: h264parse: fix test and set not using the same input

Vincent Penquerc'h vincent at kemper.freedesktop.org
Fri Jan 6 02:54:34 PST 2012


Module: gst-plugins-bad
Branch: master
Commit: 4f27b2b4fe7ece1ce47106ef55ccc0d9dbc0cedd
URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/?id=4f27b2b4fe7ece1ce47106ef55ccc0d9dbc0cedd

Author: Vincent Penquerc'h <vincent.penquerch at collabora.co.uk>
Date:   Fri Jan  6 10:52:36 2012 +0000

h264parse: fix test and set not using the same input

This fixes H.264 output not having a set framerate, and
confusing some of the elements downstream (such as interlace).

---

 gst/videoparsers/gsth264parse.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/gst/videoparsers/gsth264parse.c b/gst/videoparsers/gsth264parse.c
index 505def9..05a5bab 100644
--- a/gst/videoparsers/gsth264parse.c
+++ b/gst/videoparsers/gsth264parse.c
@@ -1060,7 +1060,7 @@ gst_h264_parse_update_src_caps (GstH264Parse * h264parse, GstCaps * caps)
       gst_caps_set_simple (caps, "width", G_TYPE_INT, sps->width,
           "height", G_TYPE_INT, sps->height, NULL);
       /* but not necessarily or reliably this */
-      if (h264parse->fps_num > 0 && h264parse->fps_den > 0)
+      if (sps->fps_num > 0 && sps->fps_den > 0)
         gst_caps_set_simple (caps, "framerate",
             GST_TYPE_FRACTION, sps->fps_num, sps->fps_den, NULL);
     }



More information about the gstreamer-commits mailing list