fix conflicts and merge with master
[ardour.git] / libs / surfaces / generic_midi / midicontrollable.cc
index f866bab2bb345ceddace5f3348e9f654941bd9fe..a26617fd676d276d89ffdf823dcef6936b27cb7e 100644 (file)
 #include "pbd/xml++.h"
 #include "pbd/stacktrace.h"
 
+#include "midi++/types.h" // Added by JE - 06-01-2009. All instances of 'byte' changed to 'MIDI::byte' (for clarification)
 #include "midi++/port.h"
 #include "midi++/channel.h"
 
+#include "ardour/async_midi_port.h"
 #include "ardour/automation_control.h"
 #include "ardour/midi_ui.h"
 #include "ardour/utils.h"
@@ -42,11 +44,11 @@ using namespace MIDI;
 using namespace PBD;
 using namespace ARDOUR;
 
-MIDIControllable::MIDIControllable (GenericMidiControlProtocol* s, Port& p, bool m)
+MIDIControllable::MIDIControllable (GenericMidiControlProtocol* s, MIDI::Parser& p, bool m)
        : _surface (s)
        , controllable (0)
        , _descriptor (0)
-       , _port (p)
+       , _parser (p)
        , _momentary (m)
 {
        _learned = false; /* from URI */
@@ -55,24 +57,25 @@ MIDIControllable::MIDIControllable (GenericMidiControlProtocol* s, Port& p, bool
        last_controllable_value = 0.0f;
        control_type = none;
        _control_description = "MIDI Control: none";
-       control_additional = (byte) -1;
+       control_additional = (MIDI::byte) -1;
        feedback = true; // for now
 }
 
-MIDIControllable::MIDIControllable (GenericMidiControlProtocol* s, Port& p, Controllable& c, bool m)
+MIDIControllable::MIDIControllable (GenericMidiControlProtocol* s, MIDI::Parser& p, Controllable& c, bool m)
        : _surface (s)
-       , controllable (&c)
        , _descriptor (0)
-       , _port (p)
+       , _parser (p)
        , _momentary (m)
 {
+       set_controllable (&c);
+       
        _learned = true; /* from controllable */
        setting = false;
        last_value = 0; // got a better idea ?
        last_controllable_value = 0.0f;
        control_type = none;
        _control_description = "MIDI Control: none";
-       control_additional = (byte) -1;
+       control_additional = (MIDI::byte) -1;
        feedback = true; // for now
 }
 
@@ -107,13 +110,31 @@ MIDIControllable::drop_external_control ()
 {
        midi_forget ();
        control_type = none;
-       control_additional = (byte) -1;
+       control_additional = (MIDI::byte) -1;
 }
 
 void
 MIDIControllable::set_controllable (Controllable* c)
 {
+       if (c == controllable) {
+               return;
+       }
+       
+       controllable_death_connection.disconnect ();
+
        controllable = c;
+
+       if (controllable) {
+               last_controllable_value = controllable->get_value();
+       } else {
+               last_controllable_value = 0.0f; // is there a better value?
+       }
+
+       if (controllable) {
+               controllable->Destroyed.connect (controllable_death_connection, MISSING_INVALIDATOR,
+                                                boost::bind (&MIDIControllable::drop_controllable, this), 
+                                                MidiControlUI::instance());
+       }
 }
 
 void
@@ -130,7 +151,7 @@ void
 MIDIControllable::learn_about_external_control ()
 {
        drop_external_control ();
-       _port.parser()->any.connect_same_thread (midi_learn_connection, boost::bind (&MIDIControllable::midi_receiver, this, _1, _2, _3));
+       _parser.any.connect_same_thread (midi_learn_connection, boost::bind (&MIDIControllable::midi_receiver, this, _1, _2, _3));
 }
 
 void
@@ -150,6 +171,14 @@ MIDIControllable::control_to_midi (float val)
        float control_max = controllable->upper ();
        const float control_range = control_max - control_min;
 
+       if (controllable->is_toggle()) {
+               if (val >= (control_min + (control_range/2.0f))) {
+                       return max_value_for_type();
+               } else {
+                       return 0;
+               }
+       }
+
        return (val - control_min) / control_range * max_value_for_type ();
 }
 
@@ -199,10 +228,7 @@ MIDIControllable::lookup_controllable()
                return -1;
        }
 
-       controllable = c.get();
-       controllable->Destroyed.connect (controllable_death_connection, MISSING_INVALIDATOR,
-                                        boost::bind (&MIDIControllable::drop_controllable, this), 
-                                        MidiControlUI::instance());
+       set_controllable (c.get ());
 
        return 0;
 }
@@ -210,8 +236,7 @@ MIDIControllable::lookup_controllable()
 void
 MIDIControllable::drop_controllable ()
 {
-       controllable_death_connection.disconnect ();
-       controllable = 0;
+       set_controllable (0);
 }
 
 void
@@ -224,7 +249,9 @@ MIDIControllable::midi_sense_note (Parser &, EventTwoBytes *msg, bool /*is_on*/)
        }
 
        if (!controllable->is_toggle()) {
-               controllable->set_value (midi_to_control (msg->note_number));
+               if (control_additional == msg->note_number) {
+                       controllable->set_value (midi_to_control (msg->velocity));
+               }
        } else {
                if (control_additional == msg->note_number) {
                        controllable->set_value (controllable->get_value() > 0.5f ? 0.0f : 1.0f);
@@ -243,6 +270,8 @@ MIDIControllable::midi_sense_controller (Parser &, EventTwoBytes *msg)
                }
        }
 
+       assert (controllable);
+
        if (controllable->touching()) {
                return; // to prevent feedback fights when e.g. dragging a UI slider
        }
@@ -250,6 +279,7 @@ MIDIControllable::midi_sense_controller (Parser &, EventTwoBytes *msg)
        if (control_additional == msg->controller_number) {
 
                if (!controllable->is_toggle()) {
+
                        float new_value = msg->value;
                        float max_value = max(last_controllable_value, new_value);
                        float min_value = min(last_controllable_value, new_value);
@@ -285,7 +315,7 @@ MIDIControllable::midi_sense_controller (Parser &, EventTwoBytes *msg)
 }
 
 void
-MIDIControllable::midi_sense_program_change (Parser &, byte msg)
+MIDIControllable::midi_sense_program_change (Parser &, MIDI::byte msg)
 {
        if (!controllable) { 
                if (lookup_controllable ()) {
@@ -321,7 +351,7 @@ MIDIControllable::midi_sense_pitchbend (Parser &, pitchbend_t pb)
 }
 
 void
-MIDIControllable::midi_receiver (Parser &, byte *msg, size_t /*len*/)
+MIDIControllable::midi_receiver (Parser &, MIDI::byte *msg, size_t /*len*/)
 {
        /* we only respond to channel messages */
 
@@ -329,12 +359,6 @@ MIDIControllable::midi_receiver (Parser &, byte *msg, size_t /*len*/)
                return;
        }
 
-       /* if the our port doesn't do input anymore, forget it ... */
-
-       if (!_port.parser()) {
-               return;
-       }
-
        bind_midi ((channel_t) (msg[0] & 0xf), eventType (msg[0] & 0xF0), msg[1]);
 
        if (controllable) {
@@ -353,49 +377,43 @@ MIDIControllable::bind_midi (channel_t chn, eventType ev, MIDI::byte additional)
        control_channel = chn;
        control_additional = additional;
 
-       if (_port.parser() == 0) {
-               return;
-       }
-
-       Parser& p = *_port.parser();
-
        int chn_i = chn;
        switch (ev) {
        case MIDI::off:
-               p.channel_note_off[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_note_off, this, _1, _2));
+               _parser.channel_note_off[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_note_off, this, _1, _2));
 
                /* if this is a togglee, connect to noteOn as well,
                   and we'll toggle back and forth between the two.
                */
 
                if (_momentary) {
-                       p.channel_note_on[chn_i].connect_same_thread (midi_sense_connection[1], boost::bind (&MIDIControllable::midi_sense_note_on, this, _1, _2));
+                       _parser.channel_note_on[chn_i].connect_same_thread (midi_sense_connection[1], boost::bind (&MIDIControllable::midi_sense_note_on, this, _1, _2));
                } 
 
                _control_description = "MIDI control: NoteOff";
                break;
 
        case MIDI::on:
-               p.channel_note_on[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_note_on, this, _1, _2));
+               _parser.channel_note_on[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_note_on, this, _1, _2));
                if (_momentary) {
-                       p.channel_note_off[chn_i].connect_same_thread (midi_sense_connection[1], boost::bind (&MIDIControllable::midi_sense_note_off, this, _1, _2));
+                       _parser.channel_note_off[chn_i].connect_same_thread (midi_sense_connection[1], boost::bind (&MIDIControllable::midi_sense_note_off, this, _1, _2));
                }
                _control_description = "MIDI control: NoteOn";
                break;
                
        case MIDI::controller:
-               p.channel_controller[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_controller, this, _1, _2));
+               _parser.channel_controller[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_controller, this, _1, _2));
                snprintf (buf, sizeof (buf), "MIDI control: Controller %d", control_additional);
                _control_description = buf;
                break;
 
        case MIDI::program:
-               p.channel_program_change[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_program_change, this, _1, _2));
+               _parser.channel_program_change[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_program_change, this, _1, _2));
                _control_description = "MIDI control: ProgramChange";
                break;
 
        case MIDI::pitchbend:
-               p.channel_pitchbend[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_pitchbend, this, _1, _2));
+               _parser.channel_pitchbend[chn_i].connect_same_thread (midi_sense_connection[0], boost::bind (&MIDIControllable::midi_sense_pitchbend, this, _1, _2));
                _control_description = "MIDI control: Pitchbend";
                break;