[Bug 751792] dashdemux: incorrect segment duration for a segment list

[GStreamer (GNOME Bugzilla)]

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

--- Comment #21 from Florin Apostol <florin.apostol at oregan.net> ---
(In reply to Vincent Penquerc'h from comment #18)
> It seems simpler this way. Adding the checks as the list is built makes the
> adding code more complex, especially as you need to know the start time of
> the next segment to know whether to add the first segment. Besides, the
> removal of clipped segments should happen only exceptionally.

My idea is to check each segment against the previous one added to the list,
not against the next one: before adding a segment to segment list, I would have
checked it against the period end and clipped if necessary. Also, I would have
checked S->t against start_time (that is, check each segment start against the
previous segment end). You still need to add this check somewhere, otherwise a
wrong S->t value will go undetected and mess up the segment list.

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