From e11c72c7e4879894b9711b5c0b8247c20c6050f6 Mon Sep 17 00:00:00 2001 From: Gary Benson Date: Sat, 31 Jan 2015 14:48:29 -0800 Subject: [PATCH] Build list of completions as symbol tables are expanded. This commit makes default_make_symbol_completion_list_break_on build the list of completions as it expands the necessary symbol tables, rather than expanding all necessary symbol tables first and then building the completion lists second. This allows for the early termination of symbol table expansion if required. gdb/ChangeLog: * symtab.c (struct add_name_data) : New field. Updated comments. (add_symtab_completions): New function. (symtab_expansion_callback): Likewise. (default_make_symbol_completion_list_break_on): Set datum.code. Move minimal symbol scan before calling expand_symtabs_matching. Scan known primary symtabs for externs and statics before calling expand_symtabs_matching. Pass symtab_expansion_callback as expansion_notify argument to expand_symtabs_matching. Do not scan primary symtabs for externs and statics after calling expand_symtabs_matching. --- gdb/ChangeLog | 14 +++++++ gdb/symtab.c | 105 +++++++++++++++++++++++++++++++------------------- 2 files changed, 80 insertions(+), 39 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index d97b9c4e3a2..ad36a702419 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,17 @@ +2015-01-31 Gary Benson + + * symtab.c (struct add_name_data) : New field. + Updated comments. + (add_symtab_completions): New function. + (symtab_expansion_callback): Likewise. + (default_make_symbol_completion_list_break_on): Set datum.code. + Move minimal symbol scan before calling expand_symtabs_matching. + Scan known primary symtabs for externs and statics before calling + expand_symtabs_matching. Pass symtab_expansion_callback as + expansion_notify argument to expand_symtabs_matching. Do not scan + primary symtabs for externs and statics after calling + expand_symtabs_matching. + 2015-01-31 Gary Benson * symfile.h (expand_symtabs_exp_notify_ftype): New typedef. diff --git a/gdb/symtab.c b/gdb/symtab.c index 47aea7664b8..abff265db28 100644 --- a/gdb/symtab.c +++ b/gdb/symtab.c @@ -5169,15 +5169,19 @@ completion_list_add_fields (struct symbol *sym, const char *sym_text, } } -/* Type of the user_data argument passed to add_macro_name or - symbol_completion_matcher. The contents are simply whatever is - needed by completion_list_add_name. */ +/* Type of the user_data argument passed to add_macro_name, + symbol_completion_matcher and symtab_expansion_callback. */ + struct add_name_data { + /* Arguments required by completion_list_add_name. */ const char *sym_text; int sym_text_len; const char *text; const char *word; + + /* Extra argument required for add_symtab_completions. */ + enum type_code code; }; /* A callback used with macro_for_each and macro_for_each_in_scope. @@ -5205,6 +5209,50 @@ symbol_completion_matcher (const char *name, void *user_data) return compare_symbol_name (name, datum->sym_text, datum->sym_text_len); } +/* Add matching symbols from SYMTAB to the current completion list. */ + +static void +add_symtab_completions (struct compunit_symtab *cust, + const char *sym_text, int sym_text_len, + const char *text, const char *word, + enum type_code code) +{ + struct symbol *sym; + const struct block *b; + struct block_iterator iter; + int i; + + for (i = GLOBAL_BLOCK; i <= STATIC_BLOCK; i++) + { + QUIT; + b = BLOCKVECTOR_BLOCK (COMPUNIT_BLOCKVECTOR (cust), i); + ALL_BLOCK_SYMBOLS (b, iter, sym) + { + if (code == TYPE_CODE_UNDEF + || (SYMBOL_DOMAIN (sym) == STRUCT_DOMAIN + && TYPE_CODE (SYMBOL_TYPE (sym)) == code)) + COMPLETION_LIST_ADD_SYMBOL (sym, + sym_text, sym_text_len, + text, word); + } + } +} + +/* Callback to add completions to the current list when symbol tables + are expanded during completion list generation. */ + +static void +symtab_expansion_callback (struct compunit_symtab *symtab, + void *user_data) +{ + struct add_name_data *datum = (struct add_name_data *) user_data; + + add_symtab_completions (symtab, + datum->sym_text, datum->sym_text_len, + datum->text, datum->word, + datum->code); +} + VEC (char_ptr) * default_make_symbol_completion_list_break_on (const char *text, const char *word, @@ -5305,17 +5353,12 @@ default_make_symbol_completion_list_break_on (const char *text, datum.sym_text_len = sym_text_len; datum.text = text; datum.word = word; - - /* Look through the partial symtabs for all symbols which begin - by matching SYM_TEXT. Expand all CUs that you find to the list. - The real names will get added by COMPLETION_LIST_ADD_SYMBOL below. */ - expand_symtabs_matching (NULL, symbol_completion_matcher, NULL, - ALL_DOMAIN, &datum); + datum.code = code; /* At this point scan through the misc symbol vectors and add each symbol you find to the list. Eventually we want to ignore anything that isn't a text symbol (everything else will be - handled by the psymtab code above). */ + handled by the psymtab code below). */ if (code == TYPE_CODE_UNDEF) { @@ -5330,6 +5373,19 @@ default_make_symbol_completion_list_break_on (const char *text, } } + /* Add completions for all currently loaded symbol tables. */ + ALL_COMPUNITS (objfile, cust) + add_symtab_completions (cust, sym_text, sym_text_len, text, word, + code); + + /* Look through the partial symtabs for all symbols which begin + by matching SYM_TEXT. Expand all CUs that you find to the list. + symtab_expansion_callback is called for each expanded symtab, + causing those symtab's completions to be added to the list too. */ + expand_symtabs_matching (NULL, symbol_completion_matcher, + symtab_expansion_callback, ALL_DOMAIN, + &datum); + /* Search upwards from currently selected frame (so that we can complete on local vars). Also catch fields of types defined in this places which match our text string. Only complete on types @@ -5379,35 +5435,6 @@ default_make_symbol_completion_list_break_on (const char *text, completion_list_add_fields (sym, sym_text, sym_text_len, text, word); } - /* Go through the symtabs and check the externs and statics for - symbols which match. */ - - ALL_COMPUNITS (objfile, cust) - { - QUIT; - b = BLOCKVECTOR_BLOCK (COMPUNIT_BLOCKVECTOR (cust), GLOBAL_BLOCK); - ALL_BLOCK_SYMBOLS (b, iter, sym) - { - if (code == TYPE_CODE_UNDEF - || (SYMBOL_DOMAIN (sym) == STRUCT_DOMAIN - && TYPE_CODE (SYMBOL_TYPE (sym)) == code)) - COMPLETION_LIST_ADD_SYMBOL (sym, sym_text, sym_text_len, text, word); - } - } - - ALL_COMPUNITS (objfile, cust) - { - QUIT; - b = BLOCKVECTOR_BLOCK (COMPUNIT_BLOCKVECTOR (cust), STATIC_BLOCK); - ALL_BLOCK_SYMBOLS (b, iter, sym) - { - if (code == TYPE_CODE_UNDEF - || (SYMBOL_DOMAIN (sym) == STRUCT_DOMAIN - && TYPE_CODE (SYMBOL_TYPE (sym)) == code)) - COMPLETION_LIST_ADD_SYMBOL (sym, sym_text, sym_text_len, text, word); - } - } - /* Skip macros if we are completing a struct tag -- arguable but usually what is expected. */ if (current_language->la_macro_expansion == macro_expansion_c -- 2.30.2