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.

Sign up for GitHub

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

Jump to bottom

Rename TypeIsOwned<bool> to TypeInRust<> #186

Closed

kenz-gelsoft opened this issue Jan 5, 2023 · 1 comment · Fixed by #187

Assignees

Owner

kenz-gelsoft commented Jan 5, 2023 •

edited

Loading

As this represents the concept clearer and can be distinguished from Rust's owned type.

also rename Unowned -> InCxx/InCpp/FromCpp

kenz-gelsoft self-assigned this

Owner Author

kenz-gelsoft commented Jan 6, 2023

I modeled downcasted instance as unowned (by Rust code), so renaming breaks the semntics.

kenz-gelsoft mentioned this issue

Rename ownership concept #187

Merged

kenz-gelsoft closed this as completed in #187

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment