Skip to content

fix parsing of isolated as an argument label #38422

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
merged 1 commit into from
Jul 19, 2021

Conversation

kavon
Copy link
Member

@kavon kavon commented Jul 16, 2021

With the introduction of isolated as
a type modifier for actor types, the
parsing of a parameter regressed such
that isolated was no longer accepted
as an ordinary argument label. This patch
fixes that and adds a little lookahead
utility to clean-up the code that
disambiguates the uses of isolated
as either a label or a type modifier.

Resolves rdar://80300022

@kavon kavon added bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. concurrency Feature: umbrella label for concurrency language features labels Jul 16, 2021
@kavon
Copy link
Member Author

kavon commented Jul 16, 2021

@swift-ci please smoke test

@kavon kavon force-pushed the isolated-argument-label branch from 3c91a64 to 0d01a37 Compare July 16, 2021 00:48
With the introduction of `isolated` as
a type modifier for actor types, the
parsing of a parameter regressed such
that `isolated` was no longer accepted
as an ordinary argument label. This patch
fixes that and adds a little lookahead
utility to clean-up the code that
disambiguates the uses of `isolated`
as either a label or a type modifier.

Resolves rdar://80300022
@kavon kavon force-pushed the isolated-argument-label branch from 0d01a37 to 5ffe8e0 Compare July 16, 2021 00:49
@kavon
Copy link
Member Author

kavon commented Jul 16, 2021

@swift-ci please smoke test

@kavon
Copy link
Member Author

kavon commented Jul 16, 2021

@swift-ci please smoke test and merge

1 similar comment
@kavon
Copy link
Member Author

kavon commented Jul 19, 2021

@swift-ci please smoke test and merge

@swift-ci swift-ci merged commit f636e9a into swiftlang:main Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A deviation from expected or documented behavior. Also: expected but undesirable behavior. concurrency Feature: umbrella label for concurrency language features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants