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

Disable long double tests and remove CMake flag groups #494

Merged
merged 129 commits into from
Mar 22, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
129 commits
Select commit Hold shift + click to select a range
23e0df3
OESS-98 convert plugin option to FetchContent, add tests
byrnHDF Dec 10, 2020
84293f5
Fixes for pkcfg files because of plugin option
byrnHDF Dec 11, 2020
544bd61
OESS-98 fix tools test for plugins
byrnHDF Dec 15, 2020
ffb182f
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 8, 2021
2561b6b
Keep doxygen comments under 100 chars long - format hint
byrnHDF Jan 8, 2021
1c73615
Whitespace
byrnHDF Jan 8, 2021
b71a853
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 11, 2021
b68973c
HDFFV-11144 - Reclassify CMake messages
byrnHDF Jan 11, 2021
f64497d
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 13, 2021
c854ffe
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 14, 2021
0f90d26
HDFFV-11099/11100 added help text
byrnHDF Jan 14, 2021
20757d6
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 15, 2021
d161733
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 20, 2021
f03b095
Reworked switch statement to compare string instead
byrnHDF Jan 22, 2021
bd52065
Fix typo
byrnHDF Jan 22, 2021
238c060
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 25, 2021
f12b2ae
Update CDash mode
byrnHDF Jan 26, 2021
673aac8
Correct name of threadsafe
byrnHDF Jan 26, 2021
97df6db
Correct option name
byrnHDF Jan 26, 2021
e48f758
Undo accidental commit
byrnHDF Jan 26, 2021
2f364d8
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 27, 2021
f42f02b
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 27, 2021
5b48b33
Note LLVM 10 to 11 format default changes
byrnHDF Jan 27, 2021
9927372
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 28, 2021
ad61d57
Update format plugin
byrnHDF Jan 28, 2021
613bc7a
Undo clang-format version 11 changes
byrnHDF Jan 28, 2021
0114098
One more correction
byrnHDF Jan 28, 2021
a685efa
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Jan 29, 2021
78f25ac
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 1, 2021
1899ba6
Update supported platforms
byrnHDF Feb 2, 2021
28d6d63
Revert whitespace changes
byrnHDF Feb 2, 2021
f1a0239
Correct whitespace
byrnHDF Feb 2, 2021
b656722
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 3, 2021
1c9eaf4
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 4, 2021
f85f7da
Changes from PR#3
byrnHDF Feb 4, 2021
464d08e
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 5, 2021
34ad32c
HDFFV-11213 added option to control gcc10 warnings diagnostics
byrnHDF Feb 5, 2021
d6db05b
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into
byrnHDF Feb 8, 2021
264caff
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 9, 2021
9b42150
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 10, 2021
4569d39
HDFFV-11212 Use the new references correctly in JNI utility and tests
byrnHDF Feb 11, 2021
4a2865c
format source
byrnHDF Feb 11, 2021
1a2dc75
Fix typo
byrnHDF Feb 11, 2021
23679fa
Add new test file
byrnHDF Feb 11, 2021
232426b
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 16, 2021
020a40c
HDFFV-11212 - update test and remove unused arg
byrnHDF Feb 16, 2021
2352972
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 17, 2021
5806658
Minor non-space formatting changes
byrnHDF Feb 18, 2021
83b619f
Use H5I_INVALID_ID instead of "-1"
byrnHDF Feb 19, 2021
fbf97be
source formatting
byrnHDF Feb 19, 2021
ea94075
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 19, 2021
797a6c2
add missing testfile, update jni function
byrnHDF Feb 19, 2021
32d6ee9
Undo commit of debug code
byrnHDF Feb 19, 2021
71c4557
remove mislocated file
byrnHDF Feb 19, 2021
2c6df43
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 19, 2021
66d54a9
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 19, 2021
08c7738
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 19, 2021
46c620f
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 22, 2021
a4132c8
Fix h5repack test for handling of fapls and id close
byrnHDF Feb 22, 2021
498d7ec
Update h5diff test files usage text
byrnHDF Feb 22, 2021
82ff537
HDFFV-11212 add new ref tests for JNI export dataset
byrnHDF Feb 22, 2021
7e81307
src format update
byrnHDF Feb 22, 2021
4bda56f
Remove blank line typo
byrnHDF Feb 22, 2021
25884a6
src format typo
byrnHDF Feb 22, 2021
07d8cf2
long double requires %Lg
byrnHDF Feb 22, 2021
8735c2a
Another long double foramt specifer S.B. %Lg
byrnHDF Feb 22, 2021
140aca0
issue with t128bit test
byrnHDF Feb 22, 2021
c790f17
Windows issue with h5dump and type.
byrnHDF Feb 22, 2021
eb3f7cc
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 23, 2021
76d4fba
Fix review issues
byrnHDF Feb 23, 2021
7b96c23
refactor function nesting and fix error checks
byrnHDF Feb 23, 2021
90bb747
format fixes
byrnHDF Feb 23, 2021
321b926
Remove untested functions and javadoc quiet comments
byrnHDF Feb 23, 2021
2d40134
Restore TRY block.
byrnHDF Feb 24, 2021
b8cda57
Change string append errors to memory exception
byrnHDF Feb 24, 2021
ec3c879
revert to H5_JNI_FATAL_ERROR - support functions need work
byrnHDF Feb 24, 2021
0a56190
Add assertion error for h5util functions
byrnHDF Feb 24, 2021
282ac70
remove duplicate function
byrnHDF Feb 24, 2021
9eae4f5
format fix
byrnHDF Feb 24, 2021
a3d7f13
Revert HD function error handling
byrnHDF Feb 24, 2021
3ea3719
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 25, 2021
2a71393
Update copyright comments
byrnHDF Feb 25, 2021
68d7443
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 25, 2021
dab6971
GH #386 java folder copyright corrections
byrnHDF Feb 26, 2021
bfa3508
Whitespace
byrnHDF Feb 26, 2021
b42646f
GH #359 implement and fix tools 1.6 API usage
byrnHDF Feb 26, 2021
f10a3a1
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Feb 26, 2021
6d17d6f
remove excessive comments
byrnHDF Feb 26, 2021
a9bd5ba
Flip inits to correct ifdef section
byrnHDF Feb 26, 2021
7037609
rework ifdef to be simpler
byrnHDF Feb 26, 2021
9e7b3d7
format issue
byrnHDF Feb 26, 2021
d76e848
Reformat ifdef inits
byrnHDF Feb 26, 2021
49e5203
remove static attribute
byrnHDF Feb 26, 2021
53a1528
format compliance
byrnHDF Feb 26, 2021
bb01c7b
Update names
byrnHDF Feb 28, 2021
70a7b81
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 1, 2021
97ed2a4
Revert because logic relies on float not being int
byrnHDF Mar 1, 2021
f19f7d6
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 1, 2021
2998e91
Changes noticed from creating merge of #412
byrnHDF Mar 2, 2021
5b6e1af
Double underscore change
byrnHDF Mar 2, 2021
e478961
Correct compiler version variable used
byrnHDF Mar 3, 2021
f00ce36
Remove header guard underscores
byrnHDF Mar 3, 2021
24db13a
Whitespace cleanup
byrnHDF Mar 3, 2021
1309524
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 4, 2021
0b04714
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 5, 2021
e460b2b
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 5, 2021
c7115cd
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 8, 2021
746ab82
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 10, 2021
d429b3f
Split format source and commit changes on repo push
byrnHDF Mar 10, 2021
9029a70
remove pre-split setting
byrnHDF Mar 10, 2021
04284c1
Change windows TS to use older VS.
byrnHDF Mar 10, 2021
33f8693
correct window os name
byrnHDF Mar 10, 2021
723aa6d
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 10, 2021
49f92f0
HDFFV-11212 JNI export util and Javadoc
byrnHDF Mar 11, 2021
b6d4127
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 11, 2021
e555eb6
Suggested review changes
byrnHDF Mar 11, 2021
720c393
Another change found
byrnHDF Mar 11, 2021
c3bd691
Committing clang-format changes
github-actions[bot] Mar 11, 2021
a57c9b7
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 12, 2021
cadc876
HDFFV-11113 long double in tools
byrnHDF Mar 12, 2021
fdd4a51
HDFFV-11113 add note
byrnHDF Mar 12, 2021
efe338b
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 15, 2021
421ab67
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 17, 2021
d67f8d3
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 18, 2021
e6972f2
Disable long double tests for now
byrnHDF Mar 22, 2021
866da1a
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 22, 2021
ef237e4
Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into d…
byrnHDF Mar 22, 2021
3691b55
HDFFV-11228 remove arbitrary CMake warning groups.
byrnHDF Mar 22, 2021
fdd06ab
Make each flag a string
byrnHDF Mar 22, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
140 changes: 30 additions & 110 deletions config/cmake/HDFCXXCompilerFlags.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -91,18 +91,18 @@ if (NOT MSVC AND NOT MINGW)
if (CMAKE_CXX_COMPILER_ID STREQUAL "Intel")
ADD_H5_FLAGS (HDF5_CMAKE_CXX_FLAGS "${HDF5_SOURCE_DIR}/config/intel-warnings/general")
if(NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 18.0)
list (APPEND H5_CXXFLAGS0 "-Wextra-tokens -Wformat -Wformat-security -Wic-pointer -Wshadow")
list (APPEND H5_CXXFLAGS0 "-Wsign-compare -Wtrigraphs -Wwrite-strings")
list (APPEND H5_CXXFLAGS "-Wextra-tokens -Wformat -Wformat-security -Wic-pointer -Wshadow")
list (APPEND H5_CXXFLAGS "-Wsign-compare -Wtrigraphs -Wwrite-strings")
endif()
elseif (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
if (CMAKE_COMPILER_IS_GNUCXX AND CMAKE_CXX_COMPILER_LOADED
AND CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 4.8)
# add the general CXX flags for g++ compiler versions 4.8 and above.
ADD_H5_FLAGS (HDF5_CMAKE_CXX_FLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-general")
if (HDF5_ENABLE_WARNINGS_AS_ERRORS)
ADD_H5_FLAGS (H5_CXXFLAGS0 "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-error-general")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-error-general")
else ()
ADD_H5_FLAGS (H5_CXXFLAGS0 "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-noerror-general")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-noerror-general")
endif ()
endif ()
elseif (CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
Expand All @@ -122,20 +122,20 @@ if (NOT MSVC AND NOT MINGW)
if (HDF5_ENABLE_DEV_WARNINGS)
message (STATUS "....HDF5 developer group warnings are enabled")
# if (CMAKE_CXX_COMPILER_ID STREQUAL "Intel")
# list (APPEND H5_CXXFLAGS0 "-Winline -Wreorder -Wport -Wstrict-aliasing")
# list (APPEND H5_CXXFLAGS "-Winline -Wreorder -Wport -Wstrict-aliasing")
# elseif (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS0 "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-general")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-general")
# elseif (CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
# ADD_H5_FLAGS (H5_CXXFLAGS0 "${HDF5_SOURCE_DIR}/config/clang-warnings/developer-general")
# ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/clang-warnings/developer-general")
endif ()
else ()
if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS0 "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-general")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-general")
# elseif (CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
# ADD_H5_FLAGS (H5_CXXFLAGS0 "${HDF5_SOURCE_DIR}/config/clang-warnings/no-developer-general")
# ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/clang-warnings/no-developer-general")
endif ()
endif ()

Expand All @@ -144,41 +144,41 @@ if (NOT MSVC AND NOT MINGW)
# we should approach them a bit cautiously... Only needed for gcc 4.X
if (CMAKE_CXX_COMPILER_VERSION VERSION_LESS 5.0 AND CMAKE_C_COMPILER_VERSION VERSION_GREATER_EQUAL 4.8)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/4.8-4.last")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/4.8-4.last")
endif ()

# Append more extra warning flags that only gcc 4.8+ know about
if (NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 4.8)
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/4.8")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/4.8")
if (HDF5_ENABLE_DEV_WARNINGS)
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-4.8")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-4.8")
else ()
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-4.8")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-4.8")
endif ()
endif ()

# Append more extra warning flags that only gcc 4.9+ know about
if (NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 4.9)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/4.9")
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-4.9")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/4.9")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-4.9")
endif ()

# Append more extra warning flags that only gcc 5.1+ know about
if (NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 5.0)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-5")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-5")
if (HDF5_ENABLE_WARNINGS_AS_ERRORS)
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-error-5")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-error-5")
else ()
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-noerror-5")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/cxx-noerror-5")
endif ()
endif ()

# Append more extra warning flags that only gcc 6.x+ know about
if (NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 6.0)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS1 "${HDF5_SOURCE_DIR}/config/gnu-warnings/6")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/6")
endif ()

# Append more extra warning flags that only gcc 7.x+ know about
Expand All @@ -187,34 +187,34 @@ if (NOT MSVC AND NOT MINGW)
ADD_H5_FLAGS (H5_CXxFLAGS2 "${HDF5_SOURCE_DIR}/config/gnu-warnings/7")
if (HDF5_ENABLE_DEV_WARNINGS)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS2 "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-7")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-7")
#else ()
# ADD_H5_FLAGS (H5_CXXFLAGS2 "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-7")
# ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-7")
endif ()
endif ()

# Append more extra warning flags that only gcc 8.x+ know about
if (NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 8.0)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS3 "${HDF5_SOURCE_DIR}/config/gnu-warnings/8")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/8")
#if (HDF5_ENABLE_WARNINGS_AS_ERRORS)
# ADD_H5_FLAGS (H5_CXXFLAGS3 "${HDF5_SOURCE_DIR}/config/gnu-warnings/error-8")
# ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/error-8")
#else ()
# ADD_H5_FLAGS (H5_CXXFLAGS3 "${HDF5_SOURCE_DIR}/config/gnu-warnings/noerror-8")
# ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/noerror-8")
#endif ()
if (HDF5_ENABLE_DEV_WARNINGS)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS3 "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-8")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/developer-8")
else ()
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS3 "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-8")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/no-developer-8")
endif ()
endif ()

# Append more extra warning flags that only gcc 9.x+ know about
if (NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 9.0)
# autotools always add the C flags with the CXX flags
ADD_H5_FLAGS (H5_CXXFLAGS4 "${HDF5_SOURCE_DIR}/config/gnu-warnings/9")
ADD_H5_FLAGS (H5_CXXFLAGS "${HDF5_SOURCE_DIR}/config/gnu-warnings/9")
endif ()
endif ()
else ()
Expand All @@ -232,97 +232,17 @@ if (HDF5_ENABLE_ALL_WARNINGS)
if (HDF5_ENABLE_DEV_WARNINGS)
if (CMAKE_CXX_COMPILER_LOADED)
string (REGEX REPLACE "(^| )([/-])W[0-9]( |$)" " " CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS}")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/Wall /wd4668")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/Wall" "/wd4668")
endif ()
else ()
if (CMAKE_CXX_COMPILER_LOADED)
string (REGEX REPLACE "(^| )([/-])W[0-9]( |$)" " " CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS}")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/W3")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/W3" "/wd4100" "/wd4706" "/wd4127")
endif ()
endif ()
else ()
if (CMAKE_CXX_COMPILER_LOADED)
list (APPEND HDF5_CMAKE_CXX_FLAGS ${H5_CXXFLAGS0} ${H5_CXXFLAGS1} ${H5_CXXFLAGS2} ${H5_CXXFLAGS3} ${H5_CXXFLAGS4})
endif ()
endif ()
endif ()

#-----------------------------------------------------------------------------
# Option to allow the user to enable warnings by groups
#-----------------------------------------------------------------------------
if (HDF5_ENABLE_GROUPZERO_WARNINGS)
message (STATUS "....Group Zero warnings are enabled")
if (MSVC)
if (CMAKE_CXX_COMPILER_LOADED)
string (REGEX REPLACE "(^| )([/-])W[0-9]( |$)" " " HDF5_CMAKE_CXX_FLAGS "${HDF5_CMAKE_CXX_FLAGS}")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/W1")
endif ()
else ()
if (CMAKE_CXX_COMPILER_LOADED)
list (APPEND HDF5_CMAKE_CXX_FLAGS ${H5_CXXFLAGS0})
endif ()
endif ()
endif ()

#-----------------------------------------------------------------------------
# Option to allow the user to enable warnings by groups
#-----------------------------------------------------------------------------
if (HDF5_ENABLE_GROUPONE_WARNINGS)
message (STATUS "....Group One warnings are enabled")
if (MSVC)
if (CMAKE_CXX_COMPILER_LOADED)
string (REGEX REPLACE "(^| )([/-])W[0-9]( |$)" " " HDF5_CMAKE_CXX_FLAGS "${HDF5_CMAKE_CXX_FLAGS}")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/W2")
endif ()
else ()
if (CMAKE_CXX_COMPILER_LOADED)
list (APPEND HDF5_CMAKE_CXX_FLAGS ${H5_CXXFLAGS1})
endif ()
endif ()
endif ()

#-----------------------------------------------------------------------------
# Option to allow the user to enable warnings by groups
#-----------------------------------------------------------------------------
if (HDF5_ENABLE_GROUPTWO_WARNINGS)
message (STATUS "....Group Two warnings are enabled")
if (MSVC)
if (CMAKE_CXX_COMPILER_LOADED)
string (REGEX REPLACE "(^| )([/-])W[0-9]( |$)" " " HDF5_CMAKE_CXX_FLAGS "${HDF5_CMAKE_CXX_FLAGS}")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/W3")
endif ()
else ()
if (CMAKE_CXX_COMPILER_LOADED)
list (APPEND HDF5_CMAKE_CXX_FLAGS ${H5_CXXFLAGS2})
endif ()
endif ()
endif ()

#-----------------------------------------------------------------------------
# Option to allow the user to enable warnings by groups
#-----------------------------------------------------------------------------
if (HDF5_ENABLE_GROUPTHREE_WARNINGS)
message (STATUS "....Group Three warnings are enabled")
if (MSVC)
if (CMAKE_CXX_COMPILER_LOADED)
string (REGEX REPLACE "(^| )([/-])W[0-9]( |$)" " " HDF5_CMAKE_CXX_FLAGS "${HDF5_CMAKE_CXX_FLAGS}")
list (APPEND HDF5_CMAKE_CXX_FLAGS "/W4")
endif ()
else ()
if (CMAKE_CXX_COMPILER_LOADED)
list (APPEND HDF5_CMAKE_CXX_FLAGS ${H5_CXXFLAGS3})
endif ()
endif ()
endif ()

#-----------------------------------------------------------------------------
# Option to allow the user to enable warnings by groups
#-----------------------------------------------------------------------------
if (HDF5_ENABLE_GROUPFOUR_WARNINGS)
message (STATUS "....Group Four warnings are enabled")
if (NOT MSVC)
if (CMAKE_CXX_COMPILER_LOADED)
list (APPEND HDF5_CMAKE_CXX_FLAGS ${H5_CXXFLAGS4})
list (APPEND HDF5_CMAKE_CXX_FLAGS ${H5_CXXFLAGS})
endif ()
endif ()
endif ()
Expand Down
Loading