I am happy to get this in as is especially as it is good to have the tests. I 
also agree that there is an underlying issue which we should look at in more 
detail. Can you add a FIXME to this effect?

REPOSITORY
  rL LLVM

================
Comment at: lib/Format/TokenAnnotator.cpp:1293
@@ -1292,3 +1292,3 @@
         InFunctionDecl && Current->Type == TT_FunctionDeclarationName &&
-        Line.Last->is(tok::l_brace))  // Only for definitions.
+        !Line.Last->is(tok::semi))  // Only for definitions.
       Current->MustBreakBefore = true;
----------------
Can you make this:

  !Line.Last->isOneOf(tok::semi, tok::comment)

That seems to be the most obvious exception I can think of.

http://reviews.llvm.org/D4886



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to