diff options
author | Albert Cheng <acheng@hdfgroup.org> | 2002-08-20 20:29:30 (GMT) |
---|---|---|
committer | Albert Cheng <acheng@hdfgroup.org> | 2002-08-20 20:29:30 (GMT) |
commit | b35b6c0411cd96f7371721ae4aafbaae927c557e (patch) | |
tree | c8920698c1e012a6548e4e66855b1854cb0cf5f0 | |
parent | 204426a530eddd798de062e816c4ad846c3802d5 (diff) | |
download | hdf5-b35b6c0411cd96f7371721ae4aafbaae927c557e.zip hdf5-b35b6c0411cd96f7371721ae4aafbaae927c557e.tar.gz hdf5-b35b6c0411cd96f7371721ae4aafbaae927c557e.tar.bz2 |
[svn-r5881] Purpose:
bug fix
Description:
Last change of FUNC_ENTER to FUNC_ENTER_NOAPI did not include
definition of variable ret_value and label done.
Solution:
Added the missing ret_value and label done.
Not sure if the logic is correct since the function will abort
if some assert fails. This fix only takes care of the syntax
error.
Platforms tested:
eirene(pp). Did not test on other machine since the syntax fix
is pretty simple.
-rw-r--r-- | src/H5B.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -2143,6 +2143,7 @@ H5B_assert(H5F_t *f, haddr_t addr, const H5B_class_t *type, void *udata) int i, ncell, cmp; static int ncalls = 0; herr_t status; + herr_t ret_value=SUCCEED; /* Return value */ /* A queue of child data */ struct child_t { @@ -2234,6 +2235,7 @@ H5B_assert(H5F_t *f, haddr_t addr, const H5B_class_t *type, void *udata) head = tmp; } - FUNC_LEAVE(SUCCEED); +done: + FUNC_LEAVE(ret_value); } #endif /* H5B_DEBUG */ |