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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
