GNOME Bugzilla – Bug 741285
dashdemux: gst_dash_demux_setup_streams fails to initialise the segment numbers for live streams
Last modified: 2014-12-10 01:55:46 UTC
Created attachment 292350 [details] [review] Set next segment number on next_streams instead of streams gst_dash_demux_setup_streams has logic to find the correct segment/fragment numbers for each stream based on the current time. However this logic does not operate on the list of newly created streams and instead operates on an empty list. This is because gst_dash_demux_setup_all_streams creates the new streams in a next_streams list and the loop in gst_dash_demux_setup_streams operates on the current streams list. Either the next_streams need propagating to the current streams list before this loop, or the loop needs to operate on next_streams instead. Included is a patch to change the loop to operate on next_streams instead.
Thanks for the fix. For future reference, please use the commit message style for the project. Just browse through git log and you will understand it. commit b03acbf584cab286b8c869f4aa3bdc8e98059f65 Author: David Waring <david.waring@rd.bbc.co.uk> Date: Fri Dec 5 16:20:08 2014 +0000 dashdemux: Find current segment in the streams that have just been setup Using demux->streams will just skip the whole loop as demux->streams will always be NULL at that point https://bugzilla.gnome.org/show_bug.cgi?id=741285