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

Conversion category for optionals #833

Closed
grisumbras opened this issue Jan 2, 2023 · 0 comments · Fixed by #923
Closed

Conversion category for optionals #833

grisumbras opened this issue Jan 2, 2023 · 0 comments · Fixed by #923

Comments

@grisumbras
Copy link
Member

#807 introduced a special case for when an struct contains an optional member. But it is special cased for std::optional. Having a similar behaviour for boost::optional is logical, but implementing it the same way introduces a dependency on Boost.Optional and it's also a bit silly to have several special cases. It appears, we need a special coversion category for optionals. We can look at Boost.Url for ideas on how to implement the check for whether something is an optional.

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 a pull request may close this issue.

1 participant