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

Consider cancellation of dispatch channel #69

Open
Rosomack opened this issue Aug 5, 2021 · 1 comment
Open

Consider cancellation of dispatch channel #69

Rosomack opened this issue Aug 5, 2021 · 1 comment

Comments

@Rosomack
Copy link
Contributor

Rosomack commented Aug 5, 2021

I have another small observation regarding the mem. leak to think about -> I’ve noticed the dispatchChannel from the RealContainer is never cancelled -> so there could be a situation, probably, where the channel would have a buffered message but the receiver coroutine is already cancelled. I guess we should cancel/close the channel when scope completes as well.

@rhmndnt
Copy link

rhmndnt commented Jul 2, 2022

https://stackoverflow.com/a/43889107

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

2 participants