[cfe-commits] [PATCH] Make parenthesis counting and aligning a bit saner

Dmitri Gribenko gribozavr at gmail.com
Tue Dec 4 06:40:42 PST 2012



================
Comment at: lib/Format/Format.cpp:151
@@ +150,3 @@
+  ///
+  /// Puts the token on the current line if \p Newline is \c and adds a
+  /// line break and necessary indentation otherwise.
----------------
... is \c true ?

================
Comment at: lib/Format/Format.cpp:154
@@ +153,3 @@
+  ///
+  /// If \p DryRun is \c true, also creates and stores the required
+  /// \c Replacement.
----------------
maybe "is \c false"?

================
Comment at: lib/Format/Format.cpp:207-208
@@ +206,4 @@
+
+  // Mark the next token as consumed in \p State and modify its stacks
+  // accordingly.
+  void moveStateToNextToken(IndentState &State) {
----------------
Three slashes, please.


http://llvm-reviews.chandlerc.com/D162



More information about the cfe-commits mailing list