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 fo serde #71

Open
mjos opened this issue May 23, 2019 · 4 comments · May be fixed by #72
Open

Support fo serde #71

mjos opened this issue May 23, 2019 · 4 comments · May be fixed by #72

Comments

@mjos
Copy link

mjos commented May 23, 2019

Hi,

Is it possible to add support for serialization and deserialization via serde derive? Serialization is briefly mentioned in Readme, but otherwise no examples given. Thanks a lot.

@gnzlbg
Copy link
Owner

gnzlbg commented May 23, 2019

We could add that via a cargo feature. SliceDeque derefs into a slice, so implementing this should be a one liner, since serde supports serializing and deserializing slices.

@mjos
Copy link
Author

mjos commented May 23, 2019

That would be great! Being a beginner I failed to implement custom deserializer (serializer was easier) for external type today, so I reverted to VecDeque for now.

@gnzlbg gnzlbg linked a pull request May 24, 2019 that will close this issue
@gnzlbg
Copy link
Owner

gnzlbg commented May 24, 2019

There is a PR open to close this in #72 . Once that is merged I'll release 0.2.5 with this. To enable serde support you'll need to enable the serde cargo feature.

@mjos
Copy link
Author

mjos commented May 24, 2019

Thanks! And I learned something.

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

Successfully merging a pull request may close this issue.

2 participants