[Bug 701404] dashdemux: should not buffer the entire Period

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=701404
  GStreamer | gst-plugins-bad | 1.x

Thiago Sousa Santos <thiago.sousa.santos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1

--- Comment #16 from Thiago Sousa Santos <thiago.sousa.santos at collabora.co.uk> 2013-07-03 20:46:56 UTC ---
I don't see much an advantage on buffering less than max_buffering_time,
switches are faster the smaller the buffer but they can also switch to worse as
much as it can switch to better. Why switching earlier is a good thing if we
don't know how will it switch? Also we aren't checking how much is buffered
downstream on this case.

If we are going to restrict to minBufferingTime I'd suggest changing
max_buffering_time to 0 by default and use it as the top limit in case the user
specifies to do so.

Check out the patch I attached, it already cares for the conditions and locking
as GstDataQueue already has it internally.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.