summaryrefslogtreecommitdiffstats
path: root/src/H5S.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/H5S.c')
-rw-r--r--src/H5S.c647
1 files changed, 297 insertions, 350 deletions
diff --git a/src/H5S.c b/src/H5S.c
index 7e48076..1e5f8b4 100644
--- a/src/H5S.c
+++ b/src/H5S.c
@@ -15,47 +15,41 @@
/* Module Setup */
/****************/
-#include "H5Smodule.h" /* This source code file is part of the H5S module */
-
+#include "H5Smodule.h" /* This source code file is part of the H5S module */
/***********/
/* Headers */
/***********/
-#include "H5private.h" /* Generic Functions */
-#include "H5Eprivate.h" /* Error handling */
-#include "H5CXprivate.h" /* API Contexts */
-#include "H5Fprivate.h" /* Files */
-#include "H5FLprivate.h" /* Free lists */
-#include "H5Iprivate.h" /* IDs */
-#include "H5MMprivate.h" /* Memory management */
-#include "H5Oprivate.h" /* Object headers */
-#include "H5Spkg.h" /* Dataspaces */
-
+#include "H5private.h" /* Generic Functions */
+#include "H5Eprivate.h" /* Error handling */
+#include "H5CXprivate.h" /* API Contexts */
+#include "H5Fprivate.h" /* Files */
+#include "H5FLprivate.h" /* Free lists */
+#include "H5Iprivate.h" /* IDs */
+#include "H5MMprivate.h" /* Memory management */
+#include "H5Oprivate.h" /* Object headers */
+#include "H5Spkg.h" /* Dataspaces */
/****************/
/* Local Macros */
/****************/
/* Version of dataspace encoding */
-#define H5S_ENCODE_VERSION 0
-
+#define H5S_ENCODE_VERSION 0
/******************/
/* Local Typedefs */
/******************/
-
/********************/
/* Local Prototypes */
/********************/
static htri_t H5S__is_simple(const H5S_t *sdim);
-
/*****************************/
/* Library Private Variables */
/*****************************/
-
/*********************/
/* Package Variables */
/*********************/
@@ -65,11 +59,11 @@ hbool_t H5_PKG_INIT_VAR = FALSE;
/* Format version bounds for dataspace */
const unsigned H5O_sdspace_ver_bounds[] = {
- H5O_SDSPACE_VERSION_1, /* H5F_LIBVER_EARLIEST */
- H5O_SDSPACE_VERSION_2, /* H5F_LIBVER_V18 */
- H5O_SDSPACE_VERSION_2, /* H5F_LIBVER_V110 */
- H5O_SDSPACE_VERSION_2, /* H5F_LIBVER_V112 */
- H5O_SDSPACE_VERSION_LATEST /* H5F_LIBVER_LATEST */
+ H5O_SDSPACE_VERSION_1, /* H5F_LIBVER_EARLIEST */
+ H5O_SDSPACE_VERSION_2, /* H5F_LIBVER_V18 */
+ H5O_SDSPACE_VERSION_2, /* H5F_LIBVER_V110 */
+ H5O_SDSPACE_VERSION_2, /* H5F_LIBVER_V112 */
+ H5O_SDSPACE_VERSION_LATEST /* H5F_LIBVER_LATEST */
};
/* Declare a free list to manage the H5S_extent_t struct */
@@ -81,33 +75,29 @@ H5FL_DEFINE(H5S_t);
/* Declare a free list to manage the array's of hsize_t's */
H5FL_ARR_DEFINE(hsize_t, H5S_MAX_RANK);
-
/*******************/
/* Local Variables */
/*******************/
/* Dataspace ID class */
static const H5I_class_t H5I_DATASPACE_CLS[1] = {{
- H5I_DATASPACE, /* ID class value */
- 0, /* Class flags */
- 2, /* # of reserved IDs for class */
- (H5I_free_t)H5S_close /* Callback routine for closing objects of this class */
+ H5I_DATASPACE, /* ID class value */
+ 0, /* Class flags */
+ 2, /* # of reserved IDs for class */
+ (H5I_free_t)H5S_close /* Callback routine for closing objects of this class */
}};
/* Dataspace selection iterator ID class */
static const H5I_class_t H5I_SPACE_SEL_ITER_CLS[1] = {{
- H5I_SPACE_SEL_ITER, /* ID class value */
- 0, /* Class flags */
- 0, /* # of reserved IDs for class */
- (H5I_free_t)H5S_sel_iter_close /* Callback routine for closing objects of this class */
+ H5I_SPACE_SEL_ITER, /* ID class value */
+ 0, /* Class flags */
+ 0, /* # of reserved IDs for class */
+ (H5I_free_t)H5S_sel_iter_close /* Callback routine for closing objects of this class */
}};
-
/* Flag indicating "top" of interface has been initialized */
static hbool_t H5S_top_package_initialize_s = FALSE;
-
-
/*--------------------------------------------------------------------------
NAME
H5S__init_package -- Initialize interface-specific information
@@ -121,17 +111,18 @@ DESCRIPTION
herr_t
H5S__init_package(void)
{
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_PACKAGE
/* Initialize the atom group for the dataspace IDs */
- if(H5I_register_type(H5I_DATASPACE_CLS) < 0)
+ if (H5I_register_type(H5I_DATASPACE_CLS) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, FAIL, "unable to initialize dataspace ID class")
/* Initialize the atom group for the dataspace selction iterator IDs */
- if(H5I_register_type(H5I_SPACE_SEL_ITER_CLS) < 0)
- HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, FAIL, "unable to initialize dataspace selection iterator ID class")
+ if (H5I_register_type(H5I_SPACE_SEL_ITER_CLS) < 0)
+ HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, FAIL,
+ "unable to initialize dataspace selection iterator ID class")
/* Mark "top" of interface as initialized, too */
H5S_top_package_initialize_s = TRUE;
@@ -140,7 +131,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S__init_package() */
-
/*--------------------------------------------------------------------------
NAME
H5S_top_term_package
@@ -162,30 +152,29 @@ done:
int
H5S_top_term_package(void)
{
- int n = 0;
+ int n = 0;
FUNC_ENTER_NOAPI_NOINIT_NOERR
- if(H5S_top_package_initialize_s) {
- if(H5I_nmembers(H5I_DATASPACE) > 0) {
+ if (H5S_top_package_initialize_s) {
+ if (H5I_nmembers(H5I_DATASPACE) > 0) {
(void)H5I_clear_type(H5I_DATASPACE, FALSE, FALSE);
n++; /*H5I*/
- } /* end if */
+ } /* end if */
- if(H5I_nmembers(H5I_SPACE_SEL_ITER) > 0) {
+ if (H5I_nmembers(H5I_SPACE_SEL_ITER) > 0) {
(void)H5I_clear_type(H5I_SPACE_SEL_ITER, FALSE, FALSE);
n++; /*H5I*/
- } /* end if */
+ } /* end if */
/* Mark "top" of interface as closed */
- if(0 == n)
+ if (0 == n)
H5S_top_package_initialize_s = FALSE;
} /* end if */
FUNC_LEAVE_NOAPI(n)
} /* end H5S_top_term_package() */
-
/*--------------------------------------------------------------------------
NAME
H5S_term_package
@@ -209,11 +198,11 @@ H5S_top_term_package(void)
int
H5S_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(0 == H5I_nmembers(H5I_DATASPACE));
HDassert(0 == H5I_nmembers(H5I_SPACE_SEL_ITER));
@@ -226,15 +215,13 @@ H5S_term_package(void)
n += (H5I_dec_type_ref(H5I_SPACE_SEL_ITER) > 0);
/* Mark interface as closed */
- if(0 == n)
+ if (0 == n)
H5_PKG_INIT_VAR = FALSE;
} /* end if */
FUNC_LEAVE_NOAPI(n)
} /* end H5S_term_package() */
-
-
/*--------------------------------------------------------------------------
NAME
H5S_get_validiated_dataspace
@@ -257,26 +244,26 @@ H5S_term_package(void)
herr_t
H5S_get_validated_dataspace(hid_t space_id, const H5S_t **space)
{
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
HDassert(space);
/* Check for invalid ID */
- if(space_id < 0)
+ if (space_id < 0)
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "invalid space_id (ID cannot be a negative number)")
/* No special dataspace struct for H5S_ALL */
- if(H5S_ALL == space_id)
+ if (H5S_ALL == space_id)
*space = NULL;
else {
/* Get the dataspace pointer */
- if(NULL == (*space = (const H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (*space = (const H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "space_id is not a dataspace ID")
/* Check for valid selection */
- if(H5S_SELECT_VALID(*space) != TRUE)
+ if (H5S_SELECT_VALID(*space) != TRUE)
HGOTO_ERROR(H5E_DATASPACE, H5E_BADRANGE, FAIL, "selection + offset not within extent")
}
@@ -284,7 +271,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_get_validated_dataspace() */
-
/*--------------------------------------------------------------------------
NAME
H5S_create
@@ -306,25 +292,25 @@ done:
H5S_t *
H5S_create(H5S_class_t type)
{
- H5S_t *new_ds = NULL; /* New dataspace created */
+ H5S_t *new_ds = NULL; /* New dataspace created */
H5S_t *ret_value = NULL; /* Return value */
FUNC_ENTER_NOAPI(NULL)
/* Create a new dataspace */
- if(NULL == (new_ds = H5FL_CALLOC(H5S_t)))
+ if (NULL == (new_ds = H5FL_CALLOC(H5S_t)))
HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, NULL, "memory allocation failed")
/* Initialize default dataspace state */
new_ds->extent.type = type;
- if(type == H5S_NULL)
+ if (type == H5S_NULL)
new_ds->extent.version = H5O_SDSPACE_VERSION_2;
else
new_ds->extent.version = H5O_SDSPACE_VERSION_1;
new_ds->extent.rank = 0;
new_ds->extent.size = new_ds->extent.max = NULL;
- switch(type) {
+ switch (type) {
case H5S_SCALAR:
new_ds->extent.nelem = 1;
break;
@@ -341,28 +327,27 @@ H5S_create(H5S_class_t type)
} /* end switch */
/* Start with "all" selection */
- if(H5S_select_all(new_ds, FALSE) < 0)
+ if (H5S_select_all(new_ds, FALSE) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTSET, NULL, "unable to set all selection")
/* Reset common selection info pointer */
new_ds->select.sel_info.hslab = NULL;
/* Reset "shared" info on extent */
- if(H5O_msg_reset_share(H5O_SDSPACE_ID, &(new_ds->extent.sh_loc)) < 0)
+ if (H5O_msg_reset_share(H5O_SDSPACE_ID, &(new_ds->extent.sh_loc)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRESET, NULL, "unable to reset shared component info")
/* Set return value */
ret_value = new_ds;
done:
- if(ret_value == NULL)
- if(new_ds && H5S_close(new_ds) < 0)
+ if (ret_value == NULL)
+ if (new_ds && H5S_close(new_ds) < 0)
HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, NULL, "unable to release dataspace")
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_create() */
-
/*--------------------------------------------------------------------------
NAME
H5Screate
@@ -384,32 +369,31 @@ done:
hid_t
H5Screate(H5S_class_t type)
{
- H5S_t *new_ds = NULL; /* New dataspace structure */
- hid_t ret_value; /* Return value */
+ H5S_t *new_ds = NULL; /* New dataspace structure */
+ hid_t ret_value; /* Return value */
FUNC_ENTER_API(FAIL)
H5TRACE1("i", "Sc", type);
/* Check args */
- if(type <= H5S_NO_CLASS || type > H5S_NULL) /* don't allow complex dataspace yet */
+ if (type <= H5S_NO_CLASS || type > H5S_NULL) /* don't allow complex dataspace yet */
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "invalid dataspace type")
- if(NULL == (new_ds = H5S_create(type)))
+ if (NULL == (new_ds = H5S_create(type)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCREATE, FAIL, "unable to create dataspace")
/* Atomize */
- if((ret_value = H5I_register(H5I_DATASPACE, new_ds, TRUE)) < 0)
+ if ((ret_value = H5I_register(H5I_DATASPACE, new_ds, TRUE)) < 0)
HGOTO_ERROR(H5E_ATOM, H5E_CANTREGISTER, FAIL, "unable to register dataspace atom")
done:
- if(ret_value < 0)
- if(new_ds && H5S_close(new_ds) < 0)
+ if (ret_value < 0)
+ if (new_ds && H5S_close(new_ds) < 0)
HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, FAIL, "unable to release dataspace")
FUNC_LEAVE_API(ret_value)
} /* end H5Screate() */
-
/*-------------------------------------------------------------------------
* Function: H5S__extent_release
*
@@ -430,20 +414,19 @@ H5S__extent_release(H5S_extent_t *extent)
HDassert(extent);
/* Release extent */
- if(extent->type == H5S_SIMPLE) {
- if(extent->size)
+ if (extent->type == H5S_SIMPLE) {
+ if (extent->size)
extent->size = H5FL_ARR_FREE(hsize_t, extent->size);
- if(extent->max)
+ if (extent->max)
extent->max = H5FL_ARR_FREE(hsize_t, extent->max);
} /* end if */
- extent->rank = 0;
+ extent->rank = 0;
extent->nelem = 0;
FUNC_LEAVE_NOAPI(SUCCEED)
} /* end H5S__extent_release() */
-
/*-------------------------------------------------------------------------
* Function: H5S_close
*
@@ -459,18 +442,18 @@ H5S__extent_release(H5S_extent_t *extent)
herr_t
H5S_close(H5S_t *ds)
{
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
HDassert(ds);
/* Release selection (this should come before the extent release) */
- if(H5S_SELECT_RELEASE(ds) < 0)
+ if (H5S_SELECT_RELEASE(ds) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, FAIL, "unable to release dataspace selection")
/* Release extent */
- if(H5S__extent_release(&ds->extent) < 0)
+ if (H5S__extent_release(&ds->extent) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, FAIL, "unable to release dataspace extent")
done:
@@ -484,7 +467,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_close() */
-
/*-------------------------------------------------------------------------
* Function: H5Sclose
*
@@ -500,24 +482,23 @@ done:
herr_t
H5Sclose(hid_t space_id)
{
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_API(FAIL)
H5TRACE1("e", "i", space_id);
/* Check args */
- if(NULL == H5I_object_verify(space_id, H5I_DATASPACE))
+ if (NULL == H5I_object_verify(space_id, H5I_DATASPACE))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a dataspace")
/* When the reference count reaches zero the resources are freed */
- if(H5I_dec_app_ref(space_id) < 0)
+ if (H5I_dec_app_ref(space_id) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDEC, FAIL, "problem freeing id")
done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sclose() */
-
/*-------------------------------------------------------------------------
* Function: H5Scopy
*
@@ -535,34 +516,33 @@ done:
hid_t
H5Scopy(hid_t space_id)
{
- H5S_t *src = NULL;
- H5S_t *dst = NULL;
- hid_t ret_value = H5I_INVALID_HID;
+ H5S_t *src = NULL;
+ H5S_t *dst = NULL;
+ hid_t ret_value = H5I_INVALID_HID;
FUNC_ENTER_API(H5I_INVALID_HID)
H5TRACE1("i", "i", space_id);
/* Check args */
- if(NULL == (src = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (src = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, H5I_INVALID_HID, "not a dataspace")
/* Copy */
- if(NULL == (dst = H5S_copy(src, FALSE, TRUE)))
+ if (NULL == (dst = H5S_copy(src, FALSE, TRUE)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, H5I_INVALID_HID, "unable to copy dataspace")
/* Atomize */
- if((ret_value = H5I_register (H5I_DATASPACE, dst, TRUE)) < 0)
+ if ((ret_value = H5I_register(H5I_DATASPACE, dst, TRUE)) < 0)
HGOTO_ERROR(H5E_ATOM, H5E_CANTREGISTER, H5I_INVALID_HID, "unable to register dataspace atom")
done:
- if(ret_value < 0)
- if(dst && H5S_close(dst) < 0)
+ if (ret_value < 0)
+ if (dst && H5S_close(dst) < 0)
HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, H5I_INVALID_HID, "unable to release dataspace")
FUNC_LEAVE_API(ret_value)
} /* end H5Scopy() */
-
/*-------------------------------------------------------------------------
* Function: H5Sextent_copy
*
@@ -576,30 +556,29 @@ done:
*-------------------------------------------------------------------------
*/
herr_t
-H5Sextent_copy(hid_t dst_id,hid_t src_id)
+H5Sextent_copy(hid_t dst_id, hid_t src_id)
{
- H5S_t *src;
- H5S_t *dst;
- herr_t ret_value = SUCCEED;
+ H5S_t *src;
+ H5S_t *dst;
+ herr_t ret_value = SUCCEED;
FUNC_ENTER_API(FAIL)
H5TRACE2("e", "ii", dst_id, src_id);
/* Check args */
- if(NULL == (src = (H5S_t *)H5I_object_verify(src_id, H5I_DATASPACE)))
+ if (NULL == (src = (H5S_t *)H5I_object_verify(src_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a dataspace")
- if(NULL == (dst = (H5S_t *)H5I_object_verify(dst_id, H5I_DATASPACE)))
+ if (NULL == (dst = (H5S_t *)H5I_object_verify(dst_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a dataspace")
/* Copy */
- if(H5S_extent_copy(dst, src) < 0)
+ if (H5S_extent_copy(dst, src) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOPY, FAIL, "can't copy extent")
done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sextent_copy() */
-
/*-------------------------------------------------------------------------
* Function: H5S_extent_copy
*
@@ -615,7 +594,7 @@ done:
herr_t
H5S_extent_copy(H5S_t *dst, const H5S_t *src)
{
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -623,20 +602,19 @@ H5S_extent_copy(H5S_t *dst, const H5S_t *src)
HDassert(src);
/* Copy extent */
- if(H5S__extent_copy_real(&(dst->extent), &(src->extent), TRUE) < 0)
+ if (H5S__extent_copy_real(&(dst->extent), &(src->extent), TRUE) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOPY, FAIL, "can't copy extent")
/* If the selection is 'all', update the number of elements selected in the
* destination space */
- if(H5S_SEL_ALL == H5S_GET_SELECT_TYPE(dst))
- if(H5S_select_all(dst, FALSE) < 0)
+ if (H5S_SEL_ALL == H5S_GET_SELECT_TYPE(dst))
+ if (H5S_select_all(dst, FALSE) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDELETE, FAIL, "can't change selection")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_extent_copy() */
-
/*-------------------------------------------------------------------------
* Function: H5S__extent_copy_real
*
@@ -653,38 +631,38 @@ herr_t
H5S__extent_copy_real(H5S_extent_t *dst, const H5S_extent_t *src, hbool_t copy_max)
{
unsigned u;
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_PACKAGE
/* Release destination extent before we copy over it */
- if(H5S__extent_release(dst) < 0)
+ if (H5S__extent_release(dst) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, FAIL, "unable to release dataspace extent")
/* Copy the regular fields */
- dst->type = src->type;
+ dst->type = src->type;
dst->version = src->version;
- dst->nelem = src->nelem;
- dst->rank = src->rank;
+ dst->nelem = src->nelem;
+ dst->rank = src->rank;
switch (src->type) {
case H5S_NULL:
case H5S_SCALAR:
dst->size = NULL;
- dst->max = NULL;
+ dst->max = NULL;
break;
case H5S_SIMPLE:
- if(src->size) {
+ if (src->size) {
dst->size = (hsize_t *)H5FL_ARR_MALLOC(hsize_t, (size_t)src->rank);
- for(u = 0; u < src->rank; u++)
+ for (u = 0; u < src->rank; u++)
dst->size[u] = src->size[u];
} /* end if */
else
dst->size = NULL;
- if(copy_max && src->max) {
+ if (copy_max && src->max) {
dst->max = (hsize_t *)H5FL_ARR_MALLOC(hsize_t, (size_t)src->rank);
- for(u = 0; u < src->rank; u++)
+ for (u = 0; u < src->rank; u++)
dst->max[u] = src->max[u];
} /* end if */
else
@@ -698,14 +676,13 @@ H5S__extent_copy_real(H5S_extent_t *dst, const H5S_extent_t *src, hbool_t copy_m
} /* end switch */
/* Copy the shared object info */
- if(H5O_set_shared(&(dst->sh_loc), &(src->sh_loc)) < 0)
+ if (H5O_set_shared(&(dst->sh_loc), &(src->sh_loc)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOPY, FAIL, "can't copy shared information")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S__extent_copy_real() */
-
/*-------------------------------------------------------------------------
* Function: H5S_copy
*
@@ -728,34 +705,33 @@ done:
H5S_t *
H5S_copy(const H5S_t *src, hbool_t share_selection, hbool_t copy_max)
{
- H5S_t *dst = NULL;
- H5S_t *ret_value = NULL; /* Return value */
+ H5S_t *dst = NULL;
+ H5S_t *ret_value = NULL; /* Return value */
FUNC_ENTER_NOAPI(NULL)
- if(NULL == (dst = H5FL_CALLOC(H5S_t)))
+ if (NULL == (dst = H5FL_CALLOC(H5S_t)))
HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, NULL, "memory allocation failed")
/* Copy the source dataspace's extent */
- if(H5S__extent_copy_real(&(dst->extent), &(src->extent), copy_max) < 0)
+ if (H5S__extent_copy_real(&(dst->extent), &(src->extent), copy_max) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOPY, NULL, "can't copy extent")
/* Copy the source dataspace's selection */
- if(H5S_select_copy(dst, src, share_selection) < 0)
+ if (H5S_select_copy(dst, src, share_selection) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOPY, NULL, "can't copy select")
/* Set the return value */
ret_value = dst;
done:
- if(NULL == ret_value)
- if(dst)
+ if (NULL == ret_value)
+ if (dst)
dst = H5FL_FREE(H5S_t, dst);
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_copy() */
-
/*-------------------------------------------------------------------------
* Function: H5S_get_simple_extent_npoints
*
@@ -777,7 +753,7 @@ done:
hssize_t
H5S_get_simple_extent_npoints(const H5S_t *ds)
{
- hssize_t ret_value = -1; /* Return value */
+ hssize_t ret_value = -1; /* Return value */
FUNC_ENTER_NOAPI(-1)
@@ -791,7 +767,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_get_simple_extent_npoints() */
-
/*-------------------------------------------------------------------------
* Function: H5Sget_simple_extent_npoints
*
@@ -808,14 +783,14 @@ done:
hssize_t
H5Sget_simple_extent_npoints(hid_t space_id)
{
- H5S_t *ds;
- hssize_t ret_value;
+ H5S_t * ds;
+ hssize_t ret_value;
FUNC_ENTER_API(FAIL)
H5TRACE1("Hs", "i", space_id);
/* Check args */
- if(NULL == (ds = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (ds = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a dataspace")
ret_value = (hssize_t)H5S_GET_EXTENT_NPOINTS(ds);
@@ -824,7 +799,6 @@ done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sget_simple_extent_npoints() */
-
/*-------------------------------------------------------------------------
* Function: H5S_get_npoints_max
*
@@ -846,15 +820,15 @@ done:
hsize_t
H5S_get_npoints_max(const H5S_t *ds)
{
- unsigned u;
- hsize_t ret_value = 0; /* Return value */
+ unsigned u;
+ hsize_t ret_value = 0; /* Return value */
FUNC_ENTER_NOAPI(0)
/* check args */
HDassert(ds);
- switch(H5S_GET_EXTENT_TYPE(ds)) {
+ switch (H5S_GET_EXTENT_TYPE(ds)) {
case H5S_NULL:
ret_value = 0;
break;
@@ -864,9 +838,9 @@ H5S_get_npoints_max(const H5S_t *ds)
break;
case H5S_SIMPLE:
- if(ds->extent.max) {
- for(ret_value = 1, u = 0; u < ds->extent.rank; u++) {
- if(H5S_UNLIMITED == ds->extent.max[u]) {
+ if (ds->extent.max) {
+ for (ret_value = 1, u = 0; u < ds->extent.rank; u++) {
+ if (H5S_UNLIMITED == ds->extent.max[u]) {
ret_value = HSIZET_MAX;
break;
}
@@ -875,7 +849,7 @@ H5S_get_npoints_max(const H5S_t *ds)
}
}
else
- for(ret_value = 1, u = 0; u < ds->extent.rank; u++)
+ for (ret_value = 1, u = 0; u < ds->extent.rank; u++)
ret_value *= ds->extent.size[u];
break;
@@ -889,7 +863,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_get_npoints_max() */
-
/*-------------------------------------------------------------------------
* Function: H5Sget_simple_extent_ndims
*
@@ -906,14 +879,14 @@ done:
int
H5Sget_simple_extent_ndims(hid_t space_id)
{
- H5S_t *ds;
- int ret_value = -1;
+ H5S_t *ds;
+ int ret_value = -1;
FUNC_ENTER_API((-1))
H5TRACE1("Is", "i", space_id);
/* Check args */
- if(NULL == (ds = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (ds = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, (-1), "not a dataspace")
ret_value = (int)H5S_GET_EXTENT_NDIMS(ds);
@@ -922,7 +895,6 @@ done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sget_simple_extent_ndims() */
-
/*-------------------------------------------------------------------------
* Function: H5S_get_simple_extent_ndims
*
@@ -945,14 +917,14 @@ done:
int
H5S_get_simple_extent_ndims(const H5S_t *ds)
{
- int ret_value = -1; /* Return value */
+ int ret_value = -1; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
/* check args */
HDassert(ds);
- switch(H5S_GET_EXTENT_TYPE(ds)) {
+ switch (H5S_GET_EXTENT_TYPE(ds)) {
case H5S_NULL:
case H5S_SCALAR:
case H5S_SIMPLE:
@@ -969,7 +941,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_get_simple_extent_ndims() */
-
/*-------------------------------------------------------------------------
* Function: H5Sget_simple_extent_dims
*
@@ -987,16 +958,16 @@ done:
*-------------------------------------------------------------------------
*/
int
-H5Sget_simple_extent_dims(hid_t space_id, hsize_t dims[]/*out*/, hsize_t maxdims[]/*out*/)
+H5Sget_simple_extent_dims(hid_t space_id, hsize_t dims[] /*out*/, hsize_t maxdims[] /*out*/)
{
- H5S_t *ds;
- int ret_value = -1;
+ H5S_t *ds;
+ int ret_value = -1;
FUNC_ENTER_API((-1))
H5TRACE3("Is", "ixx", space_id, dims, maxdims);
/* Check args */
- if(NULL == (ds = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (ds = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, (-1), "not a dataspace")
ret_value = H5S_get_simple_extent_dims(ds, dims, maxdims);
@@ -1005,7 +976,6 @@ done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sget_simple_extent_dims() */
-
/*-------------------------------------------------------------------------
* Function: H5S_extent_get_dims
*
@@ -1023,15 +993,15 @@ done:
int
H5S_extent_get_dims(const H5S_extent_t *ext, hsize_t dims[], hsize_t max_dims[])
{
- int i; /* Local index variable */
- int ret_value = -1; /* Return value */
+ int i; /* Local index variable */
+ int ret_value = -1; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
/* check args */
HDassert(ext);
- switch(ext->type) {
+ switch (ext->type) {
case H5S_NULL:
case H5S_SCALAR:
ret_value = 0;
@@ -1039,16 +1009,16 @@ H5S_extent_get_dims(const H5S_extent_t *ext, hsize_t dims[], hsize_t max_dims[])
case H5S_SIMPLE:
ret_value = (int)ext->rank;
- for(i = 0; i < ret_value; i++) {
- if(dims)
+ for (i = 0; i < ret_value; i++) {
+ if (dims)
dims[i] = ext->size[i];
- if(max_dims) {
- if(ext->max)
+ if (max_dims) {
+ if (ext->max)
max_dims[i] = ext->max[i];
else
max_dims[i] = ext->size[i];
} /* end if */
- } /* end for */
+ } /* end for */
break;
case H5S_NO_CLASS:
@@ -1061,7 +1031,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_extent_get_dims() */
-
/*-------------------------------------------------------------------------
* Function: H5S_get_simple_extent_dims
*
@@ -1079,7 +1048,7 @@ done:
int
H5S_get_simple_extent_dims(const H5S_t *ds, hsize_t dims[], hsize_t max_dims[])
{
- int ret_value = -1; /* Return value */
+ int ret_value = -1; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -1087,14 +1056,13 @@ H5S_get_simple_extent_dims(const H5S_t *ds, hsize_t dims[], hsize_t max_dims[])
HDassert(ds);
/* Get dims for extent */
- if((ret_value = H5S_extent_get_dims(&ds->extent, dims, max_dims)) < 0)
+ if ((ret_value = H5S_extent_get_dims(&ds->extent, dims, max_dims)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTGET, FAIL, "can't retrieve dataspace extent dims")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_get_simple_extent_dims() */
-
/*-------------------------------------------------------------------------
* Function: H5S_write
*
@@ -1111,7 +1079,7 @@ done:
herr_t
H5S_write(H5F_t *f, H5O_t *oh, unsigned update_flags, H5S_t *ds)
{
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -1121,14 +1089,13 @@ H5S_write(H5F_t *f, H5O_t *oh, unsigned update_flags, H5S_t *ds)
HDassert(H5S_GET_EXTENT_TYPE(ds) >= 0);
/* Write the current dataspace extent to the dataspace message */
- if(H5O_msg_write_oh(f, oh, H5O_SDSPACE_ID, 0, update_flags, &(ds->extent)) < 0)
+ if (H5O_msg_write_oh(f, oh, H5O_SDSPACE_ID, 0, update_flags, &(ds->extent)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, FAIL, "can't update simple dataspace message")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_write() */
-
/*-------------------------------------------------------------------------
* Function: H5S_append
*
@@ -1144,7 +1111,7 @@ done:
herr_t
H5S_append(H5F_t *f, H5O_t *oh, H5S_t *ds)
{
- herr_t ret_value = SUCCEED; /* Return value */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -1154,14 +1121,13 @@ H5S_append(H5F_t *f, H5O_t *oh, H5S_t *ds)
HDassert(H5S_GET_EXTENT_TYPE(ds) >= 0);
/* Add the dataspace message to the object header */
- if(H5O_msg_append_oh(f, oh, H5O_SDSPACE_ID, 0, 0, &(ds->extent)) < 0)
+ if (H5O_msg_append_oh(f, oh, H5O_SDSPACE_ID, 0, 0, &(ds->extent)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, FAIL, "can't add simple dataspace message")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_append() */
-
/*-------------------------------------------------------------------------
* Function: H5S_read
*
@@ -1179,36 +1145,35 @@ done:
H5S_t *
H5S_read(const H5O_loc_t *loc)
{
- H5S_t *ds = NULL; /* Dataspace to return */
- H5S_t *ret_value = NULL; /* Return value */
+ H5S_t *ds = NULL; /* Dataspace to return */
+ H5S_t *ret_value = NULL; /* Return value */
FUNC_ENTER_NOAPI(NULL)
/* check args */
HDassert(loc);
- if(NULL == (ds = H5FL_CALLOC(H5S_t)))
+ if (NULL == (ds = H5FL_CALLOC(H5S_t)))
HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, NULL, "memory allocation failed")
- if(NULL == H5O_msg_read(loc, H5O_SDSPACE_ID, &(ds->extent)))
+ if (NULL == H5O_msg_read(loc, H5O_SDSPACE_ID, &(ds->extent)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, NULL, "unable to load dataspace info from dataset header")
/* Default to entire dataspace being selected */
- if(H5S_select_all(ds, FALSE) < 0)
+ if (H5S_select_all(ds, FALSE) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTSET, NULL, "unable to set all selection")
/* Set the value for successful return */
ret_value = ds;
done:
- if(ret_value == NULL)
- if(ds != NULL)
+ if (ret_value == NULL)
+ if (ds != NULL)
ds = H5FL_FREE(H5S_t, ds);
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_read() */
-
/*--------------------------------------------------------------------------
NAME
H5S__is_simple
@@ -1226,7 +1191,7 @@ done:
static htri_t
H5S__is_simple(const H5S_t *sdim)
{
- htri_t ret_value = FAIL; /* Return value */
+ htri_t ret_value = FAIL; /* Return value */
FUNC_ENTER_STATIC_NOERR
@@ -1234,13 +1199,12 @@ H5S__is_simple(const H5S_t *sdim)
HDassert(sdim);
/* H5S_NULL shouldn't be simple dataspace */
- ret_value = (H5S_GET_EXTENT_TYPE(sdim) == H5S_SIMPLE ||
- H5S_GET_EXTENT_TYPE(sdim) == H5S_SCALAR) ? TRUE : FALSE;
+ ret_value =
+ (H5S_GET_EXTENT_TYPE(sdim) == H5S_SIMPLE || H5S_GET_EXTENT_TYPE(sdim) == H5S_SCALAR) ? TRUE : FALSE;
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S__is_simple() */
-
/*--------------------------------------------------------------------------
NAME
H5Sis_simple
@@ -1258,14 +1222,14 @@ H5S__is_simple(const H5S_t *sdim)
htri_t
H5Sis_simple(hid_t space_id)
{
- H5S_t *space; /* Dataspace to check */
- htri_t ret_value; /* Return value */
+ H5S_t *space; /* Dataspace to check */
+ htri_t ret_value; /* Return value */
FUNC_ENTER_API(FAIL)
H5TRACE1("t", "i", space_id);
/* Check args and all the boring stuff. */
- if(NULL == (space = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (space = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ATOM, H5E_BADATOM, FAIL, "not a dataspace")
ret_value = H5S__is_simple(space);
@@ -1274,7 +1238,6 @@ done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sis_simple() */
-
/*--------------------------------------------------------------------------
NAME
H5Sset_extent_simple
@@ -1302,44 +1265,44 @@ done:
--------------------------------------------------------------------------*/
herr_t
-H5Sset_extent_simple(hid_t space_id, int rank, const hsize_t dims[/*rank*/],
- const hsize_t max[/*rank*/])
+H5Sset_extent_simple(hid_t space_id, int rank, const hsize_t dims[/*rank*/], const hsize_t max[/*rank*/])
{
- H5S_t *space; /* Dataspace to modify */
- int u; /* Local counting variable */
- herr_t ret_value = SUCCEED; /* Return value */
+ H5S_t *space; /* Dataspace to modify */
+ int u; /* Local counting variable */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_API(FAIL)
H5TRACE4("e", "iIs*[a1]h*[a1]h", space_id, rank, dims, max);
/* Check args */
- if(NULL == (space = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (space = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ATOM, H5E_BADATOM, FAIL, "not a dataspace")
- if(rank > 0 && dims == NULL)
+ if (rank > 0 && dims == NULL)
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "no dimensions specified")
- if(rank < 0 || rank > H5S_MAX_RANK)
+ if (rank < 0 || rank > H5S_MAX_RANK)
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "invalid rank")
- if(dims)
- for(u = 0; u < rank; u++)
- if(H5S_UNLIMITED == dims[u])
- HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "current dimension must have a specific size, not H5S_UNLIMITED")
- if(max != NULL) {
- if(dims == NULL)
- HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "maximum dimension specified, but no current dimensions specified")
- for(u = 0; u < rank; u++)
- if(max[u] != H5S_UNLIMITED && max[u] < dims[u])
+ if (dims)
+ for (u = 0; u < rank; u++)
+ if (H5S_UNLIMITED == dims[u])
+ HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL,
+ "current dimension must have a specific size, not H5S_UNLIMITED")
+ if (max != NULL) {
+ if (dims == NULL)
+ HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL,
+ "maximum dimension specified, but no current dimensions specified")
+ for (u = 0; u < rank; u++)
+ if (max[u] != H5S_UNLIMITED && max[u] < dims[u])
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "invalid maximum dimension size")
}
/* Do it */
- if(H5S_set_extent_simple(space, (unsigned)rank, dims, max) < 0)
+ if (H5S_set_extent_simple(space, (unsigned)rank, dims, max) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, FAIL, "unable to set simple extent")
done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sset_extent_simple() */
-
/*-------------------------------------------------------------------------
* Function: H5S_set_extent_simple
*
@@ -1353,11 +1316,10 @@ done:
*-------------------------------------------------------------------------
*/
herr_t
-H5S_set_extent_simple(H5S_t *space, unsigned rank, const hsize_t *dims,
- const hsize_t *max)
+H5S_set_extent_simple(H5S_t *space, unsigned rank, const hsize_t *dims, const hsize_t *max)
{
- unsigned u; /* Local index variable */
- herr_t ret_value = SUCCEED; /* Return value */
+ unsigned u; /* Local index variable */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -1365,16 +1327,16 @@ H5S_set_extent_simple(H5S_t *space, unsigned rank, const hsize_t *dims,
HDassert(rank <= H5S_MAX_RANK);
/* shift out of the previous state to a "simple" dataspace. */
- if(H5S__extent_release(&space->extent) < 0)
+ if (H5S__extent_release(&space->extent) < 0)
HGOTO_ERROR(H5E_RESOURCE, H5E_CANTFREE, FAIL, "failed to release previous dataspace extent")
- if(rank == 0) { /* scalar variable */
- space->extent.type = H5S_SCALAR;
+ if (rank == 0) { /* scalar variable */
+ space->extent.type = H5S_SCALAR;
space->extent.nelem = 1;
- space->extent.rank = 0; /* set to scalar rank */
- } /* end if */
+ space->extent.rank = 0; /* set to scalar rank */
+ } /* end if */
else {
- hsize_t nelem; /* Number of elements in extent */
+ hsize_t nelem; /* Number of elements in extent */
space->extent.type = H5S_SIMPLE;
@@ -1383,7 +1345,7 @@ H5S_set_extent_simple(H5S_t *space, unsigned rank, const hsize_t *dims,
space->extent.size = (hsize_t *)H5FL_ARR_MALLOC(hsize_t, (size_t)rank);
/* Copy the dimensions & compute the number of elements in the extent */
- for(u = 0, nelem = 1; dims && (u < space->extent.rank); u++) {
+ for (u = 0, nelem = 1; dims && (u < space->extent.rank); u++) {
space->extent.size[u] = dims[u];
nelem *= dims[u];
} /* end for */
@@ -1392,10 +1354,10 @@ H5S_set_extent_simple(H5S_t *space, unsigned rank, const hsize_t *dims,
/* Copy the maximum dimensions if specified. Otherwise, the maximal dimensions are the
* same as the dimension */
space->extent.max = (hsize_t *)H5FL_ARR_MALLOC(hsize_t, (size_t)rank);
- if(max != NULL)
+ if (max != NULL)
H5MM_memcpy(space->extent.max, max, sizeof(hsize_t) * rank);
else
- for(u = 0; dims && (u < space->extent.rank); u++)
+ for (u = 0; dims && (u < space->extent.rank); u++)
space->extent.max[u] = dims[u];
} /* end else */
@@ -1406,15 +1368,14 @@ H5S_set_extent_simple(H5S_t *space, unsigned rank, const hsize_t *dims,
space->select.offset_changed = FALSE;
/* If the selection is 'all', update the number of elements selected */
- if(H5S_GET_SELECT_TYPE(space) == H5S_SEL_ALL)
- if(H5S_select_all(space, FALSE) < 0)
+ if (H5S_GET_SELECT_TYPE(space) == H5S_SEL_ALL)
+ if (H5S_select_all(space, FALSE) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDELETE, FAIL, "can't change selection")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_set_extent_simple() */
-
/*-------------------------------------------------------------------------
* Function: H5Screate_simple
*
@@ -1439,50 +1400,50 @@ done:
hid_t
H5Screate_simple(int rank, const hsize_t dims[/*rank*/], const hsize_t maxdims[/*rank*/])
{
- H5S_t *space = NULL;
- int i;
- hid_t ret_value = H5I_INVALID_HID;
+ H5S_t *space = NULL;
+ int i;
+ hid_t ret_value = H5I_INVALID_HID;
FUNC_ENTER_API(H5I_INVALID_HID)
H5TRACE3("i", "Is*[a0]h*[a0]h", rank, dims, maxdims);
/* Check arguments */
- if(rank < 0)
+ if (rank < 0)
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, H5I_INVALID_HID, "dimensionality cannot be negative")
- if(rank > H5S_MAX_RANK)
+ if (rank > H5S_MAX_RANK)
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, H5I_INVALID_HID, "dimensionality is too large")
/* We allow users to use this function to create scalar or null dataspace.
* Check DIMS isn't set when the RANK is 0.
*/
- if(!dims && rank != 0)
+ if (!dims && rank != 0)
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, H5I_INVALID_HID, "invalid dataspace information")
/* Check whether the current dimensions are valid */
- for(i = 0; i < rank; i++) {
- if(H5S_UNLIMITED == dims[i])
- HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, H5I_INVALID_HID, "current dimension must have a specific size, not H5S_UNLIMITED")
- if(maxdims && H5S_UNLIMITED != maxdims[i] && maxdims[i]<dims[i])
+ for (i = 0; i < rank; i++) {
+ if (H5S_UNLIMITED == dims[i])
+ HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, H5I_INVALID_HID,
+ "current dimension must have a specific size, not H5S_UNLIMITED")
+ if (maxdims && H5S_UNLIMITED != maxdims[i] && maxdims[i] < dims[i])
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, H5I_INVALID_HID, "maxdims is smaller than dims")
} /* end for */
/* Create the space and set the extent */
- if(NULL == (space = H5S_create_simple((unsigned)rank, dims, maxdims)))
+ if (NULL == (space = H5S_create_simple((unsigned)rank, dims, maxdims)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCREATE, H5I_INVALID_HID, "can't create simple dataspace")
/* Atomize */
- if((ret_value = H5I_register (H5I_DATASPACE, space, TRUE)) < 0)
+ if ((ret_value = H5I_register(H5I_DATASPACE, space, TRUE)) < 0)
HGOTO_ERROR(H5E_ATOM, H5E_CANTREGISTER, H5I_INVALID_HID, "unable to register dataspace ID")
done:
- if(ret_value < 0)
- if(space && H5S_close(space) < 0)
+ if (ret_value < 0)
+ if (space && H5S_close(space) < 0)
HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, H5I_INVALID_HID, "unable to release dataspace")
FUNC_LEAVE_API(ret_value)
} /* end H5Screate_simple() */
-
/*-------------------------------------------------------------------------
* Function: H5S_create_simple
*
@@ -1499,7 +1460,7 @@ done:
H5S_t *
H5S_create_simple(unsigned rank, const hsize_t dims[/*rank*/], const hsize_t maxdims[/*rank*/])
{
- H5S_t *ret_value = NULL; /* Return value */
+ H5S_t *ret_value = NULL; /* Return value */
FUNC_ENTER_NOAPI(NULL)
@@ -1507,16 +1468,15 @@ H5S_create_simple(unsigned rank, const hsize_t dims[/*rank*/], const hsize_t max
HDassert(rank <= H5S_MAX_RANK);
/* Create the space and set the extent */
- if(NULL == (ret_value = H5S_create(H5S_SIMPLE)))
+ if (NULL == (ret_value = H5S_create(H5S_SIMPLE)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCREATE, NULL, "can't create simple dataspace")
- if(H5S_set_extent_simple(ret_value, rank, dims, maxdims) < 0)
+ if (H5S_set_extent_simple(ret_value, rank, dims, maxdims) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTINIT, NULL, "can't set dimensions")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_create_simple() */
-
/*-------------------------------------------------------------------------
* Function: H5Sencode2
*
@@ -1536,28 +1496,27 @@ done:
herr_t
H5Sencode2(hid_t obj_id, void *buf, size_t *nalloc, hid_t fapl_id)
{
- H5S_t *dspace;
- herr_t ret_value=SUCCEED;
+ H5S_t *dspace;
+ herr_t ret_value = SUCCEED;
FUNC_ENTER_API(FAIL)
H5TRACE4("e", "i*x*zi", obj_id, buf, nalloc, fapl_id);
/* Check argument and retrieve object */
- if(NULL == (dspace = (H5S_t *)H5I_object_verify(obj_id, H5I_DATASPACE)))
+ if (NULL == (dspace = (H5S_t *)H5I_object_verify(obj_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a dataspace")
/* 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_FILE, H5E_CANTSET, H5I_INVALID_HID, "can't set access property list info")
- if(H5S_encode(dspace, (unsigned char **)&buf, nalloc) < 0)
+ if (H5S_encode(dspace, (unsigned char **)&buf, nalloc) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTENCODE, FAIL, "can't encode dataspace")
done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sencode2() */
-
/*-------------------------------------------------------------------------
* Function: H5S_encode
*
@@ -1575,33 +1534,33 @@ done:
herr_t
H5S_encode(H5S_t *obj, unsigned char **p, size_t *nalloc)
{
- H5F_t *f = NULL; /* Fake file structure*/
- size_t extent_size; /* Size of serialized dataspace extent */
- hssize_t sselect_size; /* Signed size of serialized dataspace selection */
- size_t select_size; /* Size of serialized dataspace selection */
- herr_t ret_value = SUCCEED; /* Return value */
+ H5F_t * f = NULL; /* Fake file structure*/
+ size_t extent_size; /* Size of serialized dataspace extent */
+ hssize_t sselect_size; /* Signed size of serialized dataspace selection */
+ size_t select_size; /* Size of serialized dataspace selection */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI_NOINIT
/* Allocate "fake" file structure */
- if(NULL == (f = H5F_fake_alloc((uint8_t)0)))
+ if (NULL == (f = H5F_fake_alloc((uint8_t)0)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTALLOC, FAIL, "can't allocate fake file struct")
/* Find out the size of buffer needed for extent */
- if((extent_size = H5O_msg_raw_size(f, H5O_SDSPACE_ID, TRUE, obj)) == 0)
+ if ((extent_size = H5O_msg_raw_size(f, H5O_SDSPACE_ID, TRUE, obj)) == 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_BADSIZE, FAIL, "can't find dataspace size")
/* Find out the size of buffer needed for selection */
- if((sselect_size = H5S_SELECT_SERIAL_SIZE(obj)) < 0)
+ if ((sselect_size = H5S_SELECT_SERIAL_SIZE(obj)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_BADSIZE, FAIL, "can't find dataspace selection size")
H5_CHECKED_ASSIGN(select_size, size_t, sselect_size, hssize_t);
/* Verify the size of buffer. If it's not big enough, simply return the
* right size without filling the buffer. */
- if(!*p || *nalloc < (extent_size + select_size + 1 + 1 + 1 + 4))
+ if (!*p || *nalloc < (extent_size + select_size + 1 + 1 + 1 + 4))
*nalloc = extent_size + select_size + 1 + 1 + 1 + 4;
else {
- unsigned char *pp = (*p); /* Local pointer for decoding */
+ unsigned char *pp = (*p); /* Local pointer for decoding */
/* Encode the type of the information */
*pp++ = H5O_SDSPACE_ID;
@@ -1616,25 +1575,24 @@ H5S_encode(H5S_t *obj, unsigned char **p, size_t *nalloc)
UINT32ENCODE(pp, extent_size);
/* Encode the extent part of dataspace */
- if(H5O_msg_encode(f, H5O_SDSPACE_ID, TRUE, pp, obj) < 0)
+ if (H5O_msg_encode(f, H5O_SDSPACE_ID, TRUE, pp, obj) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTENCODE, FAIL, "can't encode extent space")
pp += extent_size;
/* Encode the selection part of dataspace. */
*p = pp;
- if(H5S_SELECT_SERIALIZE(obj, p) < 0)
+ if (H5S_SELECT_SERIALIZE(obj, p) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTENCODE, FAIL, "can't encode select space")
} /* end else */
done:
/* Release fake file structure */
- if(f && H5F_fake_free(f) < 0)
+ if (f && H5F_fake_free(f) < 0)
HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, FAIL, "unable to release fake file struct")
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_encode() */
-
/*-------------------------------------------------------------------------
* Function: H5Sdecode
*
@@ -1653,27 +1611,26 @@ done:
hid_t
H5Sdecode(const void *buf)
{
- H5S_t *ds;
- hid_t ret_value;
+ H5S_t *ds;
+ hid_t ret_value;
FUNC_ENTER_API(H5I_INVALID_HID)
H5TRACE1("i", "*x", buf);
- if(buf == NULL)
+ if (buf == NULL)
HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, H5I_INVALID_HID, "empty buffer")
- if((ds = H5S_decode((const unsigned char **)&buf)) == NULL)
+ if ((ds = H5S_decode((const unsigned char **)&buf)) == NULL)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDECODE, H5I_INVALID_HID, "can't decode object")
/* Register the type and return the ID */
- if((ret_value = H5I_register(H5I_DATASPACE, ds, TRUE)) < 0)
+ if ((ret_value = H5I_register(H5I_DATASPACE, ds, TRUE)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTREGISTER, H5I_INVALID_HID, "unable to register dataspace")
done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sdecode() */
-
/*-------------------------------------------------------------------------
* Function: H5S_decode
*
@@ -1689,32 +1646,32 @@ done:
*
*-------------------------------------------------------------------------
*/
-H5S_t*
+H5S_t *
H5S_decode(const unsigned char **p)
{
- H5F_t *f = NULL; /* Fake file structure*/
- H5S_t *ds; /* Decoded dataspace */
- H5S_extent_t *extent; /* Entent of decoded dataspace */
- const unsigned char *pp = (*p); /* Local pointer for decoding */
- size_t extent_size; /* size of the extent message*/
- uint8_t sizeof_size; /* 'Size of sizes' for file */
- H5S_t *ret_value = NULL; /* Return value */
+ H5F_t * f = NULL; /* Fake file structure*/
+ H5S_t * ds; /* Decoded dataspace */
+ H5S_extent_t * extent; /* Entent of decoded dataspace */
+ const unsigned char *pp = (*p); /* Local pointer for decoding */
+ size_t extent_size; /* size of the extent message*/
+ uint8_t sizeof_size; /* 'Size of sizes' for file */
+ H5S_t * ret_value = NULL; /* Return value */
FUNC_ENTER_NOAPI_NOINIT
/* Decode the type of the information */
- if(*pp++ != H5O_SDSPACE_ID)
+ if (*pp++ != H5O_SDSPACE_ID)
HGOTO_ERROR(H5E_DATASPACE, H5E_BADMESG, NULL, "not an encoded dataspace")
/* Decode the version of the dataspace information */
- if(*pp++ != H5S_ENCODE_VERSION)
+ if (*pp++ != H5S_ENCODE_VERSION)
HGOTO_ERROR(H5E_DATASPACE, H5E_VERSION, NULL, "unknown version of encoded dataspace")
/* Decode the "size of size" information */
sizeof_size = *pp++;
/* Allocate "fake" file structure */
- if(NULL == (f = H5F_fake_alloc(sizeof_size)))
+ if (NULL == (f = H5F_fake_alloc(sizeof_size)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTALLOC, NULL, "can't allocate fake file struct")
/* Decode size of extent information */
@@ -1722,26 +1679,27 @@ H5S_decode(const unsigned char **p)
/* Decode the extent part of dataspace */
/* (pass mostly bogus file pointer and bogus DXPL) */
- if(NULL == (extent = (H5S_extent_t *)H5O_msg_decode(f, NULL, H5O_SDSPACE_ID, extent_size, pp)))
+ if (NULL == (extent = (H5S_extent_t *)H5O_msg_decode(f, NULL, H5O_SDSPACE_ID, extent_size, pp)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDECODE, NULL, "can't decode object")
pp += extent_size;
/* Copy the extent into dataspace structure */
- if(NULL == (ds = H5FL_CALLOC(H5S_t)))
- HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, NULL, "memory allocation failed for dataspace conversion path table")
- if(NULL == H5O_msg_copy(H5O_SDSPACE_ID, extent, &(ds->extent)))
+ if (NULL == (ds = H5FL_CALLOC(H5S_t)))
+ HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, NULL,
+ "memory allocation failed for dataspace conversion path table")
+ if (NULL == H5O_msg_copy(H5O_SDSPACE_ID, extent, &(ds->extent)))
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOPY, NULL, "can't copy object")
- if(H5S__extent_release(extent) < 0)
+ if (H5S__extent_release(extent) < 0)
HGOTO_ERROR(H5E_RESOURCE, H5E_CANTDELETE, NULL, "can't release previous dataspace")
extent = H5FL_FREE(H5S_extent_t, extent);
/* Initialize to "all" selection. Deserialization relies on valid existing selection. */
- if(H5S_select_all(ds, FALSE) < 0)
+ if (H5S_select_all(ds, FALSE) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTSET, NULL, "unable to set all selection")
/* Decode the select part of dataspace. I believe this part always exists. */
*p = pp;
- if(H5S_SELECT_DESERIALIZE(&ds, p) < 0)
+ if (H5S_SELECT_DESERIALIZE(&ds, p) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDECODE, NULL, "can't decode space selection")
/* Set return value */
@@ -1749,13 +1707,12 @@ H5S_decode(const unsigned char **p)
done:
/* Release fake file structure */
- if(f && H5F_fake_free(f) < 0)
+ if (f && H5F_fake_free(f) < 0)
HDONE_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, NULL, "unable to release fake file struct")
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_decode() */
-
/*-------------------------------------------------------------------------
* Function: H5S_get_simple_extent_type
*
@@ -1777,7 +1734,7 @@ done:
H5S_class_t
H5S_get_simple_extent_type(const H5S_t *space)
{
- H5S_class_t ret_value = H5S_NO_CLASS; /* Return value */
+ H5S_class_t ret_value = H5S_NO_CLASS; /* Return value */
FUNC_ENTER_NOAPI(H5S_NO_CLASS)
@@ -1789,7 +1746,6 @@ done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_get_simple_extent_type() */
-
/*-------------------------------------------------------------------------
* Function: H5Sget_simple_extent_type
*
@@ -1807,14 +1763,14 @@ done:
H5S_class_t
H5Sget_simple_extent_type(hid_t sid)
{
- H5S_t *space;
- H5S_class_t ret_value; /* Return value */
+ H5S_t * space;
+ H5S_class_t ret_value; /* Return value */
FUNC_ENTER_API(H5S_NO_CLASS)
H5TRACE1("Sc", "i", sid);
/* Check arguments */
- if(NULL == (space = (H5S_t *)H5I_object_verify(sid, H5I_DATASPACE)))
+ if (NULL == (space = (H5S_t *)H5I_object_verify(sid, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, H5S_NO_CLASS, "not a dataspace")
ret_value = H5S_GET_EXTENT_TYPE(space);
@@ -1823,7 +1779,6 @@ done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sget_simple_extent_type() */
-
/*--------------------------------------------------------------------------
NAME
H5Sset_extent_none
@@ -1841,18 +1796,18 @@ done:
herr_t
H5Sset_extent_none(hid_t space_id)
{
- H5S_t *space; /* Dataspace to modify */
- herr_t ret_value = SUCCEED; /* Return value */
+ H5S_t *space; /* Dataspace to modify */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_API(FAIL)
H5TRACE1("e", "i", space_id);
/* Check args */
- if(NULL == (space = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
+ if (NULL == (space = (H5S_t *)H5I_object_verify(space_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ATOM, H5E_BADATOM, FAIL, "not a dataspace")
/* Clear the previous extent from the dataspace */
- if(H5S__extent_release(&space->extent) < 0)
+ if (H5S__extent_release(&space->extent) < 0)
HGOTO_ERROR(H5E_RESOURCE, H5E_CANTDELETE, FAIL, "can't release previous dataspace")
space->extent.type = H5S_NULL;
@@ -1861,7 +1816,6 @@ done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sset_extent_none() */
-
/*-------------------------------------------------------------------------
* Function: H5S_set_extent
*
@@ -1878,7 +1832,7 @@ htri_t
H5S_set_extent(H5S_t *space, const hsize_t *size)
{
unsigned u; /* Local index variable */
- htri_t ret_value = FALSE; /* Return value */
+ htri_t ret_value = FALSE; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -1887,27 +1841,27 @@ H5S_set_extent(H5S_t *space, const hsize_t *size)
HDassert(size);
/* Verify that the dimensions being changed are allowed to change */
- for(u = 0; u < space->extent.rank; u++)
- if(space->extent.size[u] != size[u]) {
+ for (u = 0; u < space->extent.rank; u++)
+ if (space->extent.size[u] != size[u]) {
/* Check for invalid dimension size modification */
- if(space->extent.max && H5S_UNLIMITED != space->extent.max[u] &&
- space->extent.max[u] < size[u])
- HGOTO_ERROR(H5E_DATASPACE, H5E_BADVALUE, FAIL, "dimension cannot exceed the existing maximal size (new: %llu max: %llu)", (unsigned long long)size[u], (unsigned long long)space->extent.max[u])
+ if (space->extent.max && H5S_UNLIMITED != space->extent.max[u] && space->extent.max[u] < size[u])
+ HGOTO_ERROR(H5E_DATASPACE, H5E_BADVALUE, FAIL,
+ "dimension cannot exceed the existing maximal size (new: %llu max: %llu)",
+ (unsigned long long)size[u], (unsigned long long)space->extent.max[u])
/* Indicate that dimension size can be modified */
ret_value = TRUE;
} /* end if */
/* Update dimension size(s) */
- if(ret_value)
- if(H5S_set_extent_real(space, size) < 0)
- HGOTO_ERROR(H5E_DATASPACE, H5E_CANTSET, FAIL, "failed to change dimension size(s)")
+ if (ret_value)
+ if (H5S_set_extent_real(space, size) < 0)
+ HGOTO_ERROR(H5E_DATASPACE, H5E_CANTSET, FAIL, "failed to change dimension size(s)")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_set_extent() */
-
/*-------------------------------------------------------------------------
* Function: H5S_has_extent
*
@@ -1926,13 +1880,13 @@ done:
H5_ATTR_PURE hbool_t
H5S_has_extent(const H5S_t *ds)
{
- hbool_t ret_value = FALSE; /* Return value */
+ hbool_t ret_value = FALSE; /* Return value */
FUNC_ENTER_NOAPI_NOINIT_NOERR
HDassert(ds);
- if(0 == ds->extent.rank && 0 == ds->extent.nelem && H5S_NULL != ds->extent.type)
+ if (0 == ds->extent.rank && 0 == ds->extent.nelem && H5S_NULL != ds->extent.type)
ret_value = FALSE;
else
ret_value = TRUE;
@@ -1940,7 +1894,6 @@ H5S_has_extent(const H5S_t *ds)
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_has_extent() */
-
/*-------------------------------------------------------------------------
* Function: H5S_set_extent_real
*
@@ -1957,9 +1910,9 @@ H5S_has_extent(const H5S_t *ds)
herr_t
H5S_set_extent_real(H5S_t *space, const hsize_t *size)
{
- hsize_t nelem; /* Number of elements in extent */
- unsigned u; /* Local index variable */
- herr_t ret_value = SUCCEED; /* Return value */
+ hsize_t nelem; /* Number of elements in extent */
+ unsigned u; /* Local index variable */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -1968,26 +1921,25 @@ H5S_set_extent_real(H5S_t *space, const hsize_t *size)
HDassert(size);
/* Change the dataspace size & re-compute the number of elements in the extent */
- for(u = 0, nelem = 1; u < space->extent.rank; u++ ) {
+ for (u = 0, nelem = 1; u < space->extent.rank; u++) {
space->extent.size[u] = size[u];
nelem *= size[u];
} /* end for */
space->extent.nelem = nelem;
/* If the selection is 'all', update the number of elements selected */
- if(H5S_SEL_ALL == H5S_GET_SELECT_TYPE(space))
- if(H5S_select_all(space, FALSE) < 0)
+ if (H5S_SEL_ALL == H5S_GET_SELECT_TYPE(space))
+ if (H5S_select_all(space, FALSE) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTDELETE, FAIL, "can't change selection")
/* Mark the dataspace as no longer shared if it was before */
- if(H5O_msg_reset_share(H5O_SDSPACE_ID, space) < 0)
+ if (H5O_msg_reset_share(H5O_SDSPACE_ID, space) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRESET, FAIL, "can't stop sharing dataspace")
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_set_extent_real() */
-
/*-------------------------------------------------------------------------
* Function: H5Sextent_equal
*
@@ -2005,27 +1957,26 @@ done:
htri_t
H5Sextent_equal(hid_t space1_id, hid_t space2_id)
{
- const H5S_t *ds1; /* Dataspaces to compare */
- const H5S_t *ds2;
- htri_t ret_value;
+ const H5S_t *ds1; /* Dataspaces to compare */
+ const H5S_t *ds2;
+ htri_t ret_value;
FUNC_ENTER_API(FAIL)
H5TRACE2("t", "ii", space1_id, space2_id);
/* check args */
- if(NULL == (ds1 = (const H5S_t *)H5I_object_verify(space1_id, H5I_DATASPACE)) ||
- NULL == (ds2 = (const H5S_t *)H5I_object_verify(space2_id, H5I_DATASPACE)))
+ if (NULL == (ds1 = (const H5S_t *)H5I_object_verify(space1_id, H5I_DATASPACE)) ||
+ NULL == (ds2 = (const H5S_t *)H5I_object_verify(space2_id, H5I_DATASPACE)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a dataspace")
/* Check dataspaces for extent's equality */
- if((ret_value = H5S_extent_equal(ds1, ds2)) < 0)
+ if ((ret_value = H5S_extent_equal(ds1, ds2)) < 0)
HGOTO_ERROR(H5E_DATASPACE, H5E_CANTCOMPARE, FAIL, "dataspace comparison failed")
done:
FUNC_LEAVE_API(ret_value)
} /* end H5Sextent_equal() */
-
/*--------------------------------------------------------------------------
NAME
H5S_extent_equal
@@ -2042,8 +1993,8 @@ done:
H5_ATTR_PURE htri_t
H5S_extent_equal(const H5S_t *ds1, const H5S_t *ds2)
{
- unsigned u; /* Local index variable */
- htri_t ret_value = TRUE; /* Return value */
+ unsigned u; /* Local index variable */
+ htri_t ret_value = TRUE; /* Return value */
FUNC_ENTER_NOAPI_NOINIT_NOERR
@@ -2052,41 +2003,39 @@ H5S_extent_equal(const H5S_t *ds1, const H5S_t *ds2)
HDassert(ds2);
/* Make certain the dataspaces are the same type */
- if(ds1->extent.type != ds2->extent.type)
+ if (ds1->extent.type != ds2->extent.type)
HGOTO_DONE(FALSE)
/* Make certain the dataspaces are the same rank */
- if(ds1->extent.rank != ds2->extent.rank)
+ if (ds1->extent.rank != ds2->extent.rank)
HGOTO_DONE(FALSE)
/* Make certain the dataspaces' current dimensions are the same size */
- if(ds1->extent.rank > 0) {
+ if (ds1->extent.rank > 0) {
HDassert(ds1->extent.size);
HDassert(ds2->extent.size);
- for(u = 0; u < ds1->extent.rank; u++)
- if(ds1->extent.size[u] != ds2->extent.size[u])
+ for (u = 0; u < ds1->extent.rank; u++)
+ if (ds1->extent.size[u] != ds2->extent.size[u])
HGOTO_DONE(FALSE)
} /* end if */
/* Make certain the dataspaces' maximum dimensions are the same size */
- if(ds1->extent.rank > 0) {
+ if (ds1->extent.rank > 0) {
/* Check for no maximum dimensions on dataspaces */
- if(ds1->extent.max != NULL && ds2->extent.max != NULL) {
- for(u = 0; u < ds1->extent.rank; u++)
- if(ds1->extent.max[u] != ds2->extent.max[u])
+ if (ds1->extent.max != NULL && ds2->extent.max != NULL) {
+ for (u = 0; u < ds1->extent.rank; u++)
+ if (ds1->extent.max[u] != ds2->extent.max[u])
HGOTO_DONE(FALSE)
} /* end if */
- else
- if((ds1->extent.max == NULL && ds2->extent.max != NULL) ||
- (ds1->extent.max != NULL && ds2->extent.max == NULL))
- HGOTO_DONE(FALSE)
+ else if ((ds1->extent.max == NULL && ds2->extent.max != NULL) ||
+ (ds1->extent.max != NULL && ds2->extent.max == NULL))
+ HGOTO_DONE(FALSE)
} /* end if */
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_extent_equal() */
-
/*-------------------------------------------------------------------------
* Function: H5S_extent_nelem
*
@@ -2112,7 +2061,6 @@ H5S_extent_nelem(const H5S_extent_t *ext)
FUNC_LEAVE_NOAPI(ext->nelem)
} /* end H5S_extent_nelem() */
-
/*-------------------------------------------------------------------------
* Function: H5S_set_version
*
@@ -2127,8 +2075,8 @@ H5S_extent_nelem(const H5S_extent_t *ext)
herr_t
H5S_set_version(H5F_t *f, H5S_t *ds)
{
- unsigned version; /* Message version */
- herr_t ret_value = SUCCEED; /* Return value */
+ unsigned version; /* Message version */
+ herr_t ret_value = SUCCEED; /* Return value */
FUNC_ENTER_NOAPI(FAIL)
@@ -2140,7 +2088,7 @@ H5S_set_version(H5F_t *f, H5S_t *ds)
version = MAX(ds->extent.version, H5O_sdspace_ver_bounds[H5F_LOW_BOUND(f)]);
/* Version bounds check */
- if(version > H5O_sdspace_ver_bounds[H5F_HIGH_BOUND(f)])
+ if (version > H5O_sdspace_ver_bounds[H5F_HIGH_BOUND(f)])
HGOTO_ERROR(H5E_DATASET, H5E_BADRANGE, FAIL, "Dataspace version out of bounds")
/* Set the message version */
@@ -2149,4 +2097,3 @@ H5S_set_version(H5F_t *f, H5S_t *ds)
done:
FUNC_LEAVE_NOAPI(ret_value)
} /* end H5S_set_version() */
-