summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* 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
| | | | | |
| * | | | | H5vers.txt has versions v10 that should be v110 and some missing v110Larry Knox2020-01-191-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | entries.
| * | | | | Merge branch 'develop' into stack_size_warningsDana Robinson2020-01-1766-1016/+4104
| |\ \ \ \ \
| | * | | | | Squashed commit of the token_refactoring branch:Dana Robinson2020-01-1666-998/+4065
| | | |_|/ / | | |/| | |
| | * | | | H5R: set app ref when incrementing ref_count on location held by referenceJerome Soumagne2020-01-105-18/+39
| | | |/ / | | |/| | | | | | | | | | | | (fix HDFFV-10992)
| * | | | Fixed stack and frame size warnings. Not complete, but fixes most ofDana Robinson2020-01-151-16/+24
| |/ / / | | | | | | | | | | | | the easier cases.
| * | | Merge pull request #2234 in HDFFV/hdf5 from ~KMU/hdf5:bugfix/intel_warnings ↵Kimmy Mu2020-01-071-0/+1
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to develop * commit '30ac64d83aff5e022d51a17fd6c1d72903ae4f02': add new type support
| | * | | add new type supportkmu2020-01-071-0/+1
| | | | |
| * | | | Merge pull request #2071 in HDFFV/hdf5 from ~KMU/hdf5:bugfix/intel_warnings ↵Kimmy Mu2020-01-0627-55/+62
| |\ \ \ \ | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to develop * commit '0a2bb11b248df6841daabca3970df5d8504adfc7': address problems from comments fix and address comments change according to previous comments add missing piece remove unnecessary check macro fix intel compile warnings Revert "fix warnings from Intel compiler" Revert "fix warnings and some text alignment" Revert "let hdf5 pick up the right compiler in Intel environment" Revert "fix issues from previous PR comments" Revert "using a different MACRO" using a different MACRO fix issues from previous PR comments let hdf5 pick up the right compiler in Intel environment fix warnings and some text alignment fix warnings from Intel compiler
| | * | | address problems from commentskmu2019-12-114-4/+5
| | | | |
| | * | | fix and address commentskmu2019-12-115-7/+15
| | | | |
| | * | | change according to previous commentskmu2019-12-102-4/+3
| | | | |
| | * | | add missing piecekmu2019-12-101-0/+1
| | | | |
| | * | | Merge branch 'develop' into bugfix/intel_warningskmu2019-12-0954-394/+972
| | |\ \ \ | | | |/ /
| | * | | remove unnecessary check macrokmu2019-12-0622-35/+34
| | | | |
| | * | | fix intel compile warningskmu2019-12-0425-48/+47
| | | | |
| * | | | Merge pull request #2228 in HDFFV/hdf5 from ↵Quincey Koziol2020-01-0423-150/+191
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | minor_improvements_from_token_refactor to develop * commit '0225e6d59698c9a720177766794619c7ad273f4a': Small changes from the token_refactoring branch, to reduce the delta to develop
| | * | | | Small changes from the token_refactoring branch, to reduce the delta to developQuincey Koziol2020-01-0423-150/+191
| | | | | |
| * | | | | Fix some places where H5P_DEFAULT gets passed down to a VOL connectorJordan Henderson2020-01-0315-117/+102
| |/ / / /
| * | | | Remove unnecessary H5CX callQuincey Koziol2019-12-211-4/+0
| | | | |
| * | | | Refactor H5Dvlen_get_buf_size to use optional dataset operation, with ↵Quincey Koziol2019-12-217-54/+255
| | | | | | | | | | | | | | | | | | | | generic fallback for VOL connectors that don't implement operation
| * | | | Cleanups from PR reviewsQuincey Koziol2019-12-204-102/+101
| | | | |
| * | | | Refactor all the 'H5VL_*_optional' callbacks to move the type of operation outQuincey Koziol2019-12-2045-1572/+2861
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of the va_list, so it's at least possible for another connector to know what the operation is and decide whether to implement it or not. Added a new VOL sub-class called "introspect" where callbacks that report information about the connector or container can be placed. Added an 'opt_query' callback to this sub-class, for a connector to report back to the library whether a particular optional callback operation is supported. Also added a 'get_conn_cls' introspection callback, to retrieve the H5VL_class_t of a connector (either the "current" connector, H5VL_GET_CONN_LVL_CURR, or the terminal connector, H5VL_GET_CONN_LVL_TERM). Moved the "post open" operation from a file 'specific' operation to a file 'optional' operation, now that it's possible to detect (with the 'opt_query' introspection callback) whether a VOL connector implements an optional operation, without just returning an error. Added new internal VOL helper routines: H5VL_object_is_native, to determine if an object is in (or is a) native file, and H5VL_file_is_same, to determine if two objects are in (or are) the same terminal VOL connector's container. (And moved the special handling for FILE_IS_EQUAL operation out of internal VOL callback routine into H5VL_file_is_same) Made new dataset 'get' operation for H5Dvlen_get_buf_size, aligning it better with other 'get' operations in API. Fixed several issues with pass-through connectors, which are now passing the 'make check-passthrough-vol' tests again. A bunch of warning and style cleanups as well.
generated by cgit v0.12 at 2024-07-22 15:06:24 (GMT)