baseparse: Don't override gst_segment_do_seek()
authorNicolas Dufresne <nicolas.dufresne@collabora.com>
Fri, 17 Jul 2015 21:44:52 +0000 (17:44 -0400)
committerNicolas Dufresne <nicolas.dufresne@collabora.com>
Fri, 17 Jul 2015 21:44:52 +0000 (17:44 -0400)
This line has no purpose, clearly gst_segment_do_seek() is doing
the right job, also, having the start time (a timestamp) be that
same as time (the stream time) is quite odd.

https://bugzilla.gnome.org/show_bug.cgi?id=750783

libs/gst/base/gstbaseparse.c

index 146e35e..df201ab 100644 (file)
@@ -4340,7 +4340,6 @@ gst_base_parse_handle_seek (GstBaseParse * parse, GstEvent * event)
     seekpos = gst_base_parse_find_offset (parse, startpos, TRUE, &start_ts);
     seekstop = gst_base_parse_find_offset (parse, seeksegment.stop, FALSE,
         NULL);
-    seeksegment.start = seeksegment.time = seeksegment.position = start_ts;
   } else {
     if (rate >= 0)
       start_ts = seeksegment.position;
@@ -4356,7 +4355,6 @@ gst_base_parse_handle_seek (GstBaseParse * parse, GstEvent * event)
     if (!gst_base_parse_convert (parse, format, seeksegment.stop,
             GST_FORMAT_BYTES, &seekstop))
       goto convert_failed;
-
   }
 
   GST_DEBUG_OBJECT (parse,