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.

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

Reference Counting #3

Closed
iguessthislldo opened this issue Jan 12, 2020 · 1 comment
Closed

Reference Counting #3

iguessthislldo opened this issue Jan 12, 2020 · 1 comment
Milestone
v0.1

Comments

@iguessthislldo
Copy link
Member

Honestly I haven't been paying much attention to how reference counting should be done, at least not in a while. This will be something I should look into again before the initial release. Maybe I could tie some of the reference counting Python is doing with the counting OpenDDS is doing.
@iguessthislldo iguessthislldo added this to the 0.1 milestone Jan 12, 2020
@iguessthislldo iguessthislldo mentioned this issue Jan 13, 2020
Open
iguessthislldo added a commit that referenced this issue Jan 18, 2020
@iguessthislldo
Review Reference Counting in _pyopendds.cpp
7367312 
Progress towards #3. Also revisted some of it as I saw fit.
@iguessthislldo
Copy link
Member Author

This could probably use some more improvement, but I think this is mostly cleaned up.

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

No branches or pull requests
1 participant
@iguessthislldo