diff options
author | Robin Gareus <robin@gareus.org> | 2018-08-24 16:53:06 +0200 |
---|---|---|
committer | Robin Gareus <robin@gareus.org> | 2018-08-24 16:55:00 +0200 |
commit | 3136b20847ea2e2e12f707bdc82fa336f85f4801 (patch) | |
tree | 08c54a63b074f3f853227f9985dc9cf031690c8b /libs/surfaces/launch_control_xl/controllers.cc | |
parent | c46cd91d0eca0e060d3efed5162c8a5205567274 (diff) |
Cleanup various design patterns.
* do not use implicit bool-to-int-cast
* avoid C++11 member initialization in header
* always use set and access methods, remove public variable
Diffstat (limited to 'libs/surfaces/launch_control_xl/controllers.cc')
-rw-r--r-- | libs/surfaces/launch_control_xl/controllers.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libs/surfaces/launch_control_xl/controllers.cc b/libs/surfaces/launch_control_xl/controllers.cc index 4bcfc32fbc..5749b58d48 100644 --- a/libs/surfaces/launch_control_xl/controllers.cc +++ b/libs/surfaces/launch_control_xl/controllers.cc @@ -528,13 +528,13 @@ LaunchControlXL::button_track_mode(TrackMode state) void LaunchControlXL::button_select_left() { - switch_bank (max (0, bank_start - (7 + (LaunchControlXL::use_fader8master)))); + switch_bank (max (0, bank_start - (7 + (fader8master() ? 1 : 0)))); } void LaunchControlXL::button_select_right() { - switch_bank (max (0, bank_start + 7 + (LaunchControlXL::use_fader8master))); + switch_bank (max (0, bank_start + 7 + (fader8master() ? 1 : 0))); } void |