[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, 2 insertions, 2 deletions

diff --git a/src/H5P.c b/src/H5P.c
index 63537f0..f88ec60 100644
--- a/src/H5P.c
+++ b/src/H5P.c
@@ -1847,7 +1847,7 @@ H5Pget_driver(hid_t plist_id)
  *		Rewritten to use the virtual file layer.
  *
  * 		Robb Matzke, 1999-08-05
- *		If the driver ID is -2 then substitute the current value of
+ *		If the driver ID is H5FD_VFD_DEFAULT then substitute the current value of
  *		H5FD_SEC2.
  *-------------------------------------------------------------------------
  */
@@ -1874,7 +1874,7 @@ H5Pget_driver(hid_t plist_id)
 		      "not a file access or data transfer property list");
     }
 
-    if (H5P_DEFAULT==ret_value)
+    if (H5FD_VFD_DEFAULT==ret_value)
         ret_value = H5FD_SEC2;
 
     FUNC_LEAVE(ret_value);