[PATCH] Remove extra whitespace instead of breaking the line in comments when possible.

Manuel Klimek klimek at google.com
Tue Nov 12 01:23:10 PST 2013


  LG


================
Comment at: lib/Format/BreakableToken.h:64-65
@@ -63,1 +63,4 @@
 
+  /// \brief Replaces whitespace range, described with \p Split, with a single
+  /// space character.
+  virtual void replaceWhitespace(unsigned LineIndex, unsigned TailOffset,
----------------
I'd slightly reword:
Replaces the whitespace range described by \p Split with a single space.


================
Comment at: lib/Format/BreakableToken.cpp:226
@@ +225,3 @@
+                                                Split.first,
+                                       Split.second, "", "", false, 0, 0, 1);
+}
----------------
As below, it would help to have the /*Param=*/ style comments for the constants...


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

BRANCH
  svn

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list