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

Some response fields should have more semantic types #37

Open
kstep opened this issue Feb 2, 2016 · 1 comment
Open

Some response fields should have more semantic types #37

kstep opened this issue Feb 2, 2016 · 1 comment

Comments

@kstep
Copy link
Owner

kstep commented Feb 2, 2016

Some fields in response types has types, which doesn't match field semantics.

E.g. all boolean values VK passes as 0/1 integers, so all response fields which should be bool, are u8 now.

This PR to serde may help with this issue: serde-rs/serde#198
It adds an ability to apply custom deserializers to some struct fields with attributes.
If this feature (or some similar feature) will land to serde, we can use it to avoid writing
deserializers manually (which I want to avoid at all costs, or the code will explode beyond
any manageable limits).

@kstep kstep changed the title Some response fields should have more sematic types Some response fields should have more semantic types Feb 2, 2016
@kstep
Copy link
Owner Author

kstep commented Feb 22, 2016

See also #54.

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

No branches or pull requests

1 participant