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

Reply via email to