GNOME Bugzilla – Bug 735848
adaptivedemux: new base class
Last modified: 2014-12-24 04:10:56 UTC
Currently all 3 adaptive demuxers share a lot of code. This adaptive demuxer base class refactors the common code to a single place and leave the subclasses only to deal with mapping the specifics of their manifests into the fragments data (url, byte ranges, timestamps). Simple playback and seeking are working and tested. The major gap now is testing with live playback. The latest version can be found at http://cgit.freedesktop.org/~thiagoss/gst-plugins-bad/log/?h=adaptivedemux Let me know if you have another preferred way of reviewing this so I can set it up as the patches are rather large. I'm also going to attach the main patch here. Additionally, a few new features have landed in some of the adaptive demuxers in the past days that are not on the branch but those are trivial to port once we get some review on the current status.
Created attachment 285088 [details] [review] adds adaptivedemux base class
Branch updated in freedesktop. The API is not ideal yet but it is not going to be public anyway. Besides, people will only test it when it lands upstream and I have some time available to respond and fix bugs in the upcoming weeks. The overhead of backporting patches to this branch or replicating fixes across elements upstream is also another unnecessary workload that would disappear. In the long run I think it makes sense to merge already.
Created attachment 291299 [details] [review] adaptivedemux: add adaptivedemux base class Updated patch
Yes, it should land very soon if we want to get it in in this cycle, otherwise we won't have enough time to stabilise it before the release. I agree that no one is going to test it before it lands :)
Yes, merge it please :)
commit 9ec9f3f11956280de031cf3d5d1ea30c472e45d2 Author: Thiago Santos <thiagoss@osg.samsung.com> Date: Fri Nov 21 20:42:09 2014 -0300 adaptivedemux: add adaptivedemux base class https://bugzilla.gnome.org/show_bug.cgi?id=735848 commit 58a1b0d05883d795d345eb4a38275f8fa67ea36b Author: Thiago Santos <thiagoss@osg.samsung.com> Date: Tue Aug 26 16:45:46 2014 -0300 dashdemux: port to adaptive demux Adaptive demux and dashdemux port merged. Will keep this open until all ports are done.
mssdemux merged commit b418c88b26221afdefaec85e80c8f19960e71212 Author: Thiago Santos <thiagoss@osg.samsung.com> Date: Sat Nov 22 01:01:50 2014 -0300 mssdemux: port to adaptivedemux base class
commit 1e9ce11efdb09eed7632bc7c69ce5135f6eeb93b Author: Thiago Santos <thiagoss@osg.samsung.com> Date: Wed Aug 27 16:26:19 2014 -0300 hlsdemux: port to adaptive base class All ported and merged now