diff options
author | Quincey Koziol <koziol@hdfgroup.org> | 2004-05-20 15:31:12 (GMT) |
---|---|---|
committer | Quincey Koziol <koziol@hdfgroup.org> | 2004-05-20 15:31:12 (GMT) |
commit | 70686065d082ad5f3bbce84decc066a9028a0a01 (patch) | |
tree | 41e2df2eeca460e3da44ed7a7f114bb4b1430035 /src/H5Smpio.c | |
parent | aec19e771d2b5de4a38cddbfaf0cd87e0a7ff0a1 (diff) | |
download | hdf5-70686065d082ad5f3bbce84decc066a9028a0a01.zip hdf5-70686065d082ad5f3bbce84decc066a9028a0a01.tar.gz hdf5-70686065d082ad5f3bbce84decc066a9028a0a01.tar.bz2 |
[svn-r8543] Purpose:
Code optimization
Description:
Expand the use of macros to inline trivial function pointer lookup and
calls to reduce the overall number of functions invoked during normal operation
of the library.
Platforms tested:
Solaris 2.7 (arabica)
FreeBSD 4.9 (sleipnir) w/parallel
Too minor to require h5committest
Diffstat (limited to 'src/H5Smpio.c')
-rw-r--r-- | src/H5Smpio.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/H5Smpio.c b/src/H5Smpio.c index 209ca91..bea5982 100644 --- a/src/H5Smpio.c +++ b/src/H5Smpio.c @@ -344,7 +344,7 @@ H5S_mpio_hyper_type( const H5S_t *space, size_t elmt_size, * Construct contig type for inner contig dims: *******************************************************/ #ifdef H5Smpi_DEBUG - HDfprintf(stderr, "%s: Making contig type %d MPI_BYTEs\n", FUNC, elmt_size ); + HDfprintf(stderr, "%s: Making contig type %d MPI_BYTEs\n", FUNC,elmt_size ); for (i=rank-1; i>=0; --i) HDfprintf(stderr, "d[%d].xtent=%Hu \n", i, d[i].xtent); #endif @@ -363,12 +363,12 @@ H5S_mpio_hyper_type( const H5S_t *space, size_t elmt_size, #endif #ifdef H5Smpi_DEBUG - HDfprintf(stderr, "%s: i=%d Making vector-type \n", FUNC, i); + HDfprintf(stderr, "%s: i=%d Making vector-type \n", FUNC,i); #endif /**************************************** * Build vector in current dimension: ****************************************/ - mpi_code =MPI_Type_vector((int)(d[i].count), /* count */ + mpi_code =MPI_Type_vector((int)(d[i].count), /* count */ (int)(d[i].block), /* blocklength */ (int)(d[i].strid), /* stride */ inner_type, /* old type */ @@ -486,7 +486,7 @@ empty: done: /* Release selection iterator */ if(sel_iter_init) { - if (H5S_select_iter_release(&sel_iter)<0) + if (H5S_SELECT_ITER_RELEASE(&sel_iter)<0) HDONE_ERROR (H5E_DATASPACE, H5E_CANTRELEASE, FAIL, "unable to release selection iterator"); } /* end if */ @@ -629,9 +629,9 @@ done: */ static herr_t H5S_mpio_spaces_xfer(H5F_t *f, const H5O_layout_t *layout, size_t elmt_size, - const H5S_t *file_space, const H5S_t *mem_space, - hid_t dxpl_id, void *_buf /*out*/, - hbool_t do_write ) + const H5S_t *file_space, const H5S_t *mem_space, + hid_t dxpl_id, void *_buf /*out*/, + hbool_t do_write ) { haddr_t addr; /* Address of dataset (or selection) within file */ size_t mpi_buf_count, mpi_file_count; /* Number of "objects" to transfer */ @@ -837,8 +837,8 @@ H5S_mpio_opt_possible( const H5S_t *mem_space, const H5S_t *file_space, const un HGOTO_DONE(FALSE); /* Check whether both selections are "regular" */ - c1=(*file_space->select.is_regular)(file_space); - c2=(*mem_space->select.is_regular)(mem_space); + c1=H5S_SELECT_IS_REGULAR(file_space); + c2=H5S_SELECT_IS_REGULAR(mem_space); if(c1==FAIL || c2==FAIL) HGOTO_ERROR(H5E_DATASPACE, H5E_BADRANGE, FAIL, "invalid check for single selection blocks"); if(c1==FALSE || c2==FALSE) |