[Bug 723075] matroskademux: invalid handling of flushing seek without start position

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=723075
  GStreamer | gst-plugins-good | unspecified

--- Comment #6 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2014-01-31 13:27:40 UTC ---
See docs/design/part-seeking.txt:

2. GST_SEEK_FLAG_KEY_UNIT:

If the KEY_UNIT flag is specified, the demuxer/parser should adjust the
segment start to the position of the key frame closest to the requested
seek position and then start pushing out data from there. The nearest
key frame may be before or after the requested seek position, but many
implementations will only look for the closest keyframe before the
requested position.




There's some ambiguity here. SEEK_TYPE_NONE says "do not update position",
however KEY_UNIT says "position should be adjusted to closest key unit". It's
not clear to me which of the two would take preference here, but I think it is
more powerful if in this case the demuxer would go to the nearest keyframe
according to the current position. If you don't want that you would not set the
KEY_UNIT flag on the seek, and then it wouldn't go to the nearest keyframe for
the current position but instead stay exactly at the current position.

Does that make sense to you?

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.