Skip to content
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

Allow recovering at most one element of a tuple to mutable capability. #2585

Merged
merged 1 commit into from
Mar 21, 2018

Conversation

Praetonus
Copy link
Member

It is now allowed to recover one mutable element in the tuple, as long as all other elements are already safe.

In addition, the compiler now reports an error when attempting to recover unsafe tuples instead of silently accepting without actually lifting the capabilities.

It is now allowed to recover one mutable element in the tuple, as long
as all other elements are already safe.

In addition, the compiler now reports an error when attempting to
recover unsafe tuples instead of silently accepting without actually
lifting the capabilities.
@jemc
Copy link
Member

jemc commented Mar 21, 2018

@Praetonus - is this related to open issue #1972 in any way?

@Praetonus
Copy link
Member Author

Yes, this implements one of the solution we discussed in that issue.

@jemc jemc changed the title Relax the rules for tuple recovery Allow recovering at most one element of a tuple to mutable capability. Mar 21, 2018
@jemc jemc added the changelog - added Automatically add "Added" CHANGELOG entry on merge label Mar 21, 2018
@jemc jemc merged commit aec9610 into ponylang:master Mar 21, 2018
ponylang-main added a commit that referenced this pull request Mar 21, 2018
@Praetonus Praetonus deleted the relax-tuple-recovery branch March 21, 2018 19:45
dipinhora pushed a commit to dipinhora/ponyc that referenced this pull request Jun 5, 2018
Allow recovering at most one element of a tuple to mutable capability.
dipinhora pushed a commit to dipinhora/ponyc that referenced this pull request Jun 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog - added Automatically add "Added" CHANGELOG entry on merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants