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

Use a more push-based approach to handle async case. #14

Open
stevefan1999-personal opened this issue Jun 25, 2022 · 1 comment
Open

Comments

@stevefan1999-personal
Copy link
Contributor

stevefan1999-personal commented Jun 25, 2022

Judging from the discussion on #13, it seems like adding async support would be a PITA because 1. maintenance hell with two copies of almost the same thing and 2. there is no standard regarding async and the use of io::Error in no_std situation which makes it very painful to write for in the first place.

I guess its time that we should discuss for a more modular, push-based approach similar to httparse. Then the user should handle the read write outside of the stream accordingly.

This of course means the responsibility would be offloaded to user and it will be a major semver break situation.

@stevefan1999-personal
Copy link
Contributor Author

Another way of reliving this is use poll_ style...I'm still learning about this matter though.

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