summaryrefslogtreecommitdiffstats
path: root/tools/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* Patches from vtk (#358)Sean McBride2021-02-252-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Drop the export attribute on forward declaration They don't mean anything on forward declarations anyways. * Fixed -Wunused-parameter warnings in some C++ files Allowed all the attribute #defines to work in C++ also. * Fixed -Wunused-variable warnings by removing dead code * Fixed all -Wshorten-64-to-32 warnings by adjusting casts There was no truncation in fact, since the unsigned result was upcast to size_t then implicitly downcast to uint32_t. * Fixed all -Wcomma warnings by spliting statements onto own lines * Fixed all -Wself-assign warnings with different unused param suppression * Fixed all -Wextra-semi warnings * Fixed all -Wswitch-enum warnings Just added new explicit cases with the existing default case. * Fixed all -Wmissing-prototypes warnings Just made functions static. Moved them into an extern "C" block. * Reformatted source with bin/format_source using clang version 10.0.1 Co-authored-by: Larry Knox <lrknox@hdfgroup.org>
* Fixed all -Wincompatible-pointer-types-discards-qualifiers warnings (#341)Sean McBride2021-02-221-1/+3
| | | | | | | | | | | | | * Fixed various -Wincompatible-pointer-types-discards-qualifiers warnings by adding const * Fixed various -Wincompatible-pointer-types-discards-qualifiers warning by removing extraneous consts There were casts with const, but the function parameter doesn't actaully take const, so just modified the casts. In the other case, a local variable was const that should not have been, becuase its source wasn't const either. * Fixed a -Wincompatible-pointer-types-discards-qualifiers warning by strdup-ing a string Create a duplicate string instead of mutating a supposedly const one.
* Fixed uninitialized warnings (#360)Sean McBride2021-02-191-1/+1
| | | | | | | | | | * Fixed all -Wsometimes-uninitialized warnings by initializing variables * Fixed all -Wconditional-uninitialized warnings by initializing variables * Commit alignment changes from running bin/format_source with clang version 10.0.1. Co-authored-by: Larry Knox <lrknox@hdfgroup.org>
* Update license url part2 (#333)Larry Knox2021-02-1725-25/+25
| | | | | | * Modify temporary rpath for testing in java example scripts. * Update URL in source file Copyright headers for web copy of COPYING file - files not in src or test.
* develop revert source to clang-format version 11 (#293)Allen Byrne2021-01-293-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * OESS-98 convert plugin option to FetchContent, add tests * Fixes for pkcfg files because of plugin option * OESS-98 fix tools test for plugins * Keep doxygen comments under 100 chars long - format hint * Whitespace * HDFFV-11144 - Reclassify CMake messages * HDFFV-11099/11100 added help text * Reworked switch statement to compare string instead * Fix typo * Update CDash mode * Correct name of threadsafe * Correct option name * Undo accidental commit * Note LLVM 10 to 11 format default changes * Update format plugin * Undo clang-format version 11 changes * One more correction
* Small fixes (#285)Allen Byrne2021-01-271-21/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | * OESS-98 convert plugin option to FetchContent, add tests * Fixes for pkcfg files because of plugin option * OESS-98 fix tools test for plugins * Keep doxygen comments under 100 chars long - format hint * Whitespace * HDFFV-11144 - Reclassify CMake messages * HDFFV-11099/11100 added help text * Reworked switch statement to compare string instead * Fix typo * Update CDash mode * Correct name of threadsafe * Correct option name * Undo accidental commit * Small changes plus merge of tools arg parse from 1.12
* Expand 'opt_query' VOL callback to return a bitfield instead of a flag (#136)Quincey Koziol2020-11-301-5/+5
| | | | | * Expand 'opt_query' VOL callback to return a bitfield instead of a flag * Correct pastos
* Basic alignment with async branch (#115)Quincey Koziol2020-11-233-8/+8
| | | | | | | * Basic alignment with async branch - trivial changes to reduce clutter in overall diff. * Update minor error code to reflect change within library * Update the error output to match library
* Clang-format of source filesAllen Byrne2020-09-3024-6155/+6286
|
* Merge pull request #2650 in HDFFV/hdf5 from ↵David Young2020-09-282-4/+4
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ~DYOUNG/werror:rebased-fprintf-experiment to develop * commit 'c0fbc5c086566d5d3c1d1ef26baa81a53d59fc08': (24 commits) Use the right format string, "%zu", for size_t. Repair more format strings. Fix a bunch of format string errors reported by Larry. Fix some HDfprintf compilation errors: use the right format strings ("zu", PRIuHSIZE), avoid casting some printf arguments, pass the right number of arguments. Test the format string "ll" before "l", "L", and "q", like the ./configure script does. This ought to fix the compilation failure in test/dt_arith.c that Allen told me about: Cast a non-void pointer to void pointer for "%p". Use PRIu32 and "zu" formats. Delete some casts from `size_t`. I'm taking a guess that this code intended to point the 2-digit wide hexadecimal octet values, not 2 character-wide pointers to the bytes. The %02p format, which is a GNU-ism, disagreed with GCC 8.3.0 and the option flags we use. %08p is not portable, it's a GNU-ism. Use %8p, instead. Squashes a GCC error. Add format string macros PRI[doxX]HID for hid_t and use PRIdHID. Use HDva_copy() and introduce a bunch of compatbility format-string constants for uppercase hexadecimal strings, `PRIX...`. Should fix the VS2010 errors that Allen mentioned: Always #define HDfprintf as fprintf in this header. I believe this will fix the Windows build error that Allen reported. Provide an HDvasprintf implementation only if it isn't #defined. This should fix the mingw compilation issue that Allen reported. Fix va_list usage in the vasprintf(3) implementation. Promote format-string warnings to errors. Use the portable `-eq` operator instead of the bash-ism `==`. Fixes the tests on NetBSD, where /bin/sh != bash. Restore a literal percent sign ("%%") that I accidentally deleted. Fix code that made GCC complain about a NULL or `unsigned char *` arguments for "%s". Take pains to provide UINT64_MAX in all conditions. Correct a couple of format strings. ...
| * Merge remote-tracking branch 'hdffv/develop' into rebased-fprintf-experimentDavid Young2020-09-271-0/+13
| |\
| * \ Merge remote-tracking branch 'hdffv/develop' into rebased-fprintf-experimentDavid Young2020-09-035-29/+28
| |\ \
| * \ \ Merge remote-tracking branch 'hdffv/develop' into rebased-fprintf-experimentDavid Young2020-07-3012-3752/+2783
| |\ \ \
| * | | | Squash my changes on branch `fprintf-experiment` into one commit forDavid Young2020-07-072-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | reapplication to my new warnings branch, `warnings-again`. These changes are included: commit 915551b7bf64e777dd2007386ec77b1d117770da Merge: 63858c2 a8892bb Author: David Young <dyoung@hdfgroup.org> Date: Mon Nov 25 17:39:49 2019 -0600 Merge remote-tracking branch 'hdf5/develop' into fprintf-experiment commit a8892bb42d6f6e4fbc30fae0eb2b957f81c938b9 Merge: 5c911d8 f907b51 Author: David Young <dyoung@hdfgroup.org> Date: Mon Nov 25 17:33:54 2019 -0600 Merge pull request #2055 in HDFFV/hdf5 from ~DYOUNG/vchoi_fork:add-werror-and-squash-some to develop * commit 'f907b511d06612dafc7814a7c30f2f3d2b76d52b': Oops, remove more C99 designated initializers for VS 2010 compatibility. commit 63858c22e168acaec0af8ced6641f26102cc6bb0 Merge: 20ae787 5c911d8 Author: David Young <dyoung@hdfgroup.org> Date: Mon Nov 25 17:04:42 2019 -0600 Merge remote-tracking branch 'hdf5/develop' into fprintf-experiment commit 5c911d8baf3ee7fe654269088eebdf07f59a8192 Merge: b8a5671 62208b0 Author: David Young <dyoung@hdfgroup.org> Date: Mon Nov 25 16:58:27 2019 -0600 Merge pull request #2030 in HDFFV/hdf5 from ~DYOUNG/vchoi_fork:add-werror-and-squash-some to develop * commit '62208b056a09c01855fbac7f75146be58ad6bfe5': (44 commits) 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 *. Change hdset_reg_ref_t and H5R_ref_t from arrays of unsigned char to structs containing those arrays. Encapsulating the arrays in this way makes it easier to write and think about pointers to these types, casts to/from these types, etc. ... commit 20ae7877e33931b95e8c3502b027d6c3fe94a11f Merge: 46f8c61 edd5297 Author: David Young <dyoung@hdfgroup.org> Date: Fri Nov 22 15:34:09 2019 -0600 Merge remote-tracking branch 'origin/add-werror-and-squash-some' into fprintf-experiment commit 46f8c613d5117a8be5bc8385a072daa0b4262f06 Author: David Young <dyoung@hdfgroup.org> Date: Fri Nov 22 15:29:00 2019 -0600 GCC really wants us to use `ll` to format `long long`, so try to make that work before any other format modifier. Seems like we're not compiling the autoconf test program with -Werror=format ? Probably should. commit eee35b8ef3759c391327cd48a9b3c56b6f8abc99 Author: David Young <dyoung@hdfgroup.org> Date: Fri Nov 22 15:28:05 2019 -0600 It's hard to know just how wide an HDoff_t will be, and I don't think POSIX or C standards provide a PRI macro for it, so cast to intmax_t and format using PRIdMAX. commit 86eab12df7a89b546a38e99f8178dd2adbcb3433 Author: David Young <dyoung@hdfgroup.org> Date: Fri Nov 22 15:26:25 2019 -0600 URemove some casts.se the right format string for the argument. Here and there stop casting a printf argument. commit f722f7cbecbaa99449941484b014426f62f1bed5 Merge: 58e3743 6d5ec83 Author: David Young <dyoung@hdfgroup.org> Date: Fri Nov 22 14:44:16 2019 -0600 Merge branch 'add-werror-and-squash-some' into fprintf-experiment commit 58e3743b7faa9836606ee91798fe80dfc0040da7 Author: David Young <dyoung@hdfgroup.org> Date: Wed Nov 20 21:07:21 2019 -0600 Remove custom HDfprintf implementation, using the standard library's, instead. Take a swipe at repairing fprintf format strings, mainly replacing "%Hu" with "%" PRIuHSIZE, "%a" with "%" PRIuHADDR, "%Zu" with "%zu". Here and there remove an awkward cast of a printf argument to `long long` and use PRI[doux]8, PRI[doux]32, or PRI[doux]64, instead. Change occurrences of "%t" to "%s" and perform a suitable change of argument, `cond` -> `cond ? "TRUE" : "FALSE"`. Some occurrences of %Hu, %a, and %t remain, they just weren't flagged by the compiler because of #ifdef'age. commit d4366909293fa970c23512ac80e5d865d76cddbf Author: David Young <dyoung@hdfgroup.org> Date: Wed Nov 20 20:54:32 2019 -0600 Promote format-string warnigns to errors.
* | | | | OESS-98 merge with clang-format changesAllen Byrne2020-09-251-2/+2
| |_|_|/ |/| | |
* | | | CMake and script changes for clang-formatAllen Byrne2020-09-041-0/+13
| |_|/ |/| |
* | | HDFFV-9984 Add options to merge/prune external links during repackAllen Byrne2020-08-141-1/+1
| | |
* | | Removes staff email addresses from the repositoryDana Robinson2020-08-073-3/+3
| | | | | | | | | | | | Removes redundant C library headers from hl library
* | | Warning fixes in tools and h5test.cDana Robinson2020-08-061-25/+24
| |/ |/|
* | Warnings reduction in toolsAllen Byrne2020-07-308-130/+124
| |
* | Tools refactorAllen Byrne2020-07-1512-3741/+2778
|/ | | | | | Move error_stack setup to tools library Add structure for h5dump properties Eliminate duplicated h5diff functions Use properties structure in place of argument lists.
* Fixes for warnings in the tools code.Dana Robinson2020-04-251-3/+3
|
* Merge pull request #2532 in HDFFV/hdf5 from ~DEROBINS/hdf5_der:develop_minor ↵Dana Robinson2020-04-232-4/+30
|\ | | | | | | | | | | | | | | to develop * commit '785bd0975ba13c3033820d0fc809da963fcd54ca': Fixed h5diff command-line parameters to use 1 and 2 instead of src and dst. Added VOL command-line options to (p)h5diff, h5ls, h5dump, and h5mkgrp.
| * Fixed h5diff command-line parameters to use 1 and 2 instead of srcDana Robinson2020-04-222-8/+8
| | | | | | | | and dst.
| * Added VOL command-line options to (p)h5diff, h5ls, h5dump, and h5mkgrp.Dana Robinson2020-04-212-4/+30
| |
* | Merge pull request #2530 in HDFFV/hdf5 from ~DEROBINS/hdf5_der:develop_minor ↵Dana Robinson2020-04-212-182/+186
|\ \ | |/ | | | | | | | | | | to develop * commit 'c6097935d5be4c3750b9bf9167783ad28158905b': Separated VFD and VOL parameters in tools.
| * Separated VFD and VOL parameters in tools.Dana Robinson2020-04-212-182/+186
| |
* | Trim trailing whitespaceQuincey Koziol2020-04-201-2/+2
|/
* Changed identifiers and command-line options from "id" to "value" in toolsDana Robinson2020-04-142-15/+15
| | | | code to be in agreement with H5VL API terminology.
* Added support for passing connector info strings via the commandDana Robinson2020-04-092-22/+37
| | | | line to the tools internals.
* Fix for passthrough VOL not passing tools tests due to incorrectDana Robinson2020-04-081-12/+3
| | | | optional callback queries.
* Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2020-04-063-48/+76
|\ | | | | | | | | | | | | * commit '4830a1796441abbb5a4fbd2cff9a651f2a0d066e': Updated tools dump output when dataset offset is undefined. Added checks for native optional call support in some of the tools. Misc changes for h5dump VOL changes.
| * Merge branch 'develop' into tools_vol_updateDana Robinson2020-04-062-11/+11
| |\
| * | Updated tools dump output when dataset offset is undefined.Dana Robinson2020-04-021-1/+1
| | |
| * | Added checks for native optional call support in some of the tools.Dana Robinson2020-04-021-22/+40
| | |
| * | Merge branch 'develop' into tools_vol_updateDana Robinson2020-04-022-63/+65
| |\ \
| * | | Misc changes for h5dump VOL changes.Dana Robinson2020-03-303-36/+46
| | | |
* | | | Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2020-04-051-1/+1
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | * commit '5d1590079ea0061a7b86d7210cb796b6d58a17c7': Skip testing of passthrough VOL connector in a Family File tools test Restore previous tools library driver name behavior with passthrough VOL connector
| * | | Merge pull request #2490 in HDFFV/hdf5 from tools_vol_update to developJordan Henderson2020-04-031-1/+1
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | * commit 'a5a2fb44296327787476407a249d9470b3c80427': Skip testing of passthrough VOL connector in a Family File tools test Restore previous tools library driver name behavior with passthrough VOL connector
| | * | | Restore previous tools library driver name behavior with passthrough VOL ↵Jordan Henderson2020-04-031-1/+1
| | |/ / | | | | | | | | | | | | connector
* | | | TRILAB-192 add c++ and fortran warnings build systems one fileAllen Byrne2020-04-051-6/+2
|/ / /
* | | Fixed unnecessary H5R casts in the tools that were raising warnings.Dana Robinson2020-04-022-10/+10
| |/ |/|
* | Minor renaming in the tools code.Dana Robinson2020-03-312-63/+65
|/
* Merge pull request #2473 in HDFFV/hdf5 from tools_vol_update to developAllen Byrne2020-03-271-2/+3
|\ | | | | | | | | * commit 'feb0f521b16b79485c22c86a630a1d91cb680a61': Fix some missing symbols issues in tools library
| * Fix some missing symbols issues in tools libraryJordan Henderson2020-03-271-2/+3
| |
* | Merge pull request #2469 in HDFFV/hdf5 from tools_vol_update to developJordan Henderson2020-03-276-238/+688
|\ \ | |/ | | | | | | | | | | * commit 'ea952b25ba0fd5c4b5f60a94e75a1b5b78e66172': A few small tweaks to the tools from review Replace usage of some standard library functions with HD- versions Update Tools library to be better compatible with VOL connectors
| * A few small tweaks to the tools from reviewJordan Henderson2020-03-271-125/+133
| |
| * Replace usage of some standard library functions with HD- versionsJordan Henderson2020-03-261-2/+2
| |
| * Update Tools library to be better compatible with VOL connectorsJordan Henderson2020-03-176-220/+662
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Modify h5repack to integrate with VOL connectors Update tools library to accomodate VOL connectors Update logic in h5tools_fopen for VOL connectors Add command-line options to h5repack for specifying in/out VOL connectors Implement h5tools_set_vol_fapl Fix library shutdown issue Integrate ROS3 and HDFS VFDs into new h5tools_get_fapl() scheme Avoid H5Ocopy in h5repack when using different VOL connectors Update h5tools_test_utils.c for ROS3 and HDFS integration
* | unused set-variable in macroAllen Byrne2020-03-251-0/+1
| |
generated by cgit v0.12 at 2024-07-08 03:09:12 (GMT)