From 6b5a76158feca98428a70bab266358fb82fb270d Mon Sep 17 00:00:00 2001 From: Rene Gollent Date: Tue, 13 Dec 2011 15:52:05 -0500 Subject: [PATCH] Add TODO note. --- src/apps/debugger/ThreadHandler.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/apps/debugger/ThreadHandler.cpp b/src/apps/debugger/ThreadHandler.cpp index b2262f3411..99f14b5f95 100644 --- a/src/apps/debugger/ThreadHandler.cpp +++ b/src/apps/debugger/ThreadHandler.cpp @@ -481,7 +481,7 @@ ThreadHandler::_DoStepOver(CpuState* cpuState) TRACE_CONTROL(" subroutine call -- installing breakpoint at address " "%#llx\n", info.Address() + info.Size()); - + if (_InstallTemporaryBreakpoint(info.Address() + info.Size()) != B_OK) return false; @@ -565,6 +565,10 @@ ThreadHandler::_HandleBreakpointHitStep(CpuState* cpuState) _StepFallback(); return true; } + + // TODO: this needs to handle recursive cases properly as + // STEP_OUT does, we need to have exited the statement + // *and* be back in the call frame we started in. return false; case STEP_INTO: