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

Reply via email to