Add a yield point to io::copy

smol-rs / futures-lite

Futures, streams, and async I/O combinators.

Apache License 2.0

427 stars 25 forks source link

Add a yield point to io::copy #89

Closed notgull closed 6 months ago

taiki-e commented 6 months ago

If we introduce this, I would like to introduce this in all futures and streams with similar properties, rather than introducing this only in a few futures halfway. Otherwise, we will be getting feature requests endlessly. (That is what actually happening elsewhere.) https://github.com/search?q=repo%3Asmol-rs%2Ffutures-lite%20%2Floop%20%5C%7B%2F&type=code

notgull commented 6 months ago

Ugh, yeah, didn't think about that. That's going to be a headache. I'll close this for now, and if this direction is where we want to head I'll open a more all-encompassing PR.