[RFA] c++/12273

[Keith Seitz <keiths at redhat dot com>]

Hi,

This patch deals with what would appear to be regression with linespecs. 
[It's not really a regression, just an exposure of a long-standing bug.]

The short of this is that breakpoints cannot be put onto 
functions/methods with no debug information if the name is qualified by 
a class name. This happens because decode_compound, unlike 
decode_variable, does not search the minimal symbols.

Tested on x86_64-linux. If there is time for this for 7.2.1, I would 
suggest maintainers consider including it.

Comments?
Keith

ChangeLog
2010-12-14  Keith Seitz  <keiths@redhat.com>

	* linespec.c (locate_first_half): Keep overload information, too.
	(decode_compound): Use a string to represent break characters
	to escape the loop.
	If P points to a break character, do not increment it.
	For C++ and Java, keep overload information and relevant keywords.
	If we cannot find a symbol, search the minimal symbols.


testsuite/ChangeLog
2010-12-14  Keith Seitz  <keiths@redhat.com>

	PR c++/12273
	* gdb.cp/pr12273.exp: New test.
	* gdb.cp/pr12273.cc: New file.

Attachment: pr12273.patch
Description: Text document