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

Use string values for enum option in configuration file #46

Open
dweindl opened this issue Nov 8, 2017 · 2 comments
Open

Use string values for enum option in configuration file #46

dweindl opened this issue Nov 8, 2017 · 2 comments

Comments

@dweindl
Copy link
Member

dweindl commented Nov 8, 2017

Currently only integer values are used, which is hard to read and error-prone. Need string-to-enum. Available for ceres in types.cc.

@dweindl
Copy link
Member Author

dweindl commented May 29, 2019

https://github.com/Neargye/magic_enum will be helpful here if we move to C++17.

@paulstapor
Copy link
Collaborator

Sounds cool, would simplify the code when setting options, reading success/error status, etc. 👍

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

2 participants