diff options
author | Sean McBride <sean@rogue-research.com> | 2021-08-23 21:14:53 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-08-23 21:14:53 (GMT) |
commit | f6c49fe8911a39810b236f7babf09a70fcc76c7b (patch) | |
tree | 4ee702ab548adf8e619525e14716dde7d26bb331 /test/dt_arith.c | |
parent | b5f5c59f297be19096051282068cdcc14275eb34 (diff) | |
download | hdf5-f6c49fe8911a39810b236f7babf09a70fcc76c7b.zip hdf5-f6c49fe8911a39810b236f7babf09a70fcc76c7b.tar.gz hdf5-f6c49fe8911a39810b236f7babf09a70fcc76c7b.tar.bz2 |
More clang tidy (#908)
* Pacify clang-analyzer-unix.cstring.NullArg
* Apply some bugprone-suspicious-string-compare
* Apply some readability-simplify-boolean-expr
* Apply some readability-make-member-function-const
* Apple some bugprone-macro-parentheses
* Changed an f suffix to L for `long double`
* Applied some readability-uppercase-literal-suffix automatically
* Committing clang-format changes
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'test/dt_arith.c')
-rw-r--r-- | test/dt_arith.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/test/dt_arith.c b/test/dt_arith.c index 7b79102..91e31d5 100644 --- a/test/dt_arith.c +++ b/test/dt_arith.c @@ -53,7 +53,7 @@ const char *FILENAME[] = {"dt_arith1", "dt_arith2", NULL}; * endian. If local variable `endian' is H5T_ORDER_BE then the result will * be I, otherwise the result will be Z-(I+1). */ -#define ENDIAN(Z, I, E) (H5T_ORDER_BE == E ? (I) : (Z) - ((I) + 1)) +#define ENDIAN(Z, I, E) (H5T_ORDER_BE == (E) ? (I) : (Z) - ((I) + 1)) typedef enum dtype_t { INT_SCHAR, @@ -3058,10 +3058,10 @@ test_conv_flt_1(const char *name, int run_test, hid_t src, hid_t dst) /* Check the software results against the hardware */ for (j = 0; j < nelmts; j++) { underflow = 0; - hw_f = 911.0f; - hw_d = 911.0f; + hw_f = 911.0F; + hw_d = 911.0F; #if H5_SIZEOF_LONG_DOUBLE != H5_SIZEOF_DOUBLE - hw_ld = 911.0f; + hw_ld = 911.0L; #endif /* The hardware conversion */ |