This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 2/3 v2] Interleave completion list building with symbol table expansion
- From: Doug Evans <xdje42 at gmail dot com>
- To: Gary Benson <gbenson at redhat dot com>
- Cc: gdb-patches at sourceware dot org, Eli Zaretskii <eliz at gnu dot org>
- Date: Fri, 05 Dec 2014 14:58:03 -0800
- Subject: Re: [PATCH 2/3 v2] Interleave completion list building with symbol table expansion
- Authentication-results: sourceware.org; auth=none
- References: <1417094168-25868-1-git-send-email-gbenson at redhat dot com> <1417094168-25868-3-git-send-email-gbenson at redhat dot com>
Gary Benson <gbenson@redhat.com> writes:
> 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) <code>: 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.
LGTM