diff options
| author | Quincey Koziol <koziol@hdfgroup.org> | 2000-11-28 16:18:42 (GMT) |
|---|---|---|
| committer | Quincey Koziol <koziol@hdfgroup.org> | 2000-11-28 16:18:42 (GMT) |
| commit | 51f5bbc54fae647682f4d6723c5c956053e6195a (patch) | |
| tree | 0401dd21d85ca75986d222012908ac379a35d15b /src/H5FDfamily.c | |
| parent | 923c2ecb780c709a210f74d679fc4cdde08fa0d3 (diff) | |
| download | hdf5-51f5bbc54fae647682f4d6723c5c956053e6195a.zip hdf5-51f5bbc54fae647682f4d6723c5c956053e6195a.tar.gz hdf5-51f5bbc54fae647682f4d6723c5c956053e6195a.tar.bz2 | |
[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)
Diffstat (limited to 'src/H5FDfamily.c')
| -rw-r--r-- | src/H5FDfamily.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/H5FDfamily.c b/src/H5FDfamily.c index 5266336..6b6881d 100644 --- a/src/H5FDfamily.c +++ b/src/H5FDfamily.c @@ -597,9 +597,9 @@ H5FD_family_cmp(const H5FD_t *_f1, const H5FD_t *_f2) { const H5FD_family_t *f1 = (const H5FD_family_t*)_f1; const H5FD_family_t *f2 = (const H5FD_family_t*)_f2; - int ret_value=(-2); + int ret_value=(H5FD_VFD_DEFAULT); - FUNC_ENTER(H5FD_family_cmp, -2); + FUNC_ENTER(H5FD_family_cmp, H5FD_VFD_DEFAULT); assert(f1->nmembs>=1 && f1->memb[0]); assert(f2->nmembs>=1 && f2->memb[0]); |