From: Wim Taymans <wim.taymans@gmail.com>
Date: Fri, 21 Jul 2006 11:03:28 +0000 (+0000)
Subject: ext/theora/theoradec.c: Better clipping.
X-Git-Tag: 1.19.3~511^2~11729
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5f3f10861ede2157386eec35ef35df32d3e57339;p=platform%2Fupstream%2Fgstreamer.git

ext/theora/theoradec.c: Better clipping.

Original commit message from CVS:
* ext/theora/theoradec.c: (clip_buffer):
Better clipping.
---

diff --git a/ChangeLog b/ChangeLog
index 6f07d5b..5e88a27 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2006-07-21  Wim Taymans  <wim@fluendo.com>
 
+	* ext/theora/theoradec.c: (clip_buffer):
+	Better clipping.
+
+2006-07-21  Wim Taymans  <wim@fluendo.com>
+
 	* gst-libs/gst/audio/gstaudiosink.c: (audioringbuffer_thread_func),
 	(gst_audioringbuffer_finalize), (gst_audioringbuffer_acquire),
 	(gst_audioringbuffer_release), (gst_audioringbuffer_stop):
diff --git a/ext/theora/theoradec.c b/ext/theora/theoradec.c
index de9bd9c..eba33c4 100644
--- a/ext/theora/theoradec.c
+++ b/ext/theora/theoradec.c
@@ -865,35 +865,45 @@ header_read_error:
 
 /* returns TRUE if buffer is within segment, else FALSE.
  * if Buffer is on segment border, it's timestamp and duration will be clipped */
-
 static gboolean
 clip_buffer (GstTheoraDec * dec, GstBuffer * buf)
 {
-  gboolean res = FALSE;
+  gboolean res = TRUE;
+  GstClockTime in_ts, in_dur, stop;
   gint64 cstart, cstop;
 
+  in_ts = GST_BUFFER_TIMESTAMP (buf);
+  in_dur = GST_BUFFER_DURATION (buf);
+
   GST_LOG_OBJECT (dec,
       "timestamp:%" GST_TIME_FORMAT " , duration:%" GST_TIME_FORMAT,
-      GST_TIME_ARGS (GST_BUFFER_TIMESTAMP (buf)),
-      GST_TIME_ARGS (GST_BUFFER_DURATION (buf)));
+      GST_TIME_ARGS (in_ts), GST_TIME_ARGS (in_dur));
 
-  if (dec->segment.format != GST_FORMAT_TIME) {
-    res = TRUE;
+  /* can't clip without TIME segment */
+  if (dec->segment.format != GST_FORMAT_TIME)
     goto beach;
-  }
 
-  if (!(gst_segment_clip (&dec->segment, GST_FORMAT_TIME,
-              GST_BUFFER_TIMESTAMP (buf),
-              GST_BUFFER_TIMESTAMP (buf) + GST_BUFFER_DURATION (buf),
-              &cstart, &cstop)))
+  /* we need a start time */
+  if (!GST_CLOCK_TIME_IS_VALID (in_ts))
+    goto beach;
+
+  /* generate valid stop, if duration unknown, we have unknown stop */
+  stop =
+      GST_CLOCK_TIME_IS_VALID (in_dur) ? (in_ts + in_dur) : GST_CLOCK_TIME_NONE;
+
+  /* now clip */
+  if (!(res = gst_segment_clip (&dec->segment, GST_FORMAT_TIME,
+              in_ts, stop, &cstart, &cstop)))
     goto beach;
 
+  /* update timestamp and possibly duration if the clipped stop time is
+   * valid */
   GST_BUFFER_TIMESTAMP (buf) = cstart;
-  GST_BUFFER_DURATION (buf) = cstop - cstart;
-  res = TRUE;
+  if (GST_CLOCK_TIME_IS_VALID (cstop))
+    GST_BUFFER_DURATION (buf) = cstop - cstart;
 
 beach:
-  GST_LOG_OBJECT (dec, "dropping : %d", res);
+  GST_LOG_OBJECT (dec, "%sdropping", (res ? "not " : ""));
   return res;
 }