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

Implement Weak References #4

Open
mxjones opened this issue Aug 10, 2018 · 2 comments
Open

Implement Weak References #4

mxjones opened this issue Aug 10, 2018 · 2 comments
Assignees

Comments

@mxjones
Copy link
Owner

mxjones commented Aug 10, 2018

No description provided.

@mxjones mxjones self-assigned this Aug 10, 2018
@tugberkugurlu
Copy link

👋 Just came across this project, looks really good!

Can you give a bit more info on what Weak References is about?

@mxjones
Copy link
Owner Author

mxjones commented Jan 10, 2019

Hi @tugberkugurlu - thanks!

I have not looked too much into this just yet but from what I remember on this the idea was that using weak references (for the stored Action) will remove the need to explicitly unsubscribe / help avoid memory leaks caused by holding onto the target, as it the weak reference will not stop the target from being GC'd - Would also need something that dealt with 'Expired' weak references though and cleaned them up. Not 100% sure this is neccessary (or even good) so created this as a placeholder to check it out

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

2 participants