videoaggregator: Remove broken timestamps-going-backwards check
authorSebastian Dröge <sebastian@centricular.com>
Thu, 9 Apr 2015 02:29:40 +0000 (19:29 -0700)
committerSebastian Dröge <sebastian@centricular.com>
Thu, 9 Apr 2015 02:31:43 +0000 (19:31 -0700)
This would've also triggered if for some reason the segment was updated
in such a way that PTS went backwards, but the running time increased. Like
what happens when non-flushing seeks are done.

We're doing a proper buffer-from-the-past check a few lines below based on the
running time, which is the only time we should care about here.

gst-libs/gst/video/gstvideoaggregator.c

index b1cfc835ad33aeae9dffe9b2c5aa0791a8ee4005..0f845f9415ecf25525da16f7d5a99d43fb441f28 100644 (file)
@@ -990,16 +990,6 @@ gst_videoaggregator_fill_queues (GstVideoAggregator * vagg,
       vinfo = &pad->info;
 
       /* FIXME: Make all this work with negative rates */
-
-      if ((start_time < GST_BUFFER_TIMESTAMP (buf))
-          || (pad->buffer && start_time < GST_BUFFER_TIMESTAMP (pad->buffer))) {
-        GST_DEBUG_OBJECT (pad, "Buffer from the past, dropping");
-        gst_buffer_unref (buf);
-        gst_aggregator_pad_drop_buffer (bpad);
-        need_more_data = TRUE;
-        continue;
-      }
-
       end_time = GST_BUFFER_DURATION (buf);
 
       if (end_time == -1) {