mxf: check EOS cond with any segment's flag
authorStéphane Cerveau <scerveau@collabora.com>
Tue, 4 May 2021 10:29:14 +0000 (12:29 +0200)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Tue, 4 May 2021 19:11:27 +0000 (19:11 +0000)
The previous test was preventing the pad to be in EOS
when the segment position was greater than segment stop.
It ended up consuming all the data before getting in EOS.

Regarding GST_SEEK_FLAG_SEGMENT it seems to be
correctly handled later in the method.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/2173>

gst/mxf/mxfdemux.c

index 5ed054c..e5b6ed4 100644 (file)
@@ -3189,8 +3189,7 @@ gst_mxf_demux_loop (GstPad * pad)
     goto pause;
 
   /* check EOS condition */
-  if ((demux->segment.flags & GST_SEEK_FLAG_SEGMENT) &&
-      (demux->segment.stop != -1) &&
+  if ((demux->segment.stop != -1) &&
       (demux->segment.position >= demux->segment.stop)) {
     guint i;
     gboolean eos = TRUE;