Skip to content

Conversation

@seritools
Copy link
Contributor

As mentioned on twitter, here's the straight-forward impl of serde :)

The only thing to note is that deserialize_string is used (and visit_string implemented) instead of deserialize_str to give the possibility of directly passing through a String if the deserializer used happens to have an owned String anyways.

@bodil
Copy link
Owner

bodil commented Jul 5, 2020

Looks good to me! 👍

@bodil bodil merged commit 711b0d1 into bodil:master Jul 5, 2020
@seritools seritools deleted the add-serde branch July 5, 2020 21:59
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 this pull request may close these issues.

2 participants