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

Add separate functions for required and optional properties in JSON Utils #355

Open
paulbourelly999 opened this issue Oct 12, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@paulbourelly999
Copy link
Collaborator

Summary

Using a bool parameter to indicate whether property is required forces every returned property to be wrapped in a std::optional. This requires additional value() calls which make the code more verbose and harder to read.

Reasoning for new functionality

Make code more readable and less verbose

@paulbourelly999 paulbourelly999 added the enhancement New feature or request label Oct 12, 2023
@paulbourelly999 paulbourelly999 changed the title Add separate functions for required and optional parameters Add separate functions for required and optional properties in JSON Utils Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant