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