summaryrefslogtreecommitdiffstats
path: root/tools/src
Commit message (Collapse)AuthorAgeFilesLines
* Fix standalone linkAllen Byrne2020-02-251-3/+3
|
* TRILAB-142 Change minimum CMake version to 3.12Allen Byrne2020-02-2111-11/+11
|
* HDFFV-11014, fix the h5repack issue that misses a few attributes during the ↵Muqun Yang2020-02-071-1/+8
| | | | repacking. The flag that checks the object reference attribute is not updated properly. The fix is trivial. Just need to move the flag update line into the inner loop. Tested at Jelly. Also update the release.txt.
* Small fixes for tools and skip autotools tests for no filterAllen Byrne2020-01-291-2/+2
|
* Merge pull request #2320 in HDFFV/hdf5 from ↵David Young2020-01-291-1/+1
|\ | | | | | | | | | | | | ~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-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-01-291-5/+7
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 Wredundant-decls, Wswitch-default, Wdeclaration-after-statement, ↵kmu2020-01-272-5/+5
| | | | Wsign-compare, Wmisleading-indentation, Wshadow
* squash cast warning fixkmu2020-01-234-8/+8
|
* Change tools debug macros to not use error stackAllen Byrne2020-01-231-10/+10
|
* merge and fixkmu2020-01-2112-180/+251
|\
| * Merge branch 'develop' into stack_size_warningsDana Robinson2020-01-1717-283/+352
| |\
| | * Squashed commit of the token_refactoring branch:Dana Robinson2020-01-1611-161/+229
| | |
| * | Fixed stack and frame size warnings. Not complete, but fixes most ofDana Robinson2020-01-151-19/+22
| | | | | | | | | | | | the easier cases.
* | | merge and fix conflictkmu2020-01-1916-130/+126
|\ \ \ | | |/ | |/|
| * | Update tools hid_t declarations with H5I_INVALID_HIDAllen Byrne2020-01-1316-123/+124
| |/
* | merge and fix conflictskmu2020-01-1417-780/+1046
|\ \ | |/
| * Merge pull request #2071 in HDFFV/hdf5 from ~KMU/hdf5:bugfix/intel_warnings ↵Kimmy Mu2020-01-061-3/+3
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-111-1/+1
| | |
| | * fix and address commentskmu2019-12-111-2/+2
| | |
| | * Merge branch 'develop' into bugfix/intel_warningskmu2019-12-092-9/+7
| | |\
| | * | remove unnecessary check macrokmu2019-12-061-4/+1
| | | |
| | * | fix intel compile warningskmu2019-12-041-3/+6
| | | |
| * | | Small changes from the token_refactoring branch, to reduce the delta to developQuincey Koziol2020-01-042-2/+2
| | | |
| * | | Clean up tools warnings introduced by H5TOOLS_ERR_INIT macroJordan Henderson2019-12-319-399/+385
| | | |
| * | | Refactor tools library error handling macrosJordan Henderson2019-12-2810-457/+456
| | | |
| * | | HDFFV-10976,-10980 Init obj_type before calling H5Rget_obj_type3Allen Byrne2019-12-201-1/+1
| | | |
| * | | HDFFV-10980 - h5diff uses new ref APIsAllen Byrne2019-12-199-36/+45
| | | |
| * | | Update h5debug to retrieve file pointer through VOL frameworkQuincey Koziol2019-12-171-1/+7
| | | |
| * | | Address compile warningsAllen Byrne2019-12-061-18/+0
| | | |
| * | | Update h5ls help and rework dump_mem routines for attrsAllen Byrne2019-12-043-96/+57
| | | |
| * | | Add debug compile option as commentAllen Byrne2019-12-041-0/+2
| | | |
| * | | Update reference callsAllen Byrne2019-12-041-10/+21
| | | |
| * | | Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2019-12-022-9/+7
| |\ \ \ | | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit '0772b975d1d2bfa15aedeb4b6e2c2aac78c61a2f': Fix missing free in H5T__ref_mem_read() Fix bugs in H5VL file comparison code. Add short circuit success to H5VL_cmp_connector_cls(). Implement file comparison VOL callback. Other changes to allow references to work with non-native connectors. There is a bug somewhere. Add support for point selections to H5S_select_project_intersection. Remove H5VL_NATIVE_FILE_GET_FILE_ID and add H5VL_OBJECT_GET_FILE Revert "Make a squash commit of 'Quiet some warnings by adjusting warnings level and fixing some code.' (commit 5c911d8baf3)" Revert "Oops, remove more C99 designated initializers for VS 2010 compatibility." H5R: fix H5Tconv to check for null references Trivial parameter renaming in VOL API calls. Move checking for zero offset in selection adjust calls to the selection callbacks. This makes the procedure for checking it consistent across selection types and between _s and _u, ensures it is always is performed even when called within the H5S package, and removes the redundant check that would occur when callins H5S_select_adjust_s() from outside the H5S package. Replace H5Sselect_adjust_u() and H5Shyper_adjust_s() with H5Sselect_adjust. Implement "adjust_s" callback for all selection types. Add range checking to H5Sselect_adjust().
| * | | Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2019-11-262-7/+9
| |\ \ \ | | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit 'a8892bb42d6f6e4fbc30fae0eb2b957f81c938b9': (45 commits) Oops, remove more C99 designated initializers for VS 2010 compatibility. Add an #include to get a function declaration. Don't use C99 designated initializers, they're not compatible with Visual Studio 2010. Quiet some more maybe-uninitialized warnings---each is a false positive, *sigh*. This is more code that may not compile with VS2010, *sigh sigh*. Always warn on maybe-uninitialized. -Wincompatible-pointer-types was not available until GCC 5, so enable it only if that's the GCC version we're using. Only promote maybe-uninitialized warnings to errors on GCC 8. Even on GCC 8, there may be false positives at low optimization levels? I need to check. Only use -Werror=cast-function-type with GCC 8 and later. Put all of the -W options back into the order I found them in so that it's easier to compare old and new config/gnu-flags. Add new source files to CMakeLists.txt. Mention the -Werror= flags in libhdf5.settings.in. free -> HDfree Promote decleration-after-statement warnings to errors. Quiet decleration-after-statement warnings. Move a statement under some declarations since some vintages of Visual Studio don't like declarations after statements. Document H5D__chunk_mem_xfree_wrapper(). Undo accidental test deletion. Oops, delete a debug printf that snuck in here. Undo my changes to the HD macros, hadn't really intended those to be on this branch.... Make errors of some more warnings. Move disabled warnings to DEVELOPER_WARNING_CFLAGS. Put just one warning option on a line, and sort some of the options. Cast to the parameter type, H5VL_token_t *, instead of to unsigned char *. ...
| * | | Comment correctionAllen Byrne2019-11-211-1/+1
| | | |
| * | | HDFFV-10876 Update h5dump and h5ls for new ref api.Allen Byrne2019-11-0712-639/+949
| | | |
* | | | fix unused related warningskmu2020-01-131-1/+1
| | | |
* | | | remove unsed var,function,macro, etckmu2019-12-062-2/+2
| |_|/ |/| |
* | | Revert "Make a squash commit of 'Quiet some warnings by adjusting warnings ↵David Young2019-11-272-9/+7
| |/ |/| | | | | level and fixing some code.' (commit 5c911d8baf3)"
* | Change some GCC warnings to errors. Fix code to quiet some warnings.David Young2019-11-052-7/+9
|/
* Make wrappers, tests and tools use H5Treclaim() instead of H5Dvlen_reclaim()Jerome Soumagne2019-10-085-9/+11
|
* More whitespace changesAllen Byrne2019-10-031-165/+52
|
* Whitespace cleanupAllen Byrne2019-10-031-159/+157
|
* HDFFV-10919 merge synchingAllen Byrne2019-10-025-234/+122
|
* Update versioning to next major versionAllen Byrne2019-09-251-1/+2
|
* Merge branch 'develop' of ↵Jacob Smith2019-09-241-2/+2
|\ | | | | | | https://bitbucket.hdfgroup.org/scm/~jake.smith/hdf5 into bugfix/repack_external_storage
| * Fix missing option and incorrect closeAllen Byrne2019-09-241-2/+2
| |
* | Revert else-if to single line.Jacob Smith2019-09-241-22/+12
| |
* | Merge branch 'develop' of ↵Jacob Smith2019-09-2317-1408/+1794
|\ \ | |/ | | | | https://bitbucket.hdfgroup.org/scm/~jake.smith/hdf5 into bugfix/repack_external_storage