Committing clang-format changes

src/H5A.c

@@ -2545,7 +2545,8 @@ H5Aexists_by_name_async(const char *app_file, const char *app_func, unsigned app
     /* Asynchronously check if an attribute exists */
     if (H5A__exists_by_name_api_common(loc_id, obj_name, attr_name, attr_exists, lapl_id, token_ptr,
                                        &vol_obj) < 0)
-        HGOTO_ERROR(H5E_ATTR, H5E_CANTGET, FAIL, "can't asynchronously determine if attribute exists by name");
+        HGOTO_ERROR(H5E_ATTR, H5E_CANTGET, FAIL,
+                    "can't asynchronously determine if attribute exists by name");
 
     /* If a token was created, add the token to the event set */
     if (NULL != token)

src/H5ACmpio.c

@@ -2068,7 +2068,8 @@ H5AC__run_sync_point(H5F_t *f, int sync_point_op)
 
                 case H5AC_SYNC_POINT_OP__FLUSH_CACHE:
                     if (H5AC__rsp__dist_md_write__flush(f) < 0)
-                        HGOTO_ERROR(H5E_CACHE, H5E_CANTGET, FAIL, "H5AC__rsp__dist_md_write__flush() failed.");
+                        HGOTO_ERROR(H5E_CACHE, H5E_CANTGET, FAIL,
+                                    "H5AC__rsp__dist_md_write__flush() failed.");
                     break;
 
                 default:

src/H5ACproxy_entry.c

@@ -212,7 +212,8 @@ H5AC_proxy_entry_remove_parent(H5AC_proxy_entry_t *pentry, void *_parent)
     /* Remove flush dependency between the proxy entry and a parent */
     if (pentry->nchildren > 0)
         if (H5AC_destroy_flush_dependency(parent, pentry) < 0)
-            HGOTO_ERROR(H5E_CACHE, H5E_CANTUNDEPEND, FAIL, "unable to remove flush dependency on proxy entry");
+            HGOTO_ERROR(H5E_CACHE, H5E_CANTUNDEPEND, FAIL,
+                        "unable to remove flush dependency on proxy entry");
 
 done:
     FUNC_LEAVE_NOAPI(ret_value)

src/H5Adense.c

@@ -1800,7 +1800,8 @@ H5A__dense_delete(H5F_t *f, H5O_ainfo_t *ainfo)
     if (H5_addr_defined(ainfo->corder_bt2_addr)) {
         /* Delete the creation order index, without adjusting the ref. count on the attributes  */
         if (H5B2_delete(f, ainfo->corder_bt2_addr, NULL, NULL, NULL) < 0)
-            HGOTO_ERROR(H5E_ATTR, H5E_CANTDELETE, FAIL, "unable to delete v2 B-tree for creation order index");
+            HGOTO_ERROR(H5E_ATTR, H5E_CANTDELETE, FAIL,
+                        "unable to delete v2 B-tree for creation order index");
         ainfo->corder_bt2_addr = HADDR_UNDEF;
     } /* end if */
 

src/H5B.c

@@ -795,7 +795,8 @@ H5B__insert_helper(H5F_t *f, H5B_ins_ud_t *bt_ud, const H5B_class_t *type, uint8
 
     /* Get shared info for B-tree */
     if (NULL == (rc_shared = (type->get_shared)(f, udata)))
-        HGOTO_ERROR(H5E_BTREE, H5E_CANTGET, H5B_INS_ERROR, "can't retrieve B-tree's shared ref. count object");
+        HGOTO_ERROR(H5E_BTREE, H5E_CANTGET, H5B_INS_ERROR,
+                    "can't retrieve B-tree's shared ref. count object");
     shared = (H5B_shared_t *)H5UC_GET_OBJ(rc_shared);
     assert(shared);
 
@@ -886,7 +887,8 @@ H5B__insert_helper(H5F_t *f, H5B_ins_ud_t *bt_ud, const H5B_class_t *type, uint8
         /* Since we are to the left of the leftmost key there must not be a left
          * sibling */
         if (H5_addr_defined(bt->left))
-            HGOTO_ERROR(H5E_BTREE, H5E_CANTINSERT, H5B_INS_ERROR, "internal error: likely corrupt key values");
+            HGOTO_ERROR(H5E_BTREE, H5E_CANTINSERT, H5B_INS_ERROR,
+                        "internal error: likely corrupt key values");
 #endif /* H5_STRICT_FORMAT_CHECKS */
     }
     else if (cmp > 0 && idx + 1 >= bt->nchildren) {
@@ -937,7 +939,8 @@ H5B__insert_helper(H5F_t *f, H5B_ins_ud_t *bt_ud, const H5B_class_t *type, uint8
         /* Since we are to the right of the rightmost key there must not be a
          * right sibling */
         if (H5_addr_defined(bt->right))
-            HGOTO_ERROR(H5E_BTREE, H5E_CANTINSERT, H5B_INS_ERROR, "internal error: likely corrupt key values");
+            HGOTO_ERROR(H5E_BTREE, H5E_CANTINSERT, H5B_INS_ERROR,
+                        "internal error: likely corrupt key values");
 #endif /* H5_STRICT_FORMAT_CHECKS */
     }
     else if (cmp) {
@@ -1218,7 +1221,8 @@ H5B__remove_helper(H5F_t *f, haddr_t addr, const H5B_class_t *type, int level, u
 
     /* Get shared info for B-tree */
     if (NULL == (rc_shared = (type->get_shared)(f, udata)))
-        HGOTO_ERROR(H5E_BTREE, H5E_CANTGET, H5B_INS_ERROR, "can't retrieve B-tree's shared ref. count object");
+        HGOTO_ERROR(H5E_BTREE, H5E_CANTGET, H5B_INS_ERROR,
+                    "can't retrieve B-tree's shared ref. count object");
     shared = (H5B_shared_t *)H5UC_GET_OBJ(rc_shared);
     assert(shared);
 

src/H5B2hdr.c

@@ -199,7 +199,8 @@ H5B2__hdr_init(H5B2_hdr_t *hdr, const H5B2_create_t *cparam, void *ctx_udata, ui
     /* Create the callback context, if the callback exists */
     if (hdr->cls->crt_context)
         if (NULL == (hdr->cb_ctx = (*hdr->cls->crt_context)(ctx_udata)))
-            HGOTO_ERROR(H5E_BTREE, H5E_CANTCREATE, FAIL, "unable to create v2 B-tree client callback context");
+            HGOTO_ERROR(H5E_BTREE, H5E_CANTCREATE, FAIL,
+                        "unable to create v2 B-tree client callback context");
 
 done:
     if (ret_value < 0)

src/H5B2int.c

@@ -844,7 +844,8 @@ H5B2__redistribute3(H5B2_hdr_t *hdr, uint16_t depth, H5B2_internal_t *internal,
                 if (H5B2__update_child_flush_depends(hdr, depth, left_node_ptrs, (unsigned)(*left_nrec + 1),
                                                      (unsigned)(*left_nrec + moved_middle_nrec + 1),
                                                      middle_child, left_child) < 0)
-                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL, "unable to update child nodes to new parent");
+                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL,
+                                "unable to update child nodes to new parent");
 
             /* Update the current number of records in middle node */
             curr_middle_nrec = (uint16_t)(curr_middle_nrec - moved_middle_nrec);
@@ -904,7 +905,8 @@ H5B2__redistribute3(H5B2_hdr_t *hdr, uint16_t depth, H5B2_internal_t *internal,
                 if (H5B2__update_child_flush_depends(hdr, depth, right_node_ptrs, 0,
                                                      (unsigned)right_nrec_move, middle_child,
                                                      right_child) < 0)
-                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL, "unable to update child nodes to new parent");
+                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL,
+                                "unable to update child nodes to new parent");
 
             /* Update the current number of records in middle node */
             curr_middle_nrec = (uint16_t)(curr_middle_nrec - right_nrec_move);
@@ -961,7 +963,8 @@ H5B2__redistribute3(H5B2_hdr_t *hdr, uint16_t depth, H5B2_internal_t *internal,
             if (hdr->swmr_write && depth > 1)
                 if (H5B2__update_child_flush_depends(hdr, depth, middle_node_ptrs, 0,
                                                      (unsigned)left_nrec_move, left_child, middle_child) < 0)
-                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL, "unable to update child nodes to new parent");
+                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL,
+                                "unable to update child nodes to new parent");
 
             /* Update the current number of records in middle node */
             curr_middle_nrec = (uint16_t)(curr_middle_nrec + left_nrec_move);
@@ -1017,7 +1020,8 @@ H5B2__redistribute3(H5B2_hdr_t *hdr, uint16_t depth, H5B2_internal_t *internal,
                 if (H5B2__update_child_flush_depends(
                         hdr, depth, middle_node_ptrs, (unsigned)(curr_middle_nrec + 1),
                         (unsigned)(curr_middle_nrec + right_nrec_move + 1), right_child, middle_child) < 0)
-                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL, "unable to update child nodes to new parent");
+                    HGOTO_ERROR(H5E_BTREE, H5E_CANTUPDATE, FAIL,
+                                "unable to update child nodes to new parent");
 
             /* Mark nodes as dirty */
             middle_child_flags |= H5AC__DIRTIED_FLAG;

src/H5C.c

@@ -1242,7 +1242,8 @@ H5C_validate_resize_config(H5C_auto_size_ctl_t *config_ptr, unsigned int tests)
             HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL,
                         "initial_size must be in the interval [min_size, max_size]");
         if ((config_ptr->min_clean_fraction < 0.0) || (config_ptr->min_clean_fraction > 1.0))
-            HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "min_clean_fraction must be in the interval [0.0, 1.0]");
+            HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL,
+                        "min_clean_fraction must be in the interval [0.0, 1.0]");
         if (config_ptr->epoch_length < H5C__MIN_AR_EPOCH_LENGTH)
             HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, "epoch_length too small");
         if (config_ptr->epoch_length > H5C__MAX_AR_EPOCH_LENGTH)

src/H5CX.c

@@ -539,7 +539,8 @@ H5CX_init(void)
         HGOTO_ERROR(H5E_CONTEXT, H5E_CANTGET, FAIL, "Can't retrieve local cause for breaking collective I/O");
     if (H5P_get(dx_plist, H5D_MPIO_GLOBAL_NO_COLLECTIVE_CAUSE_NAME,
                 &H5CX_def_dxpl_cache.mpio_global_no_coll_cause) < 0)
-        HGOTO_ERROR(H5E_CONTEXT, H5E_CANTGET, FAIL, "Can't retrieve global cause for breaking collective I/O");
+        HGOTO_ERROR(H5E_CONTEXT, H5E_CANTGET, FAIL,
+                    "Can't retrieve global cause for breaking collective I/O");
 #endif /* H5_HAVE_PARALLEL */
 
     /* Get error detection properties */
@@ -1092,7 +1093,8 @@ H5CX_free_state(H5CX_state_t *api_state)
         if (api_state->vol_connector_prop.connector_info)
             if (H5VL_free_connector_info(api_state->vol_connector_prop.connector_id,
                                          api_state->vol_connector_prop.connector_info) < 0)
-                HGOTO_ERROR(H5E_CONTEXT, H5E_CANTRELEASE, FAIL, "unable to release VOL connector info object");
+                HGOTO_ERROR(H5E_CONTEXT, H5E_CANTRELEASE, FAIL,
+                            "unable to release VOL connector info object");
         /* Decrement connector ID */
         if (H5I_dec_ref(api_state->vol_connector_prop.connector_id) < 0)
             HDONE_ERROR(H5E_CONTEXT, H5E_CANTDEC, FAIL, "can't close VOL connector ID")

src/H5Centry.c

@@ -390,7 +390,8 @@ H5C__generate_image(H5F_t *f, H5C_t *cache_ptr, H5C_cache_entry_t *entry_ptr)
 
     if (entry_ptr->flush_dep_nparents > 0)
         if (H5C__mark_flush_dep_serialized(entry_ptr) < 0)
-            HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL, "Can't propagate serialization status to fd parents");
+            HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL,
+                        "Can't propagate serialization status to fd parents");
 
 done:
     FUNC_LEAVE_NOAPI(ret_value)
@@ -1186,7 +1187,8 @@ H5C__load_entry(H5F_t *f,
 
         /* Check for too many tries */
         if (tries == 0)
-            HGOTO_ERROR(H5E_CACHE, H5E_READERROR, NULL, "incorrect metadata checksum after all read attempts");
+            HGOTO_ERROR(H5E_CACHE, H5E_READERROR, NULL,
+                        "incorrect metadata checksum after all read attempts");
 
         /* Calculate and track the # of retries */
         retries = max_tries - tries;
@@ -2435,7 +2437,8 @@ H5C_mark_entry_dirty(void *thing)
              */
             if (entry_ptr->type->notify &&
                 (entry_ptr->type->notify)(H5C_NOTIFY_ACTION_ENTRY_DIRTIED, entry_ptr) < 0)
-                HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL, "can't notify client about entry dirty flag set");
+                HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL,
+                            "can't notify client about entry dirty flag set");
 
             /* Propagate the dirty flag up the flush dependency chain if appropriate */
             if (entry_ptr->flush_dep_nparents > 0)
@@ -2872,7 +2875,8 @@ H5C_resize_entry(void *thing, size_t new_size)
              */
             if (entry_ptr->type->notify &&
                 (entry_ptr->type->notify)(H5C_NOTIFY_ACTION_ENTRY_DIRTIED, entry_ptr) < 0)
-                HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL, "can't notify client about entry dirty flag set");
+                HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL,
+                            "can't notify client about entry dirty flag set");
 
             /* Propagate the dirty flag up the flush dependency chain if appropriate */
             if (entry_ptr->flush_dep_nparents > 0)
@@ -3569,7 +3573,8 @@ H5C_unprotect(H5F_t *f, haddr_t addr, void *thing, unsigned flags)
              */
             if (entry_ptr->type->notify &&
                 (entry_ptr->type->notify)(H5C_NOTIFY_ACTION_ENTRY_DIRTIED, entry_ptr) < 0)
-                HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL, "can't notify client about entry dirty flag set");
+                HGOTO_ERROR(H5E_CACHE, H5E_CANTNOTIFY, FAIL,
+                            "can't notify client about entry dirty flag set");
 
             /* Propagate the flush dep dirty flag up the flush dependency chain
              * if appropriate

src/H5Cimage.c

@@ -1446,7 +1446,8 @@ H5C__decode_cache_image_entry(const H5F_t *f, const H5C_t *cache_ptr, const uint
         int i; /* Local index variable */
 
         if (NULL == (fd_parent_addrs = (haddr_t *)H5MM_malloc((size_t)(fd_parent_count)*H5F_SIZEOF_ADDR(f))))
-            HGOTO_ERROR(H5E_CACHE, H5E_CANTALLOC, FAIL, "memory allocation failed for fd parent addrs buffer");
+            HGOTO_ERROR(H5E_CACHE, H5E_CANTALLOC, FAIL,
+                        "memory allocation failed for fd parent addrs buffer");
 
         for (i = 0; i < fd_parent_count; i++) {
             H5F_addr_decode(f, &p, &(fd_parent_addrs[i]));
@@ -2667,7 +2668,8 @@ H5C__reconstruct_cache_entry(const H5F_t *f, H5C_t *cache_ptr, const uint8_t **b
 
         if (NULL == (pf_entry_ptr->fd_parent_addrs = (haddr_t *)H5MM_malloc(
                          (size_t)(pf_entry_ptr->fd_parent_count) * H5F_SIZEOF_ADDR(f))))
-            HGOTO_ERROR(H5E_CACHE, H5E_CANTALLOC, NULL, "memory allocation failed for fd parent addrs buffer");
+            HGOTO_ERROR(H5E_CACHE, H5E_CANTALLOC, NULL,
+                        "memory allocation failed for fd parent addrs buffer");
 
         for (u = 0; u < pf_entry_ptr->fd_parent_count; u++) {
             H5F_addr_decode(f, &p, &(pf_entry_ptr->fd_parent_addrs[u]));

src/H5Dchunk.c

@@ -2327,7 +2327,8 @@ H5D__piece_mem_cb(void H5_ATTR_UNUSED *elem, const H5T_t H5_ATTR_UNUSED *type, u
          */
         /* Get the chunk node from the skip list */
         if (NULL == (piece_info = (H5D_piece_info_t *)H5SL_search(fm->dset_sel_pieces, &chunk_index)))
-            HGOTO_ERROR(H5E_DATASPACE, H5E_NOTFOUND, H5_ITER_ERROR, "can't locate piece in dataset skip list");
+            HGOTO_ERROR(H5E_DATASPACE, H5E_NOTFOUND, H5_ITER_ERROR,
+                        "can't locate piece in dataset skip list");
 
         /* Check if the chunk already has a memory space */
         if (NULL == piece_info->mspace)
@@ -3373,7 +3374,8 @@ H5D__chunk_io_term(H5D_io_info_t H5_ATTR_UNUSED *io_info, H5D_dset_io_info_t *di
     /* Free the memory piece dataspace template */
     if (fm->mchunk_tmpl)
         if (H5S_close(fm->mchunk_tmpl) < 0)
-            HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, FAIL, "can't release memory chunk dataspace template");
+            HGOTO_ERROR(H5E_DATASPACE, H5E_CANTRELEASE, FAIL,
+                        "can't release memory chunk dataspace template");
 
     /* Free chunk map */
     di->layout_io_info.chunk_map = H5FL_FREE(H5D_chunk_map_t, di->layout_io_info.chunk_map);
@@ -3934,7 +3936,8 @@ H5D__chunk_flush_entry(const H5D_t *dset, H5D_rdcc_ent_t *ent, hbool_t reset)
              */
             if (H5D__chunk_file_alloc(&idx_info, &(ent->chunk_block), &udata.chunk_block, &need_insert,
                                       ent->scaled) < 0)
-                HGOTO_ERROR(H5E_DATASET, H5E_CANTINSERT, FAIL, "unable to insert/resize chunk on chunk level");
+                HGOTO_ERROR(H5E_DATASET, H5E_CANTINSERT, FAIL,
+                            "unable to insert/resize chunk on chunk level");
 
             /* Update the chunk entry's info, in case it was allocated or relocated */
             ent->chunk_block.offset = udata.chunk_block.offset;
@@ -5113,7 +5116,8 @@ H5D__chunk_allocate(const H5D_t *dset, hbool_t full_overwrite, const hsize_t old
 
             /* Allocate the chunk (with all processes) */
             if (H5D__chunk_file_alloc(&idx_info, NULL, &udata.chunk_block, &need_insert, scaled) < 0)
-                HGOTO_ERROR(H5E_DATASET, H5E_CANTINSERT, FAIL, "unable to insert/resize chunk on chunk level");
+                HGOTO_ERROR(H5E_DATASET, H5E_CANTINSERT, FAIL,
+                            "unable to insert/resize chunk on chunk level");
             assert(H5_addr_defined(udata.chunk_block.offset));
 
             /* Check if fill values should be written to chunks */
@@ -6315,7 +6319,8 @@ H5D__chunk_addrmap(const H5D_t *dset, haddr_t chunk_addr[])
 
     /* Iterate over chunks to build mapping of chunk addresses */
     if ((sc->ops->iterate)(&idx_info, H5D__chunk_addrmap_cb, &udata) < 0)
-        HGOTO_ERROR(H5E_DATASET, H5E_CANTGET, FAIL, "unable to iterate over chunk index to build address map");
+        HGOTO_ERROR(H5E_DATASET, H5E_CANTGET, FAIL,
+                    "unable to iterate over chunk index to build address map");
 
 done:
     FUNC_LEAVE_NOAPI(ret_value)

src/H5Dfill.c

@@ -166,7 +166,8 @@ H5D__fill(const void *fill, const H5T_t *fill_type, void *buf, const H5T_t *buf_
 
         /* Set up type conversion function */
         if (NULL == (tpath = H5T_path_find(fill_type, buf_type)))
-            HGOTO_ERROR(H5E_DATASET, H5E_UNSUPPORTED, FAIL, "unable to convert between src and dest datatype");
+            HGOTO_ERROR(H5E_DATASET, H5E_UNSUPPORTED, FAIL,
+                        "unable to convert between src and dest datatype");
 
         /* Construct source & destination datatype IDs, if we will need them */
         if (!H5T_path_noop(tpath)) {

src/H5Dint.c

@@ -1730,7 +1730,8 @@ H5D__open_oid(H5D_t *dataset, hid_t dapl_id)
         assert(H5D_COMPACT == dataset->shared->layout.storage.type);
 
         if ((dset_nelemts = H5S_GET_EXTENT_NPOINTS(dataset->shared->space)) < 0)
-            HGOTO_ERROR(H5E_DATASET, H5E_CANTGET, FAIL, "can't get number of elements in dataset's dataspace");
+            HGOTO_ERROR(H5E_DATASET, H5E_CANTGET, FAIL,
+                        "can't get number of elements in dataset's dataspace");
 
         dset_data_size = (size_t)dset_nelemts * dset_type_size;
 
@@ -2255,7 +2256,8 @@ H5D__alloc_storage(H5D_t *dset, H5D_time_alloc_t time_alloc, hbool_t full_overwr
                     if (layout->storage.u.contig.size > 0) {
                         /* Reserve space in the file for the entire array */
                         if (H5D__contig_alloc(f, &layout->storage.u.contig /*out*/) < 0)
-                            HGOTO_ERROR(H5E_IO, H5E_CANTINIT, FAIL, "unable to initialize contiguous storage");
+                            HGOTO_ERROR(H5E_IO, H5E_CANTINIT, FAIL,
+                                        "unable to initialize contiguous storage");
 
                         /* Indicate that we should initialize storage space */
                         must_init_space = TRUE;
@@ -3026,7 +3028,8 @@ H5D__set_extent(H5D_t *dset, const hsize_t *size)
 
                 /* Compute the scaled dimension size value */
                 if (dset->shared->layout.u.chunk.dim[dim_idx] == 0)
-                    HGOTO_ERROR(H5E_DATASET, H5E_BADVALUE, FAIL, "chunk size must be > 0, dim = %u ", dim_idx);
+                    HGOTO_ERROR(H5E_DATASET, H5E_BADVALUE, FAIL, "chunk size must be > 0, dim = %u ",
+                                dim_idx);
 
                 scaled = size[dim_idx] / dset->shared->layout.u.chunk.dim[dim_idx];