diff options
| author | Nirbheek Chauhan <nirbheek@centricular.com> | 2021-04-06 17:02:34 +0530 |
|---|---|---|
| committer | Tim-Philipp Müller <tim@centricular.com> | 2021-04-13 11:20:22 +0100 |
| commit | 64b40ba76601a186288c15e7bc52f9dd19ed0a4d (patch) | |
| tree | d19b47472a02ee55a2275f7a868d61a5542f8f8e | |
| parent | a1bf3d8d540a25268d612a489e1e836d6ea737b0 (diff) | |
| download | gstreamer-plugins-good-64b40ba76601a186288c15e7bc52f9dd19ed0a4d.tar.gz | |
rtpjitterbuffer: More logging when calculating rfc7273 timestamps
This code can be fragile, since it is very exacting in the timestamps
that it will accept. Add more logging so it's easier to debug issues
and figure out whether it's a bug in the calculation or something
wrong in the incoming buffers.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/merge_requests/946>
| -rw-r--r-- | gst/rtpmanager/rtpjitterbuffer.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/gst/rtpmanager/rtpjitterbuffer.c b/gst/rtpmanager/rtpjitterbuffer.c index 6ef7df2ab..aef5cbc35 100644 --- a/gst/rtpmanager/rtpjitterbuffer.c +++ b/gst/rtpmanager/rtpjitterbuffer.c @@ -919,8 +919,14 @@ rtp_jitter_buffer_calculate_pts (RTPJitterBuffer * jbuf, GstClockTime dts, else pts = 0; - GST_DEBUG ("RFC7273 clock time %" GST_TIME_FORMAT ", out %" GST_TIME_FORMAT, - GST_TIME_ARGS (rtpsystime), GST_TIME_ARGS (pts)); + GST_DEBUG ("RFC7273 clock time %" GST_TIME_FORMAT ", ntptime %" + GST_TIME_FORMAT ", ntprtptime %" G_GUINT64_FORMAT ", rtptime %" + G_GUINT32_FORMAT ", base_time %" GST_TIME_FORMAT ", internal %" + GST_TIME_FORMAT ", external %" GST_TIME_FORMAT ", out %" + GST_TIME_FORMAT, GST_TIME_ARGS (rtpsystime), GST_TIME_ARGS (ntptime), + ntprtptime, rtptime, GST_TIME_ARGS (base_time), + GST_TIME_ARGS (internal), GST_TIME_ARGS (external), + GST_TIME_ARGS (pts)); } else { /* If we used the RFC7273 clock before and not anymore, * we need to resync it later again */ |