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.

Sign up for GitHub

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

Jump to bottom

Implement deserialize_any #95

Open

liarokapisv wants to merge 1 commit into rust-embedded-community:master from liarokapisv:feature-deserialize_any

liarokapisv commented Jan 12, 2025 •

edited

Loading

This PR adds support for deserialize_any which is useful when trying to work around the lack of untagged enum support.

Mostly follows serde-json 's approach without the inlining.

There is definitely room for improvement when it comes to the implementation:

We can inline the delegated methods and avoid redundant checks
We can probably avoid the multiple delegations to from_str implementations when parsing for numbers.

This will probably require some cleanup, it currently uses a Visitor adaptor to take advantage of the existing deserialize_fromstr macro.


          implement deserialize_any

2f75e92

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet