mirror of
https://github.com/HDFGroup/hdf5.git
synced 2025-02-11 16:01:00 +08:00
[svn-r3993] Purpose:
Bug fix for Pablo integration Description: Several API functions were using FUNC_ENTER, without using FUNC_LEAVE, HRETURN or HRETURN_ERROR to leave the function. (Using "plain" 'return' statements instead). Solution: Changed return statements to FUNC_LEAVE, HRETURN or HRETURN_ERROR, as appropriate. Platforms tested: FreeBSD 4.3 (hawkwind)
This commit is contained in:
parent
dbdeacb7ad
commit
4d245ec856
16
src/H5FD.c
16
src/H5FD.c
@ -983,17 +983,17 @@ H5FD_cmp(const H5FD_t *f1, const H5FD_t *f2)
|
||||
|
||||
FUNC_ENTER(H5FD_cmp, -1); /*return value is arbitrary*/
|
||||
|
||||
if ((!f1 || !f1->cls) && (!f2 || !f2->cls)) return 0;
|
||||
if (!f1 || !f1->cls) return -1;
|
||||
if (!f2 || !f2->cls) return 1;
|
||||
if (f1->cls < f2->cls) return -1;
|
||||
if (f1->cls > f2->cls) return 1;
|
||||
if ((!f1 || !f1->cls) && (!f2 || !f2->cls)) HRETURN(0);
|
||||
if (!f1 || !f1->cls) HRETURN(-1);
|
||||
if (!f2 || !f2->cls) HRETURN(1);
|
||||
if (f1->cls < f2->cls) HRETURN(-1);
|
||||
if (f1->cls > f2->cls) HRETURN(1);
|
||||
|
||||
/* Files are same driver; no cmp callback */
|
||||
if (!f1->cls->cmp) {
|
||||
if (f1<f2) return -1;
|
||||
if (f1>f2) return 1;
|
||||
return 0;
|
||||
if (f1<f2) HRETURN(-1);
|
||||
if (f1>f2) HRETURN(1);
|
||||
HRETURN(0);
|
||||
}
|
||||
|
||||
ret_value = (f1->cls->cmp)(f1, f2);
|
||||
|
@ -1132,7 +1132,8 @@ H5FL_arr_free(H5FL_arr_head_t *head, void *obj)
|
||||
H5MM_xfree(obj);
|
||||
#else /* NO_ARR_FREE_LISTS */
|
||||
/* The H5MM_xfree code allows obj to null */
|
||||
if (!obj) return (NULL);
|
||||
if (!obj)
|
||||
HRETURN (NULL);
|
||||
|
||||
/* Double check parameters */
|
||||
assert(head);
|
||||
|
Loading…
Reference in New Issue
Block a user