[Bug 751311] rtp: Dynamic dropout / reorder limits

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #306448|none                        |accepted-commit_after_freez
             status|                            |e

--- Comment #40 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 306448:
 --> (https://bugzilla.gnome.org/review?bug=751311&attachment=306448)

::: gst/rtpmanager/rtpstats.c
@@ +80,3 @@
+{
+  if (time_ms <= 0 || !ctx->probed) {
+    return RTP_DEF_DROPOUT;

So if we don't know anything we will return 3000 or 100 here as safe default
values, and otherwise use the calculations below ... which could get lower than
3000 or 100 (300 or 10).

We need to make sure because of that, that we never run into the cases where we
go below the default (and current value) when it's not allowed. Like during
bursts

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