diff options
| author | Neil Fortner <nfortne2@hdfgroup.org> | 2015-08-13 16:39:21 (GMT) |
|---|---|---|
| committer | Neil Fortner <nfortne2@hdfgroup.org> | 2015-08-13 16:39:21 (GMT) |
| commit | 51e16157c29eae46c391e896ed54de8e02ffdfea (patch) | |
| tree | 0341af144cc2fabc69cee4e039e6d13a58860d08 /src/H5Dvirtual.c | |
| parent | 07106823b4792cdb81bb8b6bf9b56af3e919781b (diff) | |
| download | hdf5-51e16157c29eae46c391e896ed54de8e02ffdfea.zip hdf5-51e16157c29eae46c391e896ed54de8e02ffdfea.tar.gz hdf5-51e16157c29eae46c391e896ed54de8e02ffdfea.tar.bz2 | |
[svn-r27504] Change comments/code around to reflect decision to have virtual datasets always
return TRUE when their is_alloc function is called.
Tested: ummon
Diffstat (limited to 'src/H5Dvirtual.c')
| -rw-r--r-- | src/H5Dvirtual.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/H5Dvirtual.c b/src/H5Dvirtual.c index 0b88ad5..99c090f 100644 --- a/src/H5Dvirtual.c +++ b/src/H5Dvirtual.c @@ -1933,8 +1933,12 @@ H5D__virtual_is_space_alloc(const H5O_storage_t H5_ATTR_UNUSED *storage) FUNC_ENTER_PACKAGE_NOERR - /* Need to decide what to do here. For now just return TRUE VDSINC */ - ret_value = TRUE;//storage->u.virt.serial_list_hobjid.addr != HADDR_UNDEF; + /* Just return TRUE, since the global heap object containing the mappings is + * created when the layout message is encoded, and nothing else needs to be + * allocated for virtual datasets. This also ensures that the library never + * assumes (falsely) that no data is present in the dataset, causing errors. + */ + ret_value = TRUE; FUNC_LEAVE_NOAPI(ret_value) } /* end H5D__virtual_is_space_alloc() */ |