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

Add safe alternative to get_data/set_data #360

Merged
merged 7 commits into from
Apr 23, 2020
Merged

Add safe alternative to get_data/set_data #360

merged 7 commits into from
Apr 23, 2020

Conversation

piscisaureus
Copy link
Member

No description provided.

@piscisaureus piscisaureus requested a review from ry April 22, 2020 22:46
/// This method gives ownership of value to the Isolate. Exactly one object of
/// each type can be associated with an Isolate. If called more than once with
/// an object of the same type, the earlier version will be dropped and
/// replaced.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This mechanic is pretty wacky and difficult to describe. I wish we could come up with something that was simpler.

@ry
Copy link
Member

ry commented Apr 23, 2020

This is good enough to land and iterate on.

@ry ry merged commit 0d636de into master Apr 23, 2020
@ry ry deleted the data2 branch April 23, 2020 07:34
@piscisaureus piscisaureus changed the title WIP: get/set_data_2 Add safe alternative to get_data/set_data Apr 23, 2020
@ry ry mentioned this pull request May 25, 2020
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants