Skip to content

[5.3][Sema] Maintain the implicitness of call argument tuple/parens in coerceCallArguments #31285

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

nathawes
Copy link
Contributor

Cherry-pick of #31184 (reviewed by @xedin).

If any arguments were defaulted the tuple/paren was made implicit, even though
the original tuple/paren was present in the source.

This prevented some sourcekit ASTWalkers from considering them, making
refactorings, documentation info, jump-to-definition and other features
unavailable when queried via their argument labels.

Resolves rdar://problem/62118957

…rceCallArguments

If any arguments were defaulted the tuple/paren was made implicit, even though
the original tuple/paren was present in the source.

This prevented some sourcekit ASTWalkers from considering them, making
refactorings, documentation info, jump-to-definition and other features
unavailable when queried via their argument labels.

Resolves rdar://problem/62118957
@nathawes nathawes added the r5.3 label Apr 24, 2020
@nathawes
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - b2aa159

@nathawes
Copy link
Contributor Author

@swift-ci please test OS X Platform

@nathawes nathawes changed the title [Sema] Maintain the implicitness of call argument tuple/parens in coerceCallArguments [5.3][Sema] Maintain the implicitness of call argument tuple/parens in coerceCallArguments Apr 25, 2020
@nathawes nathawes merged commit cc7cc9a into swiftlang:release/5.3 Apr 25, 2020
@nathawes nathawes deleted the defaulted-arg-list-implicitness-5.3 branch April 25, 2020 02:10
@AnthonyLatsis AnthonyLatsis added swift 5.3 🍒 release cherry pick Flag: Release branch cherry picks labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants