diff options
| author | Tom Tromey <tom@tromey.com> | 2021-04-17 09:35:04 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2021-04-17 09:35:06 -0600 |
| commit | 0b7b2c2adff43d39ee3bd39ebd91a6710d9175e4 (patch) | |
| tree | 319e0422ab99a88c82017e660b06e7b0e0f27c41 /gdb/objfiles.h | |
| parent | 90160b57032ce8c2d12864a53036bab2c5b86682 (diff) | |
| download | fsf-binutils-gdb-0b7b2c2adff43d39ee3bd39ebd91a6710d9175e4.zip fsf-binutils-gdb-0b7b2c2adff43d39ee3bd39ebd91a6710d9175e4.tar.gz fsf-binutils-gdb-0b7b2c2adff43d39ee3bd39ebd91a6710d9175e4.tar.bz2 | |
Simplify quick_symbol_functions::map_matching_symbols
quick_symbol_functions::map_matching_symbols is only used by the Ada
code. Currently, it both expands certain psymtabs and then walks over
the full symtabs -- including any already-expanded ones -- calling a
callback.
It appears to work lazily as well, in that if the callback returns
false, iteration stops. However, only the psymtab implementation does
this; the DWARF index implementations are not lazy. It turns out,
though, that the only callback that is ever passed here never returns
false.
This patch simplifies this method by removing the callback. The
method is also renamed. In the new scheme, the caller is responsible
for walking the full symtabs, which removes some redundancy as well.
gdb/ChangeLog
2021-04-17 Tom Tromey <tom@tromey.com>
* psymtab.c (psymbol_functions::expand_matching_symbols): Rename
from map_matching_symbols. Change parameters.
* psympriv.h (struct psymbol_functions) <expand_matching_symbols>:
Rename from map_matching_symbols. Change parameters.
* dwarf2/read.c (struct dwarf2_gdb_index)
<expand_matching_symbols>: Rename from map_matching_symbols.
Change parameters.
(struct dwarf2_debug_names_index) <expand_matching_symbols>:
Rename from map_matching_symbols. Change parameters.
(dwarf2_gdb_index::expand_matching_symbols): Rename from
dw2_map_matching_symbols. Change parameters.
(dwarf2_gdb_index::expand_matching_symbols): Remove old
implementation.
(dwarf2_debug_names_index::expand_matching_symbols): Rename from
map_matching_symbols. Change parameters.
* objfiles.h (struct objfile) <expand_matching_symbols>: Rename
from map_matching_symbols. Change parameters.
* symfile-debug.c (objfile::expand_matching_symbols): Rename from
map_matching_symbols. Change parameters.
* ada-lang.c (map_matching_symbols): New function.
(add_nonlocal_symbols): Update.
Diffstat (limited to 'gdb/objfiles.h')
| -rw-r--r-- | gdb/objfiles.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/gdb/objfiles.h b/gdb/objfiles.h index 90ea49e..5a8a782 100644 --- a/gdb/objfiles.h +++ b/gdb/objfiles.h @@ -621,10 +621,9 @@ public: void expand_symtabs_with_fullname (const char *fullname); /* See quick_symbol_functions. */ - void map_matching_symbols + void expand_matching_symbols (const lookup_name_info &name, domain_enum domain, int global, - gdb::function_view<symbol_found_callback_ftype> callback, symbol_compare_ftype *ordered_compare); /* See quick_symbol_functions. */ |