Code::Blocks Forums

Developer forums (C::B DEVELOPMENT STRICTLY!) => Development => Topic started by: Loaden on March 27, 2010, 05:32:40 pm

Title: Patch for support 'step in' and 'step out' when use CDB
Post by: Loaden on March 27, 2010, 05:32:40 pm

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

Code

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]

Title: Re: Patch for support 'step in' and 'step out' when use CDB
Post by: killerbot on March 27, 2010, 07:50:29 pm

Martin, will you apply this already ??

Title: Re: Patch for support 'step in' and 'step out' when use CDB
Post by: MortenMacFly on March 28, 2010, 03:37:23 pm

Quote from: killerbot on March 27, 2010, 07:50:29 pm

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.

Title: Re: Patch for support 'step in' and 'step out' when use CDB
Post by: Loaden on March 29, 2010, 07:23:29 am

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.