[clang-format] Ensure wrapped ObjC selectors with 1 arg obey IndentWrappedFunctionNames
authorBen Hamilton <benhamilton@google.com>
Fri, 30 Mar 2018 15:38:45 +0000 (15:38 +0000)
committerBen Hamilton <benhamilton@google.com>
Fri, 30 Mar 2018 15:38:45 +0000 (15:38 +0000)
commit4dc1cdc5e1c1184f5ded058d5240532a54c96bc1
treee67765317c1e1194ae54c02868fd85bb2c5c8bf9
parent4f99836a9e7228827bde6f205ec487cdd022c93f
[clang-format] Ensure wrapped ObjC selectors with 1 arg obey IndentWrappedFunctionNames

Summary:
In D43121, @Typz introduced logic to avoid indenting 2-or-more
argument ObjC selectors too far to the right if the first component
of the selector was longer than the others.

This had a small side effect of causing wrapped ObjC selectors with
exactly 1 argument to not obey IndentWrappedFunctionNames:

```
- (aaaaaaaaaa)
aaaaaaaaaa;
```

This diff fixes the issue by ensuring we align wrapped 1-argument
ObjC selectors correctly:

```
- (aaaaaaaaaa)
    aaaaaaaaaa;
```

Test Plan: New tests added. Test failed before change, passed
  after change. Ran tests with:
  % make -j12 FormatTests && ./tools/clang/unittests/Format/FormatTests

Reviewers: djasper, klimek, Typz, jolesiak

Reviewed By: djasper, jolesiak

Subscribers: cfe-commits, Typz

Differential Revision: https://reviews.llvm.org/D44994

llvm-svn: 328871
clang/lib/Format/ContinuationIndenter.cpp
clang/unittests/Format/FormatTestObjC.cpp