configure_hotkeys: Mark member variables as const where applicable in Configure()

This commit is contained in:
Lioncash 2019-04-09 19:50:14 -04:00 committed by fearlessTobi
parent 47176e19be
commit d7d1be509b

View file

@ -75,16 +75,16 @@ void ConfigureHotkeys::Configure(QModelIndex index) {
} }
index = index.sibling(index.row(), 1); index = index.sibling(index.row(), 1);
auto* model = ui->hotkey_list->model(); auto* const model = ui->hotkey_list->model();
auto previous_key = model->data(index); const auto previous_key = model->data(index);
auto* hotkey_dialog = new SequenceDialog; auto* const hotkey_dialog = new SequenceDialog;
int return_code = hotkey_dialog->exec();
auto key_sequence = hotkey_dialog->GetSequence(); const int return_code = hotkey_dialog->exec();
const auto key_sequence = hotkey_dialog->GetSequence();
if (return_code == QDialog::Rejected || key_sequence.isEmpty()) if (return_code == QDialog::Rejected || key_sequence.isEmpty()) {
return; return;
}
if (IsUsedKey(key_sequence) && key_sequence != QKeySequence(previous_key.toString())) { if (IsUsedKey(key_sequence) && key_sequence != QKeySequence(previous_key.toString())) {
QMessageBox::critical(this, tr("Error in inputted key"), QMessageBox::critical(this, tr("Error in inputted key"),