[PATCH] D67825: [AST] Extract Decl::printNestedNameSpecifier helper from Decl::printQualifiedName

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 20 05:23:24 PDT 2019


ilya-biryukov added a comment.

In D67825#1676528 <https://reviews.llvm.org/D67825#1676528>, @kadircet wrote:

> LGTM, I know it is trivial, but some more unittests wouldn't hurt anyone :D


Yeah, good point, I'll add a few besides tests for `printQualifiedName`.

> Also could you update the summary to mention "doing string manipulations in the presence of template arguments is hard?"

Done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67825/new/

https://reviews.llvm.org/D67825





More information about the cfe-commits mailing list