Patch #3370 2012-11-15 20:48

bat

Debugger/CDB Handle parameters on application start
Download
3370-Debugger_CDB_H.patch (404 bytes)
Category
Plugin::Refinement
Status
Closed
Close date
2012-12-28 21:51
Assigned to
tpetrov
Index: cdb_driver.cpp
===================================================================
--- cdb_driver.cpp    (revision 8484)
+++ cdb_driver.cpp    (working copy)
@@ -86,6 +86,12 @@
     else
         cmd << m_Target->GetParentProject()->GetBasePath() << wxT("/") << debuggee;
 
+    if (!userArguments.IsEmpty())
+    {
+        cmd << _T(' ');
+        cmd << userArguments;
+    }
+
     return cmd;
 }
 
bat 2012-11-15 20:50

Use "Settings" "Debugger", "Arguments" field. Not really good, should use "Project" "Set Program's arguments" I think ...

But waiting for redesign of debugger it should do the job

tpetrov 2012-11-21 16:12

Can you give better explanation, what bug is being fixed by this patch?

bat 2012-11-21 18:25

Hello,

When debugger is CDB, you can't start program to be debugged with a parameter. For example I have a program : test.exe, I have to test it with "test .exe -myparam". To do this parameter should be added at end of debugger command line. Here it is something as "cdb.exe -cdbparams ... test.exe -myparam"

This patch is to give half a solution to this problem. Because it add parameters, but it add parameters that come from "Settings" "Debugger", "Arguments" field.

I think it would be better if these parameter come from "Project" "Set Program's arguments", but it need a change in debugger plugin interface I think ?

tpetrov 2012-11-21 21:40

No, it doesn't, just see how it is done for GDB and provide a patch...

If you don't know there is new CDB debugger plugin in the works. See here for details: http://forums.codeblocks.org/index.php/topic,16885.0.html

bat 2012-11-22 11:28

Ok, I was not sure that "new" CDB plugin was not already in trunk. I will check all this stuff before any other modifications

Thanks !

bat 2012-12-24 09:17

Ok, I've checked new version of CDB plugin, not released for now. So this patch is already adressed in new development and should be closed