[PATCH] Split UnwrappedLineFormatter into individual components

Manuel Klimek klimek at google.com
Fri Aug 16 03:59:16 PDT 2013


  Rest looks good, assuming it's all copy-and-pasted ;)


================
Comment at: lib/Format/ContinuationIndenter.h:64
@@ +63,3 @@
+  unsigned addTokenToState(bool Newline, bool DryRun, LineState &State);
+
+  /// \brief Mark the next token as consumed in \p State and modify its stacks
----------------
I'd insert "private:" here. That would also paper over the fact that the next function has parameters in a different order for no obvious reason.


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



More information about the cfe-commits mailing list