fix PR c++/16117
authorTom Tromey <tromey@redhat.com>
Mon, 4 Nov 2013 19:38:10 +0000 (12:38 -0700)
committerTom Tromey <tromey@redhat.com>
Fri, 15 Nov 2013 15:43:14 +0000 (08:43 -0700)
commit805e1f190887b3b7dea3fd157d58bc25effcf688
treee384f11e3c9596419f0ca2588c8e2e90d32c8299
parentad8ecc8114ce17e4f8f118606225f1002d7faec2
fix PR c++/16117

This patch fixes PR c++/16117.

gdb has an extension so that users can use expressions like FILE::NAME
to choose a variable of the given name from the given file.  The bug
is that this extension takes precedence over ordinary C++ expressions
of the same form.  You might think this is merely hypothetical, but
now that C++ headers commonly do not use an extension, it is more
common.

This patch fixes the bug by making two related changes.  First, it
changes gdb to prefer the ordinary C++ meaning of a symbol over the
extended meaning.  Second, it arranges for single-quoting of the
symbol to indicate a preference for the extension.

Built and regtested on x86-64 Fedora 18.
New test case included.

2013-11-15  Tom Tromey  <tromey@redhat.com>

PR c++/16117:
* c-exp.y (lex_one_token): Add "is_quoted_name" argument.
(classify_name): Likewise.  Prefer a field of "this" over a
filename.
(classify_inner_name, yylex): Update.

2013-11-15  Tom Tromey  <tromey@redhat.com>

* gdb.texinfo (Variables): Note gdb rules for ambiguous cases.
Add example.

2013-11-15  Tom Tromey  <tromey@redhat.com>

* gdb.cp/includefile: New file.
* gdb.cp/filename.exp: New file.
* gdb.cp/filename.cc: New file.
gdb/ChangeLog
gdb/c-exp.y
gdb/doc/ChangeLog
gdb/doc/gdb.texinfo
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.cp/filename.cc [new file with mode: 0644]
gdb/testsuite/gdb.cp/filename.exp [new file with mode: 0644]
gdb/testsuite/gdb.cp/includefile [new file with mode: 0644]