[Bug 749328] hlsdemux: Simplify logic in process_manifest
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Fri Jun 5 05:44:43 PDT 2015
https://bugzilla.gnome.org/show_bug.cgi?id=749328
Thiago Sousa Santos <thiagossantos at gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|REOPENED |RESOLVED
Resolution|--- |FIXED
--- Comment #8 from Thiago Sousa Santos <thiagossantos at gmail.com> ---
The tests were failing as it would try to get the minimum bitrate variant but
would get the previous/default one with the new behavior.
It seems to me that just dropping the TODO is enough on this case, after the
connection-speed logic has been moved to the baseclass there isn't much left to
fix here.
commit 0a63fa7a01e82b00251f1a995d8cf14593d0fbfb
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date: Fri Jun 5 09:22:58 2015 -0300
hlsdemux: drop TODO that doesn't need a solution
Connection speed is only checked at that point in hlsdemux so there
is no real need to refactor it.
https://bugzilla.gnome.org/show_bug.cgi?id=749328
commit 581d8c0b8d0c3d7d4261282c8b1555e43deecb6b
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date: Fri Jun 5 09:15:34 2015 -0300
Revert "hlsdemux: Simplify logic in process_manifest"
This reverts commit 4ca3a22b6b33ad8be4383063e76f79c4d346535d.
The connection-speed=0 is used as a special value in the property
of hlsdemux to mean 'automatic' selection, m3u8.c doesn't need
to know about that as it should be as simple as possible.
So this patch hides this automatic selection documented in hlsdemux
into m3u8 logic and I think the gets harder to understand the code.
It also makes the hlsdemux unit tests work again
https://bugzilla.gnome.org/show_bug.cgi?id=749328
--
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