[Bug 751444] RTP: Keep track of all estimated round trip times observed per each report block

[GStreamer (GNOME Bugzilla)]

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

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

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

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

::: gst/rtpmanager/rtpsource.c
@@ +1362,3 @@

+  curridx = src_sender->stats.curr_rr ^ 1;
+  curr = &src_sender->stats.rr[curridx];

You now still store the rtt in the sender source, not in the receiver source.

Isn't the whole point of this to store the rtt in the receiver source (instead
of the sender source as it is now), and additionally provide a list of all rtts
in the sender source?

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