djasper added inline comments.

================
Comment at: lib/Format/UnwrappedLineParser.cpp:708
@@ +707,3 @@
+  if (Line && Line->Tokens.size() > 1) {
+    // If the token before the previous one is an '@', the previous token is an
+    // annotation and can precede another identifier/value.
----------------
How about:
* Moving this down a bit
* Checking this only if PreviousMustBeValue
* return directly

Like this, it is a bit hard to understand all the combinations of what's going 
on.


http://reviews.llvm.org/D18943



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to