From 6f06c87c1255c45d0fd020ff30a41b9a7e83a7d9 Mon Sep 17 00:00:00 2001 From: Andy Wingo Date: Wed, 16 Jul 2003 16:08:14 +0000 Subject: [PATCH] actually recurse into sndfile if we are able big ladspa cleanups, mainly to comply with the buffer-frames caps proper... Original commit message from CVS: * actually recurse into sndfile if we are able * big ladspa cleanups, mainly to comply with the buffer-frames caps property, but also general cleanups - the samplerate prop is gone, if you want to set it explicitly (as in for get-based plugins) you need to use a filtered connection, just like with buffer-frames * big float2int and int2float changes for buffer-frames compatibility - I think it's quite a bit simpler * make the ossclock general, add it to gstaudio, and use it in sndfile as well i need to update mimetypes, but that's coming soon. there are some other plugins that don't support buffer-frames, i guess i need to get around to fixing them as well. --- ext/Makefile.am | 2 +- ext/ladspa/gstladspa.c | 835 ++++++++++++++++++++----------------------------- ext/ladspa/gstladspa.h | 5 +- sys/oss/Makefile.am | 3 +- sys/oss/gstossaudio.c | 3 + sys/oss/gstossclock.c | 191 ----------- sys/oss/gstossclock.h | 82 ----- sys/oss/gstosssink.c | 13 +- sys/oss/gstosssink.h | 2 +- sys/oss/gstosssrc.c | 10 +- 10 files changed, 349 insertions(+), 797 deletions(-) delete mode 100644 sys/oss/gstossclock.c delete mode 100644 sys/oss/gstossclock.h diff --git a/ext/Makefile.am b/ext/Makefile.am index 8432632..5581964 100644 --- a/ext/Makefile.am +++ b/ext/Makefile.am @@ -273,7 +273,7 @@ SUBDIRS=$(A52DEC_DIR) $(AALIB_DIR) $(ALSA_DIR) \ $(MAD_DIR) $(MATROSKA_DIR) $(MIKMOD_DIR) \ $(MPEG2DEC_DIR) $(PANGO_DIR) $(RAW1394_DIR) \ $(SDL_DIR) $(SHOUT_DIR) $(SIDPLAY_DIR) \ - $(SMOOTHWAVE_DIR) $(SWFDEC_DIR) $(TARKIN_DIR) \ + $(SMOOTHWAVE_DIR) $(SNDFILE_DIR) $(SWFDEC_DIR) $(TARKIN_DIR) \ $(VORBIS_DIR) $(XVID_DIR) $(SNAPSHOT_DIR) DIST_SUBDIRS=\ diff --git a/ext/ladspa/gstladspa.c b/ext/ladspa/gstladspa.c index ddaa797..f4b9684 100644 --- a/ext/ladspa/gstladspa.c +++ b/ext/ladspa/gstladspa.c @@ -27,53 +27,51 @@ #include "utils.h" /* ladspa sdk utility functions */ -GST_PAD_TEMPLATE_FACTORY (ladspa_sink_factory, - "sink", +/* takes ownership of the name */ +static GstPadTemplate* +ladspa_sink_factory (gchar *name) +{ + return GST_PAD_TEMPLATE_NEW ( + name, GST_PAD_SINK, - GST_PAD_REQUEST, + GST_PAD_ALWAYS, GST_CAPS_NEW ( "ladspa_sink", "audio/x-raw-float", - "width", GST_PROPS_INT (32), - "endianness", GST_PROPS_INT (G_BYTE_ORDER), - "rate", GST_PROPS_INT_RANGE (4000, 96000), - "intercept", GST_PROPS_FLOAT(0.0), - "slope", GST_PROPS_FLOAT(1.0), - "channels", GST_PROPS_INT (1) - ) -); - -GST_PAD_TEMPLATE_FACTORY (ladspa_src_factory, - "src", + "width", GST_PROPS_INT (32), + "endianness", GST_PROPS_INT (G_BYTE_ORDER), + "rate", GST_PROPS_INT_RANGE (4000, 96000), + "buffer-frames", GST_PROPS_INT_RANGE (1, G_MAXINT), + "channels", GST_PROPS_INT (1) + ) + ); +} + +/* takes ownership of the name */ +static GstPadTemplate* +ladspa_src_factory (gchar *name) +{ + return GST_PAD_TEMPLATE_NEW ( + name, GST_PAD_SRC, - GST_PAD_REQUEST, + GST_PAD_ALWAYS, GST_CAPS_NEW ( "ladspa_src", "audio/x-raw-float", - "width", GST_PROPS_INT (32), - "endianness", GST_PROPS_INT (G_BYTE_ORDER), - "rate", GST_PROPS_INT_RANGE (4000, 96000), - "intercept", GST_PROPS_FLOAT (0.0), - "slope", GST_PROPS_FLOAT (1.0), - "channels", GST_PROPS_INT (1) - ) -); - -static GstPadTemplate *srctempl, *sinktempl; - -enum { - ARG_0, - ARG_SAMPLERATE, - ARG_BUFFERSIZE, - ARG_LAST, -}; + "width", GST_PROPS_INT (32), + "endianness", GST_PROPS_INT (G_BYTE_ORDER), + "rate", GST_PROPS_INT_RANGE (4000, 96000), + "buffer-frames", GST_PROPS_INT_RANGE (1, G_MAXINT), + "channels", GST_PROPS_INT (1) + ) + ); +} static void gst_ladspa_class_init (GstLADSPAClass *klass); static void gst_ladspa_init (GstLADSPA *ladspa); -static void gst_ladspa_update_int(const GValue *value, gpointer data); -static GstPadLinkReturn gst_ladspa_connect (GstPad *pad, GstCaps *caps); -static GstPadLinkReturn gst_ladspa_connect_get (GstPad *pad, GstCaps *caps); +static void gst_ladspa_update_int (const GValue *value, gpointer data); +static GstPadLinkReturn gst_ladspa_link (GstPad *pad, GstCaps *caps); static void gst_ladspa_force_src_caps (GstLADSPA *ladspa, GstPad *pad); static void gst_ladspa_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec); @@ -89,26 +87,15 @@ static void gst_ladspa_chain (GstPad *pad,GstBuffer *buf); static GstBuffer * gst_ladspa_get (GstPad *pad); static GstElementClass *parent_class = NULL; -/* static guint gst_ladspa_signals[LAST_SIGNAL] = { 0 }; */ static GstPlugin *ladspa_plugin; static GHashTable *ladspa_descriptors; -static GstBufferPool* -gst_ladspa_get_bufferpool (GstPad *pad) -{ - gint i; - GstBufferPool *bp; - GstLADSPA *ladspa = (GstLADSPA *) gst_pad_get_parent (pad); - GstLADSPAClass *oclass = (GstLADSPAClass *) (G_OBJECT_GET_CLASS (ladspa)); - - if (oclass->numsrcpads > 0) - for (i=0;inumsrcpads;i++) - if ((bp = gst_pad_get_bufferpool(ladspa->srcpads[i])) != NULL) - return bp; - - return NULL; -} +GST_DEBUG_CATEGORY_STATIC (ladspa_debug); +#define DEBUG(...) \ + GST_CAT_LEVEL_LOG (ladspa_debug, GST_LEVEL_DEBUG, NULL, __VA_ARGS__) +#define DEBUG_OBJ(obj,...) \ + GST_CAT_LEVEL_LOG (ladspa_debug, GST_LEVEL_DEBUG, obj, __VA_ARGS__) static void gst_ladspa_class_init (GstLADSPAClass *klass) @@ -141,23 +128,20 @@ gst_ladspa_class_init (GstLADSPAClass *klass) klass->numcontrols = 0; /* walk through the ports, count the input, output and control ports */ - for (i=0;iPortCount;i++) { - if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[i]) && - LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])){ - klass->numsinkpads++; - } - - if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[i]) && - LADSPA_IS_PORT_OUTPUT(desc->PortDescriptors[i])){ - klass->numsrcpads++; - } - - if (LADSPA_IS_PORT_CONTROL(desc->PortDescriptors[i]) && - LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])){ + for (i=0; iPortCount; i++) { + if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[i])) + if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])) + klass->numsinkpads++; + else + klass->numsrcpads++; + else if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])) klass->numcontrols++; - } } + DEBUG ("ladspa element class: init %s with %d sink, %d src, %d control\n", + g_type_name (G_TYPE_FROM_CLASS (klass)), + klass->numsinkpads, klass->numsrcpads, klass->numcontrols); + klass->srcpad_portnums = g_new0(gint,klass->numsrcpads); klass->sinkpad_portnums = g_new0(gint,klass->numsinkpads); klass->control_portnums = g_new0(gint,klass->numcontrols); @@ -167,39 +151,17 @@ gst_ladspa_class_init (GstLADSPAClass *klass) /* walk through the ports, note the portnums for srcpads, sinkpads and control params */ - for (i=0;iPortCount;i++) { - if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[i]) && - LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])){ - GST_DEBUG ("input port %d", i); - klass->sinkpad_portnums[sinkcount++] = i; - } - - if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[i]) && - LADSPA_IS_PORT_OUTPUT(desc->PortDescriptors[i])){ - GST_DEBUG ("output port %d", i); - klass->srcpad_portnums[srccount++] = i; - } - - if (LADSPA_IS_PORT_CONTROL(desc->PortDescriptors[i]) && - LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])){ - GST_DEBUG ("control port %d", i); + for (i=0; iPortCount; i++) { + if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[i])) + if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])) + klass->sinkpad_portnums[sinkcount++] = i; + else + klass->srcpad_portnums[srccount++] = i; + else if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])) klass->control_portnums[controlcount++] = i; - } } - /* no sink pads - we'll use get mode and add params for samplerate and - buffersize */ - if (klass->numsinkpads == 0 && klass->numsrcpads > 0){ - g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_SAMPLERATE, - g_param_spec_int("samplerate","samplerate","samplerate", - 0,G_MAXINT,44100,G_PARAM_READWRITE)); - g_object_class_install_property(G_OBJECT_CLASS(klass), ARG_BUFFERSIZE, - g_param_spec_int("buffersize","buffersize","buffersize", - 0,G_MAXINT,64,G_PARAM_READWRITE)); - - } - - /* now build the contorl info from the control ports */ + /* now build the control info from the control ports */ klass->control_info = g_new0(ladspa_control_info,klass->numcontrols); for (i=0;inumcontrols;i++) { @@ -235,13 +197,16 @@ gst_ladspa_class_init (GstLADSPAClass *klass) klass->control_info[i].upper = TRUE; klass->control_info[i].upperbound = desc->PortRangeHints[current_portnum].UpperBound; - if (LADSPA_IS_HINT_SAMPLE_RATE(hintdesc)) + if (LADSPA_IS_HINT_SAMPLE_RATE(hintdesc)) { klass->control_info[i].samplerate = TRUE; + klass->control_info[i].upperbound *= 44100; /* FIXME? */ + } } else { if (argtype==G_TYPE_INT) klass->control_info[i].upperbound = (gfloat)G_MAXINT; if (argtype==G_TYPE_FLOAT) klass->control_info[i].upperbound = G_MAXFLOAT; } + /* use the lowerbound as the default value */ klass->control_info[i].def = klass->control_info[i].lowerbound; #ifdef LADSPA_IS_HINT_HAS_DEFAULT @@ -284,7 +249,8 @@ gst_ladspa_class_init (GstLADSPAClass *klass) #endif /* LADSPA_IS_HINT_HAS_DEFAULT */ klass->control_info[i].def = CLAMP(klass->control_info[i].def, - klass->control_info[i].lowerbound, klass->control_info[i].upperbound); + klass->control_info[i].lowerbound, + klass->control_info[i].upperbound); if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[current_portnum])) { argperms = G_PARAM_READWRITE; @@ -324,7 +290,7 @@ gst_ladspa_class_init (GstLADSPAClass *klass) klass->control_info[i].param_name = argname; - GST_DEBUG ("adding arg %s from %s",argname, klass->control_info[i].name); + DEBUG ("adding arg %s from %s", argname, klass->control_info[i].name); if (argtype==G_TYPE_BOOLEAN){ paramspec = g_param_spec_boolean(argname,argname,argname, FALSE, argperms); @@ -343,19 +309,21 @@ gst_ladspa_class_init (GstLADSPAClass *klass) klass->control_info[i].def, argperms); } - g_object_class_install_property(G_OBJECT_CLASS(klass), i+ARG_LAST, paramspec); + /* properties have an offset of 1 */ + g_object_class_install_property(G_OBJECT_CLASS(klass), i+1, paramspec); } } static void gst_ladspa_init (GstLADSPA *ladspa) { - GstLADSPAClass *oclass = (GstLADSPAClass*)(G_OBJECT_GET_CLASS(ladspa)); + GstLADSPAClass *oclass; ladspa_control_info cinfo; - + GList *l; LADSPA_Descriptor *desc; - gint i,sinkcount,srccount,controlcount; + gint i,sinkcount,srccount; + oclass = (GstLADSPAClass*)G_OBJECT_GET_CLASS (ladspa); desc = oclass->descriptor; ladspa->descriptor = oclass->descriptor; @@ -365,39 +333,33 @@ gst_ladspa_init (GstLADSPA *ladspa) ladspa->controls = g_new(gfloat,oclass->numcontrols); ladspa->dpman = gst_dpman_new ("ladspa_dpman", GST_ELEMENT(ladspa)); - /* walk through the ports and add all the pads */ + /* set up pads */ sinkcount = 0; srccount = 0; - controlcount = 0; - for (i=0;iPortCount;i++) { - - if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[i])){ - gchar *canon_port_name = g_strdup((gchar *)desc->PortNames[i]); - g_strcanon (canon_port_name, G_CSET_A_2_Z G_CSET_a_2_z G_CSET_DIGITS "-", '-'); - if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])) { - ladspa->sinkpads[sinkcount] = gst_pad_new_from_template (sinktempl, canon_port_name); - gst_element_add_pad(GST_ELEMENT(ladspa),ladspa->sinkpads[sinkcount]); - sinkcount++; - } - if (LADSPA_IS_PORT_OUTPUT(desc->PortDescriptors[i])) { - ladspa->srcpads[srccount] = gst_pad_new_from_template (srctempl, canon_port_name); - gst_element_add_pad(GST_ELEMENT(ladspa),ladspa->srcpads[srccount]); - srccount++; - } - } - if (LADSPA_IS_PORT_CONTROL(desc->PortDescriptors[i]) && - LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])) { - cinfo = oclass->control_info[controlcount]; - /* use the lowerbound as the default value if it exists */ - ladspa->controls[controlcount]=cinfo.def; + for (l=GST_ELEMENT_CLASS (oclass)->padtemplates; l; l=l->next) { + GstPad *pad = gst_pad_new_from_template (GST_PAD_TEMPLATE (l->data), + GST_PAD_TEMPLATE_NAME_TEMPLATE (l->data)); + gst_pad_set_link_function (pad, gst_ladspa_link); + gst_element_add_pad ((GstElement*)ladspa, pad); + + if (GST_PAD_DIRECTION (pad) == GST_PAD_SINK) + ladspa->sinkpads[sinkcount++] = pad; + else + ladspa->srcpads[srccount++] = pad; + } + + /* set up dparams */ + for (i=0; inumcontrols; i++) { + if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[i])) { + cinfo = oclass->control_info[i]; + ladspa->controls[i]=cinfo.def; - /* set up dparams for this instance */ if (cinfo.toggled){ gst_dpman_add_required_dparam_callback ( ladspa->dpman, g_param_spec_int(cinfo.param_name, cinfo.name, cinfo.name, - 0, 1, (gint)(ladspa->controls[controlcount]), G_PARAM_READWRITE), - "int", gst_ladspa_update_int, &(ladspa->controls[controlcount]) + 0, 1, (gint)(ladspa->controls[i]), G_PARAM_READWRITE), + "int", gst_ladspa_update_int, &(ladspa->controls[i]) ); } else if (cinfo.integer){ @@ -405,8 +367,8 @@ gst_ladspa_init (GstLADSPA *ladspa) ladspa->dpman, g_param_spec_int(cinfo.param_name, cinfo.name, cinfo.name, (gint)cinfo.lowerbound, (gint)cinfo.upperbound, - (gint)ladspa->controls[controlcount], G_PARAM_READWRITE), - "int", gst_ladspa_update_int, &(ladspa->controls[controlcount]) + (gint)ladspa->controls[i], G_PARAM_READWRITE), + "int", gst_ladspa_update_int, &(ladspa->controls[i]) ); } else if (cinfo.samplerate){ @@ -414,8 +376,8 @@ gst_ladspa_init (GstLADSPA *ladspa) ladspa->dpman, g_param_spec_float(cinfo.param_name, cinfo.name, cinfo.name, cinfo.lowerbound, cinfo.upperbound, - ladspa->controls[controlcount], G_PARAM_READWRITE), - "hertz-rate-bound", &(ladspa->controls[controlcount]) + ladspa->controls[i], G_PARAM_READWRITE), + "hertz-rate-bound", &(ladspa->controls[i]) ); } else { @@ -423,53 +385,41 @@ gst_ladspa_init (GstLADSPA *ladspa) ladspa->dpman, g_param_spec_float(cinfo.param_name, cinfo.name, cinfo.name, cinfo.lowerbound, cinfo.upperbound, - ladspa->controls[controlcount], G_PARAM_READWRITE), - "float", &(ladspa->controls[controlcount]) + ladspa->controls[i], G_PARAM_READWRITE), + "float", &(ladspa->controls[i]) ); } - - controlcount++; } } /* nonzero default needed to instantiate() some plugins */ ladspa->samplerate = 44100; - ladspa->buffersize = 64; - ladspa->numbuffers = 16; + ladspa->buffer_frames = 0; /* should be set with caps */ ladspa->activated = FALSE; ladspa->bufpool = NULL; ladspa->inplace_broken = LADSPA_IS_INPLACE_BROKEN(ladspa->descriptor->Properties); if (sinkcount==0 && srccount == 1) { /* get mode (no sink pads) */ - GST_DEBUG ("mono get mode with 1 src pad"); - - ladspa->newcaps = TRUE; + DEBUG_OBJ (ladspa, "mono get mode with 1 src pad"); - gst_pad_set_link_function (ladspa->srcpads[0], gst_ladspa_connect_get); gst_pad_set_get_function (ladspa->srcpads[0], gst_ladspa_get); } else if (sinkcount==1){ /* with one sink we can use the chain function */ - GST_DEBUG ("chain mode"); + DEBUG_OBJ (ladspa, "chain mode"); - gst_pad_set_link_function (ladspa->sinkpads[0], gst_ladspa_connect); gst_pad_set_chain_function (ladspa->sinkpads[0], gst_ladspa_chain); - gst_pad_set_bufferpool_function (ladspa->sinkpads[0], gst_ladspa_get_bufferpool); } else if (sinkcount > 1){ /* more than one sink pad needs loop mode */ - GST_DEBUG ("loop mode with %d sink pads and %d src pads", sinkcount, srccount); + DEBUG_OBJ (ladspa, "loop mode with %d sink pads and %d src pads", sinkcount, srccount); - for (i=0;isinkpads[i], gst_ladspa_connect); - gst_pad_set_bufferpool_function (ladspa->sinkpads[i], gst_ladspa_get_bufferpool); - } gst_element_set_loop_function (GST_ELEMENT (ladspa), gst_ladspa_loop); - } - else if (sinkcount==0 && srccount == 0){ - /* for some reason these plugins exist - we'll just ignore them */ + } else if (sinkcount==0 && srccount == 0) { + /* for example, a plugin with only control inputs and output -- just ignore + * it for now */ } else { - GST_DEBUG ("%d sink pads, %d src pads not yet supported", sinkcount, srccount); + g_warning ("%d sink pads, %d src pads not yet supported", sinkcount, srccount); } gst_ladspa_instantiate (ladspa); @@ -483,169 +433,128 @@ gst_ladspa_update_int(const GValue *value, gpointer data) } static GstPadLinkReturn -gst_ladspa_connect (GstPad *pad, GstCaps *caps) +gst_ladspa_link (GstPad *pad, GstCaps *caps) { - GstLADSPA *ladspa = (GstLADSPA *) GST_PAD_PARENT (pad); - GstLADSPAClass *oclass = (GstLADSPAClass *) (G_OBJECT_GET_CLASS (ladspa)); - guint i; + GstElement *element = (GstElement*)GST_PAD_PARENT (pad); + GstLADSPA *ladspa = (GstLADSPA*)element; + const GList *l = NULL; gint rate; - g_return_val_if_fail (caps != NULL, GST_PAD_LINK_DELAYED); - g_return_val_if_fail (pad != NULL, GST_PAD_LINK_DELAYED); - - if (gst_caps_get_int (caps, "rate", &rate)){ + if (GST_CAPS_IS_FIXED (caps)) { + /* if this fails in some other plugin, the graph is left in an inconsistent + state */ + for (l=gst_element_get_pad_list (element); l; l=l->next) + if (pad != (GstPad*)l->data) + if (gst_pad_try_set_caps ((GstPad*)l->data, caps) <= 0) + return GST_PAD_LINK_REFUSED; + + /* we assume that the ladspa plugin can handle any sample rate, so this + check gets put last */ + gst_caps_get_int (caps, "rate", &rate); /* have to instantiate ladspa plugin when samplerate changes (groan) */ - if (ladspa->samplerate != rate){ + if (ladspa->samplerate != rate) { ladspa->samplerate = rate; - if (! gst_ladspa_instantiate(ladspa)) return GST_PAD_LINK_REFUSED; } - } - - /* if the caps are fixed, we are going to try to set all srcpads using this - one caps object. if any of the pads barfs, we'll refuse the connection. i'm - not sure if this is correct. */ - if (GST_CAPS_IS_FIXED (caps)) { - for (i=0;inumsrcpads;i++) { - if (gst_pad_try_set_caps (ladspa->srcpads[i], caps) <= 0) - return GST_PAD_LINK_REFUSED; - } + + gst_caps_get_int (caps, "buffer-frames", &ladspa->buffer_frames); + + if (ladspa->bufpool) + gst_buffer_pool_unref (ladspa->bufpool); + ladspa->bufpool = gst_buffer_pool_get_default (ladspa->buffer_frames * sizeof(gfloat), + 3); + + return GST_PAD_LINK_OK; } - return GST_PAD_LINK_OK; + return GST_PAD_LINK_DELAYED; } -static GstPadLinkReturn -gst_ladspa_connect_get (GstPad *pad, GstCaps *caps) +static void +gst_ladspa_force_src_caps(GstLADSPA *ladspa, GstPad *pad) { - GstLADSPA *ladspa = (GstLADSPA*)GST_OBJECT_PARENT (pad); - gint rate; - - g_return_val_if_fail (caps != NULL, GST_PAD_LINK_DELAYED); - g_return_val_if_fail (pad != NULL, GST_PAD_LINK_DELAYED); - - if (gst_caps_get_int (caps, "rate", &rate)){ - if (ladspa->samplerate != rate) { - ladspa->samplerate = rate; - if (! gst_ladspa_instantiate(ladspa)) - return GST_PAD_LINK_REFUSED; - } + if (!ladspa->buffer_frames) { + ladspa->buffer_frames = 256; /* 5 ms at 44100 kHz (just a default...) */ + g_return_if_fail (ladspa->bufpool == NULL); + ladspa->bufpool = gst_buffer_pool_get_default (ladspa->buffer_frames * sizeof(gfloat), + 3); } - return GST_PAD_LINK_OK; -} + DEBUG_OBJ (ladspa, "forcing caps with rate=%d, buffer-frames=%d", + ladspa->samplerate, ladspa->buffer_frames); -static void -gst_ladspa_force_src_caps(GstLADSPA *ladspa, GstPad *pad) -{ - GST_DEBUG ("forcing caps with rate %d", ladspa->samplerate); gst_pad_try_set_caps (pad, gst_caps_new ( "ladspa_src_caps", "audio/x-raw-float", gst_props_new ( - "width", GST_PROPS_INT (32), - "endianness", GST_PROPS_INT (G_BYTE_ORDER), - "intercept", GST_PROPS_FLOAT(0.0), - "slope", GST_PROPS_FLOAT(1.0), - "rate", GST_PROPS_INT (ladspa->samplerate), - "channels", GST_PROPS_INT (1), + "width", GST_PROPS_INT (32), + "endianness", GST_PROPS_INT (G_BYTE_ORDER), + "buffer-frames", GST_PROPS_INT (ladspa->buffer_frames), + "rate", GST_PROPS_INT (ladspa->samplerate), + "channels", GST_PROPS_INT (1), NULL ) )); - ladspa->newcaps=FALSE; } static void gst_ladspa_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec) { GstLADSPA *ladspa = (GstLADSPA*)object; - gint cid = prop_id - ARG_LAST; GstLADSPAClass *oclass; ladspa_control_info *control_info; - gfloat val=0.0; - /* these are only registered in get mode */ - switch (prop_id) { - case ARG_SAMPLERATE: - ladspa->samplerate = g_value_get_int (value); - ladspa->newcaps=TRUE; - gst_ladspa_instantiate (ladspa); - break; - case ARG_BUFFERSIZE: - ladspa->buffersize = g_value_get_int (value); - break; - } - - /* is it a ladspa plugin arg? */ - if (cid < 0) return; - oclass = (GstLADSPAClass*)(G_OBJECT_GET_CLASS (object)); - /* verify it exists and is a control (not a port) */ - g_return_if_fail(cid < oclass->numcontrols); + /* remember, properties have an offset of 1 */ + prop_id--; + + /* verify it exists */ + g_return_if_fail (prop_id < oclass->numcontrols); - control_info = &(oclass->control_info[cid]); + control_info = &(oclass->control_info[prop_id]); g_return_if_fail (control_info->name != NULL); /* check to see if it's writable */ g_return_if_fail (control_info->writable); /* now see what type it is */ - if (control_info->toggled) { - if (g_value_get_boolean (value)) - ladspa->controls[cid] = 1.0; - else - ladspa->controls[cid] = 0.0; - } else if (control_info->integer) { - val = (gfloat)g_value_get_int (value); - ladspa->controls[cid] = val; - } else { - val = g_value_get_float (value); - ladspa->controls[cid] = val; - } - - GST_DEBUG ("set arg %s to %f", control_info->name, ladspa->controls[cid]); + if (control_info->toggled) + ladspa->controls[prop_id] = g_value_get_boolean (value) ? 1.f : 0.f; + else if (control_info->integer) + ladspa->controls[prop_id] = g_value_get_int (value); + else + ladspa->controls[prop_id] = g_value_get_float (value); + + DEBUG_OBJ (object, "set arg %s to %f", control_info->name, ladspa->controls[prop_id]); } static void gst_ladspa_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec) { GstLADSPA *ladspa = (GstLADSPA*)object; - gint cid = prop_id - ARG_LAST; GstLADSPAClass *oclass = (GstLADSPAClass*)(G_OBJECT_GET_CLASS (object)); ladspa_control_info *control_info; - /* these are only registered in get mode */ - switch (prop_id){ - case ARG_SAMPLERATE: - g_value_set_int (value, ladspa->samplerate); - break; - case ARG_BUFFERSIZE: - g_value_set_int (value, ladspa->buffersize); - break; - } - - if (cid < 0) return; + /* remember, properties have an offset of 1 */ + prop_id--; - /* verify it exists and is a control (not a port) */ - if (cid >= oclass->numcontrols) return; - control_info = &(oclass->control_info[cid]); - if (control_info->name == NULL) return; + /* verify it exists */ + g_return_if_fail (prop_id < oclass->numcontrols); - GST_DEBUG ("got arg %s as %f", control_info->name, ladspa->controls[cid]); + control_info = &(oclass->control_info[prop_id]); + g_return_if_fail (control_info->name != NULL); /* now see what type it is */ - if (control_info->toggled) { - if (ladspa->controls[cid] == 1.0) - g_value_set_boolean (value, TRUE); - else - g_value_set_boolean (value, FALSE); - } else if (control_info->integer) { - g_value_set_int (value, (gint)ladspa->controls[cid]); - } else { - g_value_set_float (value, ladspa->controls[cid]); - } + if (control_info->toggled) + g_value_set_boolean (value, ladspa->controls[prop_id] == 1.0); + else if (control_info->integer) + g_value_set_int (value, (gint)ladspa->controls[prop_id]); + else + g_value_set_float (value, ladspa->controls[prop_id]); + + DEBUG_OBJ (object, "got arg %s as %f", control_info->name, ladspa->controls[prop_id]); } static gboolean @@ -666,24 +575,21 @@ gst_ladspa_instantiate (GstLADSPA *ladspa) } /* instantiate the plugin */ - GST_DEBUG ("instantiating the plugin"); + DEBUG_OBJ (ladspa, "instantiating the plugin at %d Hz", ladspa->samplerate); ladspa->handle = desc->instantiate(desc,ladspa->samplerate); g_return_val_if_fail (ladspa->handle != NULL, FALSE); - /* walk through the ports and add all the arguments */ - for (i=0;inumcontrols;i++) { - /* connect the argument to the plugin */ - GST_DEBUG ("added control port %d", oclass->control_portnums[i]); + /* connect the control ports */ + for (i=0;inumcontrols;i++) desc->connect_port(ladspa->handle, oclass->control_portnums[i], &(ladspa->controls[i])); - } /* reactivate if it was activated before the reinstantiation */ - if (was_activated){ + if (was_activated) gst_ladspa_activate(ladspa); - } + return TRUE; } @@ -694,7 +600,6 @@ gst_ladspa_change_state (GstElement *element) GstLADSPA *ladspa = (GstLADSPA*)element; desc = ladspa->descriptor; - GST_DEBUG ("changing state"); switch (GST_STATE_TRANSITION (element)) { case GST_STATE_NULL_TO_READY: gst_ladspa_activate(ladspa); @@ -713,53 +618,46 @@ gst_ladspa_change_state (GstElement *element) } static void -gst_ladspa_activate(GstLADSPA *ladspa) +gst_ladspa_activate (GstLADSPA *ladspa) { LADSPA_Descriptor *desc; desc = ladspa->descriptor; - if (ladspa->activated){ + if (ladspa->activated) gst_ladspa_deactivate(ladspa); - } - GST_DEBUG ("activating"); + DEBUG_OBJ (ladspa, "activating"); /* activate the plugin (function might be null) */ - if (desc->activate != NULL) { + if (desc->activate != NULL) desc->activate(ladspa->handle); - } ladspa->activated = TRUE; } static void -gst_ladspa_deactivate(GstLADSPA *ladspa) +gst_ladspa_deactivate (GstLADSPA *ladspa) { LADSPA_Descriptor *desc; desc = ladspa->descriptor; - GST_DEBUG ("deactivating"); + DEBUG_OBJ (ladspa, "deactivating"); /* deactivate the plugin (function might be null) */ - if (ladspa->activated && (desc->deactivate != NULL)) { + if (ladspa->activated && (desc->deactivate != NULL)) desc->deactivate(ladspa->handle); - } ladspa->activated = FALSE; } static void -gst_ladspa_loop(GstElement *element) +gst_ladspa_loop (GstElement *element) { - guint bufferbytesize, i, numsrcpads, numsinkpads, num_empty_pads; + guint i, j, numsrcpads, numsinkpads; guint num_processed, num_to_process; - GstEvent *event = NULL; - guint32 waiting; - guint32 got_bytes; + gint largest_buffer; LADSPA_Data **data_in, **data_out; GstBuffer **buffers_in, **buffers_out; - GstBufferPool *bufpool; - GstByteStream **bytestreams; GstLADSPA *ladspa = (GstLADSPA *)element; GstLADSPAClass *oclass = (GstLADSPAClass*)(G_OBJECT_GET_CLASS (ladspa)); @@ -768,212 +666,162 @@ gst_ladspa_loop(GstElement *element) numsinkpads = oclass->numsinkpads; numsrcpads = oclass->numsrcpads; + /* fixme: these mallocs need to die */ data_in = g_new0(LADSPA_Data*, numsinkpads); data_out = g_new0(LADSPA_Data*, numsrcpads); buffers_in = g_new0(GstBuffer*, numsinkpads); buffers_out = g_new0(GstBuffer*, numsrcpads); - bytestreams = g_new0(GstByteStream*, numsinkpads); - /* find a bufferpool */ - if (numsrcpads > 0 && (bufpool = gst_pad_get_bufferpool (ladspa->srcpads[0]))) { - GST_DEBUG ("Got bufferpool from first source pad"); - } else { - bufferbytesize = sizeof (LADSPA_Data) * ladspa->buffersize; - bufpool = gst_buffer_pool_get_default (bufferbytesize, ladspa->numbuffers); - GST_DEBUG ("Created default bufferpool, %d x %d bytes", ladspa->numbuffers, bufferbytesize); - } + largest_buffer = -1; - /* get the bytestreams for each pad */ - for (i=0 ; isinkpads[i]); - } - - /* since this is a loop element, we just loop here til things fall apart. */ - do { - /* we need to get the buffers_out first so we can know how many bytes to process */ - for (i=0 ; itimestamp; - data_out[i] = (LADSPA_Data *) GST_BUFFER_DATA(buffers_out[i]); - } - - bufferbytesize = GST_BUFFER_SIZE (buffers_out[0]); - ladspa->buffersize = bufferbytesize / sizeof (LADSPA_Data); - - num_empty_pads = 0; - /* first get all the necessary data from the input ports */ - for (i=0 ; isinkpads[i]); + + if (GST_IS_EVENT (buffers_in[i])) { + /* push it out on all pads */ + gst_data_ref_by_count ((GstData*)buffers_in[i], numsrcpads); + for (j=0; jsrcpads[j], buffers_in[i]); + if (GST_EVENT_TYPE (buffers_in[i]) == GST_EVENT_EOS) { + /* shut down */ + gst_element_set_eos (element); + return; } else { - data_in[i] = (LADSPA_Data *) GST_BUFFER_DATA(buffers_in[i]); - GST_BUFFER_TIMESTAMP(buffers_in[i]) = ladspa->timestamp; + goto get_buffer; } } - if (num_empty_pads > 0){ - if (num_empty_pads < numsinkpads){ - /* only some pads have EOS, need to create some empty buffers */ - for (i=0 ; ibufpool, 0, 0); - GST_BUFFER_TIMESTAMP(buffers_in[i]) = ladspa->timestamp; - data_in[i] = data = (LADSPA_Data *) GST_BUFFER_DATA(buffers_in[i]); - for (x=0 ; x < ladspa->buffersize ; x++) - data[x] = 0.0F; - - data_in[i] = (LADSPA_Data *) GST_BUFFER_DATA(buffers_in[i]); - GST_BUFFER_TIMESTAMP(buffers_in[i]) = ladspa->timestamp; - } - } - } - else { - /* all pads have EOS, time to quit */ - /* CHECKME do I have to push EOS events here? */ - GST_DEBUG ("All sink pads have EOS, finishing."); - break; - } - } - - GST_DPMAN_PREPROCESS(ladspa->dpman, ladspa->buffersize, ladspa->timestamp); - num_processed = 0; - - /* split up processing of the buffer into chunks so that dparams can - * be updated when required. - * In many cases the buffer will be processed in one chunk anyway. - */ - while(GST_DPMAN_PROCESS(ladspa->dpman, num_processed)) { - - num_to_process = GST_DPMAN_FRAMES_TO_PROCESS(ladspa->dpman); - for (i=0 ; iconnect_port (ladspa->handle, oclass->sinkpad_portnums[i], data_in[i]); - } - for (i=0 ; iconnect_port (ladspa->handle, oclass->srcpad_portnums[i], data_out[i]); - } - desc->run(ladspa->handle, num_to_process); - for (i=0 ; itimestamp; + } - num_processed += num_to_process; - } - - for (i=0 ; isrcpads[i], buffers_out[i]); - - data_out[i] = NULL; - buffers_out[i] = NULL; + if (!ladspa->bufpool) { + gst_element_error (element, "Caps were never set, bailing..."); + return; + } + + i=0; + if (!ladspa->inplace_broken) { + for (; ibufpool, 0, 0); + GST_BUFFER_TIMESTAMP (buffers_out[i]) = ladspa->timestamp; + data_out[i] = (LADSPA_Data*)GST_BUFFER_DATA (buffers_out[i]); + } + + GST_DPMAN_PREPROCESS(ladspa->dpman, largest_buffer, ladspa->timestamp); + num_processed = 0; - ladspa->timestamp += ladspa->buffersize * 10^9 / ladspa->samplerate; + /* split up processing of the buffer into chunks so that dparams can + * be updated when required. + * In many cases the buffer will be processed in one chunk anyway. + */ + while (GST_DPMAN_PROCESS (ladspa->dpman, num_processed)) { + num_to_process = GST_DPMAN_FRAMES_TO_PROCESS(ladspa->dpman); - gst_element_yield (element); - } while (TRUE); + for (i=0 ; iconnect_port (ladspa->handle, oclass->sinkpad_portnums[i], data_in[i]); + for (i=0 ; iconnect_port (ladspa->handle, oclass->srcpad_portnums[i], data_out[i]); - gst_buffer_pool_unref(bufpool); + desc->run(ladspa->handle, num_to_process); - for (i=0 ; i= numsrcpads || buffers_out[i] != buffers_in[i]) + gst_buffer_unref(buffers_in[i]); + data_in[i] = NULL; + buffers_in[i] = NULL; + } + for (i=0 ; isrcpads[i], buffers_out[i]); + + data_out[i] = NULL; + buffers_out[i] = NULL; } + + ladspa->timestamp += ladspa->buffer_frames * GST_SECOND / ladspa->samplerate; + + /* FIXME: move these mallocs and frees to the state-change handler */ g_free (buffers_out); g_free (buffers_in); g_free (data_out); g_free (data_in); - g_free (bytestreams); } static void -gst_ladspa_chain (GstPad *pad, GstBuffer *buf) +gst_ladspa_chain (GstPad *pad, GstBuffer *buffer_in) { LADSPA_Descriptor *desc; LADSPA_Data *data_in, **data_out = NULL; GstBuffer **buffers_out = NULL; - unsigned long num_samples; guint num_to_process, num_processed, i, numsrcpads; - GstLADSPA *ladspa; GstLADSPAClass *oclass; - g_return_if_fail(pad != NULL); - g_return_if_fail(GST_IS_PAD(pad)); - g_return_if_fail(buf != NULL); - - ladspa = (GstLADSPA *)gst_pad_get_parent (pad); - g_return_if_fail(ladspa != NULL); - - /* this might happen if caps nego hasn't happened */ - g_return_if_fail(ladspa->handle != NULL); - + ladspa = (GstLADSPA*)GST_OBJECT_PARENT (pad); oclass = (GstLADSPAClass *) (G_OBJECT_GET_CLASS (ladspa)); + data_in = (LADSPA_Data *) GST_BUFFER_DATA(buffer_in); + num_samples = GST_BUFFER_SIZE(buffer_in) / sizeof(gfloat); + numsrcpads = oclass->numsrcpads; + desc = ladspa->descriptor; - if (GST_IS_EVENT (buf)) { - /* push the event out all the src pads */ - for (i=0 ; inumsrcpads ; i++){ - gst_pad_push (ladspa->srcpads[0], buf); - } + /* we shouldn't get events here... */ + g_return_if_fail (GST_IS_BUFFER (buffer_in)); + + if (!ladspa->bufpool) { + gst_element_error ((GstElement*)ladspa, "Caps were never set, bailing..."); return; } - data_in = (LADSPA_Data *) GST_BUFFER_DATA(buf); - num_samples = GST_BUFFER_SIZE(buf) / sizeof(gfloat); - numsrcpads = oclass->numsrcpads; - - desc = ladspa->descriptor; - - if (numsrcpads > 0){ - guint num_created_buffers = 0; + /* FIXME: this function shouldn't need to malloc() anything */ + if (numsrcpads > 0) { buffers_out = g_new(GstBuffer*, numsrcpads); data_out = g_new(LADSPA_Data*, numsrcpads); + } - if (ladspa->inplace_broken){ - num_created_buffers = numsrcpads; - } - else { - /* we can share the buffer for input and output */ - buffers_out[0] = buf; - data_out[0] = (LADSPA_Data *)GST_BUFFER_DATA(buf); - num_created_buffers = numsrcpads - 1; - } - - if (num_created_buffers > 0){ - ladspa->bufpool = gst_buffer_pool_get_default (sizeof (LADSPA_Data) * GST_BUFFER_SIZE(buf), ladspa->numbuffers); - - for (i = numsrcpads - num_created_buffers ; ibufpool, 0, 0); - GST_BUFFER_TIMESTAMP(buffers_out[i]) = GST_BUFFER_TIMESTAMP(buf); - data_out[i] = (LADSPA_Data *) GST_BUFFER_DATA(buffers_out[i]); - } - } + i=0; + if (!ladspa->inplace_broken && numsrcpads) { + /* reuse the first (chained) buffer */ + buffers_out[i] = buffer_in; + DEBUG ("reuse: %d", GST_BUFFER_SIZE (buffer_in)); + data_out[i] = data_in; + i++; + } + for (; ibufpool, 0, 0); + /* the size of the buffer returned from the pool is the maximum size; this + chained buffer might be smaller */ + GST_BUFFER_SIZE (buffers_out[i]) = GST_BUFFER_SIZE (buffer_in); + DEBUG ("new %d", GST_BUFFER_SIZE (buffer_in)); + GST_BUFFER_TIMESTAMP (buffers_out[i]) = ladspa->timestamp; + data_out[i] = (LADSPA_Data*)GST_BUFFER_DATA (buffers_out[i]); } - GST_DPMAN_PREPROCESS(ladspa->dpman, num_samples, GST_BUFFER_TIMESTAMP(buf)); + GST_DPMAN_PREPROCESS(ladspa->dpman, num_samples, GST_BUFFER_TIMESTAMP(buffer_in)); num_processed = 0; /* split up processing of the buffer into chunks so that dparams can @@ -984,29 +832,31 @@ gst_ladspa_chain (GstPad *pad, GstBuffer *buf) num_to_process = GST_DPMAN_FRAMES_TO_PROCESS(ladspa->dpman); desc->connect_port(ladspa->handle,oclass->sinkpad_portnums[0],data_in); - for (i=0 ; iconnect_port(ladspa->handle,oclass->srcpad_portnums[i],data_out[i]); - } + desc->run(ladspa->handle, num_to_process); data_in += num_to_process; - for (i=0 ; i 0){ - for (i=0 ; isrcpads[i], buffers_out[i]); } + g_free(buffers_out); g_free(data_out); - return; } - - /* if we have reached here, there are no src pads */ - gst_buffer_unref(buf); } static GstBuffer * @@ -1014,43 +864,25 @@ gst_ladspa_get(GstPad *pad) { GstLADSPA *ladspa; GstLADSPAClass *oclass; - GstBuffer *buf; LADSPA_Data *data; LADSPA_Descriptor *desc; - guint num_to_process, num_processed; - g_return_val_if_fail(pad != NULL, NULL); - g_return_val_if_fail(GST_IS_PAD(pad), NULL); - ladspa = (GstLADSPA *)gst_pad_get_parent (pad); - g_return_val_if_fail(ladspa != NULL, NULL); - - /* this might happen if caps nego hasn't happened */ - g_return_val_if_fail(ladspa->handle != NULL, NULL); - oclass = (GstLADSPAClass*)(G_OBJECT_GET_CLASS(ladspa)); + desc = ladspa->descriptor; - /* force src pad to set caps */ - if (ladspa->newcaps) { + if (!ladspa->bufpool) { + /* capsnego hasn't happened... */ gst_ladspa_force_src_caps(ladspa, ladspa->srcpads[0]); } - /* get a bufferpool */ - if (ladspa->bufpool == NULL) { - ladspa->bufpool = gst_pad_get_bufferpool (ladspa->srcpads[0]); - if (ladspa->bufpool == NULL) { - ladspa->bufpool = gst_buffer_pool_get_default (sizeof (LADSPA_Data) * ladspa->buffersize, ladspa->numbuffers); - } - } - buf = gst_buffer_new_from_pool (ladspa->bufpool, 0, 0); GST_BUFFER_TIMESTAMP(buf) = ladspa->timestamp; data = (LADSPA_Data *) GST_BUFFER_DATA(buf); - desc = ladspa->descriptor; - GST_DPMAN_PREPROCESS(ladspa->dpman, GST_BUFFER_SIZE (buf) / sizeof (gfloat), ladspa->timestamp); + GST_DPMAN_PREPROCESS(ladspa->dpman, ladspa->buffer_frames, ladspa->timestamp); num_processed = 0; /* split up processing of the buffer into chunks so that dparams can @@ -1064,6 +896,7 @@ gst_ladspa_get(GstPad *pad) ladspa->timestamp += num_to_process * GST_SECOND / ladspa->samplerate; desc->connect_port(ladspa->handle,oclass->srcpad_portnums[0],data); + desc->run(ladspa->handle, num_to_process); data += num_to_process; @@ -1102,8 +935,8 @@ ladspa_describe_plugin(const char *pcFullFilename, gchar *type_name; /* construct the type */ - type_name = g_strdup_printf("ladspa_%s",desc->Label); - g_strcanon (type_name, G_CSET_A_2_Z G_CSET_a_2_z G_CSET_DIGITS "-_+", '-'); + type_name = g_strdup_printf("ladspa-%s",desc->Label); + g_strcanon (type_name, G_CSET_A_2_Z G_CSET_a_2_z G_CSET_DIGITS "-+", '-'); /* if it's already registered, drop it */ if (g_type_from_name(type_name)) { g_free(type_name); @@ -1116,7 +949,7 @@ ladspa_describe_plugin(const char *pcFullFilename, details = g_new0(GstElementDetails,1); details->longname = g_strdup(desc->Name); details->klass = "Filter/Audio/LADSPA"; - details->license = "LGPL"; + details->license = g_strdup (desc->Copyright); details->description = details->longname; details->version = g_strdup_printf("%ld",desc->UniqueID); details->author = g_strdup(desc->Maker); @@ -1133,32 +966,27 @@ ladspa_describe_plugin(const char *pcFullFilename, (gpointer)desc); - /* only add sink padtemplate if there are sinkpads */ for (j=0;jPortCount;j++) { - if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[j]) && - LADSPA_IS_PORT_INPUT(desc->PortDescriptors[j])) { - sinktempl = ladspa_sink_factory(); - gst_element_factory_add_pad_template (factory, sinktempl); - break; - } - } - - /* only add src padtemplate if there are srcpads */ - for (j=0;jPortCount;j++) { - if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[j]) && - LADSPA_IS_PORT_OUTPUT(desc->PortDescriptors[j])) { - srctempl = ladspa_src_factory(); - gst_element_factory_add_pad_template (factory, srctempl); - break; + if (LADSPA_IS_PORT_AUDIO(desc->PortDescriptors[j])) { + gchar *name = g_strdup((gchar *)desc->PortNames[j]); + g_strcanon (name, G_CSET_A_2_Z G_CSET_a_2_z G_CSET_DIGITS "-", '-'); + /* the factories take ownership of the name */ + if (LADSPA_IS_PORT_INPUT(desc->PortDescriptors[j])) + gst_element_factory_add_pad_template (factory, ladspa_sink_factory (name)); + else + gst_element_factory_add_pad_template (factory, ladspa_src_factory (name)); } } - } } static gboolean plugin_init (GModule *module, GstPlugin *plugin) { + GST_DEBUG_CATEGORY_INIT (ladspa_debug, "ladspa", + GST_DEBUG_FG_GREEN | GST_DEBUG_BG_BLACK | GST_DEBUG_BOLD, + "LADSPA"); + ladspa_descriptors = g_hash_table_new(NULL,NULL); parent_class = g_type_class_ref(GST_TYPE_ELEMENT); @@ -1181,4 +1009,3 @@ GstPluginDesc plugin_desc = { "ladspa", plugin_init }; - diff --git a/ext/ladspa/gstladspa.h b/ext/ladspa/gstladspa.h index 7b8c9b4..89a039e 100644 --- a/ext/ladspa/gstladspa.h +++ b/ext/ladspa/gstladspa.h @@ -60,12 +60,11 @@ struct _GstLADSPA { GstPad **sinkpads, **srcpads; - GstBufferPool *bufpool; - gboolean newcaps, activated; + gboolean activated; - gint samplerate, buffersize, numbuffers; + gint samplerate, buffer_frames; gint64 timestamp; gboolean inplace_broken; }; diff --git a/sys/oss/Makefile.am b/sys/oss/Makefile.am index be0a9ab..de37081 100644 --- a/sys/oss/Makefile.am +++ b/sys/oss/Makefile.am @@ -8,7 +8,6 @@ libgstossaudio_la_SOURCES = gstosssink.c \ gstosssrc.c \ gstossaudio.c \ gstossgst.c \ - gstossclock.c \ gstosscommon.c libgstossaudio_la_CFLAGS = $(GST_CFLAGS) @@ -18,4 +17,4 @@ libgstossaudio_la_LDFLAGS = $(GST_PLUGIN_LDFLAGS) libgstosshelper_la_SOURCES = gstosshelper.c libgstosshelper_la_LDFLAGS = $(GST_PLUGIN_LDFLAGS) -noinst_HEADERS = gstosssink.h gstosssrc.h gstossgst.h gstossclock.h gstosshelper.h gstosscommon.h +noinst_HEADERS = gstosssink.h gstosssrc.h gstossgst.h gstosshelper.h gstosscommon.h diff --git a/sys/oss/gstossaudio.c b/sys/oss/gstossaudio.c index dbb1387..e1d54c3 100644 --- a/sys/oss/gstossaudio.c +++ b/sys/oss/gstossaudio.c @@ -27,6 +27,9 @@ plugin_init (GModule *module, GstPlugin *plugin) { gboolean ret; + if (!gst_library_load ("gstaudio")) + return FALSE; + ret = gst_osssink_factory_init (plugin); g_return_val_if_fail (ret == TRUE, FALSE); diff --git a/sys/oss/gstossclock.c b/sys/oss/gstossclock.c deleted file mode 100644 index a6249f0..0000000 --- a/sys/oss/gstossclock.c +++ /dev/null @@ -1,191 +0,0 @@ -/* GStreamer - * Copyright (C) 1999,2000 Erik Walthinsen - * 2000 Wim Taymans - * - * gstclock.c: Clock subsystem for maintaining time sync - * - * This library is free software; you can redistribute it and/or - * modify it under the terms of the GNU Library General Public - * License as published by the Free Software Foundation; either - * version 2 of the License, or (at your option) any later version. - * - * This library is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Library General Public License for more details. - * - * You should have received a copy of the GNU Library General Public - * License along with this library; if not, write to the - * Free Software Foundation, Inc., 59 Temple Place - Suite 330, - * Boston, MA 02111-1307, USA. - */ - -#include "gstossclock.h" - -static void gst_oss_clock_class_init (GstOssClockClass *klass); -static void gst_oss_clock_init (GstOssClock *clock); - -static GstClockTime gst_oss_clock_get_internal_time (GstClock *clock); -static GstClockReturn gst_oss_clock_id_wait_async (GstClock *clock, - GstClockEntry *entry); -static void gst_oss_clock_id_unschedule (GstClock *clock, - GstClockEntry *entry); - -static GstSystemClockClass *parent_class = NULL; -/* static guint gst_oss_clock_signals[LAST_SIGNAL] = { 0 }; */ - -GType -gst_oss_clock_get_type (void) -{ - static GType clock_type = 0; - - if (!clock_type) { - static const GTypeInfo clock_info = { - sizeof (GstOssClockClass), - NULL, - NULL, - (GClassInitFunc) gst_oss_clock_class_init, - NULL, - NULL, - sizeof (GstOssClock), - 4, - (GInstanceInitFunc) gst_oss_clock_init, - NULL - }; - clock_type = g_type_register_static (GST_TYPE_SYSTEM_CLOCK, "GstOssClock", - &clock_info, 0); - } - return clock_type; -} - - -static void -gst_oss_clock_class_init (GstOssClockClass *klass) -{ - GObjectClass *gobject_class; - GstObjectClass *gstobject_class; - GstClockClass *gstclock_class; - - gobject_class = (GObjectClass*) klass; - gstobject_class = (GstObjectClass*) klass; - gstclock_class = (GstClockClass*) klass; - - parent_class = g_type_class_ref (GST_TYPE_SYSTEM_CLOCK); - - gstclock_class->get_internal_time = gst_oss_clock_get_internal_time; - gstclock_class->wait_async = gst_oss_clock_id_wait_async; - gstclock_class->unschedule = gst_oss_clock_id_unschedule; -} - -static void -gst_oss_clock_init (GstOssClock *clock) -{ - gst_object_set_name (GST_OBJECT (clock), "GstOssClock"); - - clock->prev1 = 0; - clock->prev2 = 0; -} - -GstOssClock* -gst_oss_clock_new (gchar *name, GstOssClockGetTimeFunc func, gpointer user_data) -{ - GstOssClock *oss_clock = GST_OSS_CLOCK (g_object_new (GST_TYPE_OSS_CLOCK, NULL)); - - oss_clock->func = func; - oss_clock->user_data = user_data; - oss_clock->adjust = 0; - - return oss_clock; -} - -void -gst_oss_clock_set_active (GstClock *clock, gboolean active) -{ - GstOssClock *oss_clock = GST_OSS_CLOCK (clock); - GTimeVal timeval; - GstClockTime time; - GstClockTime osstime; - - g_get_current_time (&timeval); - time = GST_TIMEVAL_TO_TIME (timeval); - osstime = oss_clock->func (clock, oss_clock->user_data); - - if (active) { - oss_clock->adjust = time - osstime; - } - else { - oss_clock->adjust = osstime - time; - } - - oss_clock->active = active; -} - -static GstClockTime -gst_oss_clock_get_internal_time (GstClock *clock) -{ - GstOssClock *oss_clock = GST_OSS_CLOCK (clock); - - if (oss_clock->active) { - GstClockTime osstime; - - osstime = oss_clock->func (clock, oss_clock->user_data) + oss_clock->adjust; - - return osstime; - } - else { - GstClockTime time; - GTimeVal timeval; - - g_get_current_time (&timeval); - time = GST_TIMEVAL_TO_TIME (timeval); - - return time; - } -} - -void -gst_oss_clock_update_time (GstClock *clock, GstClockTime time) -{ - GstOssClock *oss_clock = (GstOssClock*)clock; - - while (oss_clock->async_entries) { - GstClockEntry *entry = (GstClockEntry*)oss_clock->async_entries->data; - - if (entry->time > time) - break; - - entry->func (clock, time, entry, entry->user_data); - - oss_clock->async_entries = g_slist_delete_link (oss_clock->async_entries, - oss_clock->async_entries); - /* do I need to free the entry? */ - } -} - -static gint -compare_clock_entries (GstClockEntry *entry1, GstClockEntry *entry2) -{ - return entry1->time - entry2->time; -} - -static GstClockReturn -gst_oss_clock_id_wait_async (GstClock *clock, GstClockEntry *entry) -{ - GstOssClock *oss_clock = (GstOssClock*)clock; - - oss_clock->async_entries = g_slist_insert_sorted (oss_clock->async_entries, - entry, - (GCompareFunc)compare_clock_entries); - - /* is this the proper return val? */ - return GST_CLOCK_EARLY; -} - -static void -gst_oss_clock_id_unschedule (GstClock *clock, GstClockEntry *entry) -{ - GstOssClock *oss_clock = (GstOssClock*)clock; - - oss_clock->async_entries = g_slist_remove (oss_clock->async_entries, - entry); -} diff --git a/sys/oss/gstossclock.h b/sys/oss/gstossclock.h deleted file mode 100644 index 279adfa..0000000 --- a/sys/oss/gstossclock.h +++ /dev/null @@ -1,82 +0,0 @@ -/* GStreamer - * Copyright (C) 1999,2000 Erik Walthinsen - * 2000 Wim Taymans - * - * gstclock.h: Header for clock subsystem - * - * This library is free software; you can redistribute it and/or - * modify it under the terms of the GNU Library General Public - * License as published by the Free Software Foundation; either - * version 2 of the License, or (at your option) any later version. - * - * This library is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Library General Public License for more details. - * - * You should have received a copy of the GNU Library General Public - * License along with this library; if not, write to the - * Free Software Foundation, Inc., 59 Temple Place - Suite 330, - * Boston, MA 02111-1307, USA. - */ - - -#ifndef __GST_OSS_CLOCK_H__ -#define __GST_OSS_CLOCK_H__ - -#include - -#ifdef __cplusplus -extern "C" { -#endif /* __cplusplus */ - - -#define GST_TYPE_OSS_CLOCK \ - (gst_oss_clock_get_type()) -#define GST_OSS_CLOCK(obj) \ - (G_TYPE_CHECK_INSTANCE_CAST((obj),GST_TYPE_OSS_CLOCK,GstOssClock)) -#define GST_OSS_CLOCK_CLASS(klass) \ - (G_TYPE_CHECK_CLASS_CAST((klass),GST_TYPE_OSS_CLOCK,GstOssClockClass)) -#define GST_IS_OSS_CLOCK(obj) \ - (G_TYPE_CHECK_INSTANCE_TYPE((obj),GST_TYPE_OSS_CLOCK)) -#define GST_IS_OSS_CLOCK_CLASS(obj) \ - (G_TYPE_CHECK_CLASS_TYPE((klass),GST_TYPE_OSS_CLOCK)) - -typedef struct _GstOssClock GstOssClock; -typedef struct _GstOssClockClass GstOssClockClass; - -typedef GstClockTime (*GstOssClockGetTimeFunc) (GstClock *clock, gpointer user_data); - - -struct _GstOssClock { - GstSystemClock clock; - - GstOssClockGetTimeFunc func; - gpointer user_data; - - GstClockTime prev1, prev2; - GstClockTimeDiff adjust; - - GSList *async_entries; - - gboolean active; -}; - -struct _GstOssClockClass { - GstSystemClockClass parent_class; -}; - -GType gst_oss_clock_get_type (void); -GstOssClock* gst_oss_clock_new (gchar *name, GstOssClockGetTimeFunc func, - gpointer user_data); -void gst_oss_clock_set_active (GstClock *clock, gboolean active); -void gst_oss_clock_set_time (GstClock *clock, GstClockTime time); - -void gst_oss_clock_update_time (GstClock *clock, GstClockTime time); - -#ifdef __cplusplus -} -#endif /* __cplusplus */ - - -#endif /* __GST_OSS_CLOCK_H__ */ diff --git a/sys/oss/gstosssink.c b/sys/oss/gstosssink.c index 904d191..a881e08 100644 --- a/sys/oss/gstosssink.c +++ b/sys/oss/gstosssink.c @@ -243,7 +243,7 @@ gst_osssink_init (GstOssSink *osssink) osssink->mute = FALSE; osssink->sync = TRUE; osssink->sinkpool = NULL; - osssink->provided_clock = GST_CLOCK (gst_oss_clock_new ("ossclock", gst_osssink_get_time, osssink)); + osssink->provided_clock = gst_audio_clock_new ("ossclock", gst_osssink_get_time, osssink); gst_object_set_parent (GST_OBJECT (osssink->provided_clock), GST_OBJECT (osssink)); osssink->handled = 0; @@ -348,7 +348,7 @@ gst_osssink_chain (GstPad *pad, GstBuffer *buf) switch (GST_EVENT_TYPE (event)) { case GST_EVENT_EOS: ioctl (osssink->common.fd, SNDCTL_DSP_SYNC); - gst_oss_clock_set_active (osssink->provided_clock, FALSE); + gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock), FALSE); gst_pad_event_default (pad, event); return; case GST_EVENT_DISCONTINUOUS: @@ -358,7 +358,7 @@ gst_osssink_chain (GstPad *pad, GstBuffer *buf) ioctl (osssink->common.fd, SNDCTL_DSP_RESET); if (gst_event_discont_get_value (event, GST_FORMAT_TIME, &value)) { if (!gst_clock_handle_discont (osssink->clock, value)) - gst_oss_clock_set_active (osssink->provided_clock, FALSE); + gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock), FALSE); osssink->handled = 0; } osssink->resync = TRUE; @@ -403,7 +403,7 @@ gst_osssink_chain (GstPad *pad, GstBuffer *buf) if (jitter >= 0) { gst_clock_handle_discont (osssink->clock, buftime - queued + jitter); to_write = size; - gst_oss_clock_set_active (osssink->provided_clock, TRUE); + gst_audio_clock_set_active ((GstAudioClock*)osssink->provided_clock, TRUE); osssink->resync = FALSE; } } @@ -439,8 +439,7 @@ gst_osssink_chain (GstPad *pad, GstBuffer *buf) } } - if (osssink->clock) - gst_oss_clock_update_time (osssink->clock, buftime); + gst_audio_clock_update_time ((GstAudioClock*)osssink->provided_clock, buftime); gst_buffer_unref (buf); } @@ -627,7 +626,7 @@ gst_osssink_change_state (GstElement *element) { if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN)) ioctl (osssink->common.fd, SNDCTL_DSP_RESET, 0); - gst_oss_clock_set_active (osssink->provided_clock, FALSE); + gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock), FALSE); osssink->resync = TRUE; break; } diff --git a/sys/oss/gstosssink.h b/sys/oss/gstosssink.h index 689d600..7427769 100644 --- a/sys/oss/gstosssink.h +++ b/sys/oss/gstosssink.h @@ -28,7 +28,7 @@ #include #include "gstosscommon.h" -#include "gstossclock.h" +#include G_BEGIN_DECLS diff --git a/sys/oss/gstosssrc.c b/sys/oss/gstosssrc.c index 9eb24aa..0e41d24 100644 --- a/sys/oss/gstosssrc.c +++ b/sys/oss/gstosssrc.c @@ -34,7 +34,7 @@ #include #include -#include +#include /* elementfactory information */ static GstElementDetails gst_osssrc_details = { @@ -197,9 +197,7 @@ gst_osssrc_init (GstOssSrc *osssrc) osssrc->buffersize = 4096; osssrc->curoffset = 0; - osssrc->provided_clock = GST_CLOCK (gst_oss_clock_new ("ossclock", - gst_osssrc_get_time, - osssrc)); + osssrc->provided_clock = gst_audio_clock_new ("ossclock", gst_osssrc_get_time, osssrc); gst_object_set_parent (GST_OBJECT (osssrc->provided_clock), GST_OBJECT (osssrc)); osssrc->clock = NULL; @@ -443,10 +441,10 @@ gst_osssrc_change_state (GstElement *element) osssrc->curoffset = 0; break; case GST_STATE_PAUSED_TO_PLAYING: - gst_oss_clock_set_active (osssrc->provided_clock, TRUE); + gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssrc->provided_clock), TRUE); break; case GST_STATE_PLAYING_TO_PAUSED: - gst_oss_clock_set_active (osssrc->provided_clock, FALSE); + gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssrc->provided_clock), FALSE); break; case GST_STATE_PAUSED_TO_READY: if (GST_FLAG_IS_SET (element, GST_OSSSRC_OPEN)) -- 2.7.4