Fix a few Clang warnings.

[nageru] / mainwindow.cpp

diff --git a/mainwindow.cpp b/mainwindow.cpp
index 7235e8e9c5d1c74dbd1d0e7237d0d03ac3ba7d89..9f61c36ebe1b5497afd7ff5524246b4f3830646f 100644 (file)
--- a/mainwindow.cpp
+++ b/mainwindow.cpp
@@ -129,7 +129,6 @@ void slave_fader(NonLinearFader *master, NonLinearFader *slave)
        });
 }
 
-constexpr unsigned DB_NO_FLAGS = 0x0;
 constexpr unsigned DB_WITH_SIGN = 0x1;
 constexpr unsigned DB_BARE = 0x2;
 
@@ -322,8 +321,15 @@ void MainWindow::mixer_created(Mixer *mixer)
                connect(ui_display->display, &GLWidget::color_updated, this, &MainWindow::update_channel_color);
 
                // Hook up the keyboard key.
-               QShortcut *shortcut = new QShortcut(QKeySequence(Qt::Key_1 + i), this);
-               connect(shortcut, &QShortcut::activated, bind(&MainWindow::channel_clicked, this, i));
+               QShortcut *shortcut = nullptr;
+               if (i < 9) {
+                       shortcut = new QShortcut(QKeySequence(Qt::Key_1 + i), this);
+               } else if (i == 9) {
+                       shortcut = new QShortcut(QKeySequence(Qt::Key_0), this);
+               }
+               if (shortcut != nullptr) {
+                       connect(shortcut, &QShortcut::activated, bind(&MainWindow::channel_clicked, this, i));
+               }
 
                // Hook up the quick-cut key.
                if (i < strlen(qwerty)) {
@@ -370,13 +376,13 @@ void MainWindow::mixer_created(Mixer *mixer)
        });
        connect(ui->gainstaging_knob, &QAbstractSlider::valueChanged,
                bind(&MainWindow::gain_staging_knob_changed, this, simple_bus_index, _1));
-       connect(ui->gainstaging_auto_checkbox, &QCheckBox::stateChanged, [this, simple_bus_index](int state){
+       connect(ui->gainstaging_auto_checkbox, &QCheckBox::stateChanged, [this](int state){
                global_audio_mixer->set_gain_staging_auto(simple_bus_index, state == Qt::Checked);
                midi_mapper.refresh_lights();
        });
        connect(ui->compressor_threshold_knob, &QDial::valueChanged,
                bind(&MainWindow::compressor_threshold_knob_changed, this, simple_bus_index, _1));
-       connect(ui->compressor_enabled, &QCheckBox::stateChanged, [this, simple_bus_index](int state){
+       connect(ui->compressor_enabled, &QCheckBox::stateChanged, [this](int state){
                global_audio_mixer->set_compressor_enabled(simple_bus_index, state == Qt::Checked);
                midi_mapper.refresh_lights();
        });