Make MIDI channel selector rows/col constraints actually do something.
authorDavid Robillard <d@drobilla.net>
Tue, 18 Nov 2014 07:52:56 +0000 (02:52 -0500)
committerDavid Robillard <d@drobilla.net>
Tue, 18 Nov 2014 07:55:04 +0000 (02:55 -0500)
Seems kind of pointless since it's always 4 anyway, but whatever.

gtk2_ardour/midi_channel_selector.cc

index b51584aef50d1a109160a913ce62d22a22e5c31d..4884d72328fa430b8e17b1c6643e67739ef2acfb 100644 (file)
@@ -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;