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

Supporting shared objects, including cycles #11

Open
sandhawke opened this issue Dec 19, 2016 · 4 comments
Open

Supporting shared objects, including cycles #11

sandhawke opened this issue Dec 19, 2016 · 4 comments
Assignees

Comments

@sandhawke
Copy link

I did this as a weekend project. I wouldn't say it's ready to be included yet, but I'm interested in feedback. The README is pretty complete.

https://github.com/sandhawke/borc-refs

@dignifiedquire dignifiedquire self-assigned this Dec 19, 2016
@dignifiedquire
Copy link
Owner

Thanks for posting, looks pretty interesting. I will take a look in the next days

@daviddias daviddias added backlog and removed ready labels Jan 29, 2017
@daviddias daviddias added ready and removed backlog labels Oct 17, 2017
@daviddias
Copy link
Collaborator

@dignifiedquire any next steps?

@dignifiedquire
Copy link
Owner

Never got around it, would be great to figure out how to include this in an efficient and safe manner

@sandhawke
Copy link
Author

Looking over it again, I'd be inclined to switch to using a WeakMap as I considered in the issues, so there's no Symbol left on objects. The documentation could be improved, and the debugging statements taken out, but otherwise I think it's probably fine for inclusion.

As it turns out, the project I was using this for shifted focus, so I'm not currently using the code at all.

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

No branches or pull requests

3 participants