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/vol.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/vol.c')
-rw-r--r-- | test/vol.c | 48 |
1 files changed, 24 insertions, 24 deletions
@@ -708,7 +708,7 @@ test_vol_registration(void) { vol_id = H5VLregister_connector(&fake_vol_g, lapl_id); } - H5E_END_TRY; + H5E_END_TRY if (H5I_INVALID_HID != vol_id) FAIL_PUTS_ERROR("should not be able to register a connector with an incorrect property list"); if (H5Pclose(lapl_id) < 0) @@ -723,7 +723,7 @@ test_vol_registration(void) { vol_id = H5VLregister_connector(bad_fake_vol_class, H5P_DEFAULT); } - H5E_END_TRY; + H5E_END_TRY if (H5I_INVALID_HID != vol_id) FAIL_PUTS_ERROR("should not be able to register a connector with an incompatible version #"); HDfree(bad_fake_vol_class); @@ -790,7 +790,7 @@ test_vol_registration(void) { ret = H5VLunregister_connector(native_id); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to unregister the native VOL connector"); @@ -804,7 +804,7 @@ error: H5Pclose(lapl_id); H5Pclose(vipl_id); } - H5E_END_TRY; + H5E_END_TRY if (bad_fake_vol_class) HDfree(bad_fake_vol_class); @@ -1030,7 +1030,7 @@ error: H5Pclose(fapl_id2); H5Pclose(fcpl_id); } - H5E_END_TRY; + H5E_END_TRY return FAIL; @@ -1137,7 +1137,7 @@ error: H5Pclose(fapl_id); H5Pclose(gcpl_id); } - H5E_END_TRY; + H5E_END_TRY return FAIL; @@ -1325,7 +1325,7 @@ error: H5Pclose(dapl_id); H5Pclose(dcpl_id); } - H5E_END_TRY; + H5E_END_TRY return FAIL; @@ -1438,7 +1438,7 @@ error: H5Aclose(aid); H5Aclose(aid_name); } - H5E_END_TRY; + H5E_END_TRY return FAIL; @@ -1518,7 +1518,7 @@ error: H5Pclose(fapl_id); H5Gclose(gid); } - H5E_END_TRY; + H5E_END_TRY return FAIL; @@ -1597,7 +1597,7 @@ error: H5Fclose(gid); H5Pclose(fapl_id); } - H5E_END_TRY; + H5E_END_TRY return FAIL; @@ -1694,7 +1694,7 @@ error: H5Tclose(tid); H5Tclose(tid_anon); } - H5E_END_TRY; + H5E_END_TRY return FAIL; @@ -1797,7 +1797,7 @@ exercise_reg_opt_oper(hid_t fake_vol_id, hid_t reg_opt_vol_id, H5VL_subclass_t s ret = (*reg_opt_op.obj_op)(__FILE__, __func__, __LINE__, obj_id, &vol_cb_args, H5P_DEFAULT, H5ES_NONE); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to perform an optional operation with a NULL callback"); if ((-1) != fake_obj) @@ -1956,7 +1956,7 @@ test_register_opt_operation(void) { ret = H5VLregister_opt_operation(H5VL_SUBCLS_NONE, "fail", &op_val); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to register an optional operation for the 'NONE' VOL subclass"); if ((-1) != op_val) @@ -1965,7 +1965,7 @@ test_register_opt_operation(void) { ret = H5VLregister_opt_operation(H5VL_SUBCLS_INFO, "fail2", &op_val); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to register an optional operation for the 'INFO' VOL subclass"); if ((-1) != op_val) @@ -1974,7 +1974,7 @@ test_register_opt_operation(void) { ret = H5VLregister_opt_operation(H5VL_SUBCLS_WRAP, "fail3", &op_val); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to register an optional operation for the 'WRAP' VOL subclass"); if ((-1) != op_val) @@ -1983,7 +1983,7 @@ test_register_opt_operation(void) { ret = H5VLregister_opt_operation(H5VL_SUBCLS_BLOB, "fail4", &op_val); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to register an optional operation for the 'BLOB' VOL subclass"); if ((-1) != op_val) @@ -1992,7 +1992,7 @@ test_register_opt_operation(void) { ret = H5VLregister_opt_operation(H5VL_SUBCLS_TOKEN, "fail5", &op_val); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to register an optional operation for the 'TOKEN' VOL subclass"); if ((-1) != op_val) @@ -2003,7 +2003,7 @@ test_register_opt_operation(void) { ret = H5VLregister_opt_operation(H5VL_SUBCLS_FILE, "fail6", NULL); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to register an optional operation with a NULL 'op_val'"); @@ -2012,7 +2012,7 @@ test_register_opt_operation(void) { ret = H5VLfind_opt_operation(H5VL_SUBCLS_DATASET, "fail", &op_val); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to find a non-existent optional operation"); @@ -2021,7 +2021,7 @@ test_register_opt_operation(void) { ret = H5VLunregister_opt_operation(H5VL_SUBCLS_DATASET, "fail"); } - H5E_END_TRY; + H5E_END_TRY if (FAIL != ret) FAIL_PUTS_ERROR("should not be able to unregister a non-existent optional operation"); @@ -2054,7 +2054,7 @@ error: H5VLunregister_connector(fake_vol_id); H5VLunregister_connector(reg_opt_vol_id); } - H5E_END_TRY; + H5E_END_TRY return FAIL; } /* end test_register_opt_operation() */ @@ -2208,7 +2208,7 @@ error: H5Pclose(fapl_id); H5VLunregister_connector(vol_id); } - H5E_END_TRY; + H5E_END_TRY HDfree(conn_env_str); return FAIL; @@ -2313,7 +2313,7 @@ error: H5VLunregister_connector(vol_id); H5Pclose(fapl_id); } - H5E_END_TRY; + H5E_END_TRY return FAIL; } /* end test_vol_cap_flags() */ @@ -2383,7 +2383,7 @@ error: H5Fclose(file_id); H5Pclose(fapl_id); } - H5E_END_TRY; + H5E_END_TRY return FAIL; } /* end test_get_vol_name() */ |