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

Richard Smith richard at metafoo.co.uk
Thu May 29 20:05:10 PDT 2014


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/20140529/862c57cf/attachment.html>


More information about the cfe-commits mailing list