From f777c01c5d2a23632b592de679445192c892fc66 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?L=C3=A9o=20Lam?= Date: Fri, 28 Jul 2017 12:37:21 +0800 Subject: [PATCH] WX/Debugger: Clean up event handling Boot to Pause and Automatic Start are not JIT or CPU menu items, so they are supposed to be handled by OnCPUMode. --- Source/Core/DolphinWX/Debugger/CodeWindow.cpp | 8 ++------ Source/Core/DolphinWX/Debugger/CodeWindow.h | 3 +++ Source/Core/DolphinWX/Debugger/CodeWindowFunctions.cpp | 10 ++++++++++ Source/Core/DolphinWX/Globals.h | 5 +++-- 4 files changed, 18 insertions(+), 8 deletions(-) diff --git a/Source/Core/DolphinWX/Debugger/CodeWindow.cpp b/Source/Core/DolphinWX/Debugger/CodeWindow.cpp index c1e10e248e..7c4c6de534 100644 --- a/Source/Core/DolphinWX/Debugger/CodeWindow.cpp +++ b/Source/Core/DolphinWX/Debugger/CodeWindow.cpp @@ -123,6 +123,8 @@ CCodeWindow::CCodeWindow(CFrame* parent, wxWindowID id, const wxPoint& position, Bind(wxEVT_MENU, &CCodeWindow::OnJitMenu, this, IDM_CLEAR_CODE_CACHE, IDM_SEARCH_INSTRUCTION); Bind(wxEVT_MENU, &CCodeWindow::OnSymbolsMenu, this, IDM_CLEAR_SYMBOLS, IDM_PATCH_HLE_FUNCTIONS); Bind(wxEVT_MENU, &CCodeWindow::OnProfilerMenu, this, IDM_PROFILE_BLOCKS, IDM_WRITE_PROFILE); + Bind(wxEVT_MENU, &CCodeWindow::OnBootToPauseSelected, this, IDM_BOOT_TO_PAUSE); + Bind(wxEVT_MENU, &CCodeWindow::OnAutomaticStartSelected, this, IDM_AUTOMATIC_START); // Toolbar Bind(wxEVT_MENU, &CCodeWindow::OnCodeStep, this, IDM_STEP, IDM_GOTOPC); @@ -483,12 +485,6 @@ void CCodeWindow::OnCPUMode(wxCommandEvent& event) case IDM_INTERPRETER: PowerPC::SetMode(event.IsChecked() ? PowerPC::CoreMode::Interpreter : PowerPC::CoreMode::JIT); break; - case IDM_BOOT_TO_PAUSE: - SConfig::GetInstance().bBootToPause = event.IsChecked(); - return; - case IDM_AUTOMATIC_START: - SConfig::GetInstance().bAutomaticStart = event.IsChecked(); - return; case IDM_JIT_OFF: SConfig::GetInstance().bJITOff = event.IsChecked(); break; diff --git a/Source/Core/DolphinWX/Debugger/CodeWindow.h b/Source/Core/DolphinWX/Debugger/CodeWindow.h index 76024eb1d4..8a43ebcedf 100644 --- a/Source/Core/DolphinWX/Debugger/CodeWindow.h +++ b/Source/Core/DolphinWX/Debugger/CodeWindow.h @@ -133,6 +133,9 @@ private: void OnJitMenu(wxCommandEvent& event); void OnProfilerMenu(wxCommandEvent& event); + void OnBootToPauseSelected(wxCommandEvent& event); + void OnAutomaticStartSelected(wxCommandEvent& event); + void OnSymbolListChange(wxCommandEvent& event); void OnCallstackListChange(wxCommandEvent& event); void OnCallersListChange(wxCommandEvent& event); diff --git a/Source/Core/DolphinWX/Debugger/CodeWindowFunctions.cpp b/Source/Core/DolphinWX/Debugger/CodeWindowFunctions.cpp index 7bc7f07210..7214ec26b9 100644 --- a/Source/Core/DolphinWX/Debugger/CodeWindowFunctions.cpp +++ b/Source/Core/DolphinWX/Debugger/CodeWindowFunctions.cpp @@ -505,6 +505,16 @@ void CCodeWindow::OnChangeFont(wxCommandEvent& event) // TODO: Send event to all panels that tells them to reload the font when it changes. } +void CCodeWindow::OnBootToPauseSelected(wxCommandEvent& event) +{ + SConfig::GetInstance().bBootToPause = event.IsChecked(); +} + +void CCodeWindow::OnAutomaticStartSelected(wxCommandEvent& event) +{ + SConfig::GetInstance().bAutomaticStart = event.IsChecked(); +} + // Toggle windows wxPanel* CCodeWindow::GetUntypedPanel(int id) const diff --git a/Source/Core/DolphinWX/Globals.h b/Source/Core/DolphinWX/Globals.h index 0ade18f159..d85d5add8e 100644 --- a/Source/Core/DolphinWX/Globals.h +++ b/Source/Core/DolphinWX/Globals.h @@ -208,8 +208,6 @@ enum // -------------------- // CPU Mode IDM_INTERPRETER, - IDM_AUTOMATIC_START, - IDM_BOOT_TO_PAUSE, IDM_JIT_NO_BLOCK_CACHE, IDM_JIT_NO_BLOCK_LINKING, // JIT IDM_JIT_OFF, @@ -223,7 +221,10 @@ enum IDM_JIT_FP_OFF, IDM_JIT_P_OFF, IDM_JIT_SR_OFF, + IDM_FONT_PICKER, + IDM_AUTOMATIC_START, + IDM_BOOT_TO_PAUSE, // Symbols IDM_CLEAR_SYMBOLS,