[Bug 761947] rtpbasepayload: rtpbasedepayload: Add source-info property

[GStreamer (GNOME Bugzilla)]

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

--- Comment #6 from Stian Selnes (stianse) <stian.selnes at gmail.com> ---
I've updated the patch and fixed version, ABI compability and removed the
writable check as gst_buffer_add_meta already does that.

Here's why ssrc is a pointer and it makes sense to allow it to be NULL, and
also why we shouldn't use a generic RTP header meta for this use case: As
mentioned earlier the typical use case is to have a mixer that creates a
combined stream and puts the contributing sources in the CSRC. In practice that
means that GstAggregator or something equivalent will take the meta of the
incoming buffers and create a new combined meta for the outgoing buffer.
Generally it does not make sense to combine all RTP header information, but it
DOES makes sense to aggregate the sources. The outgoing buffer should have meta
with ssrc=NULL (as anything else doesn't make sense to me) and have the
contributing sources added to the list of csrc.

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