test another fix for solo-press-crashes
[ardour.git] / gtk2_ardour / route_ui.cc
index 249826d3505bcbc1a76621e09dfc152560b77ae1..842e06953ece8780731c02d64419cbadb484e1af 100644 (file)
@@ -15,7 +15,6 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <gtkmm2ext/gtk_ui.h>
@@ -48,13 +47,24 @@ using namespace Gtkmm2ext;
 using namespace ARDOUR;
 using namespace PBD;
 
-RouteUI::RouteUI (boost::shared_ptr<ARDOUR::Route> rt, ARDOUR::Session& sess, const char* m_name,
-                 const char* s_name, const char* r_name)
-       : AxisView(sess),
-         _route(rt),
-         mute_button(0),
-         solo_button(0),
-         rec_enable_button(0)
+RouteUI::RouteUI (ARDOUR::Session& sess, const char* mute_name, const char* solo_name, const char* rec_name)
+       : AxisView(sess)
+{
+       init ();
+       set_button_names (mute_name, solo_name, rec_name);
+}
+
+RouteUI::RouteUI (boost::shared_ptr<ARDOUR::Route> rt, 
+                 ARDOUR::Session& sess, const char* mute_name, const char* solo_name, const char* rec_name)
+       : AxisView(sess)
+{
+       init ();
+       set_button_names (mute_name, solo_name, rec_name);
+       set_route (rt);
+}
+
+void
+RouteUI::init ()
 {
        xml_node = 0;
        mute_menu = 0;
@@ -64,66 +74,110 @@ RouteUI::RouteUI (boost::shared_ptr<ARDOUR::Route> rt, ARDOUR::Session& sess, co
        wait_for_release = false;
        route_active_menu_item = 0;
        was_solo_safe = false;
+       polarity_menu_item = 0;
+       denormal_menu_item = 0;
 
-       if (set_color_from_route()) {
-               set_color (unique_random_color());
+       mute_button = manage (new BindableToggleButton (0, ""));
+       mute_button->set_self_managed (true);
+       mute_button->set_name ("MuteButton");
+
+       solo_button = manage (new BindableToggleButton (0, ""));
+       solo_button->set_self_managed (true);
+       solo_button->set_name ("SoloButton");
+
+       rec_enable_button = manage (new BindableToggleButton (0, ""));
+       rec_enable_button->set_name ("RecordEnableButton");
+       rec_enable_button->set_self_managed (true);
+
+       _session.SoloChanged.connect (mem_fun(*this, &RouteUI::solo_changed_so_update_mute));
+}
+
+void
+RouteUI::reset ()
+{
+       connections.clear ();
+
+       if (solo_menu) {
+               delete solo_menu;
+               solo_menu = 0;
        }
 
-       new PairedShiva<Route,RouteUI> (*_route, *this);
+       if (mute_menu) {
+               delete mute_menu;
+               mute_menu = 0;
+       }
+       
+       if (remote_control_menu) {
+               delete remote_control_menu;
+               remote_control_menu = 0;
+       }
 
-       _route->active_changed.connect (mem_fun (*this, &RouteUI::route_active_changed));
+       if (xml_node) {
+               /* do not delete the node - its owned by the route */
+               xml_node = 0;
+       }
 
-        mute_button = manage (new BindableToggleButton (_route->mute_control(), m_name ));
-        solo_button = manage (new BindableToggleButton (_route->solo_control(), s_name ));
+       route_active_menu_item = 0;
+       polarity_menu_item = 0;
+       denormal_menu_item = 0;
+}
 
-       mute_button->set_name ("MuteButton");
-       solo_button->set_name ("SoloButton");
+void
+RouteUI::set_button_names (const char* mute, const char* solo, const char* rec)
+{
+       m_name = mute;
+       s_name = solo;
+       r_name = rec;
+}
 
-       vector<Gdk::Color> colors;
-       Gdk::Color c;
+void
+RouteUI::set_route (boost::shared_ptr<Route> rp)
+{
+       reset ();
 
-       /* mute+solo buttons get 2 color states, so add one here to supplement the existing one */
-       ::set_color(c, rgba_from_style (X_("MuteButton"), 0x7f, 0xff, 0x7f, 0, "bg", Gtk::STATE_ACTIVE, false ));
-       colors.push_back (c);
-       mute_button->set_colors (colors);
+       _route = rp;
 
-       colors.clear ();
-       
-       /* mute+solo buttons get 2 color states, so add one here to supplement the existing one */
-       ::set_color(c, rgba_from_style (X_("SoloButton"), 0x7f, 0xff, 0x7f, 0, "bg", Gtk::STATE_ACTIVE, false ));
-       colors.push_back (c);
-       solo_button->set_colors (colors);
-
-       _route->mute_changed.connect (mem_fun(*this, &RouteUI::mute_changed));
-       _route->solo_changed.connect (mem_fun(*this, &RouteUI::solo_changed));
-       _route->solo_safe_changed.connect (mem_fun(*this, &RouteUI::solo_changed));
+       if (set_color_from_route()) {
+               set_color (unique_random_color());
+       }
+
+       /* no, there is no memory leak here. This object cleans itself (and other stuff)
+          up when the route is destroyed.
+       */
+
+       new PairedShiva<Route,RouteUI> (*_route, *this);
+
+       mute_button->set_controllable (&_route->mute_control());
+       mute_button->set_label (m_name);
        
+       solo_button->set_controllable (&_route->solo_control());
+       solo_button->set_label (s_name);
+
+       connections.push_back (_route->active_changed.connect (mem_fun (*this, &RouteUI::route_active_changed)));
+       connections.push_back (_route->mute_changed.connect (mem_fun(*this, &RouteUI::mute_changed)));
+       connections.push_back (_route->solo_changed.connect (mem_fun(*this, &RouteUI::solo_changed)));
+       connections.push_back (_route->solo_safe_changed.connect (mem_fun(*this, &RouteUI::solo_changed)));
+
+       /* when solo changes, update mute state too, in case the user wants us to display it */
+
        update_solo_display ();
        update_mute_display ();
 
        if (is_track()) {
                boost::shared_ptr<Track> t = boost::dynamic_pointer_cast<Track>(_route);
 
-               t->diskstream()->RecordEnableChanged.connect (mem_fun (*this, &RouteUI::route_rec_enable_changed));
-
-               _session.RecordStateChanged.connect (mem_fun (*this, &RouteUI::session_rec_enable_changed));
+               connections.push_back (t->diskstream()->RecordEnableChanged.connect (mem_fun (*this, &RouteUI::route_rec_enable_changed)));
 
-               rec_enable_button = manage (new BindableToggleButton (t->rec_enable_control(), r_name ));
+               connections.push_back (_session.RecordStateChanged.connect (mem_fun (*this, &RouteUI::session_rec_enable_changed)));
 
-               colors.clear ();
+               rec_enable_button->set_controllable (&t->rec_enable_control());
+               rec_enable_button->set_label (r_name);
 
-               /* record button has 3 color states, so we set 2 extra here */
-               ::set_color(c, rgba_from_style (X_("TrackRecordEnableButton"), 0xff, 0, 0, 0, "bg", Gtk::STATE_SELECTED, false ));
-               colors.push_back (c);
-               
-               ::set_color(c, rgba_from_style (X_("TrackRecordEnableButton"), 0xff, 0, 0, 0, "bg", Gtk::STATE_ACTIVE, false ));
-               colors.push_back (c);
-               
-               rec_enable_button->set_colors (colors);
-               
                update_rec_display ();
        } 
        
+       connections.push_back (_route->RemoteControlIDChanged.connect (mem_fun(*this, &RouteUI::refresh_remote_control_menu)));
+
        /* map the current state */
 
        map_frozen ();
@@ -132,12 +186,26 @@ RouteUI::RouteUI (boost::shared_ptr<ARDOUR::Route> rt, ARDOUR::Session& sess, co
 RouteUI::~RouteUI()
 {
        GoingAway (); /* EMIT SIGNAL */
-       delete mute_menu;
+       if (solo_menu) {
+               delete solo_menu;
+       }
+
+       if (mute_menu) {
+               delete mute_menu;
+       }
+
+       if (remote_control_menu) {
+               delete remote_control_menu;
+       }
 }
 
 bool
 RouteUI::mute_press(GdkEventButton* ev)
 {
+       if (ev->type == GDK_2BUTTON_PRESS) {
+               return true;
+       }
+
        if (!ignore_toggle) {
 
                if (Keyboard::is_context_menu_event (ev)) {
@@ -151,10 +219,10 @@ RouteUI::mute_press(GdkEventButton* ev)
                } else {
 
                        if (ev->button == 2) {
-                               // ctrl-button2 click is the midi binding click
+                               // Primary-button2 click is the midi binding click
                                // button2-click is "momentary"
                                
-                               if (!Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::Control))) {
+                               if (!Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier))) {
                                        wait_for_release = true;
                                } else {
                                        return false;
@@ -163,9 +231,9 @@ RouteUI::mute_press(GdkEventButton* ev)
 
                        if (ev->button == 1 || ev->button == 2) {
 
-                               if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::Control|Keyboard::Shift))) {
+                               if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::TertiaryModifier))) {
 
-                                       /* ctrl-shift-click applies change to all routes */
+                                       /* Primary-Tertiary-click applies change to all routes */
 
                                        _session.begin_reversible_command (_("mute change"));
                                         Session::GlobalMuteStateCommand *cmd = new Session::GlobalMuteStateCommand(_session, this);
@@ -174,10 +242,10 @@ RouteUI::mute_press(GdkEventButton* ev)
                                        _session.add_command(cmd);
                                        _session.commit_reversible_command ();
 
-                               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::Control)) {
+                               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
 
-                                       /* ctrl-click applies change to the mix group.
-                                          ctrl-button2 is MIDI learn.
+                                       /* Primary-button1 applies change to the mix group.
+                                          NOTE: Primary-button2 is MIDI learn.
                                        */
 
                                        if (ev->button == 1) {
@@ -215,6 +283,12 @@ RouteUI::mute_release(GdkEventButton* ev)
 bool
 RouteUI::solo_press(GdkEventButton* ev)
 {
+       /* ignore double clicks */
+
+       if (ev->type == GDK_2BUTTON_PRESS) {
+               return true;
+       }
+
        if (!ignore_toggle) {
 
                if (Keyboard::is_context_menu_event (ev)) {
@@ -229,10 +303,10 @@ RouteUI::solo_press(GdkEventButton* ev)
 
                        if (ev->button == 2) {
 
-                               // ctrl-button2 click is the midi binding click
+                               // Primary-button2 click is the midi binding click
                                // button2-click is "momentary"
                                
-                               if (!Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::Control))) {
+                               if (!Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier))) {
                                        wait_for_release = true;
                                } else {
                                        return false;
@@ -241,9 +315,9 @@ RouteUI::solo_press(GdkEventButton* ev)
 
                        if (ev->button == 1 || ev->button == 2) {
 
-                               if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::Control|Keyboard::Shift))) {
+                               if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::TertiaryModifier))) {
 
-                                       /* ctrl-shift-click applies change to all routes */
+                                       /* Primary-Tertiary-click applies change to all routes */
 
                                        _session.begin_reversible_command (_("solo change"));
                                         Session::GlobalSoloStateCommand *cmd = new Session::GlobalSoloStateCommand(_session, this);
@@ -252,9 +326,9 @@ RouteUI::solo_press(GdkEventButton* ev)
                                        _session.add_command (cmd);
                                        _session.commit_reversible_command ();
                                        
-                               } else if (Keyboard::modifier_state_contains (ev->state, Keyboard::ModifierMask (Keyboard::Control|Keyboard::Alt))) {
+                               } else if (Keyboard::modifier_state_contains (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::SecondaryModifier))) {
 
-                                       // ctrl-alt-click: exclusively solo this track, not a toggle */
+                                       // Primary-Secondary-click: exclusively solo this track, not a toggle */
 
                                        _session.begin_reversible_command (_("solo change"));
                                         Session::GlobalSoloStateCommand *cmd = new Session::GlobalSoloStateCommand (_session, this);
@@ -264,17 +338,17 @@ RouteUI::solo_press(GdkEventButton* ev)
                                        _session.add_command(cmd);
                                        _session.commit_reversible_command ();
 
-                               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::Shift)) {
+                               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::TertiaryModifier)) {
 
                                        // shift-click: set this route to solo safe
 
                                        _route->set_solo_safe (!_route->solo_safe(), this);
                                        wait_for_release = false;
 
-                               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::Control)) {
+                               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
 
-                                       /* ctrl-click: solo mix group.
-                                          ctrl-button2 is MIDI learn.
+                                       /* Primary-button1: solo mix group.
+                                          NOTE: Primary-button2 is MIDI learn.
                                        */
 
                                        if (ev->button == 1) {
@@ -284,7 +358,6 @@ RouteUI::solo_press(GdkEventButton* ev)
                                } else {
 
                                        /* click: solo this route */
-
                                        reversibly_apply_route_boolean ("solo change", &Route::set_solo, !_route->soloed(), this);
                                }
                        }
@@ -313,6 +386,10 @@ RouteUI::solo_release(GdkEventButton* ev)
 bool
 RouteUI::rec_enable_press(GdkEventButton* ev)
 {
+       if (ev->type == GDK_2BUTTON_PRESS) {
+               return true;
+       }
+
        if (!_session.engine().connected()) {
                MessageDialog msg (_("Not connected to JACK - cannot engage record"));
                msg.run ();
@@ -321,10 +398,12 @@ RouteUI::rec_enable_press(GdkEventButton* ev)
 
        if (!ignore_toggle && is_track() && rec_enable_button) {
 
-               if (ev->button == 2 && Keyboard::modifier_state_equals (ev->state, Keyboard::Control)) {
+               if (ev->button == 2 && Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
+
                        // do nothing on midi bind event
-               }
-               else if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::Control|Keyboard::Shift))) {
+                       return false;
+
+               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::ModifierMask (Keyboard::PrimaryModifier|Keyboard::TertiaryModifier))) {
 
                        _session.begin_reversible_command (_("rec-enable change"));
                         Session::GlobalRecordEnableStateCommand *cmd = new Session::GlobalRecordEnableStateCommand(_session, this);
@@ -339,23 +418,27 @@ RouteUI::rec_enable_press(GdkEventButton* ev)
                        _session.add_command(cmd);
                        _session.commit_reversible_command ();
 
-               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::Control)) {
+               } else if (Keyboard::modifier_state_equals (ev->state, Keyboard::PrimaryModifier)) {
+
+                       /* Primary-button1 applies change to the mix group.
+                          NOTE: Primary-button2 is MIDI learn.
+                       */
 
                        set_mix_group_rec_enable (_route, !_route->record_enabled());
 
                } else {
 
                        reversibly_apply_audio_track_boolean ("rec-enable change", &AudioTrack::set_record_enable, !audio_track()->record_enabled(), this);
-
-                       ignore_toggle = true;
-                       rec_enable_button->set_active(audio_track()->record_enabled());
-                       ignore_toggle = false;
                }
-               
-               stop_signal (*rec_enable_button, "button-press-event");
        }
 
-       return TRUE;
+       return true;
+}
+
+bool
+RouteUI::rec_enable_release (GdkEventButton* ev)
+{
+       return true;
 }
 
 void
@@ -368,32 +451,28 @@ void
 RouteUI::update_solo_display ()
 {
        bool x;
-       vector<Gdk::Color> colors;
+       vector<Gdk::Color> fg_colors;
        Gdk::Color c;
-
-       if (_route->solo_safe() != was_solo_safe){
-
-               if (_route->solo_safe()) {
-                       /* show solo safe */
-                       ::set_color(c, rgba_from_style (safe_solo_button_name(), 0x7f, 0xff, 0x7f, 0, "bg", Gtk::STATE_ACTIVE, false ));
-                       solo_button->set_name(safe_solo_button_name());
-               } else {
-                       ::set_color(c, rgba_from_style (solo_button_name(), 0x7f, 0xff, 0x7f, 0, "bg", Gtk::STATE_ACTIVE, false ));
-                       solo_button->set_name(solo_button_name());
-               }
-
-               colors.push_back (c);
-               solo_button->set_colors (colors);
-
-               was_solo_safe = !was_solo_safe;
-       }
-
+       
        if (solo_button->get_active() != (x = _route->soloed())){
                ignore_toggle = true;
                solo_button->set_active(x);
                ignore_toggle = false;
-       }
+       } 
        
+       if (_route->solo_safe()) {
+               solo_button->set_visual_state (2);
+       } else if (_route->soloed()) {
+               solo_button->set_visual_state (1);
+       } else {
+               solo_button->set_visual_state (0);
+       }
+}
+
+void
+RouteUI::solo_changed_so_update_mute ()
+{
+       Gtkmm2ext::UI::instance()->call_slot (mem_fun (*this, &RouteUI::update_mute_display));
 }
 
 void
@@ -405,13 +484,38 @@ RouteUI::mute_changed(void* src)
 void
 RouteUI::update_mute_display ()
 {
-       bool x;
+       bool model = _route->muted();
+       bool view = mute_button->get_active();
+
+       /* first make sure the button's "depressed" visual
+          is correct.
+       */
 
-       if (mute_button->get_active() != (x = _route->muted())){
+       if (model != view) {
                ignore_toggle = true;
-               mute_button->set_active(x);
+               mute_button->set_active (model);
                ignore_toggle = false;
        }
+
+       /* now attend to visual state */
+       
+       if (Config->get_show_solo_mutes()) {
+               if (_route->muted()) {
+                       mute_button->set_visual_state (2);
+               } else if (!_route->soloed() && _route->solo_muted()) {
+                       
+                       mute_button->set_visual_state (1);
+               } else {
+                       mute_button->set_visual_state (0);
+               }
+       } else {
+               if (_route->muted()) {
+                       mute_button->set_visual_state (2);
+               } else {
+                       mute_button->set_visual_state (0);
+               }
+       }
+
 }
 
 void
@@ -435,7 +539,7 @@ RouteUI::update_rec_display ()
        /* first make sure the button's "depressed" visual
           is correct.
        */
-       
+
        if (model != view) {
                ignore_toggle = true;
                rec_enable_button->set_active (model);
@@ -447,37 +551,40 @@ RouteUI::update_rec_display ()
        if (model) {
 
                switch (_session.record_status ()) {
+               case Session::Recording:
+                       rec_enable_button->set_visual_state (1);
+                       break;
+
                case Session::Disabled:
                case Session::Enabled:
-                       if (rec_enable_button->get_state() != Gtk::STATE_ACTIVE) {
-                               rec_enable_button->set_state (Gtk::STATE_ACTIVE);
-                       }
+                       rec_enable_button->set_visual_state (2);
                        break;
 
-               case Session::Recording:
-                       if (rec_enable_button->get_state() != Gtk::STATE_SELECTED) {
-                               rec_enable_button->set_state (Gtk::STATE_SELECTED);
-                       }
-                       break;
                }
 
        } else {
-               if (rec_enable_button->get_state() != Gtk::STATE_NORMAL) {
-                       rec_enable_button->set_state (Gtk::STATE_NORMAL);
-               }
+               rec_enable_button->set_visual_state (0);
        }
 }
 
 void
 RouteUI::build_remote_control_menu ()
 {
-       remote_control_menu = manage (new Menu);
+       remote_control_menu = new Menu;
        refresh_remote_control_menu ();
 }
 
 void
 RouteUI::refresh_remote_control_menu ()
 {
+       ENSURE_GUI_THREAD (mem_fun (*this, &RouteUI::refresh_remote_control_menu));
+
+       // only refresh the menu if it has been instantiated
+
+       if (remote_control_menu == 0) {
+               return;
+       }
+
        using namespace Menu_Helpers;
 
        RadioMenuItem::Group rc_group;
@@ -867,6 +974,28 @@ RouteUI::polarity_changed ()
        /* no signal for this yet */
 }
 
+void
+RouteUI::toggle_denormal_protection ()
+{
+       if (denormal_menu_item) {
+
+               bool x;
+
+               ENSURE_GUI_THREAD(mem_fun (*this, &RouteUI::toggle_denormal_protection));
+               
+               if ((x = denormal_menu_item->get_active()) != _route->denormal_protection()) {
+                       _route->set_denormal_protection (x, this);
+               }
+       }
+}
+
+void
+RouteUI::denormal_protection_changed ()
+{
+       /* no signal for this yet */
+}
+
+
 void
 RouteUI::solo_safe_toggle(void* src, Gtk::CheckMenuItem* check)
 {
@@ -935,25 +1064,25 @@ RouteUI::disconnect_output ()
 bool
 RouteUI::is_track () const
 {
-       return dynamic_cast<Track*>(_route.get()) != 0;
+       return boost::dynamic_pointer_cast<Track>(_route) != 0;
 }
 
-Track*
+boost::shared_ptr<Track>
 RouteUI::track() const
 {
-       return dynamic_cast<Track*>(_route.get());
+       return boost::dynamic_pointer_cast<Track>(_route);
 }
 
 bool
 RouteUI::is_audio_track () const
 {
-       return dynamic_cast<AudioTrack*>(_route.get()) != 0;
+       return boost::dynamic_pointer_cast<AudioTrack>(_route) != 0;
 }
 
-AudioTrack*
+boost::shared_ptr<AudioTrack>
 RouteUI::audio_track() const
 {
-       return dynamic_cast<AudioTrack*>(_route.get());
+       return boost::dynamic_pointer_cast<AudioTrack>(_route);
 }
 
 boost::shared_ptr<Diskstream>