Albert Santoni wrote:
> I think the real problem is a conceptual one - Each "mapping" of a
> <device, controltype, miditype, midino, midichan> onto a <group, key>
> should be an object, and it currently isn't.

What about ConfigValueMidi? ;) That's an object, and currently where
each mapping is stored for engine use.

> The MidiMapping class
> you're creating should contain a list of these "mappings", and perhaps
> the name of your class should be changed then too (to something more
> plural).

Sounds fine to me but I'm not up to doing that now. I just want to get
the QtScript stuff into the correct threads to try and solve the
crashing issue. Hopefully the work I have done already will be useful in
implementing what you suggest in the future. Garth and I chatted a
little about this on IRC too and he likes the idea, so long as there are
adequate interface functions the dlg can call to tell MidiMappings'
internal storage mechanism that particular keys have been added,
changed, or deleted.

> #include "configobject.h" in midiscriptengine.h. There's something weird
> about MidiCategory and I couldn't get a forward declaration for it to
> work when I was doing my MIDI stuff.

Garth took a look at it and added some forward declarations and other
tweaks and that problem is gone now. But I still have one I can't figure
out:

src/midiobject.h:126: error: field 'm_pMidiMapping' has incomplete type

What the hell is an incomplete type?! Here's the offending line:
MidiMapping m_pMidiMapping;

Garth found that if you put a * in between, it doesn't complain. But I
want that object to be under MidiObject (I think.)

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

Reply via email to