Unidata / netcdf-cxx4

Official GitHub repository for netCDF-C++ libraries and utilities.
Other
124 stars 49 forks source link

Enforce C++ std 11 in build system. #147

Closed WardF closed 3 weeks ago

WardF commented 5 months ago

It's past time to get a C++ release out; there are PR's to be gone through; I'm also putting up this PR, if it is already handled in any of the other PR's, I'll close this. I'll start reviewing them after finishing the review of the C library PR's.

ZedThree commented 5 months ago

I also added this in 4a2c6ce in #143, although I set it to C++14. I would strongly encourage setting the minimum to C++14 rather than 11. 14 is really a "bug fix release" of the standard, and is universally supported.

I would even push for setting C++17 to be the minimum standard, as it's now very widely supported by the default compiler on all major currently supported OSes and systems.