steveire marked an inline comment as done. steveire added inline comments.
================ Comment at: include/clang/AST/ASTDumperUtils.h:96 + +struct TextChildDumper { + raw_ostream &OS; ---------------- steveire wrote: > aaron.ballman wrote: > > I'm not sold on the name for this class. It's a bit too generic to > > understand what it does. How about `ASTDumpLayoutFormatter` (and > > `ASTDumpNodeFormatter` for the node dumper)? > I'll rename the class to `ASTTextTreeStructure`. Or rather, `TextTreeStructure`. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55188/new/ https://reviews.llvm.org/D55188 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits