[Bug 795424] qtdemux: Add MSE-style flush

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri May 18 16:10:33 UTC 2018


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

--- Comment #22 from Alicia Boya García <aboya at igalia.com> ---
(In reply to Sebastian Dröge (slomo) from comment #20)
> > > It requires additional code for mapping seek and flushes together though.
> > > This seems unnecessary and potentially fragile (what if a flush arrives just
> > > between the seek and upstream flushing us?)
> > 
> > That won't happen because the seek is synchronous. Indeed, this fact is used
> > to reset demux->offset_seek_seqnum after the seek. (See
> > qtdemux_seek_offset() and gst_qtdemux_do_push_seek())
> 
> There's nothing preventing another flush-start from another thread, it's an
> OOB event.

In that case gst_pad_push_event() would return FALSE before the FLUSH event is
received by qtdemux.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list