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
noescape by default: deprecate @noescape and @autoclosure(escaping) #4031
Conversation
Adds the preferred syntax for escaping autoclosures, which is @autoclosure @escaping. Deprecates @autoclosure(escaping), and provides fixits.
@NoEscape is now the default behavior, so deprecate it and offer a fixit.
@swift-ci please test |
@swift-ci please test |
|
||
* [SE-103](https://github.com/apple/swift-evolution/blob/master/proposals/0103-make-noescape-default.md) | ||
|
||
Closure parameters are non-escaping by default, rather than explicitly being |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bits of code in the changelog should be surrounded by backticks (`)
@swift-ci test linux |
4 similar comments
@swift-ci test linux |
@swift-ci test linux |
@swift-ci test linux |
@swift-ci test linux |
@swift-ci please test |
Failure unrelated: Which is a new test file that's been problematic: |
@milseman Thanks for taking the time to do the changes, the patch looks much cleaner now! |
[pull] swiftwasm from main
What's in this pull request?
Support the @autoclosure @escaping syntax, deprecate @NoEscape and @autoclosure(escaping). Fixits and tests included.
Resolved bug number: (SR-)
Before merging this pull request to apple/swift repository:
Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
A smoke test on macOS does the following:
device standard libraries are not built.
version of these tests are not run.
A smoke test on Linux does the following:
tests are not run.
Validation Testing
Lint Testing
Note: Only members of the Apple organization can trigger swift-ci.