summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Various warning fixes (#1812)Sean McBride2022-06-2237-304/+143
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixed -Wreserved-id-macro warnings from header include guards * Removed all __int64 and LL suffix stuff now that C99 is minimum requirement * Rename `H5FD_CTL__` to `H5FD_CTL_` to fix -Wreserved-id-macro warnings Double underscore is reserved in C++ and this public header should be C++ compatible. * Never define __STDC_FORMAT_MACROS anymore Defining it causes a -Wreserved-id-macro. Happily, according to the C++11 standard: "The macros defined by <stdint> are provided unconditionally. In particular, the symbols __STDC_LIMIT_MACROS and __STDC_CONSTANT_MACROS (mentioned in C99 footnotes 219, 220, and 222) play no role in C++." https://cplusplus.github.io/LWG/issue984 So looks like it's not necessary to define it with reasonably new toolchains. * Fixed some -Wunused-macros warnings, removed dead code * Fixed all -Wdouble-promotion warnings in C++ files * Fixed remaining -Wsuggest-destructor-override warnings * Committing clang-format changes Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Fix bugs in parallel selection I/O (#1803)Neil Fortner2022-06-091-40/+44
|
* Develop sync (#1793)Allen Byrne2022-06-0912-21/+37
| | | | | * Synch update * Branch sync changes
* Updates the release schedule (#1796)Dana Robinson2022-06-063-8/+7
|
* Update windows VS2019 and fix comments (#1781)Allen Byrne2022-05-203-4/+4
|
* Remove extra ')'. (#1779)Larry Knox2022-05-191-3/+3
| | | Add configure help text for TestExpress.
* Set default level to minimum time (#1722)Allen Byrne2022-05-1711-29/+79
|
* Replace obsolete docver arg with arg to supply branch name (#1771)Allen Byrne2022-05-131-31/+11
|
* Allows parallel + thread-safe combination in Autotools and CMake (#1760)Dana Robinson2022-05-132-17/+0
|
* Use windows ENV var for default install dir setting (#1763)Allen Byrne2022-05-122-2/+2
| | | | | * Use windows ENV var for default install dir setting * Change variable name
* Revert "Cleanup and shellcheck fixes in the SWMR test script (#1741)" (#1758)Dana Robinson2022-05-101-53/+56
| | | This reverts commit cea0dc4565312f519c26de17b987b75652ab95df.
* Add a workaround for buggy PGI C Compiler version 19.10, which quits (#1705)David Young2022-05-081-4/+15
| | | | with the unhelpful and misleading error message `PGC-S-0053-Illegal use of void type` when I put a `static` qualifier on my tables.
* Use git command to get branch name (#1748)Allen Byrne2022-05-071-1/+1
|
* Cleanup and shellcheck fixes in the SWMR test script (#1741)Dana Robinson2022-05-071-56/+53
|
* Re-promote some GNU warnings to errors (#1508)David Young2022-05-068-60/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Promote some GCC warnings back to errors and deal with build breakage. I tested configuring and building with GCC 8.3.0 with `.../configure --prefix=$HOME/wrk/install/manydsets --enable-build-mode=production --enable-symbols=-gdwarf-4 --enable-warnings-as-errors --enable-java --enable-hl --enable-fortran --enable-cxx --enable-mirror-vfd --enable-ros3-vfd --enable-direct-vfd` and with the same options but `--enable-build-mode=debug`. * Promote -Wrestrict to -Werror=restrict to help catch overlapping arguments to strcpy, memcpy, et cetera, at compile time. * NFCI: sort lines. * Committing clang-format changes * NFCI: remove whitespace at EOL. * Re-use temporary variable `shared`. * Remove merge oopsie. * Mention that no op codes are understood per review comment. * Change this back to the develop branch's way, since this change isn't integral to the PR. * Committing clang-format changes * NFCI: reduce differences with `develop` branch in the hope that that also kicks off CI. Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Dana Robinson <43805+derobins@users.noreply.github.com>
* Change order of naming commands (#1742)Allen Byrne2022-05-061-1/+2
|
* Minor things noticed while merging to 1.10 (#1739)Dana Robinson2022-05-066-90/+82
| | | | | * Minor things noticed while merging to 1.10 * Set HDF5_DRIVER back in string
* Fixes for S3 and HDFS VFDs with tools (#1740)jhendersonHDF2022-05-065-59/+38
|
* Fix a few Clang sanitizer warnings (#1727)jhendersonHDF2022-05-053-18/+32
|
* changes parallel tests to .F90 ext. and fixes the CALL check strings (#1725)Scot Breitenfeld2022-05-045-19/+18
|
* h5test.h testing macros get enclosed in do..while loops (#1721)Dana Robinson2022-05-0483-25356/+25428
| | | | | | | * h5test.h testing macros get enclosed in do..while loops * Adds missed macro in hl C++ code * Fixes macro in Windows code
* Fixes a bug where t_cache fails due to a string size being too small (#1720)Dana Robinson2022-05-041-4/+6
| | | | | | | | | * Fixes a bug where t_cache fails due to a string size being too small Recent warning reductions led to an incorrect string size being passed to h5_fileaccess, causing the test to silently fail. In addition to fixing the bug, the test will now fail noisily on setup failures. * Updates the t_cache test to fail noisily on setup errors
* Fixes unused/unset variable warnings from aocc (#1712)Dana Robinson2022-05-034-35/+5
| | | | | | | * Fixes unused/unset variable warnings from aocc * Committing clang-format changes Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Updates warnhist script to deal with AMD Optimizing Compiler messages (#1713)Dana Robinson2022-05-031-0/+3
|
* test/dt_arith: use `isnan` instead of self comparison (#1709)Ben Boeckel2022-05-031-5/+5
| | | | | It is more obvious what is being asked this way. Also include a `clang-format` fix.
* Fix some const cast and stack/static object size warnings (#1700)jhendersonHDF2022-05-0121-110/+282
| | | | | | | | | * Fix various warnings * Move HDfree_const to H5private.h for wider use * Print output from all ranks in parallel tests on allocation failure * Move const pointer freeing macro to h5test.h for now
* Fix object size warnings in cache.c test (#1701)jhendersonHDF2022-05-011-4051/+2717
|
* Fix SWMR/refresh bug hidden by library free lists (#1702)jhendersonHDF2022-04-301-4/+10
|
* Be a bit safer with signed arithmetic, thus quieting some signed-overflow ↵David Young2022-04-303-22/+25
| | | | | | | | | | | | | | | | | | | | | | | warnings from GCC (#1706) * Avoid a signed overflow: check the range of `entry_ptr->age` before increasing it instead of increasing it and then checking the range. This quiets a GCC warning. * Avoid the potential for signed overflow by rewriting expressions `MAX(0, fwidth - n)` as `MAX(n, fwidth) - n` for various `n`. This change quiets some GCC warnings. * Change some local variables that cannot take sensible negative values from signed to unsigned. This quiets GCC warnings about potential signed overflow. * In a handful of instances, check the range of a signed integer before increasing/decreasing it, just in case the increase/decrease overflows. This quiets a handful of GCC signed-overflow warnings. * Committing clang-format changes Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Fixes stack size warnings in set_extent test (#1698)Dana Robinson2022-04-281-77/+127
|
* Fixes stack size warnings in dtransform (#1696)Dana Robinson2022-04-271-10/+26
|
* Fixes stack size warnings in ntypes (#1695)Dana Robinson2022-04-261-15/+60
|
* Address some warnings from casting away of const (#1684)jhendersonHDF2022-04-2616-121/+97
|
* Correct concurrency bugs when running tests, along with a bugfix & small ↵Quincey Koziol2022-04-264-34/+202
| | | | | | | | | | warning cleanup (#1683) * Correct concurrency bugs when running tests, along with a bugfix & small warning cleanup. * Committing clang-format changes Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Minor things noticed while bringing VFD SWMR in line with develop (#1691)Dana Robinson2022-04-254-13/+12
|
* Fixes stack size warnings in tcoords.c (#1688)Dana Robinson2022-04-251-14/+48
|
* Clean stack size warnings in sio_engine (#1687)Dana Robinson2022-04-251-20/+35
|
* Warnings fixes (#1680)jhendersonHDF2022-04-2517-152/+169
|
* Improve handling of copying of dynamic libraries and clean them up after (#1681)Quincey Koziol2022-04-245-32/+45
| | | test finishes.
* Replace H5detect's build-time detection of C99 integer properties with a (#1400)David Young2022-04-224-718/+398
| | | | | | | * Replace H5detect's build-time detection of C99 integer properties with a table-driven routine, `H5T__init_native_int()`, that is run at library initialization time.
* Changes Github action `hdf5 dev CI` to `PR hdf5 dev CI` (#1666)Dana Robinson2022-04-201-1/+1
| | | So the PR action name is not the same as the one in main.yml
* Adds paths-ignore to the Github pull request workflow (#1663)Dana Robinson2022-04-201-1/+8
|
* Adds the -q flag to all swmr test programs, quieting noisy output (#1665)Dana Robinson2022-04-201-15/+15
|
* added HDFFV-11306 entry (#1662)Scot Breitenfeld2022-04-191-1/+6
|
* Develop clang format java (#1653)Allen Byrne2022-04-19162-10098/+13367
|
* HDFFV-11306 Fixed (#1657)Scot Breitenfeld2022-04-199-23/+139
| | | | | | | | * HDFFV-11306, * Fixed it so both h5open_f and h5close_f can be called multiple times. * Fixed an issue with open objects remaining after h5close_f was called. * Added additional tests. * comments clean-up
* Removes HDF Group paths, adds shellcheck fixes (#1656)Dana Robinson2022-04-191-95/+41
| | | | | | For more information: https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ... https://www.shellcheck.net/wiki/SC2230 -- which is non-standard. Use builti... https://www.shellcheck.net/wiki/SC2006 -- Use $(...) notation instead of le...
* Mirror vfd test fixes (#1629)Quincey Koziol2022-04-184-14/+28
| | | | | | | | | | | | | | | | | | | * Use the FAPL that was created earlier in the test (and delete an unused variable). This allows 'make check-vfd' to pass with --enable-mirror-vfd. * Check for testing directory before creating, to avoid warning from bash. Clean out .libs directory before re-using it (after a failed test), to remove any files generated by libtool. * Committing clang-format changes * Increment error count on failed file open and skip tests for VFDs that need modified filenames. * Skip the mirror VFD for 'make check-vfd' - the mirror VFD requires networking configuration parameters and can't be provided for an automated test that is configured with an environment variable. Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Cleans up some HL library code that inappropriately returns htri_t values ↵Dana Robinson2022-04-185-196/+144
| | | | | | | | | | | | cast to herr_t (#1651) * Cleans up some HL library code that inappropriately returns htri_t values cast to herr_t * Committing clang-format changes * Formatted source Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Remove H5_NO_ALIGNMENT_RESTRICTIONS (#1426)David Young2022-04-165-153/+20
| | | | | | | | | | | | | | | | | * Do not conditionally compile code that uses a pointer dereference and assignment to copy a potentially unaligned variable to aligned automatic storage, or vice versa. Instead, always use naked `memcpy(3)`s. Disassembling the generated code reveals that the `memcpy(3)`s optimize (`-O3`) to a single `mov` instruction for x86_64, which is not strict about alignment. This change reduces the size of code and scripts by 143 lines, eases our way to cross-compilation, and avoids invoking undefined behavior. * Committing clang-format changes * Per discussion, use HD and add comments. Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>