search

CGAL / cgal

The public CGAL repository, see the README below
https://github.com/CGAL/cgal#readme
Other
4.87k stars 1.38k forks source link

Add a test of `/std:c++latest` with Visual C++ 2017. #2415

Closed lrineau closed 6 years ago

lrineau commented 7 years ago

Newest versions of VC++2017 have a new flags /std:c++latest, to enable the bleeing-edge support of C++ (C++14, C++17, and later). I think one of our testsuite configuration should test that flag.

https://docs.microsoft.com/en-us/cpp/what-s-new-for-visual-cpp-in-visual-studio

See also the documentation of the /std option.

lrineau commented 7 years ago

It could be the same configuration as for /permissive- of #2414.

afabri commented 6 years ago

Note that even the latest version of boost uses std::unary_function. This can be seen in our testsuite Maybe we should pollute the namespace std::

lrineau commented 6 years ago

I reopen this issue, now that Boost.1.67 beta1 seems to be ready to support /std:c++latest.

lrineau commented 6 years ago

See also the documentation of the /std option.

/std:c++latest now enables some C++20 features implemented in MSVC 2017. There is /std:c++17 to enable only C++17.

I think some of our users will like to use /std:c++latest. We should support it, if Boost supports it as well.

afabri commented 6 years ago

@maxGimeno We have to upgrade Eigen in the Windows testsuite, as eigen-eigen-f562a193118d\eigen\src/Core/functors/StlFunctors.h(78): error C2039: 'binder2nd': is not a member of 'std' is fixed in a more recent release.

afabri commented 6 years ago

OpenMesh is not yet /std:c++latest-ready.

janetournois commented 6 years ago

just note that on Friday I added /std:c++latest to our MSVC2017-Debug testsuite (on Gauguin). So it has been tested since CGAL-4.12-Ic-225

maxGimeno commented 6 years ago

I downloaded the currently latest stable released of eigen, the 3.3.4, on gauguin, and updated the setup_common accordingly.