diff options
author | Bill Wendling <wendling@ncsa.uiuc.edu> | 2002-06-04 21:29:05 (GMT) |
---|---|---|
committer | Bill Wendling <wendling@ncsa.uiuc.edu> | 2002-06-04 21:29:05 (GMT) |
commit | dd51aae3ca540fc43ca398beaed9f6643b880265 (patch) | |
tree | 5678b1ca712e93c2d7f75daca7a7901dad1a6ff0 /src/H5Farray.c | |
parent | 56e4d8fb0cbbcd1b1036b361758ff84a40d2cb01 (diff) | |
download | hdf5-dd51aae3ca540fc43ca398beaed9f6643b880265.zip hdf5-dd51aae3ca540fc43ca398beaed9f6643b880265.tar.gz hdf5-dd51aae3ca540fc43ca398beaed9f6643b880265.tar.bz2 |
[svn-r5530] Purpose:
Code Cleanup
Description:
Removed some compiler warnings.
Solution:
In a few cases, NULL was being returned when a FAIL was supposed to
be returned instead. There were some header files which needed to be
included in a few of the sources. A couple of if-then statements had
assignments in the conditional part. The compiler warned that they
should have extra "()"s around them. Made the code check the values
instead.
Platforms tested:
Linux (parallel) Modi4 (parallel)
Diffstat (limited to 'src/H5Farray.c')
-rw-r--r-- | src/H5Farray.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/H5Farray.c b/src/H5Farray.c index 211ccaa..859f7ae 100644 --- a/src/H5Farray.c +++ b/src/H5Farray.c @@ -180,7 +180,7 @@ H5F_arr_read(H5F_t *f, hid_t dxpl_id, const H5O_layout_t *layout, /* Get the plist structure */ if(NULL == (plist = H5I_object(dxpl_id))) - HGOTO_ERROR(H5E_ATOM, H5E_BADATOM, NULL, "can't find object for ID"); + HGOTO_ERROR(H5E_ATOM, H5E_BADATOM, FAIL, "can't find object for ID"); /* Get the driver ID */ if(H5P_get(plist, H5D_XFER_VFL_ID_NAME, &driver_id)<0) @@ -430,7 +430,7 @@ H5F_arr_write(H5F_t *f, hid_t dxpl_id, const H5O_layout_t *layout, /* Get the plist structure */ if(NULL == (plist = H5I_object(dxpl_id))) - HGOTO_ERROR(H5E_ATOM, H5E_BADATOM, NULL, "can't find object for ID"); + HGOTO_ERROR(H5E_ATOM, H5E_BADATOM, FAIL, "can't find object for ID"); /* Get the driver ID */ if(H5P_get(plist, H5D_XFER_VFL_ID_NAME, &driver_id)<0) |