gst-player: fix bug with changing playback direction
authorAskar Safin <safinaskar@mail.ru>
Wed, 4 Sep 2019 09:54:17 +0000 (12:54 +0300)
committerTim-Philipp Müller <tim@centricular.com>
Fri, 6 Sep 2019 00:05:14 +0000 (01:05 +0100)
Fix gst_event_new_seek call in gst-libs/gst/player/gstplayer.c

If rate >= 0.0, then previous code doesn't set end of segment. So, the end of segment
will be in place where previous seek put it. This is not neccesary end of media file
(in case of reverse playback). So if we play video backward for some time and then
switched to forward playing, we will get EOS somewhere in the middle of media file.
This commit always sets end of segment, thus fixing this bug

gst-libs/gst/player/gstplayer.c

index af9278a..259d11e 100644 (file)
@@ -3357,7 +3357,7 @@ gst_player_seek_internal_locked (GstPlayer * self)
 
   if (rate >= 0.0) {
     s_event = gst_event_new_seek (rate, GST_FORMAT_TIME, flags,
-        GST_SEEK_TYPE_SET, position, GST_SEEK_TYPE_NONE, GST_CLOCK_TIME_NONE);
+        GST_SEEK_TYPE_SET, position, GST_SEEK_TYPE_SET, GST_CLOCK_TIME_NONE);
   } else {
     s_event = gst_event_new_seek (rate, GST_FORMAT_TIME, flags,
         GST_SEEK_TYPE_SET, G_GINT64_CONSTANT (0), GST_SEEK_TYPE_SET, position);