[PATCH] D33982: [clang-format] Fix alignment of preprocessor trailing comments

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 7 05:55:01 PDT 2017


alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

LG



================
Comment at: lib/Format/WhitespaceManager.cpp:112
+    assert(PreviousOriginalWhitespaceEndOffset <= OriginalWhitespaceStartOffset);
+    StringRef Text(SourceMgr.getCharacterData(PreviousOriginalWhitespaceEnd),
+                   SourceMgr.getCharacterData(OriginalWhitespaceStart) -
----------------
I'd pull `SourceMgr.getCharacterData(PreviousOriginalWhitespaceEnd)` out to a variable.


https://reviews.llvm.org/D33982





More information about the cfe-commits mailing list