Revert: linespec.c (iterate_name_matcher): Fix arguments to symbol_name_cmp.
authorDoug Evans <xdje42@gmail.com>
Fri, 5 Dec 2014 08:53:56 +0000 (00:53 -0800)
committerDoug Evans <xdje42@gmail.com>
Fri, 5 Dec 2014 09:04:07 +0000 (01:04 -0800)
commit642a8d806780150b62e3a0381a8d995212bdf442
treee2008c67d8eac2912bbf51d35b2d6525e8bdda64
parentddb9f679fa9f54633442c46574b8e5b0ca45a786
Revert: linespec.c (iterate_name_matcher): Fix arguments to symbol_name_cmp.

This patch causes regressions in ada's operator_bp.exp test.
That's because it uses wild_match which expects arguments in
the original order.

There is still a bug here.  It's hard to see because either minsyms
save the day, or the needed symtab gets expanded before linespecs
need it because of the call to cp_canonicalize_string_no_typedefs
in linespec.c:find_linespec_symbols.
But if you disable both of those things, then the bug is visible.

bash$ ./gdb -D ./data-directory testsuite/gdb.cp/anon-ns
(gdb) b doit(void)
Function "doit(void)" not defined.

gdb/ChangeLog:

Revert:
PR symtab/17602
* linespec.c (iterate_name_matcher): Fix arguments to symbol_name_cmp.
gdb/ChangeLog
gdb/linespec.c