diff options
| author | David Young <dyoung@hdfgroup.org> | 2020-01-29 19:51:10 (GMT) |
|---|---|---|
| committer | David Young <dyoung@hdfgroup.org> | 2020-01-29 19:51:10 (GMT) |
| commit | 1b49ccd7faaf3285ac736ac260f15fe3e6cb69f8 (patch) | |
| tree | c5f4f7b06142b328c247ecc9eb233e4a6344bb12 /tools/src/h5repack/h5repack_main.c | |
| parent | 20c09524ea348f4009555a1cba2731842b9af80e (diff) | |
| parent | f19e06b59ee53f17465a7b5974c25c3245a40d9a (diff) | |
| download | hdf5-1b49ccd7faaf3285ac736ac260f15fe3e6cb69f8.zip hdf5-1b49ccd7faaf3285ac736ac260f15fe3e6cb69f8.tar.gz hdf5-1b49ccd7faaf3285ac736ac260f15fe3e6cb69f8.tar.bz2 | |
Merge pull request #2321 in HDFFV/hdf5 from ~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].
Diffstat (limited to 'tools/src/h5repack/h5repack_main.c')
| -rw-r--r-- | tools/src/h5repack/h5repack_main.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/tools/src/h5repack/h5repack_main.c b/tools/src/h5repack/h5repack_main.c index c0c8a2a..16899a3 100644 --- a/tools/src/h5repack/h5repack_main.c +++ b/tools/src/h5repack/h5repack_main.c @@ -412,7 +412,7 @@ set_sort_order(const char *form) static int parse_command_line(int argc, const char **argv, pack_opt_t* options) { - int opt; + int bound, opt; int ret_value = 0; /* parse command line options */ @@ -492,19 +492,21 @@ int parse_command_line(int argc, const char **argv, pack_opt_t* options) break; case 'j': - options->low_bound = (H5F_libver_t)HDatoi(opt_arg); - if (options->low_bound < H5F_LIBVER_EARLIEST || options->low_bound > H5F_LIBVER_LATEST) { + bound = HDatoi(opt_arg); + if (bound < H5F_LIBVER_EARLIEST || bound > H5F_LIBVER_LATEST) { error_msg("in parsing low bound\n"); goto done; } + options->low_bound = bound; break; case 'k': - options->high_bound = (H5F_libver_t)HDatoi(opt_arg); - if (options->high_bound < H5F_LIBVER_EARLIEST || options->high_bound > H5F_LIBVER_LATEST) { + bound = HDatoi(opt_arg); + if (bound < H5F_LIBVER_EARLIEST || bound > H5F_LIBVER_LATEST) { error_msg("in parsing high bound\n"); goto done; } + options->high_bound = bound; break; case 'c': |