Add serde

Open JonathanWoollett-Light opened this issue 11 months ago • 2 comments

Implements serde::Serialize and serde::Deserialize.

This follows up https://github.com/NULLx76/ringbuffer/issues/133

Jan 15 '25 11:01 JonathanWoollett-Light

Thanks for this implementation! I've actually tried once but found it a little complicated and went and did something else but we did kind of need it... neat! :)

Jan 15 '25 14:01 jdonszelmann

though I appreciate a lot the help, I unfortunately think this implementation isn't entirely desirable, and in at least one way I could spot sofar unsound.

I sort of just threw this together for something I was testing, I expect there are probably more issues than even the ones you mentioned, I don't know if I'll find the time to get this into proper shape any time soon but maybe this is a small step.

Jan 15 '25 20:01 JonathanWoollett-Light