From 6725c09391a8c14ce84f75e1951368a8cec8a633 Mon Sep 17 00:00:00 2001 From: Jamie Bullock Date: Wed, 8 Dec 2010 10:35:48 +0000 Subject: - refactored debug statement handling svn path=/trunk/externals/postlude/; revision=14577 --- pluginhost~/makefile | 2 +- pluginhost~/src/pluginhost~.c | 448 ++++++++++++++++++------------------------ pluginhost~/src/pluginhost~.h | 3 + 3 files changed, 192 insertions(+), 261 deletions(-) diff --git a/pluginhost~/makefile b/pluginhost~/makefile index b95b51d..f85b497 100755 --- a/pluginhost~/makefile +++ b/pluginhost~/makefile @@ -5,7 +5,7 @@ LIBDIR=/usr/local/lib PDDIR=$(LIBDIR)/pd INSTALLPATH=$(PDDIR)/extra/ ARCHITECTURE=i386 -DEBUG=0 +DEBUG=1 current: pd_linux diff --git a/pluginhost~/src/pluginhost~.c b/pluginhost~/src/pluginhost~.c index b18ef23..347309d 100644 --- a/pluginhost~/src/pluginhost~.c +++ b/pluginhost~/src/pluginhost~.c @@ -1,15 +1,15 @@ -/* dssi~ - A DSSI host for PD - * - * Copyright (C) 2006 Jamie Bullock and others +/* dssi~ - A DSSI host for PD + * + * Copyright (C) 2006 Jamie Bullock and others * * This file incorporates code from the following sources: - * + * * jack-dssi-host (BSD-style license): Copyright 2004 Chris Cannam, Steve Harris and Sean Bolton. - * + * * Hexter (GPL license): Copyright (C) 2004 Sean Bolton and others. - * + * * plugin~ (GPL license): Copyright (C) 2000 Jarno Seppänen, remIXed 2005 - * + * * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License @@ -37,11 +37,6 @@ static t_class *ph_tilde_class; static char base64[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/"; -/*static void init_MidiEventBuf(snd_seq_event_t *MidiEventBuf){ - int i; - for(i = 0; i < EVENT_BUFSIZE; i++){ - MidiEventBuf[i].*/ - /* * encode_7in6 ** Taken from gui_data.c by Sean Bolton ** @@ -104,8 +99,8 @@ dx7_bulk_dump_checksum(uint8_t *data, int length) return sum & 0x7F; } -static DSSI_Descriptor * ladspa_to_dssi( - LADSPA_Descriptor *ladspaDesc){ +static DSSI_Descriptor *ladspa_to_dssi(LADSPA_Descriptor *ladspaDesc) +{ DSSI_Descriptor *dssiDesc; dssiDesc = (DSSI_Descriptor *)calloc(1, sizeof(DSSI_Descriptor)); ((DSSI_Descriptor *)dssiDesc)->DSSI_API_Version = 1; @@ -113,18 +108,7 @@ static DSSI_Descriptor * ladspa_to_dssi( (LADSPA_Descriptor *)ladspaDesc; return (DSSI_Descriptor *)dssiDesc; } -/* - static void ph_tilde_load_plugin(const char *dll_path, void **dll_handle){ - *dll_handle = dlopen(dll_path, RTLD_NOW | RTLD_LOCAL); - if (*dll_handle){ - post("%s loaded successfully", dll_path); - } - else - post("Failed: %s", dlerror()); - - } - */ static void ph_tilde_port_info(t_ph_tilde *x){ t_int i; @@ -139,9 +123,8 @@ static void ph_tilde_port_info(t_ph_tilde *x){ LADSPA_PortDescriptor pod = x->descriptor->LADSPA_Plugin->PortDescriptors[i]; -#if DEBUG - post("Port %d: %s", i, x->descriptor->LADSPA_Plugin->PortNames[i]); -#endif + ph_debug_post("Port %d: %s", i, x->descriptor->LADSPA_Plugin->PortNames[i]); + if (LADSPA_IS_PORT_AUDIO(pod)) { x->port_info[i].data_type.a_w.w_symbol = gensym("audio"); @@ -193,26 +176,23 @@ static void ph_tilde_port_info(t_ph_tilde *x){ gensym ((char *) x->descriptor->LADSPA_Plugin->PortNames[i]); } -#if DEBUG - post("%d inputs, %d outputs, %d control inputs, %d control outs", x->plugin_ins, x->plugin_outs, x->plugin_controlIns, x->plugin_controlOuts); -#endif + ph_debug_post("%d inputs, %d outputs, %d control inputs, %d control outs", x->plugin_ins, x->plugin_outs, x->plugin_controlIns, x->plugin_controlOuts); + } static void ph_tilde_assign_ports(t_ph_tilde *x){ int i; -#if DEBUG - post("%d instances", x->n_instances); -#endif + ph_debug_post("%d instances", x->n_instances); + x->plugin_ins *= x->n_instances; x->plugin_outs *= x->n_instances; x->plugin_controlIns *= x->n_instances; x->plugin_controlOuts *= x->n_instances; -#if DEBUG - post("%d plugin outs", x->plugin_outs); -#endif + ph_debug_post("%d plugin outs", x->plugin_outs); + x->plugin_InputBuffers = (float **)malloc(x->plugin_ins * sizeof(float *)); @@ -248,9 +228,8 @@ static void ph_tilde_assign_ports(t_ph_tilde *x){ x->plugin_ControlInPortNumbers = (unsigned long *)malloc(sizeof(unsigned long) * x->plugin_controlIns); -#if DEBUG - post("Buffers assigned!"); -#endif + ph_debug_post("Buffers assigned!"); + } @@ -274,9 +253,8 @@ static void ph_tilde_init_instance(t_ph_tilde *x, t_int instance){ x->instances[instance].ui_hidden = 1; x->instances[instance].ui_show = 0; x->instances[instance].gui_pid = 0; -#if DEBUG - post("Instance %d initialized!", instance); -#endif + + ph_debug_post("Instance %d initialized!", instance); } @@ -285,10 +263,9 @@ static void ph_tilde_connect_ports(t_ph_tilde *x, t_int instance){ t_int i; for(i = 0; i < (t_int)x->descriptor->LADSPA_Plugin->PortCount; i++){ -#if DEBUG - post("PortCount: %d of %d", i, + ph_debug_post("PortCount: %d of %d", i, x->descriptor->LADSPA_Plugin->PortCount); -#endif + LADSPA_PortDescriptor pod = x->descriptor->LADSPA_Plugin->PortDescriptors[i]; @@ -304,10 +281,9 @@ static void ph_tilde_connect_ports(t_ph_tilde *x, t_int instance){ x->descriptor->LADSPA_Plugin->connect_port (x->instanceHandles[instance], i, x->plugin_OutputBuffers[x->ports_out++]); -#if DEBUG - post("Audio Input port %d connected", x->ports_in); + ph_debug_post("Audio Input port %d connected", x->ports_in); post("Audio Output port %d connected", x->ports_out); -#endif + } } else if (LADSPA_IS_PORT_CONTROL(pod)) { @@ -316,10 +292,9 @@ static void ph_tilde_connect_ports(t_ph_tilde *x, t_int instance){ x->instances[instance].plugin_PortControlInNumbers[i] = x->ports_controlIn; x->plugin_ControlDataInput[x->ports_controlIn] = (t_float) get_port_default(x, i); -#if DEBUG - post("default for port %d, controlIn, %d is %.2f",i, + ph_debug_post("default for port %d, controlIn, %d is %.2f",i, x->ports_controlIn, x->plugin_ControlDataInput[x->ports_controlIn]); -#endif + x->descriptor->LADSPA_Plugin->connect_port (x->instanceHandles[instance], i, @@ -330,30 +305,26 @@ static void ph_tilde_connect_ports(t_ph_tilde *x, t_int instance){ (x->instanceHandles[instance], i, &x->plugin_ControlDataOutput[x->ports_controlOut++]); } -#if DEBUG - post("Control Input port %d connected", x->ports_controlIn); + ph_debug_post("Control Input port %d connected", x->ports_controlIn); post("Control Output port %d connected", x->ports_controlOut); -#endif + } } -#if DEBUG - post("ports connected!"); -#endif + ph_debug_post("ports connected!"); + } static void ph_tilde_activate_plugin(t_ph_tilde *x, t_int instance){ if(x->descriptor->LADSPA_Plugin->activate){ -#if DEBUG - post("trying to activate instance: %d", instance); -#endif + ph_debug_post("trying to activate instance: %d", instance); + x->descriptor->LADSPA_Plugin->activate(x->instanceHandles[instance]); } -#if DEBUG - post("plugin activated!"); -#endif + ph_debug_post("plugin activated!"); + } static void ph_tilde_deactivate_plugin(t_ph_tilde *x, t_float instance_f){ @@ -361,9 +332,8 @@ static void ph_tilde_deactivate_plugin(t_ph_tilde *x, t_float instance_f){ t_int instance = (t_int)instance_f; if(x->descriptor->LADSPA_Plugin->deactivate) x->descriptor->LADSPA_Plugin->deactivate(x->instanceHandles[instance]); -#if DEBUG - post("plugin deactivated!"); -#endif + ph_debug_post("plugin deactivated!"); + } static void osc_error(int num, const char *msg, const char *where) @@ -373,9 +343,8 @@ static void osc_error(int num, const char *msg, const char *where) static void query_programs(t_ph_tilde *x, t_int instance) { int i; -#if DEBUG - post("querying programs"); -#endif + ph_debug_post("querying programs"); + /* free old lot */ if (x->instances[instance].pluginPrograms) { for (i = 0; i < x->instances[instance].plugin_ProgramCount; i++) @@ -412,12 +381,11 @@ static void query_programs(t_ph_tilde *x, t_int instance) { descriptor->Program; x->instances[instance].pluginPrograms[i].Name = strdup(descriptor->Name); -#if DEBUG - post("program %d is MIDI bank %lu program %lu, named '%s'",i, + ph_debug_post("program %d is MIDI bank %lu program %lu, named '%s'",i, x->instances[instance].pluginPrograms[i].Bank, x->instances[instance].pluginPrograms[i].Program, x->instances[instance].pluginPrograms[i].Name); -#endif + } } /* No - it should be 0 anyway - ph_init */ @@ -531,9 +499,8 @@ static void ph_tilde_set_control_input_by_index (t_ph_tilde *x, return; } -#if DEBUG - post("ctrl input number = %d", ctrl_input_index); -#endif + ph_debug_post("ctrl input number = %d", ctrl_input_index); + port = x->plugin_ControlInPortNumbers[ctrl_input_index]; @@ -546,10 +513,9 @@ static void ph_tilde_set_control_input_by_index (t_ph_tilde *x, else portno = x->instances[instance].plugin_PortControlInNumbers[ctrl_input_index]; -#if DEBUG - post("Global ctrl input number = %d", ctrl_input_index); + ph_debug_post("Global ctrl input number = %d", ctrl_input_index); post("Global ctrl input value = %.2f", value); -#endif + @@ -560,15 +526,13 @@ static void ph_tilde_set_control_input_by_index (t_ph_tilde *x, /* Update the UI if there is one */ if(x->is_DSSI){ if(x->instances[instance].uiTarget == NULL){ -#if DEBUG - post("pluginhost~: unable to send to NULL target"); -#endif + ph_debug_post("pluginhost~: unable to send to NULL target"); + return; } if(x->instances[instance].ui_osc_control_path == NULL){ -#if DEBUG - post("pluginhost~: unable to send to NULL control path"); -#endif + ph_debug_post("pluginhost~: unable to send to NULL control path"); + return; } lo_send(x->instances[instance].uiTarget, @@ -643,9 +607,8 @@ static void ph_tilde_control (t_ph_tilde *x, return; } -#if DEBUG - post("Received LADSPA control data for instance %d", instance); -#endif + ph_debug_post("Received LADSPA control data for instance %d", instance); + if (ctrl_name->s_name == NULL || strlen (ctrl_name->s_name) == 0) { post("pluginhost~: control messages must have a name and a value"); @@ -742,9 +705,8 @@ static void ph_tilde_ladspa_describe(const char * pcFullFilename, outlet_anything (x->control_outlet, gensym ("library"), 1, at); if(is_DSSI){ -#if DEBUG - post("DSSI plugin found by listinfo"); -#endif + ph_debug_post("DSSI plugin found by listinfo"); + for (lIndex = 0; (psDescriptor = (DSSI_Descriptor *) fDescriptorFunction(lIndex)) != NULL; lIndex++) @@ -812,9 +774,8 @@ static void ph_tilde_get_current_program(t_ph_tilde *x, int instance){ static void ph_tilde_program_change(t_ph_tilde *x, int instance){ /* jack-dssi-host queues program changes by using pending program change variables. In the audio callback, if a program change is received via MIDI it over writes the pending value (if any) set by the GUI. If unset, or processed the value will default back to -1. The following call to select_program is then made. I don't think it eventually needs to be done this way - i.e. do we need 'pending'? */ -#if DEBUG - post("executing program change"); -#endif + ph_debug_post("executing program change"); + if (x->instances[instance].pendingProgramChange >= 0){ if (x->instances[instance].pendingBankLSB >= 0) { if (x->instances[instance].pendingBankMSB >= 0) { @@ -836,9 +797,8 @@ static void ph_tilde_program_change(t_ph_tilde *x, int instance){ x->instances[instance].currentBank, x->instances[instance].currentProgram); } if (x->instances[instance].uiNeedsProgramUpdate){ -#if DEBUG - post("Updating GUI program"); -#endif + ph_debug_post("Updating GUI program"); + /* FIX - this is a hack to make text ui work*/ if(x->instances[instance].uiTarget) lo_send(x->instances[instance].uiTarget, @@ -862,12 +822,11 @@ static int osc_program_handler(t_ph_tilde *x, lo_arg **argv, int instance) int i; int found = 0; -#if DEBUG - post("osc_program_hander active!"); + ph_debug_post("osc_program_hander active!"); post("%d programs", x->instances[instance].plugin_ProgramCount); -#endif + for (i = 0; i < x->instances[instance].plugin_ProgramCount; ++i) { if (x->instances[instance].pluginPrograms[i].Bank == bank && x->instances[instance].pluginPrograms[i].Program == program) { @@ -886,9 +845,8 @@ static int osc_program_handler(t_ph_tilde *x, lo_arg **argv, int instance) x->instances[instance].pendingBankMSB = bank / 128; x->instances[instance].pendingBankLSB = bank % 128; x->instances[instance].pendingProgramChange = program; -#if DEBUG - post("bank = %d, program = %d, BankMSB = %d BankLSB = %d", bank, program, x->instances[instance].pendingBankMSB, x->instances[instance].pendingBankLSB); -#endif + ph_debug_post("bank = %d, program = %d, BankMSB = %d BankLSB = %d", bank, program, x->instances[instance].pendingBankMSB, x->instances[instance].pendingBankLSB); + ph_tilde_program_change(x, instance); return 0; @@ -900,9 +858,8 @@ static int osc_control_handler(t_ph_tilde *x, lo_arg **argv, int instance) LADSPA_Data value = argv[1]->f; x->plugin_ControlDataInput[x->instances[instance].plugin_PortControlInNumbers[port]] = value; -#if DEBUG - post("OSC: port %d = %f", port, value); -#endif + ph_debug_post("OSC: port %d = %f", port, value); + return 0; } @@ -911,15 +868,13 @@ static int osc_midi_handler(t_ph_tilde *x, lo_arg **argv, t_int instance) { int ev_type = 0, chan = 0; -#if DEBUG - post("OSC: got midi request for" + ph_debug_post("OSC: got midi request for" "(%02x %02x %02x %02x)", argv[0]->m[0], argv[0]->m[1], argv[0]->m[2], argv[0]->m[3]); -#endif + chan = instance; -#if DEBUG - post("channel: %d", chan); -#endif + ph_debug_post("channel: %d", chan); + if(argv[0]->m[1] <= 239){ if(argv[0]->m[1] >= 224) @@ -949,9 +904,8 @@ static int osc_configure_handler(t_ph_tilde *x, lo_arg **argv, int instance) const char *value = (const char *)&argv[1]->s; char *message; -#if DEBUG - post("osc_configure_handler active!"); -#endif + ph_debug_post("osc_configure_handler active!"); + if (x->descriptor->configure) { @@ -977,9 +931,8 @@ static int osc_configure_handler(t_ph_tilde *x, lo_arg **argv, int instance) static int osc_exiting_handler(t_ph_tilde *x, lo_arg **argv, int instance){ -#if DEBUG - post("exiting handler called: Freeing ui_osc"); -#endif + ph_debug_post("exiting handler called: Freeing ui_osc"); + if(x->instances[instance].uiTarget){ lo_address_free(x->instances[instance].uiTarget); x->instances[instance].uiTarget = NULL; @@ -1013,9 +966,8 @@ static int osc_update_handler(t_ph_tilde *x, lo_arg **argv, int instance) p = x->configure_buffer_head; -#if DEBUG - post("OSC: got update request from <%s>, instance %d", url, instance); -#endif + ph_debug_post("OSC: got update request from <%s>, instance %d", url, instance); + if (x->instances[instance].uiTarget) lo_address_free(x->instances[instance].uiTarget); @@ -1072,9 +1024,8 @@ static int osc_update_handler(t_ph_tilde *x, lo_arg **argv, int instance) /* Send current bank/program (-FIX- another race...) */ if (x->instances[instance].pendingProgramChange >= 0) ph_tilde_program_change(x, instance); -#if DEBUG - post("pendingProgramChange = %d", x->instances[instance].pendingProgramChange); -#endif + ph_debug_post("pendingProgramChange = %d", x->instances[instance].pendingProgramChange); + if (x->instances[instance].pendingProgramChange < 0) { unsigned long bank = x->instances[instance].currentBank; unsigned long program = x->instances[instance].currentProgram; @@ -1090,9 +1041,8 @@ static int osc_update_handler(t_ph_tilde *x, lo_arg **argv, int instance) for (i = 0; i < x->plugin_controlIns; i++) { lo_send(x->instances[instance].uiTarget, x->instances[instance].ui_osc_control_path, "if", x->plugin_ControlInPortNumbers[i], x->plugin_ControlDataInput[i]); -#if DEBUG - post("Port: %d, Default value: %.2f", x->plugin_ControlInPortNumbers[i], x->plugin_ControlDataInput[i]); -#endif + ph_debug_post("Port: %d, Default value: %.2f", x->plugin_ControlInPortNumbers[i], x->plugin_ControlDataInput[i]); + } /* Send 'show' */ @@ -1111,9 +1061,8 @@ static void ph_tilde_osc_setup(t_ph_tilde *x, int instance){ x->osc_thread = lo_server_thread_new(NULL, osc_error); char *osc_url_tmp; osc_url_tmp = lo_server_thread_get_url(x->osc_thread); -#if DEBUG - post("string length of osc_url_tmp:%d", strlen(osc_url_tmp)); -#endif + ph_debug_post("string length of osc_url_tmp:%d", strlen(osc_url_tmp)); + x->osc_url_base = (char *)malloc(sizeof(char) * (strlen(osc_url_tmp) + strlen("dssi") + 1)); sprintf(x->osc_url_base, "%s%s", osc_url_tmp, "dssi"); @@ -1126,17 +1075,15 @@ static void ph_tilde_osc_setup(t_ph_tilde *x, int instance){ (strlen(x->plugin_basename) + strlen(x->descriptor->LADSPA_Plugin->Label) + strlen("chan00") + 3)); sprintf(x->instances[instance].osc_url_path, "%s/%s/chan%02d", x->plugin_basename, x->descriptor->LADSPA_Plugin->Label, instance); -#if DEBUG - post("OSC Path is: %s", x->instances[instance].osc_url_path); + ph_debug_post("OSC Path is: %s", x->instances[instance].osc_url_path); post("OSC thread started: %s", x->osc_url_base); -#endif + } static void ph_tilde_init_programs(t_ph_tilde *x, int instance){ -#if DEBUG - post("Setting up program data"); -#endif + ph_debug_post("Setting up program data"); + query_programs(x, instance); if (x->descriptor->select_program && x->instances[instance].plugin_ProgramCount > 0) { @@ -1169,16 +1116,14 @@ static void ph_tilde_load_gui(t_ph_tilde *x, int instance){ /* don't use strndup - GNU only */ /* gui_base = strndup(x->plugin_full_path, baselen);*/ -#if DEBUG - post("gui_base: %s", gui_base); -#endif + ph_debug_post("gui_base: %s", gui_base); + gui_str = (char *)malloc(sizeof(char) * (strlen("channel 00") + 1)); sprintf (gui_str,"channel %02d", instance); -#if DEBUG - post("GUI name string, %s", gui_str); -#endif + ph_debug_post("GUI name string, %s", gui_str); + if(!(dp = opendir(gui_base))){ post("pluginhost~: unable to find GUI in %s, continuing without...", gui_base); @@ -1194,9 +1139,8 @@ static void ph_tilde_load_gui(t_ph_tilde *x, int instance){ break; } } -#if DEBUG - post("GUI filename: %s", dir_entry->d_name); -#endif + ph_debug_post("GUI filename: %s", dir_entry->d_name); + } gui_path = (char *)malloc(sizeof(char) * (strlen(gui_base) + strlen("/") + @@ -1205,9 +1149,8 @@ static void ph_tilde_load_gui(t_ph_tilde *x, int instance){ sprintf(gui_path, "%s/%s", gui_base, dir_entry->d_name); free(gui_base); -#if DEBUG - post("gui_path: %s", gui_path); -#endif + ph_debug_post("gui_path: %s", gui_path); + osc_url = (char *)malloc (sizeof(char) * (strlen(x->osc_url_base) + @@ -1217,9 +1160,8 @@ static void ph_tilde_load_gui(t_ph_tilde *x, int instance){ sprintf(osc_url, "%s/%s", x->osc_url_base, x->instances[instance].osc_url_path); post("pluginhost~: instance %d URL: %s",instance, osc_url); -#if DEBUG - post("Trying to open GUI!"); -#endif + ph_debug_post("Trying to open GUI!"); + x->instances[instance].gui_pid = fork(); if (x->instances[instance].gui_pid == 0){ @@ -1230,18 +1172,16 @@ static void ph_tilde_load_gui(t_ph_tilde *x, int instance){ exit(1); /* terminates the process */ } -#if DEBUG - post("errorcode = %d", err); -#endif + ph_debug_post("errorcode = %d", err); + free(gui_path); free(osc_url); free(gui_str); if(dp){ -#if DEBUG - post("directory handle closed = %d", closedir(dp)); -#endif + ph_debug_post("directory handle closed = %d", closedir(dp)); + } } @@ -1303,10 +1243,9 @@ static void MIDIbuf(int type, int chan, int param, int val, t_ph_tilde *x){ x->instances[mapped].pendingBankLSB = (param - 1) % 128; x->instances[mapped].pendingProgramChange = val; x->instances[mapped].uiNeedsProgramUpdate = 1; -#if DEBUG - post("pgm chabge received in buffer: MSB: %d, LSB %d, prog: %d", + ph_debug_post("pgm chabge received in buffer: MSB: %d, LSB %d, prog: %d", x->instances[mapped].pendingBankMSB, x->instances[mapped].pendingBankLSB, val); -#endif + ph_tilde_program_change(x, mapped); break; } @@ -1318,10 +1257,9 @@ static void MIDIbuf(int type, int chan, int param, int val, t_ph_tilde *x){ x->midiEventBuf[x->bufWriteIndex].data.note.velocity = val; } -#if DEBUG - post("MIDI received in buffer: chan %d, param %d, val %d, mapped to %d", + ph_debug_post("MIDI received in buffer: chan %d, param %d, val %d, mapped to %d", chan, param, val, mapped); -#endif + x->bufWriteIndex = (x->bufWriteIndex + 1) % EVENT_BUFSIZE; pthread_mutex_unlock(&x->midiEventBufferMutex); /**release mutex*/ } @@ -1350,9 +1288,8 @@ static void ph_tilde_list(t_ph_tilde *x, t_symbol *s, int argc, t_atom *argv) { case ASCII_a: ev_type = SND_SEQ_EVENT_KEYPRESS; break; } -#if DEBUG - post("initial midi NOTE:, arg1 = %d, arg2 = %d, arg3 = %d, arg4 = %d",ev_type,chan,param,val); -#endif + ph_debug_post("initial midi NOTE:, arg1 = %d, arg2 = %d, arg3 = %d, arg4 = %d",ev_type,chan,param,val); + if(ev_type != 0){ if(chan >= 0) MIDIbuf(ev_type, chan, param, val, x); @@ -1405,7 +1342,6 @@ static void ph_show(t_ph_tilde *x, t_int instance, t_int toggle){ static t_int ph_tilde_configure_buffer(t_ph_tilde *x, char *key, char *value, t_int instance){ - /*#ifdef BLAH*/ t_ph_configure_pair *current, *p; t_int add_node; add_node = 0; @@ -1432,14 +1368,13 @@ static t_int ph_tilde_configure_buffer(t_ph_tilde *x, char *key, p = x->configure_buffer_head; /*FIX: eventually give ability to query this buffer (to outlet?) */ -#if DEBUG while(p){ - post("key: %s", p->key); - post("val: %s", p->value); - post("instance: %d", p->instance); + ph_debug_post("key: %s", p->key); + ph_debug_post("val: %s", p->value); + ph_debug_post("instance: %d", p->instance); p = p->next; } -#endif + return 0; } @@ -1499,9 +1434,8 @@ static void ph_tilde_search_plugin_callback ( /* Stop searching when a first matching plugin is found */ if (*out_lib_name == NULL) { -#if DEBUG - post("pluginhost~: searching plugin \"%s\"...", full_filename); -#endif + ph_debug_post("pluginhost~: searching plugin \"%s\"...", full_filename); + for(plug_index = 0;(is_DSSI ? (descriptor = (DSSI_Descriptor *)descriptor_function(plug_index)) : @@ -1509,17 +1443,15 @@ static void ph_tilde_search_plugin_callback ( ladspa_to_dssi((LADSPA_Descriptor *) descriptor_function(plug_index)))->LADSPA_Plugin)) != NULL; plug_index++){ -#if DEBUG - post("pluginhost~: label \"%s\"", descriptor->LADSPA_Plugin->Label); -#endif + ph_debug_post("pluginhost~: label \"%s\"", descriptor->LADSPA_Plugin->Label); + if (strcasecmp (name, descriptor->LADSPA_Plugin->Label) == 0) { *out_lib_name = strdup (full_filename); -#if DEBUG - post("pluginhost~: found plugin \"%s\" in library \"%s\"", + ph_debug_post("pluginhost~: found plugin \"%s\" in library \"%s\"", name, full_filename); -#endif + /* if(!is_DSSI){ free((DSSI_Descriptor *)descriptor); descriptor = NULL; @@ -1542,9 +1474,8 @@ static const char* plugin_tilde_search_plugin_by_label (t_ph_tilde *x, user_data[0] = (void*)(&lib_name); user_data[1] = (void*)name; -#if DEBUG - post("search plugin by label: '%s'\n", name); -#endif + ph_debug_post("search plugin by label: '%s'\n", name); + lib_name = NULL; LADSPAPluginSearch (ph_tilde_search_plugin_callback, @@ -1660,9 +1591,8 @@ static t_int ph_tilde_ph_methods(t_ph_tilde *x, t_symbol *s, int argc, t_atom *a return 0; } fclose(fp); -#if DEBUG - post("Patch file length is %ul", filelength); -#endif + ph_debug_post("Patch file length is %ul", filelength); + /* figure out what kind of file it is */ filename_length = strlen(filename); if (filename_length > 4 && @@ -1670,9 +1600,8 @@ static t_int ph_tilde_ph_methods(t_ph_tilde *x, t_symbol *s, int argc, t_atom *a filelength % DX7_VOICE_SIZE_PACKED == 0) { /* It's a raw DX7 patch bank */ -#if DEBUG - post("Raw DX7 format patch bank passed"); -#endif + ph_debug_post("Raw DX7 format patch bank passed"); + count = filelength / DX7_VOICE_SIZE_PACKED; if (count > maxpatches) count = maxpatches; @@ -1693,9 +1622,8 @@ static t_int ph_tilde_ph_methods(t_ph_tilde *x, t_symbol *s, int argc, t_atom *a raw_patch_data[5] == 0x00) { /* It's a DX7 sys-ex 32 voice dump */ -#if DEBUG - post("SYSEX header check passed"); -#endif + ph_debug_post("SYSEX header check passed"); + if (filelength != DX7_DUMP_SIZE_BULK || raw_patch_data[DX7_DUMP_SIZE_BULK - 1] != 0xf7) { @@ -1709,8 +1637,9 @@ static t_int ph_tilde_ph_methods(t_ph_tilde *x, t_symbol *s, int argc, t_atom *a post("pluginhost~: DX7 32 voice dump with bad checksum!"); count = 0; - #endif + + } else { count = 32; @@ -1829,9 +1758,8 @@ static t_int ph_tilde_ph_methods(t_ph_tilde *x, t_symbol *s, int argc, t_atom *a ph_tilde_configure_buffer(x, key, value, instance); } } -#if DEBUG - post("The plugin returned %s", debug); -#endif + ph_debug_post("The plugin returned %s", debug); + free(msg_type); free(patchbuf); @@ -1887,9 +1815,8 @@ static t_int *ph_tilde_perform(t_int *w) "pluginhost~: %s: discarding spurious MIDI data, for instance %d", x->descriptor->LADSPA_Plugin->Label, instance); -#if DEBUG - post("n_instances = %d", x->n_instances); -#endif + ph_debug_post("n_instances = %d", x->n_instances); + continue; } @@ -1911,18 +1838,16 @@ static t_int *ph_tilde_perform(t_int *w) x->instanceEventBuffers[instance] [x->instanceEventCounts[instance]] = x->midiEventBuf[x->bufReadIndex]; -#if DEBUG - post("%s, note received on channel %d", + ph_debug_post("%s, note received on channel %d", x->descriptor->LADSPA_Plugin->Label, x->instanceEventBuffers[instance] [x->instanceEventCounts[instance]].data.note.channel); -#endif + x->instanceEventCounts[instance]++; -#if DEBUG - post("Instance event count for instance %d of %d: %d\n", + ph_debug_post("Instance event count for instance %d of %d: %d\n", instance + 1, x->n_instances, x->instanceEventCounts[instance]); -#endif + } @@ -2054,9 +1979,8 @@ static void ph_tilde_free_plugin(t_ph_tilde *x){ while(instance--){ if(x->instances[instance].gui_pid){ -#if DEBUG - post("Killing GUI process PID = %d", x->instances[instance].gui_pid); -#endif + ph_debug_post("Killing GUI process PID = %d", x->instances[instance].gui_pid); + kill(x->instances[instance].gui_pid, SIGINT); } if (x->instances[instance].pluginPrograms) { @@ -2151,9 +2075,8 @@ static void *ph_tilde_load_plugin(t_ph_tilde *x, t_int argc, t_atom *argv){ *plugin_label, plugin_dir[MAXPDSTRING]; -#if DEBUG - post("argc = %d", argc); -#endif + ph_debug_post("argc = %d", argc); + int i, stop, fd; @@ -2183,10 +2106,9 @@ static void *ph_tilde_load_plugin(t_ph_tilde *x, t_int argc, t_atom *argv){ plugin_full_path = strdup(tmpstr); } free(argstr); -#if DEBUG - post("plugin path = %s", plugin_full_path); + ph_debug_post("plugin path = %s", plugin_full_path); post("plugin name = %s", x->plugin_label); -#endif + if(plugin_full_path != NULL){ /* First try to load as is: this will work if plugin_full_path is an @@ -2200,10 +2122,9 @@ static void *ph_tilde_load_plugin(t_ph_tilde *x, t_int argc, t_atom *argv){ plugin_dir, &plugin_basename, MAXPDSTRING, 0); if (fd >= 0) { -#if DEBUG - post("plugin directory is %s, filename is %s", + ph_debug_post("plugin directory is %s, filename is %s", plugin_dir, plugin_basename); -#endif + x->plugin_basename = strdup(plugin_basename); pathlen = strlen(plugin_dir); tmpstr = &plugin_dir[pathlen]; @@ -2230,9 +2151,8 @@ static void *ph_tilde_load_plugin(t_ph_tilde *x, t_int argc, t_atom *argv){ while(strstr(plugin_basename, ".so") == NULL) plugin_basename = strtok(NULL, "/"); x->plugin_basename = strdup(plugin_basename); -#if DEBUG - post("plugin basename = %s", x->plugin_basename); -#endif + ph_debug_post("plugin basename = %s", x->plugin_basename); + } else{ post("pluginhost~: invalid plugin path, must end in .so"); @@ -2259,17 +2179,15 @@ static void *ph_tilde_load_plugin(t_ph_tilde *x, t_int argc, t_atom *argv){ x->n_instances = 1; -#if DEBUG - post("n_instances = %d", x->n_instances); -#endif + ph_debug_post("n_instances = %d", x->n_instances); + x->instances = (t_ph_instance *)malloc(sizeof(t_ph_instance) * x->n_instances); if(x->descriptor){ -#if DEBUG - post("%s loaded successfully!", + ph_debug_post("%s loaded successfully!", x->descriptor->LADSPA_Plugin->Label); -#endif + /*allocate memory for port_info*/ @@ -2304,6 +2222,7 @@ static void *ph_tilde_load_plugin(t_ph_tilde *x, t_int argc, t_atom *argv){ for(i = 0;i < x->n_instances; i++) ph_tilde_load_gui(x, i); #endif + for(i = 0;i < x->n_instances; i++) ph_tilde_init_programs(x, i); @@ -2394,9 +2313,8 @@ static void *ph_tilde_new(t_symbol *s, t_int argc, t_atom *argv){ static void ph_tilde_free(t_ph_tilde *x){ -#if DEBUG - post("Calling ph_tilde_free"); -#endif + ph_debug_post("Calling ph_tilde_free"); + ph_tilde_quit_plugin(x); ph_tilde_free_plugin(x); @@ -2439,18 +2357,16 @@ void pluginhost_tilde_setup(void) { static int osc_message_handler(const char *path, const char *types, lo_arg **argv,int argc, void *data, void *user_data) { -#if DEBUG - post("osc_message_handler active"); -#endif + ph_debug_post("osc_message_handler active"); + int i, instance = 0; const char *method; char chantemp[2]; t_ph_tilde *x = (t_ph_tilde *)(user_data); if (strncmp(path, "/dssi/", 6)){ -#if DEBUG - post("calling osc_debug_handler"); -#endif + ph_debug_post("calling osc_debug_handler"); + return osc_debug_handler(path, types, argv, argc, data, x); } for (i = 0; i < x->n_instances; i++) { @@ -2460,59 +2376,51 @@ static int osc_message_handler(const char *path, const char *types, break; } } -#if DEBUG for(i = 0; i < argc; i++){ - post("got osc request %c from instance %d, path: %s", + ph_debug_post("got osc request %c from instance %d, path: %s", types[i],instance,path); } -#endif + if (!x->instances[instance].osc_url_path){ -#if DEBUG - post("calling osc_debug_handler"); -#endif + ph_debug_post("calling osc_debug_handler"); + return osc_debug_handler(path, types, argv, argc, data, x); } method = path + 6 + strlen(x->instances[instance].osc_url_path); if (*method != '/' || *(method + 1) == 0){ -#if DEBUG - post("calling osc_debug_handler"); -#endif + ph_debug_post("calling osc_debug_handler"); + return osc_debug_handler(path, types, argv, argc, data, x); } method++; if (!strcmp(method, "configure") && argc == 2 && !strcmp(types, "ss")) { -#if DEBUG - post("calling osc_configure_handler"); -#endif + ph_debug_post("calling osc_configure_handler"); + return osc_configure_handler(x, argv, instance); } else if (!strcmp(method, "control") && argc == 2 && !strcmp(types, "if")) { -#if DEBUG - post("calling osc_control_handler"); -#endif + ph_debug_post("calling osc_control_handler"); + return osc_control_handler(x, argv, instance); } else if (!strcmp(method, "midi") && argc == 1 && !strcmp(types, "m")) { -#if DEBUG - post("calling osc_midi_handler"); -#endif + ph_debug_post("calling osc_midi_handler"); + return osc_midi_handler(x, argv, instance); } else if (!strcmp(method, "program") && argc == 2 && !strcmp(types, "ii")){ -#if DEBUG - post("calling osc_program_handler"); -#endif + ph_debug_post("calling osc_program_handler"); + return osc_program_handler(x, argv, instance); } else if (!strcmp(method, "update") && argc == 1 && !strcmp(types, "s")){ -#if DEBUG - post("calling osc_update_handler"); -#endif + ph_debug_post("calling osc_update_handler"); + return osc_update_handler(x, argv, instance); } else if (!strcmp(method, "exiting") && argc == 0) { @@ -2523,3 +2431,23 @@ static int osc_message_handler(const char *path, const char *types, return osc_debug_handler(path, types, argv, argc, data, x); } +static void ph_debug_post(const char *fmt, ...) +{ +#if DEBUG + va_list args; + size_t fmt_length; + char newfmt[DEBUG_STRING_SIZE]; + + fmt_length = strlen(fmt); + + sprintf(newfmt, "%s: ", MY_NAME); + strncat(newfmt, fmt, fmt_length); + newfmt[strlen(MY_NAME) + 2 + fmt_length] = '\n'; + newfmt[strlen(MY_NAME) + 2 + fmt_length + 1] = '\0'; + + va_start(args, fmt); + vfprintf(stderr, newfmt, args); + va_end(args); +#endif +} + diff --git a/pluginhost~/src/pluginhost~.h b/pluginhost~/src/pluginhost~.h index e154e90..ddf439c 100644 --- a/pluginhost~/src/pluginhost~.h +++ b/pluginhost~/src/pluginhost~.h @@ -43,10 +43,12 @@ #define VERSION 0.99 +#define MY_NAME "pluginhost~" #define EVENT_BUFSIZE 1024 #define OSC_BASE_MAX 1024 #define TYPE_STRING_SIZE 20 /* Max size of event type string (must be two more bytes than needed) */ #define DIR_STRING_SIZE 1024 /* Max size of directory string */ +#define DEBUG_STRING_SIZE 1024 #define ASCII_n 110 #define ASCII_p 112 #define ASCII_c 99 @@ -180,5 +182,6 @@ static int osc_message_handler(const char *path, const char *types, lo_arg **argv, int argc, void *data, void *user_data); static LADSPA_Data get_port_default(t_ph_tilde *x, int port); static void MIDIbuf(int type, int chan, int param, int val, t_ph_tilde *x); +static void ph_debug_post(const char *fmt, ...); -- cgit v1.2.1