-
Notifications
You must be signed in to change notification settings - Fork 33
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
cxx: explicitly require c++11 and fix tests to conform
We weren't setting a C++ version for build previously, which would result in compiler specific default being used, most likely C++11 with GNU extensions on modern compilers and C++03 with GNU exts on old compilers. This is bad because it potentially breaks build on older toolchains that don't default to a modern C++. Now we enable pure C++11 without GNU exts; this resulted in some of the build breaking because of use of typeof() GNU C/C++ ext in tests code, so fix that to use standard decltype() from C++11. @fix
- Loading branch information
Daniel Kolesa
committed
Nov 12, 2018
1 parent
1af272b
commit ac95f38
Showing
2 changed files
with
25 additions
and
25 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters