summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Reduce differences between my -Werror branch and `develop`:David Young2020-05-2015-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.
* Reduce casts of HDcalloc()/HDmalloc() that -Wc++-compat required.David Young2020-05-206-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.
* testpar/t_2Gio.c: Fix a typo that I think was introduced by aDavid Young2020-05-201-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-05-205-25/+227
|
* fix Wredundant-decls, Wswitch-default, Wdeclaration-after-statement, ↵kmu2020-05-201-1/+1
| | | | Wsign-compare, Wmisleading-indentation, Wshadow
* Replaced my_isnan with C99 isnan in h5diff.Dana Robinson2020-05-201-0/+3
|
* Removed H5_DEC_ENUMDana Robinson2020-05-201-7/+0
|
* Revert "Revert "Moved -Wunsuffixed-float-constants to the developer warnings.""Dana Robinson2020-05-202-16/+1
| | | | This reverts commit 6486e06545bf637d46e18a787395542f4fca143a.
* Revert "Moved -Wunsuffixed-float-constants to the developer warnings."Dana Robinson2020-05-202-1/+16
| | | | This reverts commit 12bc75c6172ca7b64c33b17cf538a7629500cf2f.
* Moved -Wunsuffixed-float-constants to the developer warnings.Dana Robinson2020-05-202-16/+1
|
* remove unnecessary stuffkmu2020-05-201-2/+0
|
* squash cast warning fixkmu2020-05-2011-91/+95
|
* fix missing prototype warningkmu2020-05-201-1/+2
|
* more cleanupkmu2020-05-201-1/+1
|
* pick up missing piecekmu2020-05-202-2/+3
|
* fix unused related warningskmu2020-05-2040-128/+138
|
* remove unsed var,function,macro, etckmu2020-05-202-5/+2
|
* fix uninitizlized warningkmu2020-05-201-2/+2
|
* Remove irrelevant sentence from comment.Larry Knox2020-05-201-3/+1
|
* Add v111 version for H5O functions to enable version1 default for HDF5Larry Knox2020-05-201-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-05-201-2/+2
|
* H5vers.txt has versions v10 that should be v110 and some missing v110Larry Knox2020-05-201-7/+7
| | | | entries.
* Fixed stack and frame size warnings. Not complete, but fixes most ofDana Robinson2020-05-201-16/+24
| | | | the easier cases.
* Squashed commit of the token_refactoring branch:Dana Robinson2020-05-2066-996/+4062
|
* H5R: set app ref when incrementing ref_count on location held by referenceJerome Soumagne2020-05-205-18/+39
| | | | (fix HDFFV-10992)
* add new type supportkmu2020-05-201-0/+1
|
* address problems from commentskmu2020-05-204-4/+5
|
* fix and address commentskmu2020-05-205-7/+15
|
* change according to previous commentskmu2020-05-202-4/+3
|
* add missing piecekmu2020-05-201-0/+1
|
* remove unnecessary check macrokmu2020-05-2022-35/+34
|
* fix intel compile warningskmu2020-05-2025-48/+47
|
* Revert "fix warnings from Intel compiler"kmu2020-05-2027-43/+44
| | | | This reverts commit 8b9338ab57eec0cc8fa5a36c44d7b28e52e9a466.
* Revert "fix issues from previous PR comments"kmu2020-05-2025-43/+39
| | | | This reverts commit d242a900f420b040e364f6c0976c01593e955db3.
* Revert "using a different MACRO"kmu2020-05-2023-72/+70
| | | | This reverts commit fc61b7a9f3a38331809ebcb6247482943947cdb8.
* using a different MACROkmu2020-05-2023-70/+72
|
* fix issues from previous PR commentskmu2020-05-2025-39/+43
|
* fix warnings from Intel compilerkmu2020-05-2027-44/+43
|
* Small changes from the token_refactoring branch, to reduce the delta to developQuincey Koziol2020-05-2023-150/+191
|
* Fix some places where H5P_DEFAULT gets passed down to a VOL connectorJordan Henderson2020-05-2015-117/+102
|
* Remove unnecessary H5CX callQuincey Koziol2020-05-201-4/+0
|
* Refactor H5Dvlen_get_buf_size to use optional dataset operation, with ↵Quincey Koziol2020-05-207-54/+255
| | | | generic fallback for VOL connectors that don't implement operation
* Cleanups from PR reviewsQuincey Koziol2020-05-204-102/+101
|
* Refactor all the 'H5VL_*_optional' callbacks to move the type of operation outQuincey Koziol2020-05-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.
* Fixed bugs in pread/pwrite I/O in VFDs.Dana Robinson2020-05-203-15/+21
| | | | Fixes HDFFV-10945.
* More fixes for previous committed PR #2079 dated Dec 5 2019.Vailin Choi2020-05-201-1/+1
| | | | | (1) H5O_dtype_ver_bounds[] for V112 should be H5O_DTYPE_VRESION_4 (2) The tests for the new reference types should work for V112 and beyond
* Modify H5VL initialization routines to initialize all VOL-managed objectNeil Fortner2020-05-2011-2/+153
| | | | | types. Modify H5VLwrap_register() to reject non-VOL-managed object types. Also fix overisights in h5trace.c from previous changes.
* Fix 2010 compile issuesAllen Byrne2020-05-202-623/+623
|
* Minor tweaks noticed while going over VOL documentation.Dana Robinson2020-05-203-6/+6
|
* Fix compile errors - mostly in jniAllen Byrne2020-05-201-21/+17
|
generated by cgit v0.12 at 2025-01-13 09:33:02 (GMT)