[Bug 773833] rtpL16depay, rtpL24depay: support depayloading in place

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sat Nov 5 12:09:53 UTC 2016


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

--- Comment #2 from Petr Kulhavy <brain at jikos.cz> ---
Regarding the second part gst_audio_buffer_reorder_channels() already performs
the check whether any reordering is needed. So it should be either removed from
there or the gst_buffer_make_writable() moved into
gst_audio_buffer_reorder_channels(). It makes no sense that the check is done
twice. 

My preferred way would be to move gst_buffer_make_writable() into
gst_audio_buffer_reorder_channels() because the same problem is also in
gstrtpL16pay, gstrtpL24pay and gstrtpL24depay. However I don't know what APIs
would be affected by that...

If none of the above is acceptable I would suggest at least to extend the
audio-channels API with a function, which performs the check if reordering is
needed.

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