NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

Looks great to me and pretty trivial, should somebody who's more familiar with 
this code take a look?



================
Comment at: clang/lib/AST/StmtPrinter.cpp:72
     PrintingPolicy Policy;
+    std::string NewlineSymbol;
     const ASTContext *Context;
----------------
This variable is used so many times, maybe give it a shorter name?


Repository:
  rC Clang

https://reviews.llvm.org/D51824



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D51824: StmtPrint... Artem Dergachev via Phabricator via cfe-commits

Reply via email to