gst/volume/gstvolume.*: Extend float audio support (double) and some int->uint cleanups.

Original commit message from CVS:
* gst/volume/gstvolume.c: (volume_choose_func),
(volume_update_real_volume), (gst_volume_set_volume),
(gst_volume_init), (volume_process_double), (volume_process_float),
(volume_process_int16), (volume_process_int16_clamp),
(volume_set_caps), (volume_transform_ip), (volume_update_volume):
* gst/volume/gstvolume.h:
Extend float audio support (double) and some int->uint cleanups.
This commit is contained in:
Stefan Kost 2007-02-21 13:08:51 +00:00
parent 7bc6fc07a1
commit b8dfedde6a
3 changed files with 95 additions and 56 deletions

View file

@ -1,3 +1,13 @@
2007-02-21 Stefan Kost <ensonic@users.sf.net>
* gst/volume/gstvolume.c: (volume_choose_func),
(volume_update_real_volume), (gst_volume_set_volume),
(gst_volume_init), (volume_process_double), (volume_process_float),
(volume_process_int16), (volume_process_int16_clamp),
(volume_set_caps), (volume_transform_ip), (volume_update_volume):
* gst/volume/gstvolume.h:
Extend float audio support (double) and some int->uint cleanups.
2007-02-20 Edward Hervey <edward@fluendo.com> 2007-02-20 Edward Hervey <edward@fluendo.com>
* gst/playback/gstdecodebin2.c: (gst_decode_bin_dispose), * gst/playback/gstdecodebin2.c: (gst_decode_bin_dispose),

View file

@ -55,14 +55,14 @@
#include "gstvolume.h" #include "gstvolume.h"
/* some defines for audio processing */ /* some defines for audio processing */
/* the volume factor is a range from 0.0 to (arbitrary) 4.0 /* the volume factor is a range from 0.0 to (arbitrary) VOLUME_MAX_DOUBLE = 10.0
* we map 1.0 to VOLUME_UNITY_INT * we map 1.0 to VOLUME_UNITY_INT16
*/ */
#define VOLUME_UNITY_INT 8192 /* internal int for unity */ #define VOLUME_UNITY_INT16 8192 /* internal int for unity 2^13 */
#define VOLUME_UNITY_BIT_SHIFT 13 /* number of bits to shift for unity */ #define VOLUME_UNITY_INT16_BIT_SHIFT 13 /* number of bits to shift for unity */
#define VOLUME_MAX_DOUBLE 10.0 #define VOLUME_MAX_DOUBLE 10.0
#define VOLUME_MAX_INT16 32767 #define VOLUME_MAX_INT16 32767
#define VOLUME_MIN_INT16 -32768 #define VOLUME_MIN_INT16 -32768
/* number of steps we use for the mixer interface to go from 0.0 to 1.0 */ /* number of steps we use for the mixer interface to go from 0.0 to 1.0 */
# define VOLUME_STEPS 100 # define VOLUME_STEPS 100
@ -98,7 +98,7 @@ static GstStaticPadTemplate volume_sink_template =
"rate = (int) [ 1, MAX ], " "rate = (int) [ 1, MAX ], "
"channels = (int) [ 1, MAX ], " "channels = (int) [ 1, MAX ], "
"endianness = (int) BYTE_ORDER, " "endianness = (int) BYTE_ORDER, "
"width = (int) 32; " "width = (int) {32, 64}; "
"audio/x-raw-int, " "audio/x-raw-int, "
"channels = (int) [ 1, MAX ], " "channels = (int) [ 1, MAX ], "
"rate = (int) [ 1, MAX ], " "rate = (int) [ 1, MAX ], "
@ -114,7 +114,7 @@ static GstStaticPadTemplate volume_src_template = GST_STATIC_PAD_TEMPLATE
"rate = (int) [ 1, MAX ], " "rate = (int) [ 1, MAX ], "
"channels = (int) [ 1, MAX ], " "channels = (int) [ 1, MAX ], "
"endianness = (int) BYTE_ORDER, " "endianness = (int) BYTE_ORDER, "
"width = (int) 32; " "width = (int) {32, 64}; "
"audio/x-raw-int, " "audio/x-raw-int, "
"channels = (int) [ 1, MAX ], " "channels = (int) [ 1, MAX ], "
"rate = (int) [ 1, MAX ], " "rate = (int) [ 1, MAX ], "
@ -158,33 +158,44 @@ static GstFlowReturn volume_transform_ip (GstBaseTransform * base,
static gboolean volume_set_caps (GstBaseTransform * base, GstCaps * incaps, static gboolean volume_set_caps (GstBaseTransform * base, GstCaps * incaps,
GstCaps * outcaps); GstCaps * outcaps);
static void volume_process_double (GstVolume * this, gpointer bytes,
guint n_bytes);
static void volume_process_float (GstVolume * this, gpointer bytes, static void volume_process_float (GstVolume * this, gpointer bytes,
gint n_bytes); guint n_bytes);
static void volume_process_int16 (GstVolume * this, gpointer bytes, static void volume_process_int16 (GstVolume * this, gpointer bytes,
gint n_bytes); guint n_bytes);
static void volume_process_int16_clamp (GstVolume * this, gpointer bytes, static void volume_process_int16_clamp (GstVolume * this, gpointer bytes,
gint n_bytes); guint n_bytes);
/* helper functions */ /* helper functions */
static void static void
volume_choose_func (GstVolume * this) volume_choose_func (GstVolume * this)
{ {
this->process = NULL;
switch (this->format) { switch (this->format) {
case GST_VOLUME_FORMAT_INT: case GST_VOLUME_FORMAT_INT:
/* FIXME: should we also support gint8, gint32? */
/* only clamp if the gain is greater than 1.0 /* only clamp if the gain is greater than 1.0
* FIXME: real_vol_i can change while processing the buffer! * FIXME: real_vol_i can change while processing the buffer!
*/ */
if (this->real_vol_i > VOLUME_UNITY_INT) if (this->real_vol_i > VOLUME_UNITY_INT16)
this->process = volume_process_int16_clamp; this->process = volume_process_int16_clamp;
else else
this->process = volume_process_int16; this->process = volume_process_int16;
break; break;
case GST_VOLUME_FORMAT_FLOAT: case GST_VOLUME_FORMAT_FLOAT:
this->process = volume_process_float; switch (this->width) {
case 32:
this->process = volume_process_float;
break;
case 64:
this->process = volume_process_double;
break;
}
break; break;
default: default:
this->process = NULL; break;
} }
} }
@ -199,7 +210,7 @@ volume_update_real_volume (GstVolume * this)
} else { } else {
this->real_vol_f = this->volume_f; this->real_vol_f = this->volume_f;
this->real_vol_i = this->volume_i; this->real_vol_i = this->volume_i;
passthrough = (this->volume_i == VOLUME_UNITY_INT); passthrough = (this->volume_i == VOLUME_UNITY_INT16);
} }
volume_choose_func (this); volume_choose_func (this);
gst_base_transform_set_passthrough (GST_BASE_TRANSFORM (this), passthrough); gst_base_transform_set_passthrough (GST_BASE_TRANSFORM (this), passthrough);
@ -240,7 +251,7 @@ gst_volume_set_volume (GstMixer * mixer, GstMixerTrack * track, gint * volumes)
g_return_if_fail (GST_IS_VOLUME (this)); g_return_if_fail (GST_IS_VOLUME (this));
this->volume_f = (gfloat) volumes[0] / VOLUME_STEPS; this->volume_f = (gfloat) volumes[0] / VOLUME_STEPS;
this->volume_i = this->volume_f * VOLUME_UNITY_INT; this->volume_i = this->volume_f * VOLUME_UNITY_INT16;
volume_update_real_volume (this); volume_update_real_volume (this);
} }
@ -344,9 +355,9 @@ gst_volume_init (GstVolume * this, GstVolumeClass * g_class)
GstMixerTrack *track = NULL; GstMixerTrack *track = NULL;
this->mute = FALSE; this->mute = FALSE;
this->volume_i = VOLUME_UNITY_INT; this->volume_i = VOLUME_UNITY_INT16;
this->volume_f = 1.0; this->volume_f = 1.0;
this->real_vol_i = VOLUME_UNITY_INT; this->real_vol_i = VOLUME_UNITY_INT16;
this->real_vol_f = 1.0; this->real_vol_f = 1.0;
this->tracklist = NULL; this->tracklist = NULL;
this->format = GST_VOLUME_FORMAT_NONE; this->format = GST_VOLUME_FORMAT_NONE;
@ -369,12 +380,24 @@ gst_volume_init (GstVolume * this, GstVolumeClass * g_class)
*/ */
static void static void
volume_process_float (GstVolume * this, gpointer bytes, gint n_bytes) volume_process_double (GstVolume * this, gpointer bytes, guint n_bytes)
{ {
gfloat *data; gdouble *data = (gdouble *) bytes;
gint i, num_samples; guint i, num_samples;
num_samples = n_bytes / sizeof (gdouble);
for (i = 0; i < num_samples; i++) {
*data++ *= this->real_vol_f;
}
}
static void
volume_process_float (GstVolume * this, gpointer bytes, guint n_bytes)
{
gfloat *data = (gfloat *) bytes;
guint i, num_samples;
data = (gfloat *) bytes;
num_samples = n_bytes / sizeof (gfloat); num_samples = n_bytes / sizeof (gfloat);
for (i = 0; i < num_samples; i++) { for (i = 0; i < num_samples; i++) {
@ -386,32 +409,11 @@ volume_process_float (GstVolume * this, gpointer bytes, gint n_bytes)
} }
static void static void
volume_process_int16 (GstVolume * this, gpointer bytes, gint n_bytes) volume_process_int16 (GstVolume * this, gpointer bytes, guint n_bytes)
{ {
gint16 *data; gint16 *data = (gint16 *) bytes;
gint i, val, num_samples; guint i, val, num_samples;
data = (gint16 *) bytes;
num_samples = n_bytes / sizeof (gint16);
/* FIXME: need... liboil...
* oil_scalarmultiply_s16_ns ?
* https://bugs.freedesktop.org/show_bug.cgi?id=7060
*/
for (i = 0; i < num_samples; i++) {
/* we use bitshifting instead of dividing by UNITY_INT for speed */
val = (gint) * data;
*data++ = (gint16) ((this->real_vol_i * val) >> VOLUME_UNITY_BIT_SHIFT);
}
}
static void
volume_process_int16_clamp (GstVolume * this, gpointer bytes, gint n_bytes)
{
gint16 *data;
gint i, val, num_samples;
data = (gint16 *) bytes;
num_samples = n_bytes / sizeof (gint16); num_samples = n_bytes / sizeof (gint16);
/* FIXME: need... liboil... /* FIXME: need... liboil...
@ -422,8 +424,29 @@ volume_process_int16_clamp (GstVolume * this, gpointer bytes, gint n_bytes)
/* we use bitshifting instead of dividing by UNITY_INT for speed */ /* we use bitshifting instead of dividing by UNITY_INT for speed */
val = (gint) * data; val = (gint) * data;
*data++ = *data++ =
(gint16) CLAMP ((this->real_vol_i * val) >> VOLUME_UNITY_BIT_SHIFT, (gint16) ((this->real_vol_i * val) >> VOLUME_UNITY_INT16_BIT_SHIFT);
VOLUME_MIN_INT16, VOLUME_MAX_INT16); }
}
static void
volume_process_int16_clamp (GstVolume * this, gpointer bytes, guint n_bytes)
{
gint16 *data = (gint16 *) bytes;
guint i, val, num_samples;
num_samples = n_bytes / sizeof (gint16);
/* FIXME: need... liboil...
* oil_scalarmultiply_s16_ns ?
* https://bugs.freedesktop.org/show_bug.cgi?id=7060
*/
for (i = 0; i < num_samples; i++) {
/* we use bitshifting instead of dividing by UNITY_INT for speed */
val = (gint) * data;
*data++ =
(gint16) CLAMP ((this->real_vol_i *
val) >> VOLUME_UNITY_INT16_BIT_SHIFT, VOLUME_MIN_INT16,
VOLUME_MAX_INT16);
} }
} }
@ -435,19 +458,22 @@ volume_set_caps (GstBaseTransform * base, GstCaps * incaps, GstCaps * outcaps)
{ {
GstVolume *this = GST_VOLUME (base); GstVolume *this = GST_VOLUME (base);
const gchar *mimetype; const gchar *mimetype;
GstStructure *structure;
GST_DEBUG_OBJECT (this, GST_DEBUG_OBJECT (this,
"set_caps: in %" GST_PTR_FORMAT " out %" GST_PTR_FORMAT, incaps, outcaps); "set_caps: in %" GST_PTR_FORMAT " out %" GST_PTR_FORMAT, incaps, outcaps);
mimetype = gst_structure_get_name (gst_caps_get_structure (incaps, 0)); structure = gst_caps_get_structure (incaps, 0);
gst_structure_get_int (structure, "width", &this->width);
mimetype = gst_structure_get_name (structure);
/* based on mimetype, choose the correct volume_process format */ /* based on mimetype, choose the correct volume_process format */
if (strcmp (mimetype, "audio/x-raw-int") == 0) { if (strcmp (mimetype, "audio/x-raw-int") == 0) {
this->format = GST_VOLUME_FORMAT_INT; this->format = GST_VOLUME_FORMAT_INT;
GST_INFO_OBJECT (this, "use int16"); GST_DEBUG_OBJECT (this, "use int");
} else if (strcmp (mimetype, "audio/x-raw-float") == 0) { } else if (strcmp (mimetype, "audio/x-raw-float") == 0) {
this->format = GST_VOLUME_FORMAT_FLOAT; this->format = GST_VOLUME_FORMAT_FLOAT;
GST_INFO_OBJECT (this, "use float"); GST_DEBUG_OBJECT (this, "use float");
} else { } else {
this->process = NULL; this->process = NULL;
goto invalid_format; goto invalid_format;
@ -479,7 +505,9 @@ volume_transform_ip (GstBaseTransform * base, GstBuffer * outbuf)
if (gst_base_transform_is_passthrough (base)) if (gst_base_transform_is_passthrough (base))
return GST_FLOW_OK; return GST_FLOW_OK;
/* FIXME: subdivide GST_BUFFER_SIZE into small chunks for smooth fades */ /* FIXME: if controllers are bound, subdivide GST_BUFFER_SIZE into small
* chunks for smooth fades, what is small? 1/10th sec.
*/
timestamp = GST_BUFFER_TIMESTAMP (outbuf); timestamp = GST_BUFFER_TIMESTAMP (outbuf);
timestamp = timestamp =
gst_segment_to_stream_time (&base->segment, GST_FORMAT_TIME, timestamp); gst_segment_to_stream_time (&base->segment, GST_FORMAT_TIME, timestamp);
@ -519,7 +547,7 @@ volume_update_volume (const GValue * value, gpointer data)
g_return_if_fail (GST_IS_VOLUME (this)); g_return_if_fail (GST_IS_VOLUME (this));
this->volume_f = g_value_get_double (value); this->volume_f = g_value_get_double (value);
this->volume_i = this->volume_f * VOLUME_UNITY_INT; this->volume_i = this->volume_f * VOLUME_UNITY_INT16;
volume_update_real_volume (this); volume_update_real_volume (this);
} }

View file

@ -56,14 +56,15 @@ typedef enum {
struct _GstVolume { struct _GstVolume {
GstBaseTransform element; GstBaseTransform element;
void (*process)(GstVolume*, gpointer, gint); void (*process)(GstVolume*, gpointer, guint);
gboolean mute; gboolean mute;
gint volume_i, real_vol_i; /* the _i(nt) values get synchronized with the */ gint volume_i, real_vol_i; /* the _i(nt) values get synchronized with the */
gfloat volume_f, real_vol_f; /* _f(loat) values on each update */ gfloat volume_f, real_vol_f; /* _f(loat) values on each update */
GList *tracklist; GList *tracklist;
GstVolumeFormat format; GstVolumeFormat format; /* caps variables */
gint width;
}; };
struct _GstVolumeClass { struct _GstVolumeClass {