diff options
author | Sean McBride <sean@rogue-research.com> | 2023-07-24 21:18:04 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-07-24 21:18:04 (GMT) |
commit | 553e1cd31150f7ca58f87f40d63dd8f200721611 (patch) | |
tree | 12b0aa9c90279d015bf6596e2b60d443d384c0c1 /src/H5Shyper.c | |
parent | e0083c48e90beb5b56bf93eccdbfa89baa79750a (diff) | |
download | hdf5-553e1cd31150f7ca58f87f40d63dd8f200721611.zip hdf5-553e1cd31150f7ca58f87f40d63dd8f200721611.tar.gz hdf5-553e1cd31150f7ca58f87f40d63dd8f200721611.tar.bz2 |
Another round of fixing -Wextra-semi-stmt warnings (#3264)
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'src/H5Shyper.c')
-rw-r--r-- | src/H5Shyper.c | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/src/H5Shyper.c b/src/H5Shyper.c index 15186dd..0281f22 100644 --- a/src/H5Shyper.c +++ b/src/H5Shyper.c @@ -4494,7 +4494,7 @@ done: /* Free temporary space if not passed to caller (only happens on error) */ if (!*space && tmp_space) if (H5S_close(tmp_space) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "can't close dataspace") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "can't close dataspace"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S__hyper_deserialize() */ @@ -5860,7 +5860,7 @@ H5S__hyper_coord_to_span(unsigned rank, const hsize_t *coords) done: if (ret_value == NULL && down != NULL) if (H5S__hyper_free_span_info(down) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, NULL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, NULL, "unable to free span info"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S__hyper_coord_to_span() */ @@ -6204,7 +6204,7 @@ done: if (ret_value < 0) if (head) if (H5S__hyper_free_span_info(head) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S_hyper_add_span_element() */ @@ -6778,7 +6778,7 @@ done: if (ret_value < 0 && new_space->select.sel_info.hslab->span_lst) { if (new_space->select.sel_info.hslab->span_lst->head) if (H5S__hyper_free_span(new_space->select.sel_info.hslab->span_lst->head) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span"); new_space->select.sel_info.hslab->span_lst = (H5S_hyper_span_info_t *)H5FL_ARR_FREE(hbounds_t, new_space->select.sel_info.hslab->span_lst); @@ -7338,7 +7338,7 @@ done: if (ret_value < 0) if (new_span) if (H5S__hyper_free_span(new_span) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S__hyper_append_span() */ @@ -8342,7 +8342,7 @@ done: if (ret_value == NULL) if (merged_spans) if (H5S__hyper_free_span_info(merged_spans) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, NULL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, NULL, "unable to free span info"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S__hyper_merge_spans_helper() */ @@ -9519,13 +9519,13 @@ done: /* Free resources */ if (a_not_b) if (H5S__hyper_free_span_info(a_not_b) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); if (a_and_b) if (H5S__hyper_free_span_info(a_and_b) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); if (b_not_a) if (H5S__hyper_free_span_info(b_not_a) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S__fill_in_new_space() */ @@ -9613,7 +9613,7 @@ H5S__generate_hyperslab(H5S_t *space, H5S_seloper_t op, const hsize_t start[], c done: if (new_spans) if (H5S__hyper_free_span_info(new_spans) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S__generate_hyperslab() */ @@ -11352,7 +11352,7 @@ done: if (copied_span_info) { assert(ret_value < 0); if (H5S__hyper_free_span_info(copied_span_info) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); copied_span_info = NULL; } @@ -11778,14 +11778,14 @@ done: /* Free ss_span_info_buf */ if (ss_span_info_buf) { if (H5S__hyper_free_span_info(ss_span_info_buf) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); ss_span_info_buf = NULL; } /* Free ds_span_info_buf */ if (ds_span_info_buf) { if (H5S__hyper_free_span_info(ds_span_info_buf) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); ds_span_info_buf = NULL; } @@ -11797,7 +11797,7 @@ done: for (u = 0; u < udata.ds_rank; u++) if (udata.ps_span_info[u]) { if (H5S__hyper_free_span_info(udata.ps_span_info[u]) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "unable to free span info"); udata.ps_span_info[u] = NULL; } } @@ -12278,7 +12278,7 @@ done: /* Free space on error */ if (!ret_value) if (space_out && H5S_close(space_out) < 0) - HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, NULL, "unable to release dataspace") + HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, NULL, "unable to release dataspace"); FUNC_LEAVE_NOAPI(ret_value) } /* end H5S_hyper_get_unlim_block */ |