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