diff options
Diffstat (limited to 'src/H5Rint.c')
| -rw-r--r-- | src/H5Rint.c | 625 |
1 files changed, 294 insertions, 331 deletions
diff --git a/src/H5Rint.c b/src/H5Rint.c index 38eff9d..5f8c7e9 100644 --- a/src/H5Rint.c +++ b/src/H5Rint.c @@ -15,25 +15,24 @@ /* Module Setup */ /****************/ -#include "H5Rmodule.h" /* This source code file is part of the H5R module */ - +#include "H5Rmodule.h" /* This source code file is part of the H5R module */ /***********/ /* Headers */ /***********/ -#include "H5private.h" /* Generic Functions */ -#include "H5ACprivate.h" /* Metadata cache */ -#include "H5CXprivate.h" /* API Contexts */ -#include "H5Dprivate.h" /* Datasets */ -#include "H5Eprivate.h" /* Error handling */ -#include "H5Gprivate.h" /* Groups */ -#include "H5HGprivate.h" /* Global Heaps */ -#include "H5Iprivate.h" /* IDs */ -#include "H5MMprivate.h" /* Memory management */ -#include "H5Oprivate.h" /* Object headers */ -#include "H5Rpkg.h" /* References */ -#include "H5Sprivate.h" /* Dataspaces */ -#include "H5Tprivate.h" /* Datatypes */ +#include "H5private.h" /* Generic Functions */ +#include "H5ACprivate.h" /* Metadata cache */ +#include "H5CXprivate.h" /* API Contexts */ +#include "H5Dprivate.h" /* Datasets */ +#include "H5Eprivate.h" /* Error handling */ +#include "H5Gprivate.h" /* Groups */ +#include "H5HGprivate.h" /* Global Heaps */ +#include "H5Iprivate.h" /* IDs */ +#include "H5MMprivate.h" /* Memory management */ +#include "H5Oprivate.h" /* Object headers */ +#include "H5Rpkg.h" /* References */ +#include "H5Sprivate.h" /* Dataspaces */ +#include "H5Tprivate.h" /* Datatypes */ #include "H5VLnative_private.h" /* Native VOL connector */ @@ -41,87 +40,85 @@ /* Local Macros */ /****************/ -#define H5R_MAX_STRING_LEN (1 << 16) /* Max encoded string length */ +#define H5R_MAX_STRING_LEN (1 << 16) /* Max encoded string length */ /* Encode macro */ -#define H5R_ENCODE(func, val, buf, buf_size, actual, m) do {\ - size_t __nalloc = buf_size; \ - if(func(val, buf, &__nalloc) < 0) \ - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, m) \ - if(buf && buf_size >= __nalloc) { \ - buf += __nalloc; \ - buf_size -= __nalloc; \ - } \ - actual += __nalloc; \ -} while(0) - -#define H5R_ENCODE_VAR(func, var, size, buf, buf_size, actual, m) do { \ - size_t __nalloc = buf_size; \ - if(func(var, size, buf, &__nalloc) < 0) \ - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, m) \ - if(buf && buf_size >= __nalloc) { \ - p += __nalloc; \ - buf_size -= __nalloc; \ - } \ - actual += __nalloc; \ -} while(0) +#define H5R_ENCODE(func, val, buf, buf_size, actual, m) \ + do { \ + size_t __nalloc = buf_size; \ + if (func(val, buf, &__nalloc) < 0) \ + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, m) \ + if (buf && buf_size >= __nalloc) { \ + buf += __nalloc; \ + buf_size -= __nalloc; \ + } \ + actual += __nalloc; \ + } while (0) + +#define H5R_ENCODE_VAR(func, var, size, buf, buf_size, actual, m) \ + do { \ + size_t __nalloc = buf_size; \ + if (func(var, size, buf, &__nalloc) < 0) \ + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, m) \ + if (buf && buf_size >= __nalloc) { \ + p += __nalloc; \ + buf_size -= __nalloc; \ + } \ + actual += __nalloc; \ + } while (0) /* Decode macro */ -#define H5R_DECODE(func, val, buf, buf_size, actual, m) do {\ - size_t __nbytes = buf_size; \ - if(func(buf, &__nbytes, val) < 0) \ - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, m) \ - buf += __nbytes; \ - buf_size -= __nbytes; \ - actual += __nbytes; \ -} while(0) - -#define H5R_DECODE_VAR(func, var, size, buf, buf_size, actual, m) do { \ - size_t __nbytes = buf_size; \ - if(func(buf, &__nbytes, var, size) < 0) \ - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, m) \ - p += __nbytes; \ - buf_size -= __nbytes; \ - actual += __nbytes; \ -} while(0) +#define H5R_DECODE(func, val, buf, buf_size, actual, m) \ + do { \ + size_t __nbytes = buf_size; \ + if (func(buf, &__nbytes, val) < 0) \ + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, m) \ + buf += __nbytes; \ + buf_size -= __nbytes; \ + actual += __nbytes; \ + } while (0) + +#define H5R_DECODE_VAR(func, var, size, buf, buf_size, actual, m) \ + do { \ + size_t __nbytes = buf_size; \ + if (func(buf, &__nbytes, var, size) < 0) \ + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, m) \ + p += __nbytes; \ + buf_size -= __nbytes; \ + actual += __nbytes; \ + } while (0) /* Debug */ // #define H5R_DEBUG #ifdef H5R_DEBUG -#define H5R_LOG_DEBUG(...) do { \ - HDfprintf(stdout, " # %s(): ", __func__); \ - HDfprintf(stdout, __VA_ARGS__); \ - HDfprintf(stdout, "\n"); \ - HDfflush(stdout); \ - } while (0) +#define H5R_LOG_DEBUG(...) \ + do { \ + HDfprintf(stdout, " # %s(): ", __func__); \ + HDfprintf(stdout, __VA_ARGS__); \ + HDfprintf(stdout, "\n"); \ + HDfflush(stdout); \ + } while (0) static const char * -H5R__print_token(const H5O_token_t token) { +H5R__print_token(const H5O_token_t token) +{ static char string[64]; /* Print the raw token. */ HDsnprintf(string, 64, "%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X:%02X", - (unsigned char)token.__data[15], - (unsigned char)token.__data[14], - (unsigned char)token.__data[13], - (unsigned char)token.__data[12], - (unsigned char)token.__data[11], - (unsigned char)token.__data[10], - (unsigned char)token.__data[9], - (unsigned char)token.__data[8], - (unsigned char)token.__data[7], - (unsigned char)token.__data[6], - (unsigned char)token.__data[5], - (unsigned char)token.__data[4], - (unsigned char)token.__data[3], - (unsigned char)token.__data[2], - (unsigned char)token.__data[1], - (unsigned char)token.__data[0] - ); + (unsigned char)token.__data[15], (unsigned char)token.__data[14], + (unsigned char)token.__data[13], (unsigned char)token.__data[12], + (unsigned char)token.__data[11], (unsigned char)token.__data[10], + (unsigned char)token.__data[9], (unsigned char)token.__data[8], (unsigned char)token.__data[7], + (unsigned char)token.__data[6], (unsigned char)token.__data[5], (unsigned char)token.__data[4], + (unsigned char)token.__data[3], (unsigned char)token.__data[2], (unsigned char)token.__data[1], + (unsigned char)token.__data[0]); return string; } #else -#define H5R_LOG_DEBUG(...) do { } while (0) +#define H5R_LOG_DEBUG(...) \ + do { \ + } while (0) #endif /******************/ @@ -132,8 +129,10 @@ H5R__print_token(const H5O_token_t token) { /* Local Prototypes */ /********************/ -static herr_t H5R__encode_obj_token(const H5O_token_t *obj_token, size_t token_size, unsigned char *buf, size_t *nalloc); -static herr_t H5R__decode_obj_token(const unsigned char *buf, size_t *nbytes, H5O_token_t *obj_token, uint8_t *token_size); +static herr_t H5R__encode_obj_token(const H5O_token_t *obj_token, size_t token_size, unsigned char *buf, + size_t *nalloc); +static herr_t H5R__decode_obj_token(const unsigned char *buf, size_t *nbytes, H5O_token_t *obj_token, + uint8_t *token_size); static herr_t H5R__encode_region(H5S_t *space, unsigned char *buf, size_t *nalloc); static herr_t H5R__decode_region(const unsigned char *buf, size_t *nbytes, H5S_t **space_ptr); static herr_t H5R__encode_string(const char *string, unsigned char *buf, size_t *nalloc); @@ -157,7 +156,6 @@ hbool_t H5_PKG_INIT_VAR = FALSE; /* Flag indicating "top" of interface has been initialized */ static hbool_t H5R_top_package_initialize_s = FALSE; - /*-------------------------------------------------------------------------- NAME H5R__init_package -- Initialize interface-specific information @@ -184,7 +182,6 @@ H5R__init_package(void) FUNC_LEAVE_NOAPI(SUCCEED) } /* end H5R__init_package() */ - /*-------------------------------------------------------------------------- NAME H5R_top_term_package @@ -206,19 +203,18 @@ H5R__init_package(void) int H5R_top_term_package(void) { - int n = 0; + int n = 0; FUNC_ENTER_NOAPI_NOINIT_NOERR /* Mark closed if initialized */ - if(H5R_top_package_initialize_s) - if(0 == n) + if (H5R_top_package_initialize_s) + if (0 == n) H5R_top_package_initialize_s = FALSE; FUNC_LEAVE_NOAPI(n) } /* end H5R_top_term_package() */ - /*-------------------------------------------------------------------------- NAME H5R_term_package @@ -242,23 +238,22 @@ H5R_top_term_package(void) int H5R_term_package(void) { - int n = 0; + int n = 0; FUNC_ENTER_NOAPI_NOINIT_NOERR - if(H5_PKG_INIT_VAR) { + if (H5_PKG_INIT_VAR) { /* Sanity checks */ HDassert(FALSE == H5R_top_package_initialize_s); /* Mark closed */ - if(0 == n) + if (0 == n) H5_PKG_INIT_VAR = FALSE; } FUNC_LEAVE_NOAPI(n) } /* end H5R_term_package() */ - /*------------------------------------------------------------------------- * Function: H5R__create_object * @@ -269,8 +264,7 @@ H5R_term_package(void) *------------------------------------------------------------------------- */ herr_t -H5R__create_object(const H5O_token_t *obj_token, size_t token_size, - H5R_ref_priv_t *ref) +H5R__create_object(const H5O_token_t *obj_token, size_t token_size, H5R_ref_priv_t *ref) { size_t encode_size; herr_t ret_value = SUCCEED; /* Return value */ @@ -281,25 +275,24 @@ H5R__create_object(const H5O_token_t *obj_token, size_t token_size, /* Create new reference */ ref->info.obj.filename = NULL; - ref->loc_id = H5I_INVALID_HID; - ref->type = (uint8_t)H5R_OBJECT2; - if(H5R__set_obj_token(ref, obj_token, token_size) < 0) + ref->loc_id = H5I_INVALID_HID; + ref->type = (uint8_t)H5R_OBJECT2; + if (H5R__set_obj_token(ref, obj_token, token_size) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, FAIL, "unable to set object token") /* Cache encoding size (assume no external reference) */ - if(H5R__encode(NULL, ref, NULL, &encode_size, 0) < 0) + if (H5R__encode(NULL, ref, NULL, &encode_size, 0) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, "unable to determine encoding size") ref->encode_size = (uint32_t)encode_size; H5R_LOG_DEBUG("Created object reference, %d, filename=%s, obj_addr=%s, encode size=%u", - (int)sizeof(H5R_ref_priv_t), ref->info.obj.filename, H5R__print_token(ref->info.obj.token), - ref->encode_size); + (int)sizeof(H5R_ref_priv_t), ref->info.obj.filename, H5R__print_token(ref->info.obj.token), + ref->encode_size); done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__create_object() */ - /*------------------------------------------------------------------------- * Function: H5R__create_region * @@ -310,8 +303,7 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__create_region(const H5O_token_t *obj_token, size_t token_size, - H5S_t *space, H5R_ref_priv_t *ref) +H5R__create_region(const H5O_token_t *obj_token, size_t token_size, H5S_t *space, H5R_ref_priv_t *ref) { size_t encode_size; herr_t ret_value = SUCCEED; /* Return value */ @@ -323,26 +315,26 @@ H5R__create_region(const H5O_token_t *obj_token, size_t token_size, /* Create new reference */ ref->info.obj.filename = NULL; - if(NULL == (ref->info.reg.space = H5S_copy(space, FALSE, TRUE))) + if (NULL == (ref->info.reg.space = H5S_copy(space, FALSE, TRUE))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOPY, FAIL, "unable to copy dataspace") ref->loc_id = H5I_INVALID_HID; - ref->type = (uint8_t)H5R_DATASET_REGION2; - if(H5R__set_obj_token(ref, obj_token, token_size) < 0) + ref->type = (uint8_t)H5R_DATASET_REGION2; + if (H5R__set_obj_token(ref, obj_token, token_size) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, FAIL, "unable to set object token") /* Cache encoding size (assume no external reference) */ - if(H5R__encode(NULL, ref, NULL, &encode_size, 0) < 0) + if (H5R__encode(NULL, ref, NULL, &encode_size, 0) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, "unable to determine encoding size") ref->encode_size = (uint32_t)encode_size; H5R_LOG_DEBUG("Created region reference, %d, filename=%s, obj_addr=%s, encode size=%u", - (int)sizeof(H5R_ref_priv_t), ref->info.obj.filename, H5R__print_token(ref->info.obj.token), - ref->encode_size); + (int)sizeof(H5R_ref_priv_t), ref->info.obj.filename, H5R__print_token(ref->info.obj.token), + ref->encode_size); done: - if(ret_value < 0) - if(ref->info.reg.space) { + if (ret_value < 0) + if (ref->info.reg.space) { H5S_close(ref->info.reg.space); ref->info.reg.space = NULL; } /* end if */ @@ -350,7 +342,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* H5R__create_region */ - /*------------------------------------------------------------------------- * Function: H5R__create_attr * @@ -361,8 +352,7 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__create_attr(const H5O_token_t *obj_token, size_t token_size, - const char *attr_name, H5R_ref_priv_t *ref) +H5R__create_attr(const H5O_token_t *obj_token, size_t token_size, const char *attr_name, H5R_ref_priv_t *ref) { size_t encode_size; herr_t ret_value = SUCCEED; /* Return value */ @@ -373,30 +363,31 @@ H5R__create_attr(const H5O_token_t *obj_token, size_t token_size, HDassert(ref); /* Make sure that attribute name is not longer than supported encode size */ - if(HDstrlen(attr_name) > H5R_MAX_STRING_LEN) - HGOTO_ERROR(H5E_REFERENCE, H5E_ARGS, FAIL, "attribute name too long (%d > %d)", (int)HDstrlen(attr_name), H5R_MAX_STRING_LEN) + if (HDstrlen(attr_name) > H5R_MAX_STRING_LEN) + HGOTO_ERROR(H5E_REFERENCE, H5E_ARGS, FAIL, "attribute name too long (%d > %d)", + (int)HDstrlen(attr_name), H5R_MAX_STRING_LEN) /* Create new reference */ ref->info.obj.filename = NULL; - if(NULL == (ref->info.attr.name = HDstrdup(attr_name))) + if (NULL == (ref->info.attr.name = HDstrdup(attr_name))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOPY, FAIL, "Cannot copy attribute name") ref->loc_id = H5I_INVALID_HID; - ref->type = (uint8_t)H5R_ATTR; - if(H5R__set_obj_token(ref, obj_token, token_size) < 0) + ref->type = (uint8_t)H5R_ATTR; + if (H5R__set_obj_token(ref, obj_token, token_size) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, FAIL, "unable to set object token") /* Cache encoding size (assume no external reference) */ - if(H5R__encode(NULL, ref, NULL, &encode_size, 0) < 0) + if (H5R__encode(NULL, ref, NULL, &encode_size, 0) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, "unable to determine encoding size") ref->encode_size = (uint32_t)encode_size; H5R_LOG_DEBUG("Created attribute reference, %d, filename=%s, obj_addr=%s, attr name=%s, encode size=%u", - (int)sizeof(H5R_ref_priv_t), ref->info.obj.filename, H5R__print_token(ref->info.obj.token), - ref->info.attr.name, ref->encode_size); + (int)sizeof(H5R_ref_priv_t), ref->info.obj.filename, H5R__print_token(ref->info.obj.token), + ref->info.attr.name, ref->encode_size); done: - if(ret_value < 0) { + if (ret_value < 0) { H5MM_xfree(ref->info.attr.name); ref->info.attr.name = NULL; } /* end if */ @@ -404,7 +395,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* H5R__create_attr */ - /*------------------------------------------------------------------------- * Function: H5R__destroy * @@ -423,19 +413,18 @@ H5R__destroy(H5R_ref_priv_t *ref) HDassert(ref != NULL); - H5R_LOG_DEBUG("Destroying reference, filename=%s, obj_addr=%s, encode size=%u", - ref->info.obj.filename, H5R__print_token(ref->info.obj.token), - ref->encode_size); + H5R_LOG_DEBUG("Destroying reference, filename=%s, obj_addr=%s, encode size=%u", ref->info.obj.filename, + H5R__print_token(ref->info.obj.token), ref->encode_size); H5MM_xfree(ref->info.obj.filename); ref->info.obj.filename = NULL; - switch(ref->type) { + switch (ref->type) { case H5R_OBJECT2: break; case H5R_DATASET_REGION2: - if(H5S_close(ref->info.reg.space) < 0) + if (H5S_close(ref->info.reg.space) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTFREE, FAIL, "Cannot close dataspace") ref->info.reg.space = NULL; break; @@ -459,12 +448,13 @@ H5R__destroy(H5R_ref_priv_t *ref) } /* end switch */ /* Decrement refcount of attached loc_id */ - if(ref->type && (ref->loc_id != H5I_INVALID_HID)) { - if(ref->app_ref) { - if(H5I_dec_app_ref(ref->loc_id) < 0) + if (ref->type && (ref->loc_id != H5I_INVALID_HID)) { + if (ref->app_ref) { + if (H5I_dec_app_ref(ref->loc_id) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDEC, FAIL, "decrementing location ID failed") - } else { - if(H5I_dec_ref(ref->loc_id) < 0) + } + else { + if (H5I_dec_ref(ref->loc_id) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDEC, FAIL, "decrementing location ID failed") } } @@ -473,7 +463,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__destroy() */ - /*------------------------------------------------------------------------- * Function: H5R__set_loc_id * @@ -495,12 +484,13 @@ H5R__set_loc_id(H5R_ref_priv_t *ref, hid_t id, hbool_t inc_ref, hbool_t app_ref) /* If a location ID was previously assigned, decrement refcount and * assign new one */ - if((ref->loc_id != H5I_INVALID_HID)) { - if(ref->app_ref) { - if(H5I_dec_app_ref(ref->loc_id) < 0) + if ((ref->loc_id != H5I_INVALID_HID)) { + if (ref->app_ref) { + if (H5I_dec_app_ref(ref->loc_id) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDEC, FAIL, "decrementing location ID failed") - } else { - if(H5I_dec_ref(ref->loc_id) < 0) + } + else { + if (H5I_dec_ref(ref->loc_id) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDEC, FAIL, "decrementing location ID failed") } } @@ -510,7 +500,7 @@ H5R__set_loc_id(H5R_ref_priv_t *ref, hid_t id, hbool_t inc_ref, hbool_t app_ref) * set app_ref if necessary as references are exposed to users and are * expected to be destroyed, this allows the loc_id to be cleanly released * on shutdown if users fail to call H5Rdestroy(). */ - if(inc_ref && H5I_inc_ref(ref->loc_id, app_ref) < 0) + if (inc_ref && H5I_inc_ref(ref->loc_id, app_ref) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTINC, FAIL, "incrementing location ID failed") ref->app_ref = app_ref; @@ -518,7 +508,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__set_loc_id() */ - /*------------------------------------------------------------------------- * Function: H5R__get_loc_id * @@ -531,7 +520,7 @@ done: hid_t H5R__get_loc_id(const H5R_ref_priv_t *ref) { - hid_t ret_value = H5I_INVALID_HID; /* Return value */ + hid_t ret_value = H5I_INVALID_HID; /* Return value */ FUNC_ENTER_PACKAGE_NOERR @@ -542,7 +531,6 @@ H5R__get_loc_id(const H5R_ref_priv_t *ref) FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__get_loc_id() */ - /*------------------------------------------------------------------------- * Function: H5R__reopen_file * @@ -555,63 +543,66 @@ H5R__get_loc_id(const H5R_ref_priv_t *ref) hid_t H5R__reopen_file(H5R_ref_priv_t *ref, hid_t fapl_id) { - H5P_genplist_t *plist; /* Property list for FAPL */ - void *new_file = NULL; /* File object opened */ + H5P_genplist_t * plist; /* Property list for FAPL */ + void * new_file = NULL; /* File object opened */ H5VL_connector_prop_t connector_prop; /* Property for VOL connector ID & info */ - H5VL_object_t *vol_obj = NULL; /* VOL object for file */ - hbool_t supported; /* Whether 'post open' operation is supported by VOL connector */ - hid_t ret_value = H5I_INVALID_HID; + H5VL_object_t * vol_obj = NULL; /* VOL object for file */ + hbool_t supported; /* Whether 'post open' operation is supported by VOL connector */ + hid_t ret_value = H5I_INVALID_HID; FUNC_ENTER_PACKAGE /* TODO add search path */ /* Verify access property list and set up collective metadata if appropriate */ - if(H5CX_set_apl(&fapl_id, H5P_CLS_FACC, H5I_INVALID_HID, TRUE) < 0) + if (H5CX_set_apl(&fapl_id, H5P_CLS_FACC, H5I_INVALID_HID, TRUE) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, H5I_INVALID_HID, "can't set access property list info") /* Get the VOL info from the fapl */ - if(NULL == (plist = (H5P_genplist_t *)H5I_object(fapl_id))) + if (NULL == (plist = (H5P_genplist_t *)H5I_object(fapl_id))) HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, H5I_INVALID_HID, "not a file access property list") - if(H5P_peek(plist, H5F_ACS_VOL_CONN_NAME, &connector_prop) < 0) + if (H5P_peek(plist, H5F_ACS_VOL_CONN_NAME, &connector_prop) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTGET, H5I_INVALID_HID, "can't get VOL connector info") - /* Stash a copy of the "top-level" connector property, before any pass-through + /* Stash a copy of the "top-level" connector property, before any pass-through * connectors modify or unwrap it. */ - if(H5CX_set_vol_connector_prop(&connector_prop) < 0) - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, H5I_INVALID_HID, "can't set VOL connector info in API context") + if (H5CX_set_vol_connector_prop(&connector_prop) < 0) + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, H5I_INVALID_HID, + "can't set VOL connector info in API context") /* Open the file */ /* (Must open file read-write to allow for object modifications) */ - if(NULL == (new_file = H5VL_file_open(&connector_prop, H5R_REF_FILENAME(ref), H5F_ACC_RDWR, fapl_id, H5P_DATASET_XFER_DEFAULT, H5_REQUEST_NULL))) + if (NULL == (new_file = H5VL_file_open(&connector_prop, H5R_REF_FILENAME(ref), H5F_ACC_RDWR, fapl_id, + H5P_DATASET_XFER_DEFAULT, H5_REQUEST_NULL))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTOPENFILE, H5I_INVALID_HID, "unable to open file") /* Get an ID for the file */ - if((ret_value = H5VL_register_using_vol_id(H5I_FILE, new_file, connector_prop.connector_id, TRUE)) < 0) + if ((ret_value = H5VL_register_using_vol_id(H5I_FILE, new_file, connector_prop.connector_id, TRUE)) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTREGISTER, H5I_INVALID_HID, "unable to atomize file handle") /* Get the file object */ - if(NULL == (vol_obj = H5VL_vol_object(ret_value))) + if (NULL == (vol_obj = H5VL_vol_object(ret_value))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTGET, H5I_INVALID_HID, "invalid object identifier") /* Make the 'post open' callback */ supported = FALSE; - if(H5VL_introspect_opt_query(vol_obj, H5VL_SUBCLS_FILE, H5VL_NATIVE_FILE_POST_OPEN, &supported) < 0) + if (H5VL_introspect_opt_query(vol_obj, H5VL_SUBCLS_FILE, H5VL_NATIVE_FILE_POST_OPEN, &supported) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTGET, H5I_INVALID_HID, "can't check for 'post open' operation") - if(supported) - if(H5VL_file_optional(vol_obj, H5VL_NATIVE_FILE_POST_OPEN, H5P_DATASET_XFER_DEFAULT, H5_REQUEST_NULL) < 0) - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTINIT, H5I_INVALID_HID, "unable to make file 'post open' callback") + if (supported) + if (H5VL_file_optional(vol_obj, H5VL_NATIVE_FILE_POST_OPEN, H5P_DATASET_XFER_DEFAULT, + H5_REQUEST_NULL) < 0) + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTINIT, H5I_INVALID_HID, + "unable to make file 'post open' callback") /* Attach loc_id to reference */ - if(H5R__set_loc_id((H5R_ref_priv_t *)ref, ret_value, FALSE, TRUE) < 0) + if (H5R__set_loc_id((H5R_ref_priv_t *)ref, ret_value, FALSE, TRUE) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, H5I_INVALID_HID, "unable to attach location id to reference") done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__reopen_file() */ - /*------------------------------------------------------------------------- * Function: H5R__get_type * @@ -635,7 +626,6 @@ H5R__get_type(const H5R_ref_priv_t *ref) FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__get_type() */ - /*------------------------------------------------------------------------- * Function: H5R__equal * @@ -656,33 +646,33 @@ H5R__equal(const H5R_ref_priv_t *ref1, const H5R_ref_priv_t *ref2) HDassert(ref2 != NULL); /* Compare reference types */ - if(ref1->type != ref2->type) + if (ref1->type != ref2->type) HGOTO_DONE(FALSE); /* Compare object addresses */ - if(ref1->token_size != ref2->token_size) + if (ref1->token_size != ref2->token_size) HGOTO_DONE(FALSE); - if(0 != HDmemcmp(&ref1->info.obj.token, &ref2->info.obj.token, ref1->token_size)) + if (0 != HDmemcmp(&ref1->info.obj.token, &ref2->info.obj.token, ref1->token_size)) HGOTO_DONE(FALSE); /* Compare filenames */ - if((ref1->info.obj.filename && (NULL == ref2->info.obj.filename)) - || ((NULL == ref1->info.obj.filename) && ref2->info.obj.filename)) + if ((ref1->info.obj.filename && (NULL == ref2->info.obj.filename)) || + ((NULL == ref1->info.obj.filename) && ref2->info.obj.filename)) HGOTO_DONE(FALSE); - if(ref1->info.obj.filename && ref1->info.obj.filename - && (0 != HDstrcmp(ref1->info.obj.filename, ref2->info.obj.filename))) + if (ref1->info.obj.filename && ref1->info.obj.filename && + (0 != HDstrcmp(ref1->info.obj.filename, ref2->info.obj.filename))) HGOTO_DONE(FALSE); - switch(ref1->type) { + switch (ref1->type) { case H5R_OBJECT2: break; case H5R_DATASET_REGION2: - if((ret_value = H5S_extent_equal(ref1->info.reg.space, ref2->info.reg.space)) < 0) + if ((ret_value = H5S_extent_equal(ref1->info.reg.space, ref2->info.reg.space)) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOMPARE, FAIL, "cannot compare dataspace extents") break; case H5R_ATTR: HDassert(ref1->info.attr.name && ref2->info.attr.name); - if(0 != HDstrcmp(ref1->info.attr.name, ref2->info.attr.name)) + if (0 != HDstrcmp(ref1->info.attr.name, ref2->info.attr.name)) HGOTO_DONE(FALSE); break; case H5R_OBJECT1: @@ -700,7 +690,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__equal() */ - /*------------------------------------------------------------------------- * Function: H5R__copy * @@ -721,18 +710,18 @@ H5R__copy(const H5R_ref_priv_t *src_ref, H5R_ref_priv_t *dst_ref) H5MM_memcpy(&dst_ref->info.obj.token, &src_ref->info.obj.token, sizeof(H5O_token_t)); dst_ref->encode_size = src_ref->encode_size; - dst_ref->type = src_ref->type; - dst_ref->token_size = src_ref->token_size; + dst_ref->type = src_ref->type; + dst_ref->token_size = src_ref->token_size; - switch(src_ref->type) { + switch (src_ref->type) { case H5R_OBJECT2: break; case H5R_DATASET_REGION2: - if(NULL == (dst_ref->info.reg.space = H5S_copy(src_ref->info.reg.space, FALSE, TRUE))) + if (NULL == (dst_ref->info.reg.space = H5S_copy(src_ref->info.reg.space, FALSE, TRUE))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOPY, FAIL, "unable to copy dataspace") break; case H5R_ATTR: - if(NULL == (dst_ref->info.attr.name = HDstrdup(src_ref->info.attr.name))) + if (NULL == (dst_ref->info.attr.name = HDstrdup(src_ref->info.attr.name))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOPY, FAIL, "Cannot copy attribute name") break; case H5R_OBJECT1: @@ -747,10 +736,10 @@ H5R__copy(const H5R_ref_priv_t *src_ref, H5R_ref_priv_t *dst_ref) } /* end switch */ /* We only need to keep a copy of the filename if we don't have the loc_id */ - if(src_ref->loc_id == H5I_INVALID_HID) { + if (src_ref->loc_id == H5I_INVALID_HID) { HDassert(src_ref->info.obj.filename); - if(NULL == (dst_ref->info.obj.filename = HDstrdup(src_ref->info.obj.filename))) + if (NULL == (dst_ref->info.obj.filename = HDstrdup(src_ref->info.obj.filename))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOPY, FAIL, "Cannot copy filename") dst_ref->loc_id = H5I_INVALID_HID; } @@ -758,7 +747,7 @@ H5R__copy(const H5R_ref_priv_t *src_ref, H5R_ref_priv_t *dst_ref) dst_ref->info.obj.filename = NULL; /* Set location ID and hold reference to it */ - if(H5R__set_loc_id(dst_ref, src_ref->loc_id, TRUE, TRUE) < 0) + if (H5R__set_loc_id(dst_ref, src_ref->loc_id, TRUE, TRUE) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, FAIL, "cannot set reference location ID") } @@ -766,7 +755,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__copy() */ - /*------------------------------------------------------------------------- * Function: H5R__get_obj_token * @@ -777,8 +765,7 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__get_obj_token(const H5R_ref_priv_t *ref, H5O_token_t *obj_token, - size_t *token_size) +H5R__get_obj_token(const H5R_ref_priv_t *ref, H5O_token_t *obj_token, size_t *token_size) { herr_t ret_value = SUCCEED; /* Return value */ @@ -787,19 +774,18 @@ H5R__get_obj_token(const H5R_ref_priv_t *ref, H5O_token_t *obj_token, HDassert(ref != NULL); HDassert(ref->token_size <= H5O_MAX_TOKEN_SIZE); - if(obj_token) { - if(0 == ref->token_size) + if (obj_token) { + if (0 == ref->token_size) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOPY, FAIL, "NULL token size") H5MM_memcpy(obj_token, &ref->info.obj.token, sizeof(H5O_token_t)); } - if(token_size) + if (token_size) *token_size = ref->token_size; done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__get_obj_token() */ - /*------------------------------------------------------------------------- * Function: H5R__set_obj_token * @@ -810,8 +796,7 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__set_obj_token(H5R_ref_priv_t *ref, const H5O_token_t *obj_token, - size_t token_size) +H5R__set_obj_token(H5R_ref_priv_t *ref, const H5O_token_t *obj_token, size_t token_size) { herr_t ret_value = SUCCEED; /* Return value */ @@ -829,7 +814,6 @@ H5R__set_obj_token(H5R_ref_priv_t *ref, const H5O_token_t *obj_token, FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__set_obj_token() */ - /*------------------------------------------------------------------------- * Function: H5R__get_region * @@ -853,14 +837,13 @@ H5R__get_region(const H5R_ref_priv_t *ref, H5S_t *space) HDassert(space); /* Copy reference selection to destination */ - if(H5S_select_copy(space, ref->info.reg.space, FALSE) < 0) + if (H5S_select_copy(space, ref->info.reg.space, FALSE) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTCOPY, FAIL, "unable to copy selection") done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__get_region() */ - /*------------------------------------------------------------------------- * Function: H5R__get_file_name * @@ -874,8 +857,8 @@ done: ssize_t H5R__get_file_name(const H5R_ref_priv_t *ref, char *buf, size_t size) { - size_t copy_len; - ssize_t ret_value = -1; /* Return value */ + size_t copy_len; + ssize_t ret_value = -1; /* Return value */ FUNC_ENTER_PACKAGE @@ -883,7 +866,7 @@ H5R__get_file_name(const H5R_ref_priv_t *ref, char *buf, size_t size) HDassert(ref != NULL); /* Return if that reference has no filename set */ - if(!ref->info.obj.filename) + if (!ref->info.obj.filename) HGOTO_ERROR(H5E_REFERENCE, H5E_ARGS, (-1), "no filename available for that reference") /* Get the file name length */ @@ -891,7 +874,7 @@ H5R__get_file_name(const H5R_ref_priv_t *ref, char *buf, size_t size) HDassert(copy_len <= H5R_MAX_STRING_LEN); /* Copy the file name */ - if(buf) { + if (buf) { copy_len = MIN(copy_len, size - 1); H5MM_memcpy(buf, ref->info.obj.filename, copy_len); buf[copy_len] = '\0'; @@ -902,7 +885,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__get_file_name() */ - /*------------------------------------------------------------------------- * Function: H5R__get_attr_name * @@ -915,8 +897,8 @@ done: ssize_t H5R__get_attr_name(const H5R_ref_priv_t *ref, char *buf, size_t size) { - ssize_t ret_value = -1; /* Return value */ - size_t attr_name_len; /* Length of the attribute name */ + ssize_t ret_value = -1; /* Return value */ + size_t attr_name_len; /* Length of the attribute name */ FUNC_ENTER_PACKAGE_NOERR @@ -929,7 +911,7 @@ H5R__get_attr_name(const H5R_ref_priv_t *ref, char *buf, size_t size) HDassert(attr_name_len <= H5R_MAX_STRING_LEN); /* Get the attribute name */ - if(buf) { + if (buf) { size_t copy_len = MIN(attr_name_len, size - 1); H5MM_memcpy(buf, ref->info.attr.name, copy_len); buf[copy_len] = '\0'; @@ -940,7 +922,6 @@ H5R__get_attr_name(const H5R_ref_priv_t *ref, char *buf, size_t size) FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__get_attr_name() */ - /*------------------------------------------------------------------------- * Function: H5R__encode * @@ -951,12 +932,12 @@ H5R__get_attr_name(const H5R_ref_priv_t *ref, char *buf, size_t size) *------------------------------------------------------------------------- */ herr_t -H5R__encode(const char *filename, const H5R_ref_priv_t *ref, unsigned char *buf, - size_t *nalloc, unsigned flags) +H5R__encode(const char *filename, const H5R_ref_priv_t *ref, unsigned char *buf, size_t *nalloc, + unsigned flags) { - uint8_t *p = (uint8_t *)buf; - size_t buf_size = 0, encode_size = 0; - herr_t ret_value = SUCCEED; + uint8_t *p = (uint8_t *)buf; + size_t buf_size = 0, encode_size = 0; + herr_t ret_value = SUCCEED; FUNC_ENTER_PACKAGE @@ -976,7 +957,7 @@ H5R__encode(const char *filename, const H5R_ref_priv_t *ref, unsigned char *buf, */ /* Don't encode if buffer size isn't big enough or buffer is empty */ - if(buf && *nalloc >= H5R_ENCODE_HEADER_SIZE) { + if (buf && *nalloc >= H5R_ENCODE_HEADER_SIZE) { /* Encode the type of the reference */ *p++ = (uint8_t)ref->type; @@ -988,8 +969,8 @@ H5R__encode(const char *filename, const H5R_ref_priv_t *ref, unsigned char *buf, encode_size += H5R_ENCODE_HEADER_SIZE; /* Encode object token */ - H5R_ENCODE_VAR(H5R__encode_obj_token, &ref->info.obj.token, ref->token_size, - p, buf_size, encode_size, "Cannot encode object address"); + H5R_ENCODE_VAR(H5R__encode_obj_token, &ref->info.obj.token, ref->token_size, p, buf_size, encode_size, + "Cannot encode object address"); /** * TODO Encode VOL info @@ -999,25 +980,24 @@ H5R__encode(const char *filename, const H5R_ref_priv_t *ref, unsigned char *buf, * - avoid duplicating VOL info on each reference * - must query terminal VOL connector to avoid passthrough confusion */ - if(flags & H5R_IS_EXTERNAL) + if (flags & H5R_IS_EXTERNAL) /* Encode file name */ - H5R_ENCODE(H5R__encode_string, filename, p, buf_size, encode_size, - "Cannot encode filename"); + H5R_ENCODE(H5R__encode_string, filename, p, buf_size, encode_size, "Cannot encode filename"); - switch(ref->type) { + switch (ref->type) { case H5R_OBJECT2: break; case H5R_DATASET_REGION2: /* Encode dataspace */ - H5R_ENCODE(H5R__encode_region, ref->info.reg.space, p, buf_size, - encode_size, "Cannot encode region"); + H5R_ENCODE(H5R__encode_region, ref->info.reg.space, p, buf_size, encode_size, + "Cannot encode region"); break; case H5R_ATTR: /* Encode attribute name */ - H5R_ENCODE(H5R__encode_string, ref->info.attr.name, p, buf_size, - encode_size, "Cannot encode attribute name"); + H5R_ENCODE(H5R__encode_string, ref->info.attr.name, p, buf_size, encode_size, + "Cannot encode attribute name"); break; case H5R_OBJECT1: @@ -1032,9 +1012,8 @@ H5R__encode(const char *filename, const H5R_ref_priv_t *ref, unsigned char *buf, HGOTO_ERROR(H5E_REFERENCE, H5E_UNSUPPORTED, FAIL, "internal error (unknown reference type)") } /* end switch */ - H5R_LOG_DEBUG("Encoded reference, filename=%s, obj_addr=%s, encode size=%u", - ref->info.obj.filename, H5R__print_token(ref->info.obj.token), - encode_size); + H5R_LOG_DEBUG("Encoded reference, filename=%s, obj_addr=%s, encode size=%u", ref->info.obj.filename, + H5R__print_token(ref->info.obj.token), encode_size); *nalloc = encode_size; @@ -1042,7 +1021,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__encode() */ - /*------------------------------------------------------------------------- * Function: H5R__decode * @@ -1055,10 +1033,10 @@ done: herr_t H5R__decode(const unsigned char *buf, size_t *nbytes, H5R_ref_priv_t *ref) { - const uint8_t *p = (const uint8_t *)buf; - size_t buf_size = 0, decode_size = 0; - uint8_t flags; - herr_t ret_value = SUCCEED; + const uint8_t *p = (const uint8_t *)buf; + size_t buf_size = 0, decode_size = 0; + uint8_t flags; + herr_t ret_value = SUCCEED; FUNC_ENTER_PACKAGE @@ -1068,12 +1046,12 @@ H5R__decode(const unsigned char *buf, size_t *nbytes, H5R_ref_priv_t *ref) buf_size = *nbytes; /* Don't decode if buffer size isn't big enough */ - if(buf_size < H5R_ENCODE_HEADER_SIZE) + if (buf_size < H5R_ENCODE_HEADER_SIZE) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") /* Set new reference */ ref->type = (int8_t)*p++; - if(ref->type <= H5R_BADTYPE || ref->type >= H5R_MAXTYPE) + if (ref->type <= H5R_BADTYPE || ref->type >= H5R_MAXTYPE) HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "invalid reference type") /* Set flags */ @@ -1083,30 +1061,30 @@ H5R__decode(const unsigned char *buf, size_t *nbytes, H5R_ref_priv_t *ref) decode_size += H5R_ENCODE_HEADER_SIZE; /* Decode object token */ - H5R_DECODE_VAR(H5R__decode_obj_token, &ref->info.obj.token, &ref->token_size, - p, buf_size, decode_size, "Cannot decode object address"); + H5R_DECODE_VAR(H5R__decode_obj_token, &ref->info.obj.token, &ref->token_size, p, buf_size, decode_size, + "Cannot decode object address"); /* We do not need to store the filename if the reference is internal */ - if(flags & H5R_IS_EXTERNAL) { + if (flags & H5R_IS_EXTERNAL) { /* Decode file name */ - H5R_DECODE(H5R__decode_string, &ref->info.obj.filename, p, buf_size, - decode_size, "Cannot decode filename"); + H5R_DECODE(H5R__decode_string, &ref->info.obj.filename, p, buf_size, decode_size, + "Cannot decode filename"); } else ref->info.obj.filename = NULL; - switch(ref->type) { + switch (ref->type) { case H5R_OBJECT2: break; case H5R_DATASET_REGION2: /* Decode dataspace */ - H5R_DECODE(H5R__decode_region, &ref->info.reg.space, p, buf_size, - decode_size, "Cannot decode region"); + H5R_DECODE(H5R__decode_region, &ref->info.reg.space, p, buf_size, decode_size, + "Cannot decode region"); break; case H5R_ATTR: /* Decode attribute name */ - H5R_DECODE(H5R__decode_string, &ref->info.attr.name, p, buf_size, - decode_size, "Cannot decode attribute name"); + H5R_DECODE(H5R__decode_string, &ref->info.attr.name, p, buf_size, decode_size, + "Cannot decode attribute name"); break; case H5R_OBJECT1: case H5R_DATASET_REGION1: @@ -1125,9 +1103,8 @@ H5R__decode(const unsigned char *buf, size_t *nbytes, H5R_ref_priv_t *ref) /* Set encoding size */ ref->encode_size = (uint32_t)decode_size; - H5R_LOG_DEBUG("Decoded reference, filename=%s, obj_addr=%s, encode size=%u", - ref->info.obj.filename, H5R__print_token(ref->info.obj.token), - ref->encode_size); + H5R_LOG_DEBUG("Decoded reference, filename=%s, obj_addr=%s, encode size=%u", ref->info.obj.filename, + H5R__print_token(ref->info.obj.token), ref->encode_size); *nbytes = decode_size; @@ -1135,7 +1112,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__decode() */ - /*------------------------------------------------------------------------- * Function: H5R__encode_obj_token * @@ -1146,8 +1122,7 @@ done: *------------------------------------------------------------------------- */ static herr_t -H5R__encode_obj_token(const H5O_token_t *obj_token, size_t token_size, - unsigned char *buf, size_t *nalloc) +H5R__encode_obj_token(const H5O_token_t *obj_token, size_t token_size, unsigned char *buf, size_t *nalloc) { herr_t ret_value = SUCCEED; @@ -1156,7 +1131,7 @@ H5R__encode_obj_token(const H5O_token_t *obj_token, size_t token_size, HDassert(nalloc); /* Don't encode if buffer size isn't big enough or buffer is empty */ - if(buf && *nalloc >= token_size) { + if (buf && *nalloc >= token_size) { uint8_t *p = (uint8_t *)buf; /* Encode token size */ @@ -1170,7 +1145,6 @@ H5R__encode_obj_token(const H5O_token_t *obj_token, size_t token_size, FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__encode_obj_token() */ - /*------------------------------------------------------------------------- * Function: H5R__decode_obj_token * @@ -1181,11 +1155,10 @@ H5R__encode_obj_token(const H5O_token_t *obj_token, size_t token_size, *------------------------------------------------------------------------- */ static herr_t -H5R__decode_obj_token(const unsigned char *buf, size_t *nbytes, - H5O_token_t *obj_token, uint8_t *token_size) +H5R__decode_obj_token(const unsigned char *buf, size_t *nbytes, H5O_token_t *obj_token, uint8_t *token_size) { - const uint8_t *p = (const uint8_t *)buf; - herr_t ret_value = SUCCEED; + const uint8_t *p = (const uint8_t *)buf; + herr_t ret_value = SUCCEED; FUNC_ENTER_STATIC @@ -1195,12 +1168,12 @@ H5R__decode_obj_token(const unsigned char *buf, size_t *nbytes, HDassert(token_size); /* Don't decode if buffer size isn't big enough */ - if(*nbytes < H5_SIZEOF_UINT8_T) + if (*nbytes < H5_SIZEOF_UINT8_T) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") /* Get token size */ *token_size = *p++; - if(*token_size > sizeof(H5O_token_t)) + if (*token_size > sizeof(H5O_token_t)) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Invalid token size (%u)", *token_size) /* Make sure that token is initialized */ @@ -1215,7 +1188,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__decode_obj_token() */ - /*------------------------------------------------------------------------- * Function: H5R__encode_region * @@ -1228,9 +1200,9 @@ done: static herr_t H5R__encode_region(H5S_t *space, unsigned char *buf, size_t *nalloc) { - uint8_t *p = NULL; /* Pointer to data to store */ - hssize_t buf_size = 0; - herr_t ret_value = SUCCEED; + uint8_t *p = NULL; /* Pointer to data to store */ + hssize_t buf_size = 0; + herr_t ret_value = SUCCEED; FUNC_ENTER_STATIC @@ -1238,11 +1210,12 @@ H5R__encode_region(H5S_t *space, unsigned char *buf, size_t *nalloc) HDassert(nalloc); /* Get the amount of space required to serialize the selection */ - if((buf_size = H5S_SELECT_SERIAL_SIZE(space)) < 0) - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, "Cannot determine amount of space needed for serializing selection") + if ((buf_size = H5S_SELECT_SERIAL_SIZE(space)) < 0) + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, + "Cannot determine amount of space needed for serializing selection") /* Don't encode if buffer size isn't big enough or buffer is empty */ - if(buf && *nalloc >= ((size_t)buf_size + 2 * H5_SIZEOF_UINT32_T)) { + if (buf && *nalloc >= ((size_t)buf_size + 2 * H5_SIZEOF_UINT32_T)) { int rank; p = (uint8_t *)buf; @@ -1250,12 +1223,12 @@ H5R__encode_region(H5S_t *space, unsigned char *buf, size_t *nalloc) UINT32ENCODE(p, (uint32_t)buf_size); /* Encode the extent rank */ - if((rank = H5S_get_simple_extent_ndims(space)) < 0) + if ((rank = H5S_get_simple_extent_ndims(space)) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTGET, FAIL, "can't get extent rank for selection") UINT32ENCODE(p, (uint32_t)rank); /* Serialize the selection */ - if(H5S_SELECT_SERIALIZE(space, (unsigned char **)&p) < 0) + if (H5S_SELECT_SERIALIZE(space, (unsigned char **)&p) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTENCODE, FAIL, "can't serialize selection") } /* end if */ *nalloc = (size_t)buf_size + 2 * H5_SIZEOF_UINT32_T; @@ -1264,7 +1237,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__encode_region() */ - /*------------------------------------------------------------------------- * Function: H5R__decode_region * @@ -1277,11 +1249,11 @@ done: static herr_t H5R__decode_region(const unsigned char *buf, size_t *nbytes, H5S_t **space_ptr) { - const uint8_t *p = (const uint8_t *)buf; - size_t buf_size = 0; - unsigned rank; - H5S_t *space; - herr_t ret_value = SUCCEED; + const uint8_t *p = (const uint8_t *)buf; + size_t buf_size = 0; + unsigned rank; + H5S_t * space; + herr_t ret_value = SUCCEED; FUNC_ENTER_STATIC @@ -1290,7 +1262,7 @@ H5R__decode_region(const unsigned char *buf, size_t *nbytes, H5S_t **space_ptr) HDassert(space_ptr); /* Don't decode if buffer size isn't big enough */ - if(*nbytes < (2 * H5_SIZEOF_UINT32_T)) + if (*nbytes < (2 * H5_SIZEOF_UINT32_T)) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") /* Decode the selection size */ @@ -1302,25 +1274,24 @@ H5R__decode_region(const unsigned char *buf, size_t *nbytes, H5S_t **space_ptr) buf_size += H5_SIZEOF_UINT32_T; /* Don't decode if buffer size isn't big enough */ - if(*nbytes < buf_size) + if (*nbytes < buf_size) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") /* Deserialize the selection (dataspaces need the extent rank information) */ - if(NULL == (space = H5S_create(H5S_SIMPLE))) + if (NULL == (space = H5S_create(H5S_SIMPLE))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") - if(H5S_set_extent_simple(space, rank, NULL, NULL) < 0) + if (H5S_set_extent_simple(space, rank, NULL, NULL) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTSET, FAIL, "can't set extent rank for selection") - if(H5S_SELECT_DESERIALIZE(&space, &p) < 0) + if (H5S_SELECT_DESERIALIZE(&space, &p) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "can't deserialize selection") - *nbytes = buf_size; + *nbytes = buf_size; *space_ptr = space; done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__decode_region() */ - /*------------------------------------------------------------------------- * Function: H5R__encode_string * @@ -1343,13 +1314,13 @@ H5R__encode_string(const char *string, unsigned char *buf, size_t *nalloc) /* Get the amount of space required to serialize the string */ string_len = HDstrlen(string); - if(string_len > H5R_MAX_STRING_LEN) + if (string_len > H5R_MAX_STRING_LEN) HGOTO_ERROR(H5E_REFERENCE, H5E_ARGS, FAIL, "string too long") /* Compute buffer size, allow for the attribute name length and object address */ buf_size = string_len + sizeof(uint16_t); - if(buf && *nalloc >= buf_size) { + if (buf && *nalloc >= buf_size) { uint8_t *p = (uint8_t *)buf; /* Serialize information for string length into the buffer */ UINT16ENCODE(p, string_len); @@ -1362,7 +1333,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__encode_string() */ - /*------------------------------------------------------------------------- * Function: H5R__decode_string * @@ -1376,9 +1346,9 @@ static herr_t H5R__decode_string(const unsigned char *buf, size_t *nbytes, char **string_ptr) { const uint8_t *p = (const uint8_t *)buf; - size_t string_len; - char *string = NULL; - herr_t ret_value = SUCCEED; + size_t string_len; + char * string = NULL; + herr_t ret_value = SUCCEED; FUNC_ENTER_STATIC @@ -1387,7 +1357,7 @@ H5R__decode_string(const unsigned char *buf, size_t *nbytes, char **string_ptr) HDassert(string_ptr); /* Don't decode if buffer size isn't big enough */ - if(*nbytes < sizeof(uint16_t)) + if (*nbytes < sizeof(uint16_t)) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") /* Get the string length */ @@ -1395,21 +1365,20 @@ H5R__decode_string(const unsigned char *buf, size_t *nbytes, char **string_ptr) HDassert(string_len <= H5R_MAX_STRING_LEN); /* Allocate the string */ - if(NULL == (string = (char *)H5MM_malloc(string_len + 1))) + if (NULL == (string = (char *)H5MM_malloc(string_len + 1))) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTALLOC, FAIL, "Cannot allocate string") - /* Copy the string */ - H5MM_memcpy(string, p, string_len); - string[string_len] = '\0'; + /* Copy the string */ + H5MM_memcpy(string, p, string_len); + string[string_len] = '\0'; - *string_ptr = string; - *nbytes = sizeof(uint16_t) + string_len; + *string_ptr = string; + *nbytes = sizeof(uint16_t) + string_len; done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__decode_string() */ - /*------------------------------------------------------------------------- * Function: H5R__encode_heap * @@ -1420,8 +1389,7 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__encode_heap(H5F_t *f, unsigned char *buf, size_t *nalloc, - const unsigned char *data, size_t data_size) +H5R__encode_heap(H5F_t *f, unsigned char *buf, size_t *nalloc, const unsigned char *data, size_t data_size) { size_t buf_size; herr_t ret_value = SUCCEED; @@ -1432,12 +1400,12 @@ H5R__encode_heap(H5F_t *f, unsigned char *buf, size_t *nalloc, HDassert(nalloc); buf_size = H5HG_HEAP_ID_SIZE(f); - if(buf && *nalloc >= buf_size) { - H5HG_t hobjid; + if (buf && *nalloc >= buf_size) { + H5HG_t hobjid; uint8_t *p = (uint8_t *)buf; /* Write the reference information to disk (allocates space also) */ - if(H5HG_insert(f, data_size, data, &hobjid) < 0) + if (H5HG_insert(f, data_size, data, &hobjid) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_WRITEERROR, FAIL, "Unable to write reference information") /* Encode the heap information */ @@ -1450,7 +1418,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__encode_heap() */ - /*------------------------------------------------------------------------- * Function: H5R__decode_heap * @@ -1461,13 +1428,13 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__decode_heap(H5F_t *f, const unsigned char *buf, size_t *nbytes, - unsigned char **data_ptr, size_t *data_size) +H5R__decode_heap(H5F_t *f, const unsigned char *buf, size_t *nbytes, unsigned char **data_ptr, + size_t *data_size) { const uint8_t *p = (const uint8_t *)buf; - H5HG_t hobjid; - size_t buf_size; - herr_t ret_value = SUCCEED; + H5HG_t hobjid; + size_t buf_size; + herr_t ret_value = SUCCEED; FUNC_ENTER_PACKAGE @@ -1478,17 +1445,17 @@ H5R__decode_heap(H5F_t *f, const unsigned char *buf, size_t *nbytes, buf_size = H5HG_HEAP_ID_SIZE(f); /* Don't decode if buffer size isn't big enough */ - if(*nbytes < buf_size) + if (*nbytes < buf_size) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") /* Get the heap information */ H5F_addr_decode(f, &p, &(hobjid.addr)); - if(!H5F_addr_defined(hobjid.addr) || hobjid.addr == 0) + if (!H5F_addr_defined(hobjid.addr) || hobjid.addr == 0) HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "Undefined reference pointer") UINT32DECODE(p, hobjid.idx); /* Read the information from disk */ - if(NULL == (*data_ptr = (unsigned char *)H5HG_read(f, &hobjid, (void *)*data_ptr, data_size))) + if (NULL == (*data_ptr = (unsigned char *)H5HG_read(f, &hobjid, (void *)*data_ptr, data_size))) HGOTO_ERROR(H5E_REFERENCE, H5E_READERROR, FAIL, "Unable to read reference data") *nbytes = buf_size; @@ -1497,7 +1464,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__decode_heap() */ - /*------------------------------------------------------------------------- * Function: H5R__encode_token_obj_compat * @@ -1508,8 +1474,8 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__encode_token_obj_compat(const H5O_token_t *obj_token, size_t token_size, - unsigned char *buf, size_t *nalloc) +H5R__encode_token_obj_compat(const H5O_token_t *obj_token, size_t token_size, unsigned char *buf, + size_t *nalloc) { herr_t ret_value = SUCCEED; @@ -1520,7 +1486,7 @@ H5R__encode_token_obj_compat(const H5O_token_t *obj_token, size_t token_size, HDassert(nalloc); /* Don't encode if buffer size isn't big enough or buffer is empty */ - if(buf && *nalloc >= token_size) + if (buf && *nalloc >= token_size) H5MM_memcpy(buf, obj_token, token_size); *nalloc = token_size; @@ -1528,7 +1494,6 @@ H5R__encode_token_obj_compat(const H5O_token_t *obj_token, size_t token_size, FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__encode_token_obj_compat() */ - /*------------------------------------------------------------------------- * Function: H5R__decode_token_obj_compat * @@ -1539,8 +1504,8 @@ H5R__encode_token_obj_compat(const H5O_token_t *obj_token, size_t token_size, *------------------------------------------------------------------------- */ herr_t -H5R__decode_token_obj_compat(const unsigned char *buf, size_t *nbytes, - H5O_token_t *obj_token, size_t token_size) +H5R__decode_token_obj_compat(const unsigned char *buf, size_t *nbytes, H5O_token_t *obj_token, + size_t token_size) { herr_t ret_value = SUCCEED; @@ -1552,7 +1517,7 @@ H5R__decode_token_obj_compat(const unsigned char *buf, size_t *nbytes, HDassert(token_size); /* Don't decode if buffer size isn't big enough */ - if(*nbytes < token_size) + if (*nbytes < token_size) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "Buffer size is too small") H5MM_memcpy(obj_token, buf, token_size); @@ -1563,7 +1528,6 @@ done: FUNC_LEAVE_NOAPI(ret_value) } /* H5R__decode_token_obj_compat() */ - /*------------------------------------------------------------------------- * Function: H5R__decode_token_region_compat * @@ -1575,16 +1539,15 @@ done: *------------------------------------------------------------------------- */ herr_t -H5R__decode_token_region_compat(H5F_t *f, const unsigned char *buf, - size_t *nbytes, H5O_token_t *obj_token, size_t token_size, - H5S_t **space_ptr) +H5R__decode_token_region_compat(H5F_t *f, const unsigned char *buf, size_t *nbytes, H5O_token_t *obj_token, + size_t token_size, H5S_t **space_ptr) { - unsigned char *data = NULL; - H5O_token_t token = { 0 }; - size_t data_size; + unsigned char *data = NULL; + H5O_token_t token = {0}; + size_t data_size; const uint8_t *p; - H5S_t *space = NULL; - herr_t ret_value = SUCCEED; + H5S_t * space = NULL; + herr_t ret_value = SUCCEED; FUNC_ENTER_PACKAGE @@ -1594,7 +1557,7 @@ H5R__decode_token_region_compat(H5F_t *f, const unsigned char *buf, HDassert(token_size); /* Read from heap */ - if(H5R__decode_heap(f, buf, nbytes, &data, &data_size) < 0) + if (H5R__decode_heap(f, buf, nbytes, &data, &data_size) < 0) HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "invalid location identifier") /* Get object address */ @@ -1602,37 +1565,37 @@ H5R__decode_token_region_compat(H5F_t *f, const unsigned char *buf, H5MM_memcpy(&token, p, token_size); p += token_size; - if(space_ptr) { + if (space_ptr) { H5O_loc_t oloc; /* Object location */ /* Initialize the object location */ H5O_loc_reset(&oloc); oloc.file = f; - if(H5VL_native_token_to_addr(f, H5I_FILE, token, &oloc.addr) < 0) - HGOTO_ERROR(H5E_REFERENCE, H5E_CANTUNSERIALIZE, FAIL, "can't deserialize object token into address") + if (H5VL_native_token_to_addr(f, H5I_FILE, token, &oloc.addr) < 0) + HGOTO_ERROR(H5E_REFERENCE, H5E_CANTUNSERIALIZE, FAIL, + "can't deserialize object token into address") /* Open and copy the dataset's dataspace */ - if(NULL == (space = H5S_read(&oloc))) + if (NULL == (space = H5S_read(&oloc))) HGOTO_ERROR(H5E_REFERENCE, H5E_NOTFOUND, FAIL, "not found") /* Unserialize the selection */ - if(H5S_SELECT_DESERIALIZE(&space, &p) < 0) + if (H5S_SELECT_DESERIALIZE(&space, &p) < 0) HGOTO_ERROR(H5E_REFERENCE, H5E_CANTDECODE, FAIL, "can't deserialize selection") *space_ptr = space; } - if(obj_token) + if (obj_token) H5MM_memcpy(obj_token, &token, sizeof(H5O_token_t)); done: H5MM_free(data); - if(ret_value < 0) { - if(space && H5S_close(space) < 0) + if (ret_value < 0) { + if (space && H5S_close(space) < 0) HDONE_ERROR(H5E_DATASET, H5E_CLOSEERROR, FAIL, "unable to release dataspace") } FUNC_LEAVE_NOAPI(ret_value) } /* end H5R__decode_token_region_compat() */ - |