GNOME Bugzilla – Bug 707734
rtp: add payloader and depayloader for 24bit raw audio
Last modified: 2013-09-18 16:52:17 UTC
I don't think GStreamer currently supports this, so I copied and modified the elements supporting 16 bit raw audio in order to support 24 bit payloads.
Created attachment 254430 [details] [review] Add rtpL24depay / rtpL24pay elements
This is part of RFC 3190, which also lists DAT12 and L20 as extensions. I'm just considering if we need an ARAW de/payloader that handles all these raw formats or keep on duplicating the audio de/payloaders. It would make for less code but then there would not be a direct mapping between the encoding-name and the payloader anymore. For now, let's just duplicate it to keep things consistent.
Would, - renaming these elements to something more generic (rtparawpay / rtparawdepay?) - requiring that a 'sample format' be specified (but initially only allow the value S24BE) - otherwise keeping the code the same make for an easier transition to one-element-for-all-formats, or just be too confusing? I don't mind doing some basic rearrangement if it makes the code more useful -- bearing in mind that I am new to gstreamer and glib APIs!
commit a956a6ceb2deb87cc1361aee1d6626449f46dab2 Author: David Holroyd <dave@badgers-in-foil.co.uk> Date: Mon Sep 9 11:16:40 2013 +0200 rtp: add L24 pay and depayloader Fixes https://bugzilla.gnome.org/show_bug.cgi?id=707734
(In reply to comment #3) > Would, > - renaming these elements to something more generic (rtparawpay / > rtparawdepay?) > - requiring that a 'sample format' be specified (but initially only allow the > value S24BE) > - otherwise keeping the code the same I think it's a bit confusing for now.