Théodore Prévot
Théodore Prévot
Is it planned / doable to make a `RawValue` type specifically for `serde_rmp` to deserialize only one part of a payload or to pre-serialize some data ?
Your solution also work, it has the advantage that it is also working on the browser whereas mine only works for node.js (because it uses the official stream implementation). This...
If someone want to contribute for this issue: * Move the state typemap to the `Client` struct and add a clone bound. * Pass the typemap to each handler. *...
Do you think it would be possible to split the `Vec` to `Bytes` modification in another PR? Because I'm sure I will merge this (it was on my to-do list)...
> But I put it in its own commit and I'm happy to change it back to the simpler API surface if you'd prefer. I think it is a really...
> Rebased against main, but now I'm seeing a failure in the new `concurrent_emit` test. The error I'm seeing (and CI is seeing) is that somehow it first gets `Binary(b"\x01\x02\x03")`...
The 2nd solution seems much better, with this implementation is it possible to keep the same kind of layout ? Like `TestPayload(String, Bytes, Bytes)` or `TestPayload([String, Bytes, Bytes])`?
I dig a bit the issue of the `emit` with `serde_json::Value`. I'm afraid the solution with `create_binary_placeholder` would disturb the user because it is not intuitive. What I propose is...
> Yes, that's correct, and there's really no way around that if we want the new features to work. The problem is that we can't actually tell if the user...
> @Totodore just wanted to leave a note that I haven't abandoned this -- I'm getting married next week, so my time for open source has been unsurprisingly limited lately....