[gst-cvs] gst-plugins-bad: interlace: Fix buffer timestamp and duration

Robert Swain robswain at kemper.freedesktop.org
Wed Nov 24 03:19:07 PST 2010


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

Author: Robert Swain <robert.swain at collabora.co.uk>
Date:   Mon Nov 22 15:26:06 2010 +0100

interlace: Fix buffer timestamp and duration

The field rate is twice the frame rate of the src pad and so the duration of
one output buffer is src_fps_d / (2 * src_fps_n).

---

 gst/interlace/gstinterlace.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/gst/interlace/gstinterlace.c b/gst/interlace/gstinterlace.c
index d8b7f2e..c6181ec 100644
--- a/gst/interlace/gstinterlace.c
+++ b/gst/interlace/gstinterlace.c
@@ -327,13 +327,14 @@ static void
 gst_interlace_decorate_buffer (GstInterlace * interlace, GstBuffer * buf,
     int n_fields)
 {
+  /* field duration = src_fps_d / (2 * src_fps_n) */
   GST_BUFFER_TIMESTAMP (buf) = interlace->timebase +
       gst_util_uint64_scale (GST_SECOND,
       interlace->src_fps_d * interlace->fields_since_timebase,
-      interlace->src_fps_n);
+      interlace->src_fps_n * 2);
   GST_BUFFER_DURATION (buf) =
       gst_util_uint64_scale (GST_SECOND, interlace->src_fps_d * n_fields,
-      interlace->src_fps_n);
+      interlace->src_fps_n * 2);
   /* increment the buffer timestamp by duration for the next buffer */
   gst_buffer_set_caps (buf, interlace->srccaps);
 





More information about the Gstreamer-commits mailing list