summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Minor refactoring to the VFD info free call.Dana Robinson2020-05-201-10/+8
|
* Updated the 'const memory free' changes based on PR feedback.Dana Robinson2020-05-209-45/+27
|
* Added a free wrapper that lets us free constant pointers withoutDana Robinson2020-05-208-25/+62
| | | | | | | generating warnings. Also, brought the const-ness of the VOL connector info in line with the VFD info (not visible externally).
* VS2010 needs an underscoreAllen Byrne2020-05-201-0/+6
|
* Cleaned up remaining warnings in high-level library.Dana Robinson2020-05-207-718/+988
|
* Small fixes for tools and skip autotools tests for no filterAllen Byrne2020-05-204-29/+28
|
* Reduce differences between my -Werror branch and `develop`:David Young2020-05-2026-276/+265
| | | | | | | | Rename index -> idx, fileno -> fnumber, fileno -> fno to avoid GCC shadowed declaration warnings about index(3). Convert #pragma GCC diagnostic push/pop/ignored to the HDF5 library's H5_GCC_DIAG_OFF()/H5_GCC_DIAG_ON() macros.
* Reduce casts of HDcalloc()/HDmalloc() that -Wc++-compat required.David Young2020-05-209-83/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce gratuitous casts---e.g., (size_t)1. Use the right format string for a pointer. In the H5C sanity checks, change a "size increase" variable from ssize_t (too narrow) to int64_t (wide enough). Parenthesize every appearance of `storage` in the macro `H5D_CHUNK_STORAGE_INDEX_CHK(storage)` so that you can pass in an expression like &sc and it works properly. Disallow re-assignment of the `dset` parameter to H5D__chunk_init() because it helped assure me that it's safe to replace the repeating expression `&dset->shared->layout.storage.u.chunk` with `sc` throughout. Replace lengthy expressions such as `&dset->shared->layout.storage.u.chunk` with `sc` throughout several functions in H5Dchunk.c ISTR that the compiler warned that `sc` was declared but unused in a couple of functions, and then I found that `sc` could be used in many places. Maybe the disused `sc` appeared because a bunch of code was copied and pasted, I don't know. Anyway, it's a lot tighter code now that I use `sc`. In H5D__chunk_update_old_edge_chunks() and H5D__chunk_delete() I actually expand `sc` and another temporary variable, `pline`, because they're used only in !defined(NDEBUG) code. This squashes unused-variable warnings in the defined(NDEBUG) configuration. Don't drop the `volatile` qualification with a cast in tools/src/h5import/h5import.c.
* testpar/t_2Gio.c: Fix a typo that I think was introduced by aDavid Young2020-05-2014-55/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | previous warnings PR. An array element was assigned to itself---shape[2]Â =Â shape[2];---instead of being assigned to chunk[2]. fortran/src/H5Pf.c: move conditional compilation controlled by H5_NO_DEPRECATED_SYMBOLS outside of a function for readability. fortran/src/H5match_types.c: put a variable's declaration under the same conditional compilation (H5_FORTRAN_HAVE_C_LONG_DOUBLE) as its use. For now, skip compilation of some unused debug dump routines in the JNI. While I'm in the JNI, delete a set-but-unused variable. src/H5Z.c: condition a variable declaration on H5_NO_DEPRECATED_SYMBOLS so that it's not declared but unused or vice versa. test/cache_common.h: add an #include in to get some symbols we need to avoid implicit declaration warnings. test/dsets.c: use a more conventional conditional-compilation syntax. test/dt_arith.c, test/fillval.c: initialize a bunch of uninitialized variables before use. test/vfd.c: pass the expected type of `void **` to posix_memalign(3) instead of `int **`. testpar/t_bigio.c: explicitly compare with 0 instead of using ! when "equal to 0?" is the question not "is false?" Repair some indentation while I'm here. testpar/testpar.h: repair misaligned line-continuation backslashes in a macro that probably should be a function so that we don't have to fiddle with the line continuation to begin with. tools/src/h5repack/h5repack_main.c: fix some compiler fussing about enums. tools/test/perform/pio_engine.c: the compiler fusses if you cast a function call returning double directly to off_t. It's ok if you cast a variable that's a double to off_t, however. Write and use a new function, sqrto(), to avoid the cast warnings.
* fix bad function cast warningkmu2020-05-201-2/+2
|
* initialization discards const warningkmu2020-05-201-7/+7
|
* fix unused function warningkmu2020-05-202-40/+69
|
* HDFFV-11011 - merge include listAllen Byrne2020-05-201-2/+1
|
* HDFFV-11011 correct issues with target file extensionAllen Byrne2020-05-2020-248/+166
|
* Introduce new H5VL _by_value routinesJordan Henderson2020-05-2017-91/+545
|
* Correctly manage warnings for new testsAllen Byrne2020-05-202-38/+34
|
* Ext libs ZLIB and SZIP warnings are separate concernsAllen Byrne2020-05-201-0/+1
|
* Remove commented warnings lineAllen Byrne2020-05-201-6/+0
|
* rename macrokmu2020-05-201-2/+2
|
* change conditionkmu2020-05-201-1/+1
|
* pick up from Dave's fixkmu2020-05-202-241/+244
|
* leave Wswitch-default for later fixkmu2020-05-202-37/+13
|
* fix Wredundant-decls, Wswitch-default, Wdeclaration-after-statement, ↵kmu2020-05-2018-159/+231
| | | | Wsign-compare, Wmisleading-indentation, Wshadow
* fix issues from test failkmu2020-05-201-12/+15
|
* Replaced my_isnan with C99 isnan in h5diff.Dana Robinson2020-05-202-116/+45
|
* Add another warning variationAllen Byrne2020-05-201-0/+1
|
* Correct ignore warningsAllen Byrne2020-05-201-2/+4
|
* Only use sanitizer with ClangAllen Byrne2020-05-201-1/+1
|
* Remove cmake macros from valgrind testsAllen Byrne2020-05-203-124/+109
|
* Restore fix of test assert fail eror undone by later commit.Larry Knox2020-05-201-1/+1
|
* Cleaned up warnings in test/tid.c.Dana Robinson2020-05-201-9/+16
|
* fix assert failkmu2020-05-201-1/+1
|
* Optimized the floating point comparisons a little bit.Dana Robinson2020-05-201-24/+24
|
* Fix for failing h5diff tests involving floating-point compares.Dana Robinson2020-05-201-6/+18
|
* Removed H5_DEC_ENUMDana Robinson2020-05-201-7/+0
|
* Revert "Revert "Moved -Wunsuffixed-float-constants to the developer warnings.""Dana Robinson2020-05-2011-51/+38
| | | | This reverts commit 6486e06545bf637d46e18a787395542f4fca143a.
* Revert "Moved -Wunsuffixed-float-constants to the developer warnings."Dana Robinson2020-05-2011-38/+51
| | | | This reverts commit 12bc75c6172ca7b64c33b17cf538a7629500cf2f.
* Moved -Wunsuffixed-float-constants to the developer warnings.Dana Robinson2020-05-2011-51/+38
|
* Fixed a bug in testpar/t_cache.c concerning checking expectedDana Robinson2020-05-201-2/+2
| | | | vs. actual cache entry reads and writes.
* remove unnecessary stuffkmu2020-05-201-2/+0
|
* squash cast warning fixkmu2020-05-2044-653/+687
|
* Change tools debug macros to not use error stackAllen Byrne2020-05-2010-203/+142
|
* Max Library "index" should be 7.Larry Knox2020-05-201-1/+1
|
* not use hdf5 private headerkmu2020-05-201-1/+3
|
* fix float type cmp warningkmu2020-05-201-0/+2
|
* fix float type cmp warningkmu2020-05-202-7/+8
|
* fix missing prototype warningkmu2020-05-203-4/+3
|
* more cleanupkmu2020-05-205-11/+7
|
* change it backkmu2020-05-201-2/+1
|
* pick up missing piecekmu2020-05-202-2/+3
|