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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D33982: [clang... Alexander Kornienko via Phabricator via cfe-commits

Reply via email to