summaryrefslogtreecommitdiffstats
path: root/src/H5Fsuper_cache.c
diff options
context:
space:
mode:
authorQuincey Koziol <koziol@hdfgroup.org>2010-04-02 05:28:33 (GMT)
committerQuincey Koziol <koziol@hdfgroup.org>2010-04-02 05:28:33 (GMT)
commitc2550f345494667d4786c677f8da7068eed499f0 (patch)
tree2b7d84e605c6fe018fcdc822fa7e1fd2e7f421f6 /src/H5Fsuper_cache.c
parent0611a912634742279a12fe34494cfb239466bc6b (diff)
downloadhdf5-c2550f345494667d4786c677f8da7068eed499f0.zip
hdf5-c2550f345494667d4786c677f8da7068eed499f0.tar.gz
hdf5-c2550f345494667d4786c677f8da7068eed499f0.tar.bz2
[svn-r18505] Description:
Move code to free superblock data structure into separate routine, in preparation for upcoming metadata journaling -> trunk merging. Tested on: FreeBSD/32 6.3 (duty) in debug mode FreeBSD/64 6.3 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (jam) w/PGI compilers, w/default API=1.8.x, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (amani) w/Intel compilers, w/default API=1.6.x, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.4 (tg-login3) w/parallel, w/FORTRAN, in debug mode Linux/64-amd64 2.6 (abe) w/parallel, w/FORTRAN, in production mode Mac OS X/32 10.6.3 (amazon) in debug mode
Diffstat (limited to 'src/H5Fsuper_cache.c')
-rw-r--r--src/H5Fsuper_cache.c15
1 files changed, 8 insertions, 7 deletions
diff --git a/src/H5Fsuper_cache.c b/src/H5Fsuper_cache.c
index cbd9c31..c341c6c 100644
--- a/src/H5Fsuper_cache.c
+++ b/src/H5Fsuper_cache.c
@@ -613,7 +613,7 @@ H5F_sblock_load(H5F_t *f, hid_t dxpl_id, haddr_t UNUSED addr, const void UNUSED
done:
/* Release the [possibly partially initialized] superblock on errors */
if(!ret_value && sblock)
- if(H5F_sblock_dest(f, sblock) < 0)
+ if(H5F_super_free(sblock) < 0)
HDONE_ERROR(H5E_FILE, H5E_CANTFREE, NULL, "unable to destroy superblock data")
FUNC_LEAVE_NOAPI(ret_value)
@@ -840,18 +840,19 @@ done:
static herr_t
H5F_sblock_dest(H5F_t UNUSED *f, H5F_super_t* sblock)
{
- FUNC_ENTER_NOAPI_NOINIT_NOFUNC(H5F_sblock_dest)
+ herr_t ret_value = SUCCEED; /* Return value */
+
+ FUNC_ENTER_NOAPI_NOINIT(H5F_sblock_dest)
/* Sanity check */
HDassert(sblock);
- /* Free root group symbol table entry, if any */
- sblock->root_ent = (H5G_entry_t *)H5MM_xfree(sblock->root_ent);
-
/* Free superblock */
- sblock = (H5F_super_t *)H5FL_FREE(H5F_super_t, sblock);
+ if(H5F_super_free(sblock) < 0)
+ HGOTO_ERROR(H5E_FILE, H5E_CANTFREE, FAIL, "unable to destroy superblock")
- FUNC_LEAVE_NOAPI(SUCCEED)
+done:
+ FUNC_LEAVE_NOAPI(ret_value)
} /* end H5F_sblock_dest() */