diff options
author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2022-07-11 17:27:40 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-07-11 17:27:40 (GMT) |
commit | 0412d3f292b255da700d865fd1eb990e05c038bb (patch) | |
tree | 1d4ad4ab8d32b0452a30abae4c52ebd9fed17f43 /test/cache_common.c | |
parent | fa7caf843508b250f085e88cf5edfc2606da1205 (diff) | |
download | hdf5-0412d3f292b255da700d865fd1eb990e05c038bb.zip hdf5-0412d3f292b255da700d865fd1eb990e05c038bb.tar.gz hdf5-0412d3f292b255da700d865fd1eb990e05c038bb.tar.bz2 |
Fixes for production mode gcc warnings (#1873)
* Fixes for production mode gcc warnings
With the strict-overflow changes, this brings the number of warnings
in the C library w/ gcc 12 to zero.
* Fix typo
* Committing clang-format changes
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'test/cache_common.c')
-rw-r--r-- | test/cache_common.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/test/cache_common.c b/test/cache_common.c index 9becfa8..41aa128 100644 --- a/test/cache_common.c +++ b/test/cache_common.c @@ -4977,13 +4977,17 @@ check_and_validate_cache_hit_rate(hid_t file_id, double *hit_rate_ptr, hbool_t d else { cache_ptr = file_ptr->shared->cache; + if (NULL == cache_ptr) { + pass = FALSE; + failure_mssg = "NULL cache pointer"; + } } } /* verify that we can access the cache data structure */ if (pass) { - if ((cache_ptr == NULL) || (cache_ptr->magic != H5C__H5C_T_MAGIC)) { + if (cache_ptr->magic != H5C__H5C_T_MAGIC) { pass = FALSE; failure_mssg = "Can't access cache resize_ctl."; @@ -5102,13 +5106,17 @@ check_and_validate_cache_size(hid_t file_id, size_t *max_size_ptr, size_t *min_c else { cache_ptr = file_ptr->shared->cache; + if (NULL == cache_ptr) { + pass = FALSE; + failure_mssg = "NULL cache pointer"; + } } } /* verify that we can access the cache data structure */ if (pass) { - if ((cache_ptr == NULL) || (cache_ptr->magic != H5C__H5C_T_MAGIC)) { + if (cache_ptr->magic != H5C__H5C_T_MAGIC) { pass = FALSE; failure_mssg = "Can't access cache data structure."; |