[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)

1 files changed, 8 insertions, 8 deletions

diff --git a/src/H5FD.c b/src/H5FD.c
index 5fc0c02..9dbdfcb 100644
--- a/src/H5FD.c
+++ b/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);