[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 #333070|none                        |needs-work
             status|                            |

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

Looks generally good but:

::: gst/rtpmanager/rtpstats.c
@@ +72,2 @@
   } else {
+    ctx->avg_packet_rate = (ctx->avg_packet_rate + new_packet_rate) / 2;

Why? The idea here was to round up.

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