From 242bce27a1a1bd000e5c0e5e063be431092169ec Mon Sep 17 00:00:00 2001 From: sowens99 Date: Mon, 27 Sep 2021 14:28:08 -0400 Subject: [PATCH] Do not accept on hotkey syntax errors Previously you could type whatever gibberish you wanted into the formula bar, press OK, and receive a modal syntax error window. Closing the syntax error window would cause the hotkey config window to close as well, and your gibberish would be applied to the hotkey assignment. This commit requires that a syntax error does not occur before accept() is called. --- Source/Core/DolphinQt/Config/Mapping/IOWindow.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/Source/Core/DolphinQt/Config/Mapping/IOWindow.cpp b/Source/Core/DolphinQt/Config/Mapping/IOWindow.cpp index 0deb4f5293..9f3c596463 100644 --- a/Source/Core/DolphinQt/Config/Mapping/IOWindow.cpp +++ b/Source/Core/DolphinQt/Config/Mapping/IOWindow.cpp @@ -515,9 +515,11 @@ void IOWindow::OnDialogButtonPressed(QAbstractButton* button) { ModalMessageBox::warning(this, tr("Error"), tr("The expression contains a syntax error.")); } - - // must be the OK button - accept(); + else + { + // must be the OK button + accept(); + } } void IOWindow::OnDetectButtonPressed()