Debugger: Fixed forbid breakpoints not working properly with "break o…

…n" options
SourMesen · Oct 15, 2024 · ef302de · ef302de
1 parent 3dee924
commit ef302de
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/Core/Debugger/Debugger.cpp b/Core/Debugger/Debugger.cpp
@@ -457,6 +457,7 @@ void Debugger::SleepUntilResume(CpuType sourceCpu, BreakSource source, MemoryOpe
 		//If SleepUntilResume was called outside of ProcessInstruction, keep running
 		return;
 	} else if(IsBreakpointForbidden(source, sourceCpu, operation)) {
+		ClearPendingBreakRequests();
 		return;
 	}
 
@@ -655,6 +656,15 @@ void Debugger::Run()
 	_waitForBreakResume = false;
 }
 
+void Debugger::ClearPendingBreakRequests()
+{
+	for(int i = 0; i <= (int)DebugUtilities::GetLastCpuType(); i++) {
+		if(_debuggers[i].Debugger) {
+			_debuggers[i].Debugger->Run();
+		}
+	}
+}
+
 void Debugger::PauseOnNextFrame()
 {
 	//Use BreakSource::PpuStep to prevent "Run single frame" from triggering the "bring to front on pause" feature

diff --git a/Core/Debugger/Debugger.h b/Core/Debugger/Debugger.h
@@ -99,6 +99,8 @@ class Debugger
 	bool IsBreakOptionEnabled(BreakSource src);
 	template<CpuType type> void SleepOnBreakRequest();
 
+	void ClearPendingBreakRequests();
+
 	bool IsBreakpointForbidden(BreakSource source, CpuType sourceCpu, MemoryOperationInfo* operation);
 
 public: