diff options
| author | Larry Knox <lrknox@hdfgroup.org> | 2022-07-26 19:30:04 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-07-26 19:30:04 (GMT) |
| commit | d0739f13c131fd8d85c4b3247c26ca2bca515ece (patch) | |
| tree | a70ce4e47b6689ae6b9aa5d47c9e143347998a97 /tools | |
| parent | 6eaa59d2f83a89e9b4808f04caa48d313e658df1 (diff) | |
| download | hdf5-d0739f13c131fd8d85c4b3247c26ca2bca515ece.zip hdf5-d0739f13c131fd8d85c4b3247c26ca2bca515ece.tar.gz hdf5-d0739f13c131fd8d85c4b3247c26ca2bca515ece.tar.bz2 | |
Revert "1.12 eliminate unnecessary errors in the error stack (#1880)" (#1935)
This reverts commit f6997681335f0b5fe2e8904f9108a71c5200fb2d.
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/lib/h5tools_str.c | 34 | ||||
| -rw-r--r-- | tools/lib/h5tools_str.h | 3 |
2 files changed, 17 insertions, 20 deletions
diff --git a/tools/lib/h5tools_str.c b/tools/lib/h5tools_str.c index daeeb95..626adfa 100644 --- a/tools/lib/h5tools_str.c +++ b/tools/lib/h5tools_str.c @@ -1182,7 +1182,7 @@ h5tools_str_sprint(h5tools_str_t *str, const h5tool_format_t *info, hid_t contai /* if(nsize == H5R_DSET_REG_REF_BUF_SIZE) */ H5TOOLS_DEBUG("H5T_REFERENCE:H5T_STD_REF_DSETREG"); h5tools_str_append(str, H5_TOOLS_DATASET); - h5tools_str_sprint_old_reference(str, container, H5R_DATASET_REGION, vp); + h5tools_str_sprint_old_reference(str, container, vp); } else if (H5Tequal(type, H5T_STD_REF_OBJ)) { /* if (nsize == H5R_OBJ_REF_BUF_SIZE) */ @@ -1231,7 +1231,7 @@ h5tools_str_sprint(h5tools_str_t *str, const h5tool_format_t *info, hid_t contai obj_tok_str = NULL; } - h5tools_str_sprint_old_reference(str, container, H5R_OBJECT, vp); + h5tools_str_sprint_old_reference(str, container, vp); } /* end else if (H5Tequal(type, H5T_STD_REF_OBJ)) */ } break; @@ -1382,7 +1382,7 @@ h5tools_str_sprint(h5tools_str_t *str, const h5tool_format_t *info, hid_t contai *------------------------------------------------------------------------- */ void -h5tools_str_sprint_old_reference(h5tools_str_t *str, hid_t container, H5R_type_t ref_type, void *vp) +h5tools_str_sprint_old_reference(h5tools_str_t *str, hid_t container, void *vp) { hid_t obj = H5I_INVALID_HID; hid_t region = H5I_INVALID_HID; @@ -1391,23 +1391,21 @@ h5tools_str_sprint_old_reference(h5tools_str_t *str, hid_t container, H5R_type_t H5TOOLS_START_DEBUG(" "); h5tools_str_append(str, " \""); - if (ref_type == H5R_DATASET_REGION) { - obj = H5Rdereference2(container, H5P_DEFAULT, ref_type, vp); - if (obj >= 0) { - region = H5Rget_region(container, ref_type, vp); - if (region >= 0) { - H5Rget_name(obj, ref_type, vp, (char *)ref_name, 1024); - h5tools_str_append(str, "%s", ref_name); + obj = H5Rdereference2(container, H5P_DEFAULT, H5R_DATASET_REGION, vp); + if (obj >= 0) { + region = H5Rget_region(container, H5R_DATASET_REGION, vp); + if (region >= 0) { + H5Rget_name(obj, H5R_DATASET_REGION, vp, (char *)ref_name, 1024); + h5tools_str_append(str, "%s", ref_name); - H5Sclose(region); - } /* end if (region >= 0) */ - H5Dclose(obj); - } /* end if (obj >= 0) */ - } - else if (ref_type == H5R_OBJECT) { - obj = H5Rdereference2(container, H5P_DEFAULT, ref_type, vp); + H5Sclose(region); + } /* end if (region >= 0) */ + H5Dclose(obj); + } /* end if (obj >= 0) */ + else { + obj = H5Rdereference2(container, H5P_DEFAULT, H5R_OBJECT, vp); if (obj >= 0) { - H5Rget_name(obj, ref_type, vp, (char *)ref_name, 1024); + H5Rget_name(obj, H5R_OBJECT, vp, (char *)ref_name, 1024); h5tools_str_append(str, "%s", ref_name); H5Dclose(obj); } diff --git a/tools/lib/h5tools_str.h b/tools/lib/h5tools_str.h index 27adf46..15834bb 100644 --- a/tools/lib/h5tools_str.h +++ b/tools/lib/h5tools_str.h @@ -41,8 +41,7 @@ H5TOOLS_DLL void h5tools_str_dump_space_slabs(h5tools_str_t *, hid_t, const h5t h5tools_context_t *ctx); H5TOOLS_DLL void h5tools_str_dump_space_blocks(h5tools_str_t *, hid_t, const h5tool_format_t *); H5TOOLS_DLL void h5tools_str_dump_space_points(h5tools_str_t *, hid_t, const h5tool_format_t *); -H5TOOLS_DLL void h5tools_str_sprint_old_reference(h5tools_str_t *str, hid_t container, H5R_type_t ref_type, - void *vp); +H5TOOLS_DLL void h5tools_str_sprint_old_reference(h5tools_str_t *str, hid_t container, void *vp); H5TOOLS_DLL void h5tools_str_sprint_reference(h5tools_str_t *str, H5R_ref_t *vp); H5TOOLS_DLL char *h5tools_str_sprint(h5tools_str_t *str, const h5tool_format_t *info, hid_t container, hid_t type, void *vp, h5tools_context_t *ctx); |