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

Support for async #62

Open
asdfgasdfsafgsdfa opened this issue Apr 10, 2017 · 1 comment
Open

Support for async #62

asdfgasdfsafgsdfa opened this issue Apr 10, 2017 · 1 comment

Comments

@asdfgasdfsafgsdfa
Copy link

When reading and writing to network connections (using NetworkStream) we often want to use async as it scales a lot better than having one thread for every client.

Assuming we'd want to read directly from a NetworkStream class using NetSerializer it would be essential to have async versions of Deserialize / Serialize (which internally use the ReadAsync and WriteAsync methods of the given stream).

Is that planned? If not, could you give me a rough outline what the best way to add it would be? (like what to keep in mind) @tomba

@tomba
Copy link
Owner

tomba commented Apr 17, 2017

I don't exactly know what it would need, but I have a hunch that it would slow down NetSerializer quite a lot. But feel free to try and prove me wrong =).

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

No branches or pull requests

2 participants