public inbox for gdb@sourceware.org
 help / color / mirror / Atom feed
* [mingw,gdb CVS HEAD] can't set breakpoint
@ 2012-07-27  4:04 asmwarrior
  2012-07-27  5:02 ` asmwarrior
  0 siblings, 1 reply; 3+ messages in thread
From: asmwarrior @ 2012-07-27  4:04 UTC (permalink / raw)
  To: gdb

I just see this message: (when debugging under Code::Blocks)

[debug]> break "E:/code/cb/test_code/Toggle/MyWXMain.cpp:66"
[debug]Function "66" not defined in "E:/code/cb/test_code/Toggle/MyWXMain.cpp".
[debug]Breakpoint 2 ("E:/code/cb/test_code/Toggle/MyWXMain.cpp:66") pending.

Why this command is regard as "Function 66".
When the app started, it can't hit the bp.

It works OK with gdb cvs 2012-07-25.

Yuanhui Zhang


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [mingw,gdb CVS HEAD] can't set breakpoint
  2012-07-27  4:04 [mingw,gdb CVS HEAD] can't set breakpoint asmwarrior
@ 2012-07-27  5:02 ` asmwarrior
  2012-07-29 19:01   ` Keith Seitz
  0 siblings, 1 reply; 3+ messages in thread
From: asmwarrior @ 2012-07-27  5:02 UTC (permalink / raw)
  To: gdb

On 2012-7-27 12:04, asmwarrior wrote:
> I just see this message: (when debugging under Code::Blocks)
>
> [debug]> break "E:/code/cb/test_code/Toggle/MyWXMain.cpp:66"
> [debug]Function "66" not defined in "E:/code/cb/test_code/Toggle/MyWXMain.cpp".
> [debug]Breakpoint 2 ("E:/code/cb/test_code/Toggle/MyWXMain.cpp:66") pending.
>
> Why this command is regard as "Function 66".
> When the app started, it can't hit the bp.
>
> It works OK with gdb cvs 2012-07-25.
>
> Yuanhui Zhang
>
>

I believe this issue is caused by the last two commits in file "linespec.c".

Revision: 5134bbd001531c5c6d02573d4d7fb86b17a549ec
Author: Keith Seitz
Date: 2012-7-27 0:22:44
Message:
* linespec.c (linespec_lexer_lex_number): The input

is also a valid number if the next character is a comma
	or colon.
----
Modified: gdb/ChangeLog
Modified: gdb/linespec.c

and
Revision: 4643fc6023b9d9aad57359b5624756b880c085d1
Author: Hui Zhu
Date: 2012-7-25 20:26:23
Message:
2012-07-25  Hui Zhu  <hui_zhu@mentor.com>

* linespec.c (linespec_lexer_lex_number): Update comments,
	change the return and add check to make sure the input is
	the decimal numbers.
	(linespec_lexer_lex_one): If linespec_lexer_lex_number return
	false, call linespec_lexer_lex_string.
----
Modified: gdb/ChangeLog
Modified: gdb/linespec.c

I revert these two changes, and the breakpoint command works OK.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [mingw,gdb CVS HEAD] can't set breakpoint
  2012-07-27  5:02 ` asmwarrior
@ 2012-07-29 19:01   ` Keith Seitz
  0 siblings, 0 replies; 3+ messages in thread
From: Keith Seitz @ 2012-07-29 19:01 UTC (permalink / raw)
  To: asmwarrior; +Cc: gdb

On 07/26/2012 10:01 PM, asmwarrior wrote:
> On 2012-7-27 12:04, asmwarrior wrote:
>> I just see this message: (when debugging under Code::Blocks)
>>
>> [debug]> break "E:/code/cb/test_code/Toggle/MyWXMain.cpp:66"
>> [debug]Function "66" not defined in
>> "E:/code/cb/test_code/Toggle/MyWXMain.cpp".
>> [debug]Breakpoint 2 ("E:/code/cb/test_code/Toggle/MyWXMain.cpp:66")
>> pending.

The problem is that the linespec is quoted. Quotation is not needed 
anymore (and even in the case you have sighted, it was never needed in 
any 7.x version of gdb).

I will work on a patch.

Thank you for reporting this.
Keith

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-07-29 19:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-27  4:04 [mingw,gdb CVS HEAD] can't set breakpoint asmwarrior
2012-07-27  5:02 ` asmwarrior
2012-07-29 19:01   ` Keith Seitz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).