diff options
| author | Neil Fortner <nfortne2@hdfgroup.org> | 2015-03-31 20:20:10 (GMT) |
|---|---|---|
| committer | Neil Fortner <nfortne2@hdfgroup.org> | 2015-03-31 20:20:10 (GMT) |
| commit | 95983cbdfe74dc991c190cb04e2ff76202cac31c (patch) | |
| tree | 0c3003f7e96082c196266e0b039450e20c4e317a /src/H5S.c | |
| parent | 904c84b2a2590124713f4187a074c8d89c79b2c1 (diff) | |
| download | hdf5-95983cbdfe74dc991c190cb04e2ff76202cac31c.zip hdf5-95983cbdfe74dc991c190cb04e2ff76202cac31c.tar.gz hdf5-95983cbdfe74dc991c190cb04e2ff76202cac31c.tar.bz2 | |
[svn-r26680] Implement minor suggestions from 3/26/15 code review.
Note make check still fails in h5dump test (unrelated to this checkin).
Tested: ummon
Diffstat (limited to 'src/H5S.c')
| -rw-r--r-- | src/H5S.c | 6 |
1 files changed, 1 insertions, 5 deletions
@@ -479,7 +479,7 @@ H5Sextent_copy(hid_t dst_id,hid_t src_id) HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a dataspace") /* Copy */ - if(H5S_extent_copy_real(&(dst->extent), &(src->extent), TRUE) < 0) + if(H5S_extent_copy(dst, src) < 0) HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOPY, FAIL, "can't copy extent") done: @@ -521,10 +521,6 @@ H5S_extent_copy(H5S_t *dst, const H5S_t *src) if(H5S_select_all(dst, FALSE) < 0) HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDELETE, FAIL, "can't change selection") - /* Mark the destination space as no longer shared if it was before */ - if(H5O_msg_reset_share(H5O_SDSPACE_ID, dst) < 0) - HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRESET, FAIL, "can't stop sharing dataspace") - done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5S_extent_copy() */ |