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

Replace enum with Typedef where possible #88

Open
vkkhare opened this issue May 18, 2020 · 0 comments
Open

Replace enum with Typedef where possible #88

vkkhare opened this issue May 18, 2020 · 0 comments
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: New Feature ➕ Introduction of a completely new addition to the codebase
Milestone

Comments

@vkkhare
Copy link
Member

vkkhare commented May 18, 2020

For basic int/string constants we would be happier if we used typedef instead of enum class to reduce the memory footprint.

@vkkhare vkkhare added Type: New Feature ➕ Introduction of a completely new addition to the codebase Good first issue 🎓 Perfect for beginners, welcome to OpenMined! labels May 18, 2020
@cereallarceny cereallarceny added this to the 0.2.0 milestone Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Type: New Feature ➕ Introduction of a completely new addition to the codebase
Projects
None yet
Development

No branches or pull requests

2 participants