Skip to content

Conversation

@ThierryBerger
Copy link
Contributor

@ThierryBerger ThierryBerger commented Nov 15, 2022

Fix #40.

Follow up to #41.

Currently just hacking stuff together, I dropped forbid_unsafe for my tests ; but ultimately the plan is to upstream that work to bytemuck.

Maybe https://jack.wrenn.fyi/blog/deflect/ contains helpful insights.

@ThierryBerger ThierryBerger force-pushed the 40-support-enum-with-fields branch from c1b58f3 to 0d086ab Compare November 15, 2022 14:52
@ThierryBerger
Copy link
Contributor Author

To properly test the enum handling, test_synchronize_p2p_sessions Should have its enum equivalent to call into poll_remote_clients which continues until it hits a CheckedBitPattern call.

@gschup
Copy link
Owner

gschup commented Dec 15, 2024

#82 is merged and allows for enums, since serde is much less strict than bytemuck

@gschup gschup closed this Dec 15, 2024
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.

Support enum in Config::Input

2 participants