[PATCH] Major refactoring of BreakableToken.

Manuel Klimek klimek at google.com
Fri May 24 13:04:17 PDT 2013


Hi djasper, alexfh,

Unify handling of whitespace when breaking protruding tokens with other
whitespace replacements.

As a side effect, the BreakableToken structure changed significantly:
- have a common base class for single-line breakable tokens, as they are
  much more similar
- revamp handling of multi-line comments; we now calculate the information
  about lines in multi-line comments similar to normal tokens, and always
  issue replacements

As a result, we were able to get rid of special casing of trailing whitespace
deletion for comments in the whitespace manager and the BreakableToken and
fixed bugs related to tab handling and escaped newlines.

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

Files:
  lib/Format/BreakableToken.cpp
  lib/Format/BreakableToken.h
  lib/Format/Format.cpp
  lib/Format/UnwrappedLineParser.h
  lib/Format/WhitespaceManager.cpp
  lib/Format/WhitespaceManager.h
  unittests/Format/FormatTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D864.1.patch
Type: text/x-patch
Size: 46197 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130524/f359bfdc/attachment.bin>


More information about the cfe-commits mailing list