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/H5FDmpio.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/H5FDmpio.c')
| -rw-r--r-- | src/H5FDmpio.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/H5FDmpio.c b/src/H5FDmpio.c index 501ce17..69e2b49 100644 --- a/src/H5FDmpio.c +++ b/src/H5FDmpio.c @@ -439,7 +439,7 @@ H5FD_mpio_communicator(H5FD_t *_file) { H5FD_mpio_t *file = (H5FD_mpio_t*)_file; - FUNC_ENTER_NOAPI(H5FD_mpio_communicator, NULL); + FUNC_ENTER_NOAPI(H5FD_mpio_communicator, MPI_COMM_NULL); assert(file); assert(H5FD_MPIO==file->pub.driver_id); @@ -1627,10 +1627,12 @@ H5FD_mpio_flush(H5FD_t *_file, unsigned closing) { H5FD_mpio_t *file = (H5FD_mpio_t*)_file; int mpi_code; /* mpi return code */ - uint8_t byte=0; - MPI_Status mpi_stat = {0}; MPI_Offset mpi_off; herr_t ret_value=SUCCEED; +#ifdef OLD_WAY + uint8_t byte=0; + MPI_Status mpi_stat = {0}; +#endif /* OLD_WAY */ FUNC_ENTER_NOAPI(H5FD_mpio_flush, FAIL); |