Patch for support 'step in' and 'step out' when use CDB

[Loaden]

Hi, I make a patch for support 'step in' and 'step out', when use CDB debugger.
Now, it's works well.

Code Select Expand

Index: src/plugins/debuggergdb/cdb_driver.cpp
===================================================================
--- src/plugins/debuggergdb/cdb_driver.cpp (revision 6196)
+++ src/plugins/debuggergdb/cdb_driver.cpp (working copy)
@@ -147,14 +147,15 @@
void CDB_driver::StepIn()
{
     ResetCursor();
-//    QueueCommand(new DebuggerCmd(this, _T("step")));
-    NOT_IMPLEMENTED();
+    QueueCommand(new DebuggerCmd(this, _T("t")));
+    Step();
}

void CDB_driver::StepOut()
{
     ResetCursor();
-    NOT_IMPLEMENTED();
+    QueueCommand(new DebuggerCmd(this, _T("gu")));
+    Step();
}

void CDB_driver::Backtrace()



[attachment deleted by admin]

[killerbot]

Martin, will you apply this already ??

[MortenMacFly]

Martin, will you apply this already ??

I am not sure. CDB has changed a lot recently. I don't know if this works reliable with all CDB's provided with the major Windows SDK's (XP/Vista/7/ServerXXX).

In addition it should be done in the debugger branch as this will be the next step when it comes to the debugger upgrades. If you mean that I should apply it in this branch, then I am happy to do this after checking back with obfuscated.

[Loaden]

I test this use VC9 and VC10, pass all when use the latest version of WinDbg.
CDB(WinDbg) is very stable, I think the other versions will be able to work too.