[PATCH] Fixes incorrect indentation of line comments after break and re-alignment.
Alexander Kornienko
alexfh at google.com
Mon Jun 17 05:57:44 PDT 2013
================
Comment at: lib/Format/WhitespaceManager.cpp:71
@@ -70,2 +70,3 @@
// correctly store a tok::TokenKind instead of rolling our own enum.
- tok::unknown, InPPDirective && !Tok.IsFirst));
+ Tok.Type == TT_LineComment && Newlines > 0 ? tok::comment : tok::unknown,
+ InPPDirective && !Tok.IsFirst));
----------------
Manuel Klimek wrote:
> Alexander Kornienko wrote:
> > Manuel Klimek wrote:
> > > Why does Newlines > 0 make a difference here?
> > This should be tok::comment only if we create the new comment. Currently in case Newlines == 0 we're just inserting one space, no new comment is created.
> Makes sense, please add a comment :)
How about this?
// If we don't add a newline this change doesn't start start a comment.
// When we align line comments, we don't need to treat this change as one.
// FIXME: We still need to take this change in account to properly
// calculate the new length of the comment when aligning comments.
http://llvm-reviews.chandlerc.com/D980
BRANCH
svn
ARCANIST PROJECT
clang
More information about the cfe-commits
mailing list