Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

break "filename:line" will not work in the futher gdb

<< < (3/6) > >>

ollydbg:

--- Quote from: oBFusCATed on March 26, 2012, 01:40:16 pm ---The quote was added because some people have their projects inside paths containing "code::blocks" and this is regarded as method/function specification.

--- End quote ---
Ok, thanks, here is the new test from command line(under c::b debugger plugin)


--- Code: ---[debug]> break "E:/code/cb/test_code/DebugDLLTest/TestDLL/dllmain1.cpp":29
[debug]No source file named E:/code/cb/test_code/DebugDLLTest/TestDLL/dllmain1.cpp.
[debug]Breakpoint 6 ("E:/code/cb/test_code/DebugDLLTest/TestDLL/dllmain1.cpp":29) pending.
[debug]>>>>>>cb_gdb:

No source file named E:/code/cb/test_code/DebugDLLTest/TestDLL/dllmain1.cpp.
Breakpoint 6 ("E:/code/cb/test_code/DebugDLLTest/TestDLL/dllmain1.cpp":29) pending.
> break "E:/code/cb/test_code/DebugDLLTest/TestDLL/dllmain.cpp":35

[debug]> break "E:/code/cb/test_code/DebugDLLTest/TestDLL/dllmain.cpp":35
[debug]Breakpoint 7 at 0x684c1814: file E:\code\cb\test_code\DebugDLLTest\TestDLL\dllmain.cpp, line 35.
[debug]>>>>>>cb_gdb:

Breakpoint 7 at 0x684c1814: file E:\code\cb\test_code\DebugDLLTest\TestDLL\dllmain.cpp, line 35.


--- End code ---

So, I'm now trying to adjust the reg. :)

oBFusCATed:

--- Quote from: ollydbg on March 26, 2012, 01:50:01 pm ---So, I'm now trying to adjust the reg. :)

--- End quote ---
Good luck...

ollydbg:

--- Quote from: oBFusCATed on March 26, 2012, 02:08:00 pm ---
--- Quote from: ollydbg on March 26, 2012, 01:50:01 pm ---So, I'm now trying to adjust the reg. :)

--- End quote ---
Good luck...

--- End quote ---
Thanks, Done. Here is the new regular expression and patch:

--- Code: ---Index: E:/code/cb/cb_debugger_branch/src/plugins/debuggergdb/gdb_commands.h
===================================================================
--- E:/code/cb/cb_debugger_branch/src/plugins/debuggergdb/gdb_commands.h (revision 7909)
+++ E:/code/cb/cb_debugger_branch/src/plugins/debuggergdb/gdb_commands.h (working copy)
@@ -131,7 +131,7 @@
 // Breakpoint 1 at 0x4013d6: file main.cpp, line 8.
 static wxRegEx reBreakpoint(_T("Breakpoint ([0-9]+) at (0x[0-9A-Fa-f]+)"));
 // Breakpoint 1 ("/home/jens/codeblocks-build/codeblocks-1.0svn/src/plugins/debuggergdb/gdb_commands.h:125) pending.
-static wxRegEx rePendingBreakpoint(_T("Breakpoint ([0-9]+)[ \t]\\(\\\"(.+):([0-9]+)\\)[ \t]pending\\."));
+static wxRegEx rePendingBreakpoint(_T("Breakpoint ([0-9]+)[ \t]\\(\\\"(.+)\\\":([0-9]+)\\)[ \t]pending\\."));
 // Hardware assisted breakpoint 1 at 0x4013d6: file main.cpp, line 8.
 static wxRegEx reHWBreakpoint(_T("Hardware assisted breakpoint ([0-9]+) at (0x[0-9A-Fa-f]+)"));
 // Hardware watchpoint 1: expr
@@ -448,7 +448,7 @@
                         m_Cmd << _T("break ");
                     else
                         m_Cmd << _T("tbreak ");
-                    m_Cmd << _T('"') << out << _T(":") << wxString::Format(_T("%d"), m_BP->line) << _T('"');
+                    m_Cmd << _T('"') << out << _T("\":") << wxString::Format(_T("%d"), m_BP->line);
                 }
                 else if (m_BP->type == DebuggerBreakpoint::bptData)
                 {

--- End code ---

ollydbg:
FYI:
The command:

--- Code: ---break "filename:line"

--- End code ---
is reserved for compatibility.

In the future, the gdb bp pending response will have quote balance.

See:Re: [RFA 1/2] Linespec rewrite (update 2)

ollydbg:
Again. The new linespec feature is now in GDB CVS HEAD, which means it will be official in GDB 7.5 (It also support using filename with spaces). I think we need to adjust the regex a little to support the new pattern and also keep the old GDB compatible. This should be easy.  :)

Navigation

[0] Message Index

[#] Next page

[*] Previous page