public inbox for gdb-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/23444] Syntax error for watch -l and print
[not found] <bug-23444-4717@http.sourceware.org/bugzilla/>
@ 2021-03-03 19:04 ` autkin at undo dot io
2021-03-04 16:36 ` autkin at undo dot io
` (4 subsequent siblings)
5 siblings, 0 replies; 6+ messages in thread
From: autkin at undo dot io @ 2021-03-03 19:04 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=23444
--- Comment #4 from Andrey Utkin <autkin at undo dot io> ---
Created attachment 13280
--> https://sourceware.org/bugzilla/attachment.cgi?id=13280&action=edit
a prototype fix
This patch fixes the immediate issue with `watch -l`.
This doesn't fix `print *(NS::T *) field` which fails the same way as before;
only `print *(::NS::T *) field` works. I haven't investigated this deeply
enough to be sure about whether it should work at all.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/23444] Syntax error for watch -l and print
[not found] <bug-23444-4717@http.sourceware.org/bugzilla/>
2021-03-03 19:04 ` [Bug c++/23444] Syntax error for watch -l and print autkin at undo dot io
@ 2021-03-04 16:36 ` autkin at undo dot io
2021-03-05 13:03 ` autkin at undo dot io
` (3 subsequent siblings)
5 siblings, 0 replies; 6+ messages in thread
From: autkin at undo dot io @ 2021-03-04 16:36 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=23444
--- Comment #5 from Andrey Utkin <autkin at undo dot io> ---
Unfortunately my above patch breaks putting watchpoints of elementary types.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/23444] Syntax error for watch -l and print
[not found] <bug-23444-4717@http.sourceware.org/bugzilla/>
2021-03-03 19:04 ` [Bug c++/23444] Syntax error for watch -l and print autkin at undo dot io
2021-03-04 16:36 ` autkin at undo dot io
@ 2021-03-05 13:03 ` autkin at undo dot io
2021-03-05 13:03 ` autkin at undo dot io
` (2 subsequent siblings)
5 siblings, 0 replies; 6+ messages in thread
From: autkin at undo dot io @ 2021-03-05 13:03 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=23444
--- Comment #6 from Andrey Utkin <autkin at undo dot io> ---
Created attachment 13290
--> https://sourceware.org/bugzilla/attachment.cgi?id=13290&action=edit
extended test
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/23444] Syntax error for watch -l and print
[not found] <bug-23444-4717@http.sourceware.org/bugzilla/>
` (2 preceding siblings ...)
2021-03-05 13:03 ` autkin at undo dot io
@ 2021-03-05 13:03 ` autkin at undo dot io
2021-03-12 19:18 ` tromey at sourceware dot org
2021-03-13 0:37 ` autkin at undo dot io
5 siblings, 0 replies; 6+ messages in thread
From: autkin at undo dot io @ 2021-03-05 13:03 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=23444
--- Comment #7 from Andrey Utkin <autkin at undo dot io> ---
There are more problems than that.
See my test file attached.
Putting a location watchpoint on dm__t2, gl__ns_t2, gl__cc_ns_t2 reproduces the
original "syntax error" problem.
But some others: dm__t, dm__ns_t, dm__ns_ns_t, gl__ns_ns_t show a different
error: No type "NS" within class or namespace "NS::NS".
Which is silly for variables which are validly defined in a well-formed,
compilable and executable program.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/23444] Syntax error for watch -l and print
[not found] <bug-23444-4717@http.sourceware.org/bugzilla/>
` (3 preceding siblings ...)
2021-03-05 13:03 ` autkin at undo dot io
@ 2021-03-12 19:18 ` tromey at sourceware dot org
2021-03-13 0:37 ` autkin at undo dot io
5 siblings, 0 replies; 6+ messages in thread
From: tromey at sourceware dot org @ 2021-03-12 19:18 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=23444
Tom Tromey <tromey at sourceware dot org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |tromey at sourceware dot org
--- Comment #8 from Tom Tromey <tromey at sourceware dot org> ---
Hi. Thank you for the patch.
GDB generally does patch review via the mailing list.
Could you send your patch there?
Here are some instructions on how to do so:
https://sourceware.org/gdb/wiki/ContributionChecklist
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/23444] Syntax error for watch -l and print
[not found] <bug-23444-4717@http.sourceware.org/bugzilla/>
` (4 preceding siblings ...)
2021-03-12 19:18 ` tromey at sourceware dot org
@ 2021-03-13 0:37 ` autkin at undo dot io
5 siblings, 0 replies; 6+ messages in thread
From: autkin at undo dot io @ 2021-03-13 0:37 UTC (permalink / raw)
To: gdb-prs
https://sourceware.org/bugzilla/show_bug.cgi?id=23444
--- Comment #9 from Andrey Utkin <autkin at undo dot io> ---
(In reply to Tom Tromey from comment #8)
> Hi. Thank you for the patch.
> GDB generally does patch review via the mailing list.
> Could you send your patch there?
Hi, that patch doesn't work.
If I get something working, I will send it.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-03-13 0:37 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-23444-4717@http.sourceware.org/bugzilla/>
2021-03-03 19:04 ` [Bug c++/23444] Syntax error for watch -l and print autkin at undo dot io
2021-03-04 16:36 ` autkin at undo dot io
2021-03-05 13:03 ` autkin at undo dot io
2021-03-05 13:03 ` autkin at undo dot io
2021-03-12 19:18 ` tromey at sourceware dot org
2021-03-13 0:37 ` autkin at undo dot io
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).