diff options
author | Sean McBride <sean@rogue-research.com> | 2023-06-16 04:49:02 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-06-16 04:49:02 (GMT) |
commit | 68eba3da69e659fab69bbef5901ce42e82cea1dc (patch) | |
tree | f36805f798df6c04092c14dc6803d2cb82f6eff2 /test/onion.c | |
parent | 10093f7c4345061bdbebc12888debbf08fe85a5b (diff) | |
download | hdf5-68eba3da69e659fab69bbef5901ce42e82cea1dc.zip hdf5-68eba3da69e659fab69bbef5901ce42e82cea1dc.tar.gz hdf5-68eba3da69e659fab69bbef5901ce42e82cea1dc.tar.bz2 |
Many clang -Wextra-semi-stmt fixes (#2537)
* Adds semicolons to function-like macros
* Adds a do..while(0) loop to some macros
* Removes semicolons when inappropriate, especially H5E_TRY_BEGIN/END
Diffstat (limited to 'test/onion.c')
-rw-r--r-- | test/onion.c | 68 |
1 files changed, 34 insertions, 34 deletions
diff --git a/test/onion.c b/test/onion.c index 03730a7..830220d 100644 --- a/test/onion.c +++ b/test/onion.c @@ -734,7 +734,7 @@ test_fapl(void) { ret = H5Pset_fapl_onion(H5I_INVALID_HID, &info_in); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; @@ -743,7 +743,7 @@ test_fapl(void) { ret = H5Pset_fapl_onion(fapl_id, NULL); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; @@ -753,7 +753,7 @@ test_fapl(void) { ret = H5Pset_fapl_onion(fapl_id, &info_in); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; info_in.version--; @@ -764,7 +764,7 @@ test_fapl(void) { ret = H5Pset_fapl_onion(fapl_id, &info_in); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; @@ -774,7 +774,7 @@ test_fapl(void) { ret = H5Pset_fapl_onion(fapl_id, &info_in); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; info_in.page_size = ONION_TEST_PAGE_SIZE_1; @@ -785,7 +785,7 @@ test_fapl(void) { ret = H5Pset_fapl_onion(fapl_id, &info_in); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; @@ -795,7 +795,7 @@ test_fapl(void) { ret = H5Pset_fapl_onion(fapl_id, &info_in); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; info_in.backing_fapl_id = H5P_DEFAULT; @@ -810,7 +810,7 @@ test_fapl(void) { ret = H5Pget_fapl_onion(fapl_id, NULL); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; @@ -819,7 +819,7 @@ test_fapl(void) { ret = H5Pget_fapl_onion(H5I_INVALID_HID, &info_out); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; @@ -828,7 +828,7 @@ test_fapl(void) { ret = H5Pget_fapl_onion(fapl_id_sec2, &info_out); } - H5E_END_TRY; + H5E_END_TRY if (SUCCEED == ret) TEST_ERROR; @@ -945,7 +945,7 @@ test_header_encode_decode(void) { size_ret = H5FD__onion_header_decode(exp, &hdr_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; @@ -959,7 +959,7 @@ test_header_encode_decode(void) { size_ret = H5FD__onion_header_decode(exp, &hdr_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; @@ -968,7 +968,7 @@ test_header_encode_decode(void) { size_ret = H5FD__onion_header_decode(exp, &hdr_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; @@ -1059,7 +1059,7 @@ test_history_encode_decode_empty(void) { size_ret = H5FD__onion_history_decode(exp, &history_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; @@ -1072,7 +1072,7 @@ test_history_encode_decode_empty(void) { size_ret = H5FD__onion_history_decode(exp, &history_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; @@ -1081,7 +1081,7 @@ test_history_encode_decode_empty(void) { size_ret = H5FD__onion_history_decode(exp, &history_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; @@ -1421,7 +1421,7 @@ test_revision_record_encode_decode(void) { size_ret = H5FD__onion_revision_record_decode(exp, &r_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; exp[2] = 'R'; /* reset */ @@ -1432,7 +1432,7 @@ test_revision_record_encode_decode(void) { size_ret = H5FD__onion_revision_record_decode(exp, &r_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; @@ -1442,7 +1442,7 @@ test_revision_record_encode_decode(void) { size_ret = H5FD__onion_revision_record_decode(exp, &r_out); } - H5E_END_TRY; + H5E_END_TRY if (0 != size_ret) TEST_ERROR; exp[4] = H5FD_ONION_REVISION_RECORD_VERSION_CURR; /* reset */ @@ -1858,7 +1858,7 @@ verify_stored_onion_create_0_open(struct onion_filepaths *paths, H5FD_onion_fapl { err_ret = H5FDread(file, H5FD_MEM_DRAW, H5P_DEFAULT, 0, 1, act_buf); } - H5E_END_TRY; + H5E_END_TRY if (err_ret != FAIL) TEST_ERROR; @@ -1972,7 +1972,7 @@ test_create_oniontarget(hbool_t truncate_canonical, hbool_t with_initial_data) { vfile_raw = H5FDopen(paths->canon, H5F_ACC_RDONLY, fapl_id, HADDR_UNDEF); } - H5E_END_TRY; + H5E_END_TRY /* Check if onion history for onion-open created file */ if (NULL != vfile_raw) @@ -2008,7 +2008,7 @@ test_create_oniontarget(hbool_t truncate_canonical, hbool_t with_initial_data) { vfile_ro = H5FDopen(paths->canon, H5F_ACC_RDONLY, fapl_id, HADDR_UNDEF); } - H5E_END_TRY; + H5E_END_TRY /* check if onionization (creation) not complete; nothing to open */ if (vfile_ro != NULL) TEST_ERROR; @@ -2095,7 +2095,7 @@ test_create_oniontarget(hbool_t truncate_canonical, hbool_t with_initial_data) { vfile_raw = H5FDopen(paths->recovery, H5F_ACC_RDONLY, onion_info.backing_fapl_id, HADDR_UNDEF); } - H5E_END_TRY; + H5E_END_TRY if (NULL != vfile_raw) TEST_ERROR; @@ -2192,7 +2192,7 @@ error: { H5Pclose(fapl_id); } - H5E_END_TRY; + H5E_END_TRY return -1; } /* end test_create_oniontarget() */ @@ -2546,7 +2546,7 @@ error: H5Pclose(fapl_id); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY return -1; } /* end test_several_revisions_with_logical_gaps() */ @@ -2643,7 +2643,7 @@ error: { H5Pclose(fapl_id); } - H5E_END_TRY; + H5E_END_TRY HDfree(buf_vfy); @@ -2854,7 +2854,7 @@ error: H5Pclose(fapl_id); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY return -1; } /* end test_page_aligned_history_create() */ @@ -3190,7 +3190,7 @@ error: H5Pclose(fapl_id); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY HDfree(wdata); HDfree(rdata); @@ -3449,7 +3449,7 @@ error: H5Pclose(fapl_id); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY HDfree(wdata); HDfree(rdata); @@ -3811,7 +3811,7 @@ error: H5Sclose(space); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY return -1; } /* end test_integration_create_delete_objects */ @@ -4086,7 +4086,7 @@ error: H5Pclose(fapl_id); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY return -1; } /* end test_integration_dset_extension */ @@ -4266,7 +4266,7 @@ error: H5Pclose(fapl_id); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY return -1; } @@ -4648,7 +4648,7 @@ error: H5Pclose(fapl_id); H5Pclose(onion_info.backing_fapl_id); } - H5E_END_TRY; + H5E_END_TRY return -1; } @@ -4888,7 +4888,7 @@ error: H5Fclose(file_id); H5Pclose(fapl_id); } - H5E_END_TRY; + H5E_END_TRY HDfree(wdata); HDfree(rdata); |