[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