summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* TRILAB-142 Change minimum CMake version to 3.12Allen Byrne2020-02-211-1/+1
|
* Merge developJordan Henderson2020-02-21208-6115/+17611
|\
| * Replace the old H5F_DECODE_LENGTH() implementation with one thatDavid Young2020-02-191-1/+1
| | | | | | | | initializes the variable it's loading on every path.
| * Merge pull request #2367 in HDFFV/hdf5 from ~DYOUNG/werror:unused to developDavid Young2020-02-198-23/+53
| |\ | | | | | | | | | | | | | | | | | | * commit '50aac126d7c4082c2aeb0f3a68cad26bd1cd9783': Start to document the function/parameter/variable attribute macros. Filter all of the new _USED / _UNUSED type modifiers. H5_HAVE_PARALLEL, _DEBUG_API, etc. Add attributes to some variables and parameters that are unused under some configurations.
| | * H5_HAVE_PARALLEL, _DEBUG_API, etc. Add attributes to some variables andDavid Young2020-02-108-23/+53
| | | | | | | | | | | | parameters that are unused under some configurations.
| * | Merge pull request #2369 in HDFFV/hdf5 from ~DYOUNG/vs2010:develop to developDavid Young2020-02-173-13/+41
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merged per discussion with Elena. * commit 'b4697f969295245840350e17d75d92a0fdd7e7a9': Use HD prefix. Delete unhelpful comment per Jordan's question. Make sure that H5TS_thread_id() is available as either a function or a macro in all configurations. Provide local copies of err(3)- and errx(3)-alike functions for Visual Studio compatibility. Provide C99/POSIX.1 format strings PRI[doux]{8,16,32,64,MAX,PTR} on systems that are missing <inttypes.h>.
| | * \ Make sure that H5TS_thread_id() is available as either a function or a macro inDavid Young2020-02-102-12/+2
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | all configurations. (Merge remote-tracking branch 'origin/thread_id_everywhere' into vs2010-develop)
| | | * | Delete unhelpful comment per Jordan's question.David Young2020-02-101-3/+0
| | | | |
| | | * | Make sure that H5TS_thread_id() is available as either a function or a macro inDavid Young2020-02-072-12/+5
| | | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | all configurations. Previously it was neither declared nor defined in --disable-threadsafety builds. The compiler's warning got lost in the noise---I first saw the issue because my -Werror branch stopped compiling cold---and the tests still linked and ran.
| | * | Provide C99/POSIX.1 format strings PRI[doux]{8,16,32,64,MAX,PTR} on systemsDavid Young2020-02-071-1/+39
| | |/ | | | | | | | | | that are missing <inttypes.h>.
| * | Merge pull request #2325 in HDFFV/hdf5 from ~DYOUNG/werror:h5t_copy to developDavid Young2020-02-1715-194/+480
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merged per discussion with Elena. * commit '3f903a441ad84001ea66589728bd8b036b6fdfca': Take out the temporary performance tests. Make calls through a function pointer. Use the same number of arguments, always. Increase iterations, provide a baseline for no-op, simplify the overhead case a bit. Temporarily add some code that measures the time to run the simplest possible H5T__copy_all()-like routine 10 million times and then measures the version with FUNC_ENTER_STATIC/_LEAVE_NOAPI and a HGOTO_ERROR() statement. H5T_copy() constification plus Quincey's contributions.
| | * | H5T_copy() constification plus Quincey's contributions.David Young2020-01-2915-194/+480
| | | |
| * | | Fix improper S3 signing key null check (HDFFV-11015).Jacob Smith2020-02-071-2/+2
| | |/ | |/|
| * | If H5_HAVE_THREADSAFE is not #defined, define nothing but a stub implementationDavid Young2020-02-051-0/+7
| | | | | | | | | | | | of H5TS_thread_id().
| * | src/H5Eint.c: #include H5TSprivate.h for H5TS_thread_id() definitions.David Young2020-02-031-0/+1
| | | | | | | | | | | | test/thread_id.c: move threads_failure() inside #ifdefs.
| * | Change thread IDs to uint64_t from unsigned long, per Quincey's suggestion.David Young2020-02-034-13/+13
| | | | | | | | | | | | | | | Fix a typo in the H5TS_thread_init() comment and reword some ID properties.
| * | Replace pthread_self_ulong() with H5TS_thread_id(). The POSIX ThreadsDavid Young2020-02-036-21/+124
| | | | | | | | | | | | | | | implementation ought to be portable to any system that has POSIX Threads. On Windows, I use the same API call as before.
| * | Merge pull request #2329 in HDFFV/hdf5 from ~DEROBINS/hdf5_der:H5MM_const to ↵Dana Robinson2020-01-3110-50/+67
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | develop * commit '10301154215fb6a22590e80f7b5ed0005f3e1786': Minor refactoring to the VFD info free call. Updated the 'const memory free' changes based on PR feedback. Added a free wrapper that lets us free constant pointers without generating warnings.
| | * | Minor refactoring to the VFD info free call.Dana Robinson2020-01-311-10/+8
| | | |
| | * | Updated the 'const memory free' changes based on PR feedback.Dana Robinson2020-01-319-45/+27
| | | |
| | * | Added a free wrapper that lets us free constant pointers withoutDana Robinson2020-01-308-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-01-301-0/+6
| |/ /
| * | Merge pull request #2318 in HDFFV/hdf5 from ↵David Young2020-01-2915-127/+125
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | ~DYOUNG/werror:reduce-werror-diffs-1 to develop * commit 'fa1b2f6ef35c459c9f74ead885c119ae7f61c056': Reduce differences between my -Werror branch and `develop`:
| | * | Reduce differences between my -Werror branch and `develop`:David Young2020-01-2915-127/+125
| | |/ | | | | | | | | | | | | | | | | | | | | | 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.
| * | Merge pull request #2320 in HDFFV/hdf5 from ↵David Young2020-01-296-73/+67
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | ~DYOUNG/werror:reduce-werror-diffs-2 to develop * commit 'a5f236e83bdd29e660a55c5577a69a3809605cda': Reduce casts of HDcalloc()/HDmalloc() that -Wc++-compat required.
| | * | Reduce casts of HDcalloc()/HDmalloc() that -Wc++-compat required.David Young2020-01-296-73/+67
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Merge pull request #2321 in HDFFV/hdf5 from ↵David Young2020-01-291-1/+3
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | ~DYOUNG/werror:reduce-werror-diffs-3 to develop * commit 'f19e06b59ee53f17465a7b5974c25c3245a40d9a': testpar/t_2Gio.c: Fix a typo that I think was introduced by a previous warnings PR. An array element was assigned to itself---shape[2]Â =Â shape[2];---instead of being assigned to chunk[2].
| | * | testpar/t_2Gio.c: Fix a typo that I think was introduced by aDavid Young2020-01-291-1/+3
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | Introduce new H5VL _by_value routinesJordan Henderson2020-01-295-25/+227
| |/
| * Merge pull request #2311 in HDFFV/hdf5 from ~KMU/hdf5:misc to developKimmy Mu2020-01-281-1/+1
| |\ | | | | | | | | | | | | | | | | | | | | | | | | * commit '406330addf45529ecdd088f4d22e9aab9a03661d': rename macro change condition pick up from Dave's fix leave Wswitch-default for later fix fix Wredundant-decls, Wswitch-default, Wdeclaration-after-statement, Wsign-compare, Wmisleading-indentation, Wshadow
| | * fix Wredundant-decls, Wswitch-default, Wdeclaration-after-statement, ↵kmu2020-01-271-1/+1
| | | | | | | | | | | | Wsign-compare, Wmisleading-indentation, Wshadow
| * | Replaced my_isnan with C99 isnan in h5diff.Dana Robinson2020-01-241-0/+3
| | |
| * | Removed H5_DEC_ENUMDana Robinson2020-01-231-7/+0
| | |
| * | Revert "Revert "Moved -Wunsuffixed-float-constants to the developer warnings.""Dana Robinson2020-01-232-16/+1
| |/ | | | | | | This reverts commit 6486e06545bf637d46e18a787395542f4fca143a.
| * remove unnecessary stuffkmu2020-01-231-2/+0
| |
| * squash cast warning fixkmu2020-01-2311-91/+95
| |
| * merge and fix conflictkmu2020-01-23150-3016/+7804
| |\
| | * Merge pull request #2292 in HDFFV/hdf5 from ~KMU/hdf5:bugfix/prototype to ↵Kimmy Mu2020-01-231-1/+2
| | |\ | | | | | | | | | | | | | | | | | | | | | | | | develop * commit '9f80ffa1f1908975b3100814b2529bf779521e13': fix missing prototype warning
| | | * fix missing prototype warningkmu2020-01-211-1/+2
| | | |
| | * | Merge pull request #2094 in HDFFV/hdf5 from ~KMU/hdf5:bugfix/unused to developKimmy Mu2020-01-2343-145/+151
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit '145ef3ceee20c28a443bd11507c58858bea3f889': more cleanup change it back pick up missing piece fix unused related warnings removed unused parameter more fix and address comments remove unsed var,function,macro, etc
| | | * | more cleanupkmu2020-01-231-1/+1
| | | | |
| | | * | merge and fixkmu2020-01-2167-1014/+4089
| | | |\ \
| | | * \ \ merge and fix conflictkmu2020-01-1942-123/+142
| | | |\ \ \
| | | * \ \ \ merge and fix conflictskmu2020-01-14101-2516/+4199
| | | |\ \ \ \
| | | * | | | | pick up missing piecekmu2020-01-132-2/+3
| | | | | | | |
| | | * | | | | fix unused related warningskmu2020-01-1340-128/+138
| | | | | | | |
| | | * | | | | remove unsed var,function,macro, etckmu2019-12-062-16/+2
| | | | | | | |
| | * | | | | | Remove irrelevant sentence from comment.Larry Knox2020-01-221-3/+1
| | | | | | | |
| | * | | | | | Add v111 version for H5O functions to enable version1 default for HDF5Larry Knox2020-01-211-5/+16
| | | |_|_|/ / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | 1.8-1.10 and version 3 default for HDF5 1.12. Version 2 functions are available (deprecated) but will not be the default for any version.
| | * | | | | Only 2 versions of H5O_info_t and H5O_iterate_t.Larry Knox2020-01-201-2/+2
| | | | | | |