diff options
| author | Daniel Jasper <djasper@google.com> | 2013-01-09 10:40:23 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2013-01-09 10:40:23 +0000 |
| commit | 3fc0bb779e751de71166c20a365cd5c28c3de8ed (patch) | |
| tree | f09df1a015223a83b19cc52b9139fc240fd21798 /unittests/Format | |
| parent | ceb99ab9d721848c0851c59f3fdd9c1ca85c58be (diff) | |
Only align after assignments on the top level.
This fixes llvm.org/PR14870 and we no longer mess up:
template <typename T1, typename T2 = char, typename T3 = char,
typename T4 = char>
void f();
It removes the nice aligment for assignments inside other expressions,
but I am not sure those are actually practically relevant. If so, we can
fix those later.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@171966 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'unittests/Format')
| -rw-r--r-- | unittests/Format/FormatTest.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/unittests/Format/FormatTest.cpp b/unittests/Format/FormatTest.cpp index 495fc7fee1..d4e5a51bc3 100644 --- a/unittests/Format/FormatTest.cpp +++ b/unittests/Format/FormatTest.cpp @@ -816,8 +816,9 @@ TEST_F(FormatTest, UnderstandsEquals) { "}"); verifyFormat( + // FIXME: Does an expression like this ever make sense? If yes, fix. "if (int aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa = 100000000 +\n" - " 10000000) {\n" + " 10000000) {\n" "}"); } @@ -872,7 +873,10 @@ TEST_F(FormatTest, WrapsTemplateDeclarations) { verifyFormat("template <typename T>\n" "void aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa(\n" " int aaaaaaaaaaaaaaaaa);"); - + verifyFormat( + "template <typename T1, typename T2 = char, typename T3 = char,\n" + " typename T4 = char>\n" + "void f();"); } TEST_F(FormatTest, UnderstandsTemplateParameters) { |