[svn-r3008] Purpose:

    Code cleanup
Description:
    Several places in the code were using -2 as a default value for various
    features.  However, when a default value is returned from a function that
    is supposed to return negative on failure, it was causing confusion and
    extra work for users to check for the special value.
Solution:
    Replaced hard-coded -2 values in the code with symbolic names, then changed
    symbolic names to 0 instead of -2.
Platforms tested:
    FreeBSD 4.2 (hawkwind)

1 files changed, 1 insertions, 1 deletions

diff --git a/src/H5.c b/src/H5.c
index a74c41c..ad27399 100644
--- a/src/H5.c
+++ b/src/H5.c
@@ -1689,7 +1689,7 @@ H5_trace (hbool_t returning, const char *func, const char *type, ...)
 		}
 	    } else {
 		hid_t obj = va_arg (ap, hid_t);
-		if (-2 == obj) {
+		if (H5P_DEFAULT == obj) {
 		    fprintf (out, "H5P_DEFAULT");
 		} else if (obj<0) {
 		    fprintf (out, "FAIL");