diff options
| author | Ryan Schmidt <ryandesign@macports.org> | 2022-11-17 22:28:44 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-11-17 22:28:44 (GMT) |
| commit | 9dd36f016a4316b94f5df15856d228de01d304c6 (patch) | |
| tree | 9899d3084d1c95c442517bf9a6180b079ce69cc7 | |
| parent | 169896aedb4bc870c8adac191407325b80840510 (diff) | |
| download | hdf5-9dd36f016a4316b94f5df15856d228de01d304c6.zip hdf5-9dd36f016a4316b94f5df15856d228de01d304c6.tar.gz hdf5-9dd36f016a4316b94f5df15856d228de01d304c6.tar.bz2 | |
Simplify & fix check for szlib encoder (#2263)
Return the result rather than setting the exit code. "return" is a
language keyword whereas "exit" is a function for which the <stdlib.h>
header has to be included which it wasn't in this test, therefore the
test would previously fail to identify that the encoder was enabled if
"-Werror=implicit-function-declaration" was used, which it is by default
with clang from Xcode 12 and later.
Fixes #2262
| -rw-r--r-- | configure.ac | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/configure.ac b/configure.ac index 7a0d043..a7d1129 100644 --- a/configure.ac +++ b/configure.ac @@ -1737,10 +1737,7 @@ if test "x$HAVE_SZLIB" = "xyes" -a "x$HAVE_SZLIB_H" = "xyes"; then #include "szlib.h" ],[[ /* SZ_encoder_enabled returns 1 if encoder is present */ - if(SZ_encoder_enabled() == 1) - exit(0); - else - exit(1); + return SZ_encoder_enabled() != 1; ]])] , [hdf5_cv_szlib_can_encode=yes], [hdf5_cv_szlib_can_encode=no],)] ) |