configure_hotkeys: Avoid dialog memory leak within Configure()
Without a parent, this dialog won't have its memory freed when it happens to get destroyed.
This commit is contained in:
parent
dbf13f8169
commit
b47c0c8a80
|
@ -74,10 +74,10 @@ void ConfigureHotkeys::Configure(QModelIndex index) {
|
||||||
auto* const model = ui->hotkey_list->model();
|
auto* const model = ui->hotkey_list->model();
|
||||||
const auto previous_key = model->data(index);
|
const auto previous_key = model->data(index);
|
||||||
|
|
||||||
auto* const hotkey_dialog = new SequenceDialog;
|
SequenceDialog hotkey_dialog;
|
||||||
|
|
||||||
const int return_code = hotkey_dialog->exec();
|
const int return_code = hotkey_dialog.exec();
|
||||||
const auto key_sequence = hotkey_dialog->GetSequence();
|
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;
|
||||||
}
|
}
|
||||||
|
|
Reference in New Issue