Fwd: [PATCH][REVIEW REQUEST] Fix for libclang completion of already declared template friends.

Francisco Lopes francisco.mailing.lists at oblita.com
Wed Mar 19 15:09:19 PDT 2014


---------- Forwarded message ----------
From: Francisco Lopes <francisco.mailing.lists at oblita.com>
Date: 2014-03-14 10:59 GMT-03:00
Subject: Re: [PATCH][REVIEW REQUEST] Fix for libclang completion of already
declared template friends.
To: Richard Smith <richard at metafoo.co.uk>


hi, thanks for the comment but, for example, as I want to add the call to
getCanonicalDecl
for this situation of friends solely, don't I need to check whether it's in
friend name space too?

I'm not sure whether you meant to replace the two first checks, or just the
second.

Regards.



2014-03-13 20:59 GMT-03:00 Richard Smith <richard at metafoo.co.uk>:

Your visibility check seems more complex than necessary. I think this
> should do what you want:
>
> if (ND->getMostRecentDecl()->isInIdentifierNamespace(Decl::IDNS_Ordinary |
> Decl::IDNS_Tag))
>   // visible
>
>
> On Wed, Mar 12, 2014 at 2:12 PM, Francisco Lopes <
> francisco.mailing.lists at oblita.com> wrote:
>
>> Ping
>>
>>
>> 2014-03-07 14:47 GMT-03:00 Francisco Lopes <
>> francisco.mailing.lists at oblita.com>:
>>
>>> Hi,
>>>
>>> attached is a patch that tries to fix libclang bug 13699<http://llvm.org/bugs/show_bug.cgi?id=13699>
>>> .
>>> Please review.
>>>
>>> --
>>> Francisco Lopes
>>>
>>> PS:
>>> I have requested commit access in late 2012 but never made a test commit
>>> or anything.
>>>
>>> At the time I have received from Chris Lattner:
>>> "I'm sorry for the delay, I've been fighting mailing list issues.
>>> Commit after approval access is granted.  Please try a test commit!"
>>>
>>> I'm not sure whether it's still valid.
>>>
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140319/14ae8e75/attachment.html>


More information about the cfe-commits mailing list