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.

Sign up for GitHub

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

Jump to bottom

Replace djinterop::stdx::optional with std::optional #92

Closed
mr-smidge opened this issue Jun 6, 2023 · 1 comment
Closed

Replace djinterop::stdx::optional with std::optional #92

mr-smidge opened this issue Jun 6, 2023 · 1 comment
Assignees
@mr-smidge
Labels
good first issue Good for newcomers

Comments

@mr-smidge
Copy link
Contributor

mr-smidge commented Jun 6, 2023 •
edited
Loading

The library implements a wraparound djinterop::stdx::optional, which is aliased either to std::experimental::optional or std::optional, depending on what is available.

However, it is suggested that std::optional should be available in all modern compiler suites, and hence this level of indirection can be removed.
@mr-smidge mr-smidge added the good first issue Good for newcomers label Jun 6, 2023
@mr-smidge
Copy link
Contributor Author

Resolved by #121

@mr-smidge mr-smidge self-assigned this Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mr-smidge mr-smidge

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mr-smidge