[Bug 790762] rtpsession: do not report internal sources

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Dec 1 12:19:55 UTC 2017


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

Håvard Graff (hgr) <havard.graff at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |havard.graff at gmail.com

--- Comment #11 from Håvard Graff (hgr) <havard.graff at gmail.com> ---
Hi!

First, good work Miguel and team, this looks like real improvements to the RTP
stack!

What I really really would like is for the following patch to be merged first,
and then have the test-changes (and potentially the new tests) use the new
SessionHarness instead:
https://bugzilla.gnome.org/show_bug.cgi?id=791070

As an example, the whole crank_harness_rtcp block would just be a matter of
calling the provided session_harness_produce_rtcp.

The patch "make test harness always sender and receiver" would conflict
horribly with the SessionHarness changes, and I think it achieves exactly the
same thing but with much less code to worry about. (SessionHarness does set up
both rtcp and rtp send/recv)

Would this be acceptable to you Miguel? I am happy to help out converting your
test to the new SessionHarness, and I think you will find it only makes things
clearer and easier?

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