[cfe-commits] r82293 - in /cfe/trunk: include/clang/Sema/CodeCompleteConsumer.h lib/Sema/CodeCompleteConsumer.cpp test/CodeCompletion/functions.cpp

[Chris Lattner]

On Sep 21, 2009, at 3:44 PM, Chris Lattner wrote:
>> I like it. We currently provide qualification when the name in the
>> base class (or an outer context) is *hidden*, because in that case
>> we show the qualification is needed for us to have any chance of
>> doing the right thing.
>
> Right, I'd expect to see something like this in the 'popup' for P->
>
> field1
> field2         - Base class 'foo'
> Base::field2   - Base class 'Base', hidden by 'foo::field2'
> field3
>
> etc, where field1/field3 are direct fields in *p.  This way you don't
> get noise when dealing with simple structs with no base classes, etc.

Actually, after thinking about it more, instead of being a new Chunk  
kind, maybe this is just an extra string that is one per each result?

-Chris