[PINGv2][PATCH v3 0/2] Improve handling of using directives

[Bruno Larsen <blarsen@redhat.com>]

Ping!
On 07/12/2022 14:53, Bruno Larsen wrote:
> Ping!
>
>
> -- 
> Cheers,
> Bruno
>
> On 22/11/2022 12:33, Bruno Larsen wrote:
>> This series tries to fix two of the main issues I saw when trying to get
>> gdb.cp/nsusing.exp working with clang. The first issue was that GDB
>> wouldn't care about when the 'using' directive happened in the code,
>> even if we were stopped before it, it was considered valid. The second
>> was that GDB would find the first reasonable variable in the imported
>> declarations and leave early, not caring about whether that variable was
>> ambiguous or not. Each of my patches fixes one of those issues.
>>
>> Changelog for v3:
>>   * changed map key to use std::string, so we get an ordered output.
>>   * Improved documentation on some spots
>>   * Fix bug where gdb could return a partial list of the ambiguous 
>> symbols
>>
>> Changelog for v2:
>>   * factored out some code to avoid unnecessary repetition.
>>   * made it so ambiguous variables are explicitly reported
>>   * fixed formatting issues.
>>
>> Bruno Larsen (2):
>>    gdb/c++: validate 'using' directives based on the current line
>>    gdb/c++: Detect ambiguous variables in imported namespaces
>>
>>   gdb/cp-namespace.c               | 99 ++++++++++++++++++++++++--------
>>   gdb/dwarf2/read.c                | 30 +++++++++-
>>   gdb/namespace.c                  | 25 ++++++++
>>   gdb/namespace.h                  | 16 +++++-
>>   gdb/testsuite/gdb.cp/nsusing.cc  |  3 +-
>>   gdb/testsuite/gdb.cp/nsusing.exp | 25 +++++++-
>>   6 files changed, 167 insertions(+), 31 deletions(-)


-- 
Cheers,
Bruno