diff options
author | Scott Wegner <swegner@hdfgroup.org> | 2008-07-08 17:03:45 (GMT) |
---|---|---|
committer | Scott Wegner <swegner@hdfgroup.org> | 2008-07-08 17:03:45 (GMT) |
commit | 81de0a75cc6500bf6e1cbc1d7fc6cf21ac3a91a5 (patch) | |
tree | 2b1d4c85a641fa8ae110409f5969d8c58ee5ecf2 | |
parent | d989ab71f4444cfa5a93061eebdbd9df2c0efe28 (diff) | |
download | hdf5-81de0a75cc6500bf6e1cbc1d7fc6cf21ac3a91a5.zip hdf5-81de0a75cc6500bf6e1cbc1d7fc6cf21ac3a91a5.tar.gz hdf5-81de0a75cc6500bf6e1cbc1d7fc6cf21ac3a91a5.tar.bz2 |
[svn-r15338] Purpose: Reset fill value in correct hyperslab selection (fixes bug 1155)
Description:
This bug was previously fixed, but the fixed was modified slightly from the 1.8 branch and the trunk. The problem is that after we write a particular byte to manipulate the fill values, we need to write it back before reading and comparing to the expected value. The previous change makes the second write after the hyperslab is manipulated, so the initial hyperslab is still incorrect.
Tested:
VS2005 on WinXP (32- and 64-bit)
h5committest
-rw-r--r-- | test/fillval.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/fillval.c b/test/fillval.c index 7c72531..6a939c6 100644 --- a/test/fillval.c +++ b/test/fillval.c @@ -1722,6 +1722,9 @@ test_extend_cases(hid_t file, hid_t _dcpl, const char *dset_name, /* Verify the element read in is the value written out */ if(verify_rtn((unsigned)__LINE__, hs_offset, val_rd, buf) < 0) TEST_ERROR + /* Set the element back to fillval */ + if(H5Dwrite(dset, dtype, mspace, fspace, H5P_DEFAULT, fillval) < 0) TEST_ERROR + /* Release any VL components */ if(H5Dvlen_reclaim(dtype, mspace, H5P_DEFAULT, val_rd) < 0) TEST_ERROR @@ -1741,9 +1744,6 @@ test_extend_cases(hid_t file, hid_t _dcpl, const char *dset_name, /* Verify the element read in is the fill-value */ if(verify_rtn((unsigned)__LINE__, hs_offset, val_rd, fillval) < 0) TEST_ERROR - /* Set the element back to fillval */ - if(H5Dwrite(dset, dtype, mspace, fspace, H5P_DEFAULT, fillval) < 0) TEST_ERROR - /* Release any VL components */ if(H5Dvlen_reclaim(dtype, mspace, H5P_DEFAULT, val_rd) < 0) TEST_ERROR |