-
Notifications
You must be signed in to change notification settings - Fork 4
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 serialize_packed? #2
Comments
Right, async packed serialization is not implemented yet. It would probably be a fascinating project involving some rather hairy state machines, and I'd be thrilled if you wanted to attack it. Note that the reference C++ implementation also does not have async packed serialization yet, so if we get this done first we'll have some bragging rights! |
Haha, I guess I didn't realize what I might be getting into. I may have some time to take a look in a week or so. Mind giving me a quick overview of why it's so challenging? |
I'm not sure that it's necessarily super challenging. It's just that nobody has needed it enough to bother yet. The goal is to have versions of
I have a vague worry that (2) might not work for some reason, but I might misremembering something. |
This will be nigh-trivial once Rust gets generators and corresponding async/await. In the mean time, it is already possible to implement this, at the cost of an extra copy, by serializing to an in-memory buffer (synchronously) and then writing out the buffer. |
It looks like this only supports using serialize right now. Is there a way to use serialize_packed? If not, I'm happy to help out with the implementation.
The text was updated successfully, but these errors were encountered: