r182024 - Add additional test-case for one-line-function.

Daniel Jasper djasper at google.com
Thu May 16 09:54:34 PDT 2013


Author: djasper
Date: Thu May 16 11:54:34 2013
New Revision: 182024

URL: http://llvm.org/viewvc/llvm-project?rev=182024&view=rev
Log:
Add additional test-case for one-line-function.

This ensures that we format:
void longFunctionName {
} // long comment here

And not:
void longFunctionName {}
// long comment here

As requested in post-commit-review.

Modified:
    cfe/trunk/unittests/Format/FormatTest.cpp

Modified: cfe/trunk/unittests/Format/FormatTest.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Format/FormatTest.cpp?rev=182024&r1=182023&r2=182024&view=diff
==============================================================================
--- cfe/trunk/unittests/Format/FormatTest.cpp (original)
+++ cfe/trunk/unittests/Format/FormatTest.cpp Thu May 16 11:54:34 2013
@@ -3066,6 +3066,9 @@ TEST_F(FormatTest, PullTrivialFunctionDe
                "}");
   verifyFormat("void f() {} // comment");
   verifyFormat("void f() { int a; } // comment");
+  verifyFormat("void f() {\n"
+               "} // comment",
+               getLLVMStyleWithColumns(15));
 
   verifyFormat("void f() { return 42; }", getLLVMStyleWithColumns(23));
   verifyFormat("void f() {\n  return 42;\n}", getLLVMStyleWithColumns(22));





More information about the cfe-commits mailing list