Replace futures-util with atomic-waker and manual poll_fn

Open notgull opened this issue 2 years ago • 3 comments

This removes ones of the heavier dependencies. The goal is to also remove futures-util from the rest of the dependency tree, which is hard to justify when futures-util is used in more fundamental dependencies.

Closes #507

Signed-off-by: John Nunley [email protected]

Oct 28 '23 20:10 notgull

@seanmonstar Any chance you can review this?

Nov 15 '23 04:11 notgull

Sorry, yes I will take a look, but I want to check out this separate dependency, and just didn't have time while trying to cram all the prep work to release hyper 1.0 (today).

Nov 15 '23 17:11 seanmonstar

@seanmonstar Has hyper v1.0 craziness cooled off enough for you to have time to review this?

Dec 30 '23 17:12 notgull

@seanmonstar Just following up on this. Have you had to chance to consider this yet?

Apr 19 '24 03:04 notgull