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

Using c++17 standard library #7526

Merged
merged 53 commits into from
Aug 18, 2023
Merged

Conversation

sloriot
Copy link
Member

@sloriot sloriot commented Jun 15, 2023

use std::shared_ptr, std::optional, std::variant, ...

These are important changes since they appear in the CGAL API and thus are breaking changes.

includes #7416

@sloriot sloriot added Modern C++ (C++11 and later) depends on another PR This pull-request should only be merged after other ones. labels Jun 15, 2023
@sloriot sloriot force-pushed the CGAL-using_c++17_stl branch from 318315a to 76f16d7 Compare June 20, 2023 16:33
@sloriot sloriot force-pushed the CGAL-using_c++17_stl branch from 86a13fa to e759fb4 Compare June 29, 2023 10:58
@sloriot sloriot added this to the 6.0-beta milestone Aug 18, 2023
@sloriot sloriot self-assigned this Aug 18, 2023
@sloriot
Copy link
Member Author

sloriot commented Aug 18, 2023

Successfully tested in CGAL-6.0-Ic-43

@sloriot sloriot added the Tested label Aug 18, 2023
@github-actions github-actions bot removed the Tested label Aug 18, 2023
@github-actions

This comment was marked as off-topic.

@sloriot sloriot force-pushed the CGAL-using_c++17_stl branch from 21ee608 to d6c04e3 Compare August 18, 2023 07:34
@sloriot sloriot added the Tested label Aug 18, 2023
@sloriot sloriot merged commit 5a962eb into CGAL:master Aug 18, 2023
@sloriot sloriot deleted the CGAL-using_c++17_stl branch August 18, 2023 07:35
MaelRL added a commit to MaelRL/cgal that referenced this pull request Jan 16, 2025
sloriot added a commit that referenced this pull request Jan 20, 2025
## Summary of Changes

Some occurrences that slipped through.

See #7526

## Release Management

* Affected package(s): `Tetrahedral_remeshing`, `Triangulation_3`
* Issue(s) solved (if any): -
* Feature/Small Feature (if any): - 
* License and copyright ownership: no change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants