videoaggregator: Do not wrongly set the aggregator.segment
authorThibault Saunier <tsaunier@gnome.org>
Tue, 23 Sep 2014 13:59:10 +0000 (15:59 +0200)
committerThibault Saunier <tsaunier@gnome.org>
Mon, 6 Oct 2014 16:57:28 +0000 (18:57 +0200)
The aggregator.segment is not to be initialized by the subclasses but
by the aggregator itself. Moreover, initializing it on start would make
us loose the information coming from the initial seek.

gst-libs/gst/video/gstvideoaggregator.c

index ff7ea3b..4464b40 100644 (file)
@@ -782,7 +782,6 @@ gst_videoaggregator_reset (GstVideoAggregator * vagg)
   vagg->priv->ts_offset = 0;
   vagg->priv->nframes = 0;
 
-  gst_segment_init (&agg->segment, GST_FORMAT_TIME);
   agg->segment.position = -1;
 
   gst_videoaggregator_reset_qos (vagg);
@@ -1638,7 +1637,6 @@ gst_videoaggregator_start (GstAggregator * agg)
   if (!GST_AGGREGATOR_CLASS (gst_videoaggregator_parent_class)->start (agg))
     return FALSE;
 
-  gst_segment_init (&agg->segment, GST_FORMAT_TIME);
   gst_caps_replace (&vagg->priv->current_caps, NULL);
 
   return TRUE;