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

Stop leaking file descriptors #387

Open

thomasqueirozb wants to merge 3 commits into cloudflare:master from thomasqueirozb:fd-leak

thomasqueirozb commented Jan 20, 2024 •

edited

Fixes #170

~~The thread::sleep isn't ideal but I didn't see a simple way to wait until the queue is dropped to check the number of fds.~~

thomasqueirozb added 3 commits

January 20, 2024 02:57


          epoll: return unit type whenever possible

7d7e2b4

This makes it more clear to see where EventRef is actually being used


          epoll: drop fds when dropping EventPoll

1229fe5


          test_fd_leaks: use ref count instead of sleeping

1aeec7e

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