================
Comment at: lib/AST/StmtDumper.cpp:33
@@ -32,2 +32,3 @@
unsigned IndentLevel;
+ bool NeedNewLine;
----------------
I think it's not immediately clear what cases are handled by this bool - a
comment would help me here...
After reading the whole patch, it looks like this is mainly there for the first
newline... If that's true, I'd probably call it IsFirstLine or something.
================
Comment at: lib/AST/StmtDumper.cpp:92
@@ +91,3 @@
+ void flush() {
+ if (NeedNewLine)
+ OS << "\n";
----------------
Can that ever be false in flush()
http://llvm-reviews.chandlerc.com/D95
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
