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

Simplify deserialize on Variant!(void, T) #28

Merged
merged 1 commit into from
Oct 10, 2022

Conversation

WebFreak001
Copy link
Contributor

There was a special case for Nullable, I also applied this to Variants with void in first place. This is used heavily in serve-d.

Additionally it seems Variants don't actually support the serdeEnumProxy properly that was present in the here adjusted test. However with Nullable and now with Variant!(void, T) they at least work, which should cover a lot of cases. In serve-d there is only a single exception to this, to which I simply implemented a custom wrapper struct for serialization around.

There was a special case for Nullable, I also applied this to Variants
with void in first place. This is used heavily in serve-d.

Additionally it seems Variants don't actually support the serdeEnumProxy
properly that's in the here adjusted test. However with Nullable and now
with Variant!(void, T) they at least work, which should cover a lot of
cases. In serve-d there is only a single exception to this, to which I
simply implemented a custom wrapper struct for serialization around.
@9il 9il merged commit 8138827 into libmir:master Oct 10, 2022
@WebFreak001 WebFreak001 deleted the quick-void-deserialize branch October 21, 2022 01:36
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