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

Premature release of $strong_self #25

Open
michaelfig opened this issue Mar 23, 2017 · 1 comment
Open

Premature release of $strong_self #25

michaelfig opened this issue Mar 23, 2017 · 1 comment

Comments

@michaelfig
Copy link

michaelfig commented Mar 23, 2017

I commented on the commit that introduced weaken $self.

With the code I attached in there, I don't see the problem of $self going to undef while commands are still pending.

Without my patch, if I have a long-running AnyEvent process that fires off commands and lets the handle go out of scope, I get undefined $self errors even though $cmd_cv has not yet been sent.

Thanks,
Michael.

@michaelfig
Copy link
Author

#26 introduces the needed change.

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

No branches or pull requests

1 participant