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.
- 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!
Author: David Holroyd <firstname.lastname@example.org>
Date: Mon Sep 9 11:16:40 2013 +0200
rtp: add L24 pay and depayloader
(In reply to comment #3)
> - renaming these elements to something more generic (rtparawpay /
> - 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.