[cfe-dev] Code completion: remove type scopes?

Sam McCall via cfe-dev cfe-dev at lists.llvm.org
Wed Nov 8 11:21:31 PST 2017


Indeed, sorry I missed that. Thanks!

On Nov 8, 2017 19:26, "Alex L" <arphaman at gmail.com> wrote:

>
> Hi Sam,
>
> I believe that was already fixed in r317677 (https://reviews.llvm.org/
> D38538).
>
> Cheers,
> Alex
>
> On 8 November 2017 at 10:15, Sam McCall <sammccall at google.com> wrote:
>
>> I'm experimenting with how completions are displayed in clangd, and
>> trying to reduce the noise, particularly for templates.
>>
>> e.g. this example
>>   std::vector<int> X;
>>   X.push_back|
>> currently produces
>>   void push_back(std::vector<int, std::allocator<int> >::value_type &&__x)
>> and i'd prefer
>>   void push_back(value_type &&__x)
>>
>> I can get this by enabling PrintingPolicy.SuppressScope in
>> SemaCodeComplete.
>> My initial thought was just to do this locally in clangd, but maybe:
>>  - there are known cases where this falls down terribly
>>  - this is just better and we should turn it on for other users (xcode?)
>> too
>>
>> The global change only breaks a few tests, in the obvious way: either a
>> return type or parameter of a completion is no longer qualified.
>>
>> (There are other things that are noisy here -- int would be better than
>> value_type -- but that's a different question)
>>
>> Cheers, Sam
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20171108/0d71cc07/attachment-0001.html>


More information about the cfe-dev mailing list