[Bug 762259] rtpjitterbuffer: Add RFC7273 media clock handling

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Mar 24 13:25:54 UTC 2016


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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #321878|none                        |reviewed
             status|                            |

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

::: gst/rtpmanager/rtpjitterbuffer.c
@@ +900,3 @@
+      nsrtptimediff = 0;
+
+    rtpntptime = nsrtptimediff + jbuf->media_clock_base_time;

It might make sense to update the base times here every now and then, e.g. to
the average of all in the last 10 seconds or something. Otherwise we will have
a base in the past, and small inaccuracies in the netclock sync will result in
huge differences as the distance from the base is big.

-- 
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