summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/H5AC.c173
-rw-r--r--src/H5ACprivate.h5
-rw-r--r--src/H5F.c47
-rw-r--r--src/H5Gnode.c2
-rw-r--r--src/H5MPprivate.h10
-rw-r--r--src/H5Obogus.c2
6 files changed, 96 insertions, 143 deletions
diff --git a/src/H5AC.c b/src/H5AC.c
index 3f8ff53..a051460 100644
--- a/src/H5AC.c
+++ b/src/H5AC.c
@@ -37,7 +37,7 @@
*
* John Mainzer, 6/4/04
* Factored the new cache code into a separate file (H5C.c) to
- * faciltate re-use. Re-worked this file again to use H5C.
+ * facilitate re-use. Re-worked this file again to use H5C.
*
*-------------------------------------------------------------------------
*/
@@ -70,8 +70,7 @@
*/
/* Default dataset transfer property list for metadata I/O calls */
-/* (Collective set, "block before metadata write" set and */
-/* "library internal" set) */
+/* (Collective set, "block before metadata write" set and "library internal" set) */
/* (Global variable definition, declaration is in H5ACprivate.h also) */
hid_t H5AC_dxpl_id=(-1);
@@ -96,15 +95,15 @@ static herr_t H5AC_check_if_write_permitted(H5F_t *f,
/*-------------------------------------------------------------------------
- * Function: H5AC_init
+ * Function: H5AC_init
*
- * Purpose: Initialize the interface from some other layer.
+ * Purpose: Initialize the interface from some other layer.
*
- * Return: Success: non-negative
+ * Return: Success: non-negative
*
- * Failure: negative
+ * Failure: negative
*
- * Programmer: Quincey Koziol
+ * Programmer: Quincey Koziol
* Saturday, January 18, 2003
*
* Modifications:
@@ -351,11 +350,10 @@ const char * H5AC_entry_type_names[H5AC_NTYPES] =
};
int
-H5AC_create(const H5F_t *f,
- int UNUSED size_hint)
+H5AC_create(const H5F_t *f, int UNUSED size_hint)
{
- H5AC_t * cache_ptr = NULL;
- int ret_value=1; /* Return value */
+ H5AC_t *cache = NULL;
+ int ret_value=SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(H5AC_create, FAIL)
@@ -366,19 +364,19 @@ H5AC_create(const H5F_t *f,
* in proper size hints.
* -- JRM
*/
- cache_ptr = H5C_create(H5C__DEFAULT_MAX_CACHE_SIZE,
+ cache = H5C_create(H5C__DEFAULT_MAX_CACHE_SIZE,
H5C__DEFAULT_MIN_CLEAN_SIZE,
(H5AC_NTYPES - 1),
(const char **)H5AC_entry_type_names,
H5AC_check_if_write_permitted);
- if ( NULL == cache_ptr ) {
+ if ( NULL == cache ) {
HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, FAIL, "memory allocation failed")
} else {
- f->shared->cache = (H5AC_t *)cache_ptr;
+ f->shared->cache = cache;
}
@@ -386,9 +384,9 @@ done:
if ( ret_value < 0 ) {
- if ( cache_ptr != NULL ) {
+ if ( cache != NULL ) {
- H5C_dest_empty(cache_ptr);
+ H5C_dest_empty(cache);
f->shared->cache = NULL;
} /* end if */
@@ -427,21 +425,20 @@ done:
*-------------------------------------------------------------------------
*/
herr_t
-H5AC_dest(H5F_t *f,
- hid_t dxpl_id)
+H5AC_dest(H5F_t *f, hid_t dxpl_id)
{
- H5AC_t *cache_ptr = NULL;
+ H5AC_t *cache = NULL;
herr_t ret_value=SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(H5AC_dest, FAIL)
assert(f);
assert(f->shared->cache);
- cache_ptr = (H5AC_t *)(f->shared->cache);
+ cache = f->shared->cache;
f->shared->cache = NULL;
- if ( H5C_dest(f, dxpl_id, H5AC_noblock_dxpl_id, cache_ptr) < 0 ) {
+ if ( H5C_dest(f, dxpl_id, H5AC_noblock_dxpl_id, cache) < 0 ) {
HGOTO_ERROR(H5E_CACHE, H5E_CANTFREE, FAIL, "can't destroy cache")
}
@@ -517,29 +514,20 @@ done:
*-------------------------------------------------------------------------
*/
herr_t
-H5AC_flush(H5F_t *f,
- hid_t dxpl_id,
- const H5AC_class_t *type,
- haddr_t addr,
- unsigned flags)
+H5AC_flush(H5F_t *f, hid_t dxpl_id, unsigned flags)
{
- herr_t status;
- herr_t ret_value = SUCCEED;
- H5AC_t * cache_ptr;
+ herr_t status;
+ herr_t ret_value=SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(H5AC_flush, FAIL)
HDassert(f);
HDassert(f->shared->cache);
- HDassert(type == NULL);
- HDassert(!H5F_addr_defined(addr));
-
- cache_ptr = (H5AC_t *)(f->shared->cache);
status = H5C_flush_cache(f,
dxpl_id,
H5AC_noblock_dxpl_id,
- cache_ptr,
+ f->shared->cache,
flags);
if ( status < 0 ) {
@@ -592,18 +580,13 @@ done:
*
*-------------------------------------------------------------------------
*/
-
herr_t
-H5AC_set(H5F_t *f,
- hid_t dxpl_id,
- const H5AC_class_t *type,
- haddr_t addr,
- void *thing)
+H5AC_set(H5F_t *f, hid_t dxpl_id, const H5AC_class_t *type, haddr_t addr, void *thing)
{
herr_t result;
- herr_t ret_value = SUCCEED; /* Return value */
- H5AC_info_t * info_ptr;
- H5AC_t * cache_ptr;
+ H5AC_info_t *info;
+ H5AC_t *cache;
+ herr_t ret_value=SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(H5AC_set, FAIL)
@@ -615,12 +598,13 @@ H5AC_set(H5F_t *f,
HDassert(H5F_addr_defined(addr));
HDassert(thing);
- cache_ptr = (H5AC_t *)(f->shared->cache);
- info_ptr = (H5AC_info_t *)thing;
+ /* Get local copy of this information */
+ cache = f->shared->cache;
+ info = (H5AC_info_t *)thing;
- info_ptr->addr = addr;
- info_ptr->type = type;
- info_ptr->is_protected = FALSE;
+ info->addr = addr;
+ info->type = type;
+ info->is_protected = FALSE;
#ifdef H5_HAVE_PARALLEL
#ifdef H5_HAVE_FPHDF5
@@ -668,8 +652,8 @@ H5AC_set(H5F_t *f,
/* write the metadata to the SAP. */
- result = (info_ptr->type->flush)(f, dxpl_id, TRUE,
- info_ptr->addr, info_ptr);
+ result = (info->type->flush)(f, dxpl_id, TRUE,
+ info->addr, info);
if ( result < 0 ) {
HGOTO_ERROR(H5E_CACHE, H5E_CANTFLUSH, FAIL, \
@@ -695,7 +679,7 @@ H5AC_set(H5F_t *f,
result = H5C_insert_entry(f,
dxpl_id,
H5AC_noblock_dxpl_id,
- cache_ptr,
+ cache,
type,
addr,
thing);
@@ -742,17 +726,12 @@ done:
*
*-------------------------------------------------------------------------
*/
-
herr_t
-H5AC_rename(H5F_t *f,
- hid_t UNUSED dxpl_id,
- const H5AC_class_t *type,
- haddr_t old_addr,
+H5AC_rename(H5F_t *f, hid_t UNUSED dxpl_id, const H5AC_class_t *type, haddr_t old_addr,
haddr_t new_addr)
{
herr_t result;
- herr_t ret_value = SUCCEED; /* Return value */
- H5AC_t * cache_ptr;
+ herr_t ret_value=SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(H5AC_rename, FAIL)
@@ -763,8 +742,6 @@ H5AC_rename(H5F_t *f,
HDassert(H5F_addr_defined(new_addr));
HDassert(H5F_addr_ne(old_addr, new_addr));
- cache_ptr = (H5AC_t *)(f->shared->cache);
-
#ifdef H5_HAVE_PARALLEL
#ifdef H5_HAVE_FPHDF5
/* In the flexible parallel case, the cache is always empty.
@@ -786,7 +763,7 @@ H5AC_rename(H5F_t *f,
#endif /* H5_HAVE_PARALLEL */
result = H5C_rename_entry(f,
- cache_ptr,
+ f->shared->cache,
type,
old_addr,
new_addr);
@@ -863,7 +840,6 @@ done:
*
*-------------------------------------------------------------------------
*/
-
void *
H5AC_protect(H5F_t *f,
hid_t dxpl_id,
@@ -878,7 +854,6 @@ H5AC_protect(H5F_t *f,
rw)
{
void * thing = NULL;
- H5AC_t * cache_ptr = NULL;
void * ret_value; /* Return value */
FUNC_ENTER_NOAPI(H5AC_protect, NULL)
@@ -891,8 +866,6 @@ H5AC_protect(H5F_t *f,
HDassert(type->load);
HDassert(H5F_addr_defined(addr));
- cache_ptr = (H5AC_t *)(f->shared->cache);
-
#ifdef H5_HAVE_PARALLEL
#ifdef H5_HAVE_FPHDF5
/* The following code to support flexible parallel is a direct copy
@@ -904,7 +877,7 @@ H5AC_protect(H5F_t *f,
H5FD_t * lf;
unsigned req_id;
H5FP_status_t status;
- H5AC_info_t * info_ptr;
+ H5AC_info_t * info;
HDassert(f->shared->lf);
@@ -960,26 +933,26 @@ H5AC_protect(H5F_t *f,
HGOTO_DONE(NULL);
}
- info_ptr = (H5AC_info_t *)thing;
+ info = (H5AC_info_t *)thing;
- HDassert(info_ptr->is_dirty == FALSE);
+ HDassert(info->is_dirty == FALSE);
- info_ptr->addr = addr;
- info_ptr->type = type;
- info_ptr->is_protected = TRUE;
+ info->addr = addr;
+ info->type = type;
+ info->is_protected = TRUE;
- if ( (type->size)(f, thing, &(info_ptr->size)) < 0 ) {
+ if ( (type->size)(f, thing, &(info->size)) < 0 ) {
HGOTO_ERROR(H5E_RESOURCE, H5E_CANTGETSIZE, NULL, \
"Can't get size of thing")
}
- HDassert(info_ptr->size < H5C_MAX_ENTRY_SIZE);
+ HDassert(info->size < H5C_MAX_ENTRY_SIZE);
- info_ptr->next = NULL;
- info_ptr->prev = NULL;
- info_ptr->aux_next = NULL;
- info_ptr->aux_prev = NULL;
+ info->next = NULL;
+ info->prev = NULL;
+ info->aux_next = NULL;
+ info->aux_prev = NULL;
HGOTO_DONE(thing);
}
@@ -990,7 +963,7 @@ H5AC_protect(H5F_t *f,
thing = H5C_protect(f,
dxpl_id,
H5AC_noblock_dxpl_id,
- cache_ptr,
+ f->shared->cache,
type,
addr,
udata1,
@@ -1052,11 +1025,11 @@ done:
* Sep 2 1997
*
* Modifications:
- * Robb Matzke, 1999-07-27
- * The ADDR argument is passed by value.
+ * Robb Matzke, 1999-07-27
+ * The ADDR argument is passed by value.
*
- * Quincey Koziol, 2003-03-19
- * Added "deleted" argument
+ * Quincey Koziol, 2003-03-19
+ * Added "deleted" argument
*
* Bill Wendling, 2003-09-18
* If this is an FPHDF5 driver and the data is dirty,
@@ -1073,17 +1046,10 @@ done:
*-------------------------------------------------------------------------
*/
herr_t
-H5AC_unprotect(H5F_t *f,
- hid_t dxpl_id,
- const H5AC_class_t *type,
- haddr_t addr,
- void *thing,
- hbool_t deleted)
+H5AC_unprotect(H5F_t *f, hid_t dxpl_id, const H5AC_class_t *type, haddr_t addr, void *thing, hbool_t deleted)
{
herr_t result;
- herr_t ret_value = SUCCEED; /* Return value */
- H5AC_info_t * info_ptr;
- H5AC_t * cache_ptr = NULL;
+ herr_t ret_value=SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(H5AC_unprotect, FAIL)
@@ -1094,12 +1060,8 @@ H5AC_unprotect(H5F_t *f,
HDassert(type->flush);
HDassert(H5F_addr_defined(addr));
HDassert(thing);
-
- cache_ptr = (H5AC_t *)(f->shared->cache);
- info_ptr = (H5AC_info_t *)thing;
-
- HDassert( info_ptr->addr == addr );
- HDassert( info_ptr->type == type );
+ HDassert( ((H5AC_info_t *)thing)->addr == addr );
+ HDassert( ((H5AC_info_t *)thing)->type == type );
#ifdef H5_HAVE_PARALLEL
#ifdef H5_HAVE_FPHDF5
@@ -1119,9 +1081,9 @@ H5AC_unprotect(H5F_t *f,
if ( H5FD_is_fphdf5_driver(lf) ) {
- HDassert( info_ptr->is_protected );
+ HDassert( ((H5AC_info_t *)thing)->is_protected );
- info_ptr->is_protected = FALSE;
+ ((H5AC_info_t *)thing)->is_protected = FALSE;
/*
* FIXME: If the metadata is *really* deleted at this point
@@ -1155,7 +1117,7 @@ H5AC_unprotect(H5F_t *f,
}
/* Exit now. The FPHDF5 stuff is finished. */
- HGOTO_DONE(SUCCEED);
+ HGOTO_DONE(SUCCEED)
}
}
#endif /* H5_HAVE_FPHDF5 */
@@ -1164,7 +1126,7 @@ H5AC_unprotect(H5F_t *f,
result = H5C_unprotect(f,
dxpl_id,
H5AC_noblock_dxpl_id,
- cache_ptr,
+ f->shared->cache,
type,
addr,
thing,
@@ -1204,21 +1166,17 @@ done:
*
*-------------------------------------------------------------------------
*/
-
herr_t
H5AC_stats(H5F_t UNUSED *f)
{
herr_t ret_value = SUCCEED; /* Return value */
- H5AC_t * cache_ptr;
FUNC_ENTER_NOAPI(H5AC_stats, FAIL)
HDassert(f);
HDassert(f->shared->cache);
- cache_ptr = (H5AC_t *)(f->shared->cache);
-
- H5C_stats(cache_ptr, f->name, FALSE); /* at present, this can't fail */
+ H5C_stats(f->shared->cache, f->name, FALSE); /* at present, this can't fail */
done:
FUNC_LEAVE_NOAPI(ret_value)
@@ -1312,4 +1270,3 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* H5AC_check_if_write_permitted() */
-
diff --git a/src/H5ACprivate.h b/src/H5ACprivate.h
index bf1cffe..214f38e 100644
--- a/src/H5ACprivate.h
+++ b/src/H5ACprivate.h
@@ -171,14 +171,11 @@ H5_DLL void *H5AC_protect(H5F_t *f, hid_t dxpl_id, const H5AC_class_t *type,
H5AC_protect_t rw);
H5_DLL herr_t H5AC_unprotect(H5F_t *f, hid_t dxpl_id, const H5AC_class_t *type, haddr_t addr,
void *thing, hbool_t deleted);
-H5_DLL herr_t H5AC_flush(H5F_t *f, hid_t dxpl_id, const H5AC_class_t *type, haddr_t addr,
- unsigned flags);
+H5_DLL herr_t H5AC_flush(H5F_t *f, hid_t dxpl_id, unsigned flags);
H5_DLL herr_t H5AC_rename(H5F_t *f, hid_t dxpl_id, const H5AC_class_t *type,
haddr_t old_addr, haddr_t new_addr);
H5_DLL herr_t H5AC_dest(H5F_t *f, hid_t dxpl_id);
-
H5_DLL herr_t H5AC_stats(H5F_t *f);
-
#endif /* !_H5ACprivate_H */
diff --git a/src/H5F.c b/src/H5F.c
index 9eee843..798a28e 100644
--- a/src/H5F.c
+++ b/src/H5F.c
@@ -1130,7 +1130,7 @@ H5F_get_objects_cb(void *obj_ptr, hid_t obj_id, void *key)
if( (!olist->shared && olist->obj_type==H5I_DATATYPE && H5T_is_immutable((H5T_t*)obj_ptr)==FALSE)
|| (!olist->shared && olist->obj_type!=H5I_DATATYPE)
- || (ent && ent->file->shared == olist->shared) ) {
+ || (ent && ent->file && ent->file->shared == olist->shared) ) {
/* Add the object's ID to the ID list, if appropriate */
if(olist->obj_id_list) {
olist->obj_id_list[olist->list_index] = obj_id;
@@ -3033,7 +3033,7 @@ H5F_flush(H5F_t *f, hid_t dxpl_id, H5F_scope_t scope, unsigned flags)
* allocates object headers (calls the H5O_init function...via a
* lot of other functions first)....
*/
- if (H5AC_flush(f, dxpl_id, NULL, HADDR_UNDEF, flags & (H5F_FLUSH_INVALIDATE | H5F_FLUSH_CLEAR_ONLY)) < 0)
+ if (H5AC_flush(f, dxpl_id, flags & (H5F_FLUSH_INVALIDATE | H5F_FLUSH_CLEAR_ONLY)) < 0)
HGOTO_ERROR(H5E_CACHE, H5E_CANTFLUSH, FAIL, "unable to flush meta data cache")
/* Write the superblock to disk */
@@ -3306,7 +3306,7 @@ H5F_close(H5F_t *f)
if (H5F_flush(f, H5AC_dxpl_id, H5F_SCOPE_LOCAL,
H5F_FLUSH_INVALIDATE | H5F_FLUSH_CLOSING | H5F_FLUSH_CLEAR_ONLY) < 0)
HGOTO_ERROR(H5E_CACHE, H5E_CANTFLUSH, FAIL, "unable to flush cache")
- }
+ } /* end if */
#endif /* H5_HAVE_FPHDF5 */
} /* end if */
@@ -3426,8 +3426,8 @@ H5F_mount(H5G_entry_t *loc, const char *name, H5F_t *child,
parent = H5G_fileof(mount_point);
mp_ent = H5G_entof(mount_point);
for (ancestor=parent; ancestor; ancestor=ancestor->mtab.parent) {
- if (ancestor==child)
- HGOTO_ERROR(H5E_FILE, H5E_MOUNT, FAIL, "mount would introduce a cycle")
+ if (ancestor==child)
+ HGOTO_ERROR(H5E_FILE, H5E_MOUNT, FAIL, "mount would introduce a cycle")
}
/*
@@ -3439,29 +3439,29 @@ H5F_mount(H5G_entry_t *loc, const char *name, H5F_t *child,
rt=parent->mtab.nmounts;
cmp = -1;
while (lt<rt && cmp) {
- md = (lt+rt)/2;
- ent = H5G_entof(parent->mtab.child[md].group);
- cmp = H5F_addr_cmp(mp_ent->header, ent->header);
- if (cmp<0) {
- rt = md;
- } else if (cmp>0) {
- lt = md+1;
- }
+ md = (lt+rt)/2;
+ ent = H5G_entof(parent->mtab.child[md].group);
+ cmp = H5F_addr_cmp(mp_ent->header, ent->header);
+ if (cmp<0) {
+ rt = md;
+ } else if (cmp>0) {
+ lt = md+1;
+ }
}
if (cmp>0)
md++;
if (!cmp)
- HGOTO_ERROR(H5E_FILE, H5E_MOUNT, FAIL, "mount point is already in use")
+ HGOTO_ERROR(H5E_FILE, H5E_MOUNT, FAIL, "mount point is already in use")
/* Make room in the table */
if (parent->mtab.nmounts>=parent->mtab.nalloc) {
- unsigned n = MAX(16, 2*parent->mtab.nalloc);
- H5F_mount_t *x = H5MM_realloc(parent->mtab.child,
+ unsigned n = MAX(16, 2*parent->mtab.nalloc);
+ H5F_mount_t *x = H5MM_realloc(parent->mtab.child,
n*sizeof(parent->mtab.child[0]));
- if (!x)
- HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, FAIL, "memory allocation failed for mount table")
- parent->mtab.child = x;
- parent->mtab.nalloc = n;
+ if (!x)
+ HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, FAIL, "memory allocation failed for mount table")
+ parent->mtab.child = x;
+ parent->mtab.nalloc = n;
}
/* Insert into table */
@@ -3478,9 +3478,9 @@ H5F_mount(H5G_entry_t *loc, const char *name, H5F_t *child,
name_r=H5RS_wrap(name);
assert(name_r);
if (H5G_replace_name( H5G_UNKNOWN, loc, name_r, NULL, NULL, NULL, OP_MOUNT )<0)
- HGOTO_ERROR(H5E_FILE, H5E_MOUNT, FAIL, "unable to replace name")
+ HGOTO_ERROR(H5E_FILE, H5E_MOUNT, FAIL, "unable to replace name")
if(H5RS_decr(name_r)<0)
- HGOTO_ERROR(H5E_FILE, H5E_CANTDEC, FAIL, "unable to decrement name string")
+ HGOTO_ERROR(H5E_FILE, H5E_CANTDEC, FAIL, "unable to decrement name string")
done:
if (ret_value<0 && mount_point)
@@ -3550,7 +3550,7 @@ H5F_unmount(H5G_entry_t *loc, const char *name, hid_t dxpl_id)
ent = H5G_entof(child->shared->root_grp);
if (child->mtab.parent &&
- H5F_addr_eq(mnt_ent->header, ent->header)) {
+ H5F_addr_eq(mnt_ent->header, ent->header)) {
/*
* We've been given the root group of the child. We do a reverse
* lookup in the parent's mount table to find the correct entry.
@@ -4805,4 +4805,3 @@ H5Fget_name(hid_t obj_id, char *name/*out*/, size_t size)
done:
FUNC_LEAVE_API(ret_value);
} /* end H5Fget_name() */
-
diff --git a/src/H5Gnode.c b/src/H5Gnode.c
index bdb4ae6..2f83d28 100644
--- a/src/H5Gnode.c
+++ b/src/H5Gnode.c
@@ -610,7 +610,7 @@ H5G_node_dest(H5F_t UNUSED *f, H5G_node_t *sym)
assert(sym);
/* Verify that node is clean */
- assert (sym->cache_info.is_dirty==0);
+ assert (sym->cache_info.is_dirty==FALSE);
if(sym->entry)
sym->entry = H5FL_SEQ_FREE(H5G_entry_t,sym->entry);
diff --git a/src/H5MPprivate.h b/src/H5MPprivate.h
index 7df3154..97e8ccb 100644
--- a/src/H5MPprivate.h
+++ b/src/H5MPprivate.h
@@ -96,12 +96,15 @@
#define color_H5Dset_extent "red"
#define color_H5Dget_offset "red"
-#define color_H5Eclear "red"
#define color_H5Eclear_stack "red"
#define color_H5Eclose_msg "red"
#define color_H5Eclose_stack "red"
#define color_H5Ecreate_msg "red"
+#define color_H5Eset_auto "red"
#define color_H5Eget_auto "red"
+#define color_H5Eclear "red"
+#define color_H5Eprint "red"
+#define color_H5Ewalk "red"
#define color_H5Eget_auto_stack "red"
#define color_H5Eget_class_name "red"
#define color_H5Eget_current_stack "red"
@@ -110,16 +113,13 @@
#define color_H5Eget_msg "red"
#define color_H5Eget_num "red"
#define color_H5Epop "red"
-#define color_H5Eprint "red"
#define color_H5Eprint_stack "red"
#define color_H5Epush "red"
#define color_H5Epush_stack "red"
#define color_H5Eregister_class "red"
-#define color_H5Eset_auto "red"
#define color_H5Eset_auto_stack "red"
#define color_H5Eset_current_stack "red"
#define color_H5Eunregister_class "red"
-#define color_H5Ewalk "red"
#define color_H5Ewalk_stack "red"
#define color_H5Eauto_is_stack "red"
@@ -313,7 +313,6 @@
#define color_H5Pisa_class "red"
#define color_H5Piterate "red"
#define color_H5Pcopy_prop "red"
-#define color_H5Premove_filter "red"
#define color_H5Premove "red"
#define color_H5Punregister "red"
#define color_H5Pclose_class "red"
@@ -354,6 +353,7 @@
#define color_H5Pget_nfilters "red"
#define color_H5Pget_filter "red"
#define color_H5Pget_filter_by_id "red"
+#define color_H5Premove_filter "red"
#define color_H5Pset_deflate "red"
#define color_H5Pset_fletcher32 "red"
#define color_H5Pset_szip "red"
diff --git a/src/H5Obogus.c b/src/H5Obogus.c
index ddd0a2c..e6f7931 100644
--- a/src/H5Obogus.c
+++ b/src/H5Obogus.c
@@ -179,7 +179,7 @@ H5O_bogus_size(H5F_t UNUSED *f, const void UNUSED *mesg)
/* check args */
assert(f);
- FUNC_LEAVE(4);
+ FUNC_LEAVE_NOAPI(4);
} /* end H5O_bogus_size() */