Nick Guenther wrote: > I don't really know what's going on but: why are you changing the > list?! Make the list once, keep it in place. Keeping data static when > you can avoids the entire class of bugs that dealing with state in > multiple places brings.
I agree, but for the record, dlgprefmidibindings currently uses a QTableWidget to store values during run time. Removing that essentially involves rewriting most of dlgprefmidibindings. My goal was just to get the XML parsing out of it both for organization as well as run order reasons. (This is all part of trying to solve the crashing problem with MIDI scripting.) > Don't you have to say > bool execute(QString function, char channel, QString device, char > control, char value, enum MidiCategory category); I don't think so, as it's not written that way in MidiObject.h, line 103: void receive(MidiCategory category, char channel, char control, char value, QString device); And that works just fine. But I'll try it anyway. Sean <<--------------------------------------------------------------------------------->> This E-Mail message has been scanned for viruses and cleared by >>SmartMail<< from Smarter Technology, Inc. <<--------------------------------------------------------------------------------->> ------------------------------------------------------------------------------ This SF.net email is sponsored by: SourcForge Community SourceForge wants to tell your story. http://p.sf.net/sfu/sf-spreadtheword _______________________________________________ Mixxx-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mixxx-devel
