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/H5Fistore.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/H5Fistore.c')
| -rw-r--r-- | src/H5Fistore.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/H5Fistore.c b/src/H5Fistore.c index 38d7a1f..d966edd 100644 --- a/src/H5Fistore.c +++ b/src/H5Fistore.c @@ -2465,7 +2465,7 @@ H5F_istore_allocate(H5F_t *f, hid_t dxpl_id, const H5O_layout_t *layout, if(H5P_get(dc_plist, H5D_CRT_DATA_PIPELINE_NAME, &pline) < 0) HGOTO_ERROR(H5E_STORAGE, H5E_CANTGET, FAIL, "can't get data pipeline"); if(H5P_get(dc_plist, H5D_CRT_FILL_TIME_NAME, &fill_time) < 0) - HGOTO_ERROR(H5E_PLIST, H5E_CANTGET, NULL, "can't retrieve fill time"); + HGOTO_ERROR(H5E_PLIST, H5E_CANTGET, FAIL, "can't retrieve fill time"); /* Get necessary properties from dataset transfer property list */ if (TRUE!=H5P_isa_class(dxpl_id,H5P_DATASET_XFER) || NULL == (dx_plist = H5I_object(dxpl_id))) |