[Bug 752085] dashdemux: revert changes introduced in fix for bug 751850

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Jul 7 10:18:13 PDT 2015


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

--- Comment #6 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to Florin Apostol from comment #4)
> The definition of segment_index is:
> struct _GstActiveStream
> {
> ...
> gint segment_index;                         /* index of next sequence chunk
> */
> ...
> }

I would argue that this comment is either outdated, or things are inconsistent
between adaptivedemux and dashdemux/mpdparser. It all seems correct in the code
with the interpretation of segment_index being the *current* index though.


Especially it is initialized at 0 and not at 1, it must be initialized to 1 if
the above comment was correct.

-- 
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