ardour/livetrax
13
0
Fork 0
Code

Fix duplicate if(A) else if (A) condition

Browse Source 
Issue came to be from consolidating Session and UI configuration.
Found by PVS-Studio - https://www.viva64.com/en/b/0540/
This commit is contained in:
Robin Gareus 2017-11-23 09:22:21 +01:00
parent 7d4353dddc
commit 55ee5d2e50
2 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gtk2_ardour/ardour_ui_options.cc
View File

@ -514,10 +514,6 @@ ARDOUR_UI::parameter_changed (std::string p)
}
} else if (p == "layered-record-mode") {
layered_button.set_active (_session->config.get_layered_record_mode ());
} else if (p == "show-waveform-clipping") {
ArdourWaveView::WaveView::set_global_show_waveform_clipping (UIConfiguration::instance().get_show_waveform_clipping());
} else if (p == "waveform-gradient-depth") {
ArdourWaveView::WaveView::set_global_gradient_depth (UIConfiguration::instance().get_waveform_gradient_depth());
} else if (p == "flat-buttons") {
bool flat = UIConfiguration::instance().get_flat_buttons();
if (ArdourButton::flat_buttons () != flat) {

3
gtk2_ardour/mixer_strip.cc
View File

@ -2388,6 +2388,7 @@ MixerStrip::parameter_changed (string p)
_visibility.set_state (UIConfiguration::instance().get_mixer_strip_visibility ());
} else if (p == "track-name-number") {
name_changed ();
update_track_number_visibility();
} else if (p == "use-monitor-bus") {
if (monitor_section_button) {
if (mute_button->get_parent()) {
@ -2406,8 +2407,6 @@ MixerStrip::parameter_changed (string p)
mute_button->show();
}
}
} else if (p == "track-name-number") {
update_track_number_visibility();
}
}