add editor to Theme Manager for modifiers

[ardour.git] / gtk2_ardour / midi_channel_selector.cc

diff --git a/gtk2_ardour/midi_channel_selector.cc b/gtk2_ardour/midi_channel_selector.cc
index 2f5ca729a6b942480d51ae963304670f181189d6..4884d72328fa430b8e17b1c6643e67739ef2acfb 100644 (file)
--- a/gtk2_ardour/midi_channel_selector.cc
+++ b/gtk2_ardour/midi_channel_selector.cc
@@ -44,14 +44,11 @@ using namespace Gtk;
 using namespace ARDOUR;
 
 MidiChannelSelector::MidiChannelSelector(int n_rows, int n_columns, int start_row, int start_column)
-       : Table(n_rows, n_columns, true)
+       : Table(std::max(4, std::max(n_rows,    start_row    + 4)),
+               std::max(4, std::max(n_columns, start_column + 4)),
+               true)
        , _recursion_counter(0)
 {
-       n_rows    = std::max(4, n_rows);
-       n_rows    = std::max(4, start_row + 4);
-       n_columns = std::max(4, n_columns);
-       n_columns = std::max(4, start_column + 4);
-
        property_column_spacing() = 0;
        property_row_spacing() = 0;
 
@@ -516,7 +513,7 @@ MidiChannelSelectorWindow::set_playback_selected_channels (uint16_t mask)
        case ForceChannel:
                /* only set the lowest set channel in the mask as active */
                for (uint16_t i = 0; i < 16; i++) {
-                       playback_buttons[i]->set_active (i == (ffs (mask) - 1));
+                       playback_buttons[i]->set_active (i == (PBD::ffs (mask) - 1));
                }
                break;
        }
@@ -539,7 +536,7 @@ MidiChannelSelectorWindow::set_capture_selected_channels (uint16_t mask)
        case ForceChannel:
                /* only set the lowest set channel in the mask as active */
                for (uint16_t i = 0; i < 16; i++) {
-                       capture_buttons[i]->set_active (i == (ffs (mask) - 1));
+                       capture_buttons[i]->set_active (i == (PBD::ffs (mask) - 1));
                }
                break;
        }
@@ -595,7 +592,7 @@ MidiChannelSelectorWindow::playback_mode_changed ()
        case ForceChannel:
                if (last_drawn_playback_mode == AllChannels || last_drawn_playback_mode == FilterChannels) {
                        playback_buttons.clear ();
-                       first_channel = ffs (track->get_playback_channel_mask()) - 1;
+                       first_channel = PBD::ffs (track->get_playback_channel_mask()) - 1;
                }
                for (vector<Widget*>::iterator i = playback_mask_controls.begin(); i != playback_mask_controls.end(); ++i) {
                        (*i)->set_sensitive (false);
@@ -693,7 +690,7 @@ MidiChannelSelectorWindow::capture_mode_changed ()
        case ForceChannel:
                if (last_drawn_capture_mode == AllChannels || last_drawn_capture_mode == FilterChannels) {
                        capture_buttons.clear ();
-                       first_channel = ffs (track->get_capture_channel_mask()) - 1;
+                       first_channel = PBD::ffs (track->get_capture_channel_mask()) - 1;
                }
                for (vector<Widget*>::iterator i = capture_mask_controls.begin(); i != capture_mask_controls.end(); ++i) {
                        (*i)->set_sensitive (false);