diff options
author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2022-07-11 17:28:02 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-07-11 17:28:02 (GMT) |
commit | e269f9036474b9d5b3c88d865e7bb20ecad5c54c (patch) | |
tree | aa5313f198b044605d6b135cd4499cc2f91eacee /tools/lib/h5diff_array.c | |
parent | 0412d3f292b255da700d865fd1eb990e05c038bb (diff) | |
download | hdf5-e269f9036474b9d5b3c88d865e7bb20ecad5c54c.zip hdf5-e269f9036474b9d5b3c88d865e7bb20ecad5c54c.tar.gz hdf5-e269f9036474b9d5b3c88d865e7bb20ecad5c54c.tar.bz2 |
Misc clang warning fixes (#1869)
* Minor clang warning fixes in src and test
* Fixes clang warnings in high-level library
* clang format-nonliteral warning fixes
* Committing clang-format changes
* Fixed int -> float clang cast warnings in h5diff
* Quiets some warnings in flex/bison generated code
* Suppress overlong string warning for libinfo on clang
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'tools/lib/h5diff_array.c')
-rw-r--r-- | tools/lib/h5diff_array.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tools/lib/h5diff_array.c b/tools/lib/h5diff_array.c index 9fa3ccc..0c28789 100644 --- a/tools/lib/h5diff_array.c +++ b/tools/lib/h5diff_array.c @@ -2734,7 +2734,7 @@ diff_long_element(unsigned char *mem1, unsigned char *mem2, hsize_t elem_idx, di HDmemcpy(&temp2_long, mem2, sizeof(long)); /* -d and !-p */ if (opts->delta_bool && !opts->percent_bool) { - if (ABS(temp1_long - temp2_long) > opts->delta) { + if (ABS(temp1_long - temp2_long) > (long)opts->delta) { opts->print_percentage = 0; print_pos(opts, elem_idx, 0); if (print_data(opts)) { @@ -2776,7 +2776,7 @@ diff_long_element(unsigned char *mem1, unsigned char *mem2, hsize_t elem_idx, di } nfound++; } - else if (per > opts->percent && ABS(temp1_long - temp2_long) > opts->delta) { + else if (per > opts->percent && ABS(temp1_long - temp2_long) > (long)opts->delta) { opts->print_percentage = 1; print_pos(opts, elem_idx, 0); if (print_data(opts)) { @@ -2822,7 +2822,7 @@ diff_ulong_element(unsigned char *mem1, unsigned char *mem2, hsize_t elem_idx, d HDmemcpy(&temp2_ulong, mem2, sizeof(unsigned long)); /* -d and !-p */ if (opts->delta_bool && !opts->percent_bool) { - if (PDIFF(temp1_ulong, temp2_ulong) > opts->delta) { + if (PDIFF(temp1_ulong, temp2_ulong) > (unsigned long)opts->delta) { opts->print_percentage = 0; print_pos(opts, elem_idx, 0); if (print_data(opts)) { @@ -2866,7 +2866,7 @@ diff_ulong_element(unsigned char *mem1, unsigned char *mem2, hsize_t elem_idx, d } nfound++; } - else if (per > opts->percent && PDIFF(temp1_ulong, temp2_ulong) > opts->delta) { + else if (per > opts->percent && PDIFF(temp1_ulong, temp2_ulong) > (unsigned long)opts->delta) { opts->print_percentage = 1; print_pos(opts, elem_idx, 0); if (print_data(opts)) { @@ -2913,7 +2913,7 @@ diff_llong_element(unsigned char *mem1, unsigned char *mem2, hsize_t elem_idx, d /* -d and !-p */ if (opts->delta_bool && !opts->percent_bool) { - if (ABS(temp1_llong - temp2_llong) > opts->delta) { + if (ABS(temp1_llong - temp2_llong) > (long long)opts->delta) { opts->print_percentage = 0; print_pos(opts, elem_idx, 0); if (print_data(opts)) { @@ -2957,7 +2957,7 @@ diff_llong_element(unsigned char *mem1, unsigned char *mem2, hsize_t elem_idx, d } nfound++; } - else if (per > opts->percent && ABS(temp1_llong - temp2_llong) > opts->delta) { + else if (per > opts->percent && ABS(temp1_llong - temp2_llong) > (long long)opts->delta) { opts->print_percentage = 1; print_pos(opts, elem_idx, 0); if (print_data(opts)) { |