Skip to content

Makes all objects in connection have a stable address #285

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
Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Makes all objects in connection have a stable address #285

wants to merge 1 commit into from

Conversation

anarthal
Copy link
Collaborator

No description provided.

@anarthal
Copy link
Collaborator Author

This is intended to alleviate one of the problems you had in #283, and make reasoning about lifetimes trivial.

Before I add some tests to verify that moving connections is fine, do you think this is the way to go?

I could also make exec_op, reader_op, writer_op and friends operate on connection_impl rather than connection, which should be marginally more efficient, but requires more work. Let me know if you think it's worth it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@anarthal