diff options
| author | Ben Hamilton <benhamilton@google.com> | 2018-03-20 14:53:25 +0000 |
|---|---|---|
| committer | Ben Hamilton <benhamilton@google.com> | 2018-03-20 14:53:25 +0000 |
| commit | ad991868c1c4d4e778f41ebccc1b4ad8ce978d31 (patch) | |
| tree | e5faa78d8597ac1e2caf8966aa9f1234e54013ea /clang/lib/CodeGen/CGCleanup.cpp | |
| parent | 63cc8e96c331b536eb59bc543b10cc4036e1c2a8 (diff) | |
| download | bcm5719-llvm-ad991868c1c4d4e778f41ebccc1b4ad8ce978d31.tar.gz bcm5719-llvm-ad991868c1c4d4e778f41ebccc1b4ad8ce978d31.zip | |
[clang-format] Fix ObjC selectors with multiple params passed to macro
Summary:
Objective-C selectors with arguments take the form of:
foo:
foo:bar:
foo:bar:baz:
These can be passed to a macro, like NS_SWIFT_NAME():
https://developer.apple.com/library/content/documentation/Swift/Conceptual/BuildingCocoaApps/MixandMatch.html
and must never have spaces inserted around the colons.
Previously, there was logic in TokenAnnotator's tok::colon parser to
handle the single-argument case, but it failed for the
multiple-argument cases.
This diff fixes the bug and adds more tests.
Test Plan: New tests added. Ran tests with:
% make -j12 FormatTests && ./tools/clang/unittests/Format/FormatTests
Reviewers: jolesiak, djasper, Wizard
Reviewed By: jolesiak, Wizard
Subscribers: klimek, cfe-commits
Differential Revision: https://reviews.llvm.org/D44638
llvm-svn: 327986
Diffstat (limited to 'clang/lib/CodeGen/CGCleanup.cpp')
0 files changed, 0 insertions, 0 deletions

