[PATCH] Take into account PrintingPolicy::SuppressUnwrittenScope in NamedDecl::printQualifiedName.

Volodymyr Sapsai volodymyr.sapsai at gmail.com
Fri May 30 05:34:06 PDT 2014


Please, commit for me.


On Fri, May 30, 2014 at 6:05 AM, Richard Smith <richard at metafoo.co.uk>
wrote:

> Do you have commit access, or would you like me to commit this for you?
>
>
> On Thu, May 29, 2014 at 8:04 PM, Richard Smith <richard at metafoo.co.uk>
> wrote:
>
>> LGTM
>>
>> ================
>> Comment at: lib/AST/Decl.cpp:1307
>> @@ +1306,3 @@
>> +      if (P.SuppressUnwrittenScope &&
>> +          (ND->isAnonymousNamespace() || ND->isInlineNamespace()))
>> +        continue;
>> ----------------
>> Just use `isInline` here since you already know you've got a namespace.
>>
>> http://reviews.llvm.org/D3922
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140530/dc7eb3cd/attachment.html>


More information about the cfe-commits mailing list