Fix typos found in the rest of the hdf5 code-base (#1985)

* Fix typos found in the rest of the hdf5 code-base

* Typo in tool.cmake

* Revert "Typo in tool.cmake"

This reverts commit 06b8403d99757c7d20f1eed530cf317c3d84de4b.

* Fix clang-format styling

18 files changed, 35 insertions, 36 deletions

diff --git a/test/cache.c b/test/cache.c
index 3c6c818..e564289 100644
--- a/test/cache.c
+++ b/test/cache.c
@@ -9861,7 +9861,7 @@ check_flush_cache__flush_op_eviction_test(H5F_t *file_ptr)
  *         Added tests for pinned entries.
  *
  *         JRM -- 5/17/06
- *         Complete reqrite of pinned entry tests to accommodate
+ *         Complete rewrite of pinned entry tests to accommodate
  *         the new H5C_mark_entry_dirty() call.
  *
  *-------------------------------------------------------------------------
@@ -33494,7 +33494,7 @@ cedds__H5C_flush_invalidate_cache__bucket_scan(H5F_t *file_ptr)
 
     if (pass) {
 
-        /* load the entry that will have a flush dependencey with (MET, 0),
+        /* load the entry that will have a flush dependency with (MET, 0),
          * thus preventing it from being flushed on the first pass through
          * the skip list.
          */
diff --git a/test/cache_common.c b/test/cache_common.c
index 5146073..8a56027 100644
--- a/test/cache_common.c
+++ b/test/cache_common.c
@@ -922,7 +922,7 @@ notify_image_len(const void *thing, size_t *image_length)
 /*-------------------------------------------------------------------------
  * Function:    pre_serialize & friends
  *
- * Purpose:    Pre_serialize the supplied entry.  For now this consistes of
+ * Purpose:    Pre_serialize the supplied entry.  For now this consists of
  *         executing any flush operations and loading the appropriate
  *        values into *new_addr_ptr, *new_len_ptr, and *flags_ptr.
  *
@@ -1086,7 +1086,7 @@ notify_pre_serialize(H5F_t *f, void *thing, haddr_t addr, size_t len, haddr_t *n
 /*-------------------------------------------------------------------------
  * Function:    serialize & friends
  *
- * Purpose:    Serialize the supplied entry.  For now this consistes of
+ * Purpose:    Serialize the supplied entry.  For now this consists of
  *         loading the type and index of the entry into the first
  *         three bytes of the image (if it is long enough -- if not
  *         just load the low order byte of the index into the first
diff --git a/test/cache_common.h b/test/cache_common.h
index acc05f0..24ce509 100644
--- a/test/cache_common.h
+++ b/test/cache_common.h
@@ -253,7 +253,7 @@ typedef struct flush_op {
 
 typedef enum test_entry_action_t {
     TEST_ENTRY_ACTION_NUL = 0, /* No action on entry */
-    TEST_ENTRY_ACTION_MOVE     /* Entry is beging moved */
+    TEST_ENTRY_ACTION_MOVE     /* Entry is being moved */
 } test_entry_action_t;
 
 typedef struct test_entry_t {
@@ -372,7 +372,7 @@ typedef struct test_entry_t {
                                                       * been resized by a resize flush
                                                       * op, and the flush function has
                                                       * not yet returned,  This field is
-                                                      * used to turn off overactive santity
+                                                      * used to turn off overactive sanity
                                                       * checking code that would otherwise
                                                       * cause a false test failure.
                                                       */
diff --git a/test/chunk_info.c b/test/chunk_info.c
index 3fdc77d..877a182 100644
--- a/test/chunk_info.c
+++ b/test/chunk_info.c
@@ -922,7 +922,7 @@ test_chunk_info_single_chunk(const char *filename, hid_t fapl)
     if (ret != FAIL)
         TEST_ERROR;
 
-    /* Release resourse */
+    /* Release resource */
     if (H5Dclose(dset) < 0)
         TEST_ERROR;
     if (H5Sclose(dspace) < 0)
@@ -1039,7 +1039,7 @@ test_chunk_info_implicit(char *filename, hid_t fapl)
                 FAIL_PUTS_ERROR("Verification of H5Dget_chunk_info_by_coord failed\n");
         }
 
-    /* Release resourse */
+    /* Release resource */
     if (H5Dclose(dset) < 0)
         TEST_ERROR;
     if (H5Sclose(dspace) < 0)
@@ -1184,7 +1184,7 @@ test_chunk_info_fixed_array(const char *filename, hid_t fapl)
     /* Read and verify values of selected chunks */
     if (verify_selected_chunks(dset, H5P_DEFAULT, start, end) < 0)
 
-        /* Release resourse */
+        /* Release resource */
         if (H5Dclose(dset) < 0)
             TEST_ERROR;
     if (H5Sclose(dspace) < 0)
@@ -1333,7 +1333,7 @@ test_chunk_info_extensible_array(const char *filename, hid_t fapl)
     /* Read and verify values of selected chunks */
     if (verify_selected_chunks(dset, H5P_DEFAULT, start, end) < 0)
 
-        /* Release resourse */
+        /* Release resource */
         if (H5Dclose(dset) < 0)
             TEST_ERROR;
     if (H5Sclose(dspace) < 0)
@@ -1482,7 +1482,7 @@ test_chunk_info_version2_btrees(const char *filename, hid_t fapl)
     /* Read and verify values of selected chunks */
     if (verify_selected_chunks(dset, H5P_DEFAULT, start, end) < 0)
 
-        /* Release resourse */
+        /* Release resource */
         if (H5Dclose(dset) < 0)
             TEST_ERROR;
     if (H5Sclose(dspace) < 0)
@@ -1746,7 +1746,7 @@ test_basic_query(hid_t fapl)
     if (cptr != &(chunk_infos[1]))
         FAIL_PUTS_ERROR("Verification of halted iterator failed");
 
-    /* Release resourse */
+    /* Release resource */
     if (H5Dclose(dset) < 0)
         TEST_ERROR;
     if (H5Sclose(dspace) < 0)
@@ -1875,7 +1875,7 @@ test_failed_attempts(const char *filename, hid_t fapl)
     if (ret != FAIL)
         FAIL_PUTS_ERROR("    Attempt a chunk query function on a contiguous dataset.");
 
-    /* Release resourse */
+    /* Release resource */
     if (H5Dclose(dset) < 0)
         TEST_ERROR;
     if (H5Sclose(dspace) < 0)
@@ -2151,7 +2151,7 @@ test_flt_msk_with_skip_compress(hid_t fapl)
     if (verify_get_chunk_info_by_coord(dset, offset, CHK_SIZE, flt_msk) == FAIL)
         FAIL_PUTS_ERROR("Verification of H5Dget_chunk_info_by_coord failed\n");
 
-    /* Release resourse */
+    /* Release resource */
     if (H5Dclose(dset) < 0)
         TEST_ERROR;
     if (H5Sclose(mem_space) < 0)
diff --git a/test/dsets.c b/test/dsets.c
index 6397d3d..d26e2c7 100644
--- a/test/dsets.c
+++ b/test/dsets.c
@@ -1827,7 +1827,7 @@ filter_corrupt(unsigned int flags, size_t cd_nelmts, const unsigned int *cd_valu
         TEST_ERROR;
     HDmemset(data, (int)value, (size_t)length);
 
-    if (flags & H5Z_FLAG_REVERSE) { /* Varify data is actually corrupted during read */
+    if (flags & H5Z_FLAG_REVERSE) { /* Verify data is actually corrupted during read */
         dst += offset;
         if (HDmemcmp(data, dst, (size_t)length) != 0)
             TEST_ERROR;
@@ -7722,7 +7722,7 @@ test_missing_chunk(hid_t file)
     if (H5Dread(did2, H5T_NATIVE_INT, H5S_ALL, H5S_ALL, H5P_DEFAULT, rdata2_bytes) < 0)
         TEST_ERROR;
 
-    /* Validata values read for the 1-D dataset */
+    /* Validate values read for the 1-D dataset */
     for (u = 0; u < MISSING_CHUNK_DIM; u++) {
         if ((u % 10) >= 5) {
             if (rdata[u] != 911) {
@@ -7739,7 +7739,7 @@ test_missing_chunk(hid_t file)
         }     /* end else */
     }         /* end for */
 
-    /* Validata values read for the 2-D dataset */
+    /* Validate values read for the 2-D dataset */
     for (i = 0; i < MISSING_CHUNK_DIM; i++) {
         for (j = 0; j < MISSING_CHUNK_DIM; j++) {
 
diff --git a/test/file_image.c b/test/file_image.c
index d25f131..a36892a 100644
--- a/test/file_image.c
+++ b/test/file_image.c
@@ -610,7 +610,7 @@ test_core(void)
     VERIFY((udata->used_callbacks == MALLOC) || (udata->used_callbacks == (MALLOC | UDATA_COPY | UDATA_FREE)),
            "opening a core file used the wrong callbacks");
     VERIFY(udata->malloc_src == H5FD_FILE_IMAGE_OP_FILE_OPEN,
-           "Malloc callback came from wrong sourc in core open");
+           "Malloc callback came from wrong source in core open");
 
     /* Close file */
     reset_udata(udata);
@@ -618,7 +618,7 @@ test_core(void)
     VERIFY(ret >= 0, "H5Fclose failed");
     VERIFY(udata->used_callbacks == FREE, "Closing a core file used the wrong callbacks");
     VERIFY(udata->free_src == H5FD_FILE_IMAGE_OP_FILE_CLOSE,
-           "Free callback came from wrong sourc in core close");
+           "Free callback came from wrong source in core close");
 
     /* Reopen file */
     file = H5Fopen(copied_filename, H5F_ACC_RDWR, fapl);
@@ -654,7 +654,7 @@ test_core(void)
     VERIFY(ret >= 0, "H5Fclose failed");
     VERIFY(udata->used_callbacks == (FREE), "Closing a core file used the wrong callbacks");
     VERIFY(udata->free_src == H5FD_FILE_IMAGE_OP_FILE_CLOSE,
-           "Free callback came from wrong sourc in core close");
+           "Free callback came from wrong source in core close");
 
     /* Create file image buffer */
     fd = HDopen(copied_filename, O_RDONLY);
diff --git a/test/h5test.c b/test/h5test.c
index 3df2807..87758f6 100644
--- a/test/h5test.c
+++ b/test/h5test.c
@@ -1386,7 +1386,7 @@ h5_szip_can_encode(void)
  *     further calls to getenv_all take place.
  *
  * Return:  pointer to a string containing the value of the environment variable
- *     NULL if the varialbe doesn't exist in task 'root's environment.
+ *     NULL if the variable doesn't exist in task 'root's environment.
  *
  * Programmer:  Leon Arber
  *              4/4/05
diff --git a/test/hdfs.c b/test/hdfs.c
index f114ed1..06cd7cc 100644
--- a/test/hdfs.c
+++ b/test/hdfs.c
@@ -52,7 +52,7 @@
  *     4) Configurable expected-actual order in generated comparison strings.
  *        Some prefer `VERIFY(expected, actual)`, others
  *        `VERIFY(actual, expected)`. Provide preprocessor ifdef switch
- *        to satifsy both parties, assuming one paradigm per test file.
+ *        to satisfy both parties, assuming one paradigm per test file.
  *        (One could #undef and redefine the flag through the file as desired,
  *         but _why_.)
  *
diff --git a/test/objcopy.c b/test/objcopy.c
index beb272b..72ebbcc 100644
--- a/test/objcopy.c
+++ b/test/objcopy.c
@@ -3998,7 +3998,7 @@ test_copy_dataset_chunked_empty(hid_t fcpl_src, hid_t fcpl_dst, hid_t src_fapl,
     if ((did = H5Dopen2(fid_src, NAME_DATASET_CHUNKED3, H5P_DEFAULT)) < 0)
         TEST_ERROR;
 
-    /* open the copied dataset NAME_DATASET_CHUNKED3 at destinaion */
+    /* open the copied dataset NAME_DATASET_CHUNKED3 at destination */
     if ((did2 = H5Dopen2(fid_dst, NAME_DATASET_CHUNKED3, H5P_DEFAULT)) < 0)
         TEST_ERROR;
 
@@ -11069,7 +11069,7 @@ typedef struct cmpd_vl_t {
  *              Failure:        number of errors
  *
  * Programmer:  Neil Fortner
- *              Tuseday, September 29, 2009
+ *              Tuesday, September 29, 2009
  *
  *-------------------------------------------------------------------------
  */
diff --git a/test/onion.c b/test/onion.c
index 96a4970..e731b6f 100644
--- a/test/onion.c
+++ b/test/onion.c
@@ -2120,7 +2120,7 @@ test_create_oniontarget(hbool_t truncate_canonical, hbool_t with_initial_data)
         TEST_ERROR;
     vfile_raw = NULL;
 
-    /* R/O open the file with Onion VFD; inspect logial file */
+    /* R/O open the file with Onion VFD; inspect logical file */
 
     vfile_ro = H5FDopen(paths->canon, H5F_ACC_RDONLY, fapl_id, HADDR_UNDEF);
     if (NULL == vfile_ro)
diff --git a/test/reserved.c b/test/reserved.c
index 08e747e..22f1764 100644
--- a/test/reserved.c
+++ b/test/reserved.c
@@ -509,8 +509,7 @@ main(void)
         return num_errs;
     }
     else {
-        HDputs(
-            "All address space reservation tests skippped - Incompatible with current Virtual File Driver");
+        HDputs("All address space reservation tests skipped - Incompatible with current Virtual File Driver");
     }
 #endif /* BROKEN */
 
diff --git a/test/ros3.c b/test/ros3.c
index 2bfca96..bc9eb86 100644
--- a/test/ros3.c
+++ b/test/ros3.c
@@ -56,7 +56,7 @@
  *     4) Configurable expected-actual order in generated comparison strings.
  *        Some prefer `VERIFY(expected, actual)`, others
  *        `VERIFY(actual, expected)`. Provide preprocessor ifdef switch
- *        to satifsy both parties, assuming one paradigm per test file.
+ *        to satisfy both parties, assuming one paradigm per test file.
  *        (One could #undef and redefine the flag through the file as desired,
  *         but _why_.)
  *
diff --git a/test/s3comms.c b/test/s3comms.c
index d3998f3..d68cc84 100644
--- a/test/s3comms.c
+++ b/test/s3comms.c
@@ -47,7 +47,7 @@
  *     4) Configurable expected-actual order in generated comparison strings.
  *        Some prefer `VERIFY(expected, actual)`, others
  *        `VERIFY(actual, expected)`. Provide preprocessor ifdef switch
- *        to satifsy both parties, assuming one paradigm per test file.
+ *        to satisfy both parties, assuming one paradigm per test file.
  *        (One could #undef and redefine the flag through the file as desired,
  *         but _why_.)
  *        Provided as courtesy, per consideration for inclusion in the library
@@ -1374,7 +1374,7 @@ test_nlowercase(void)
         HDfree(dest);
     } /* end for each testcase */
 
-    JSVERIFY(FAIL, H5FD_s3comms_nlowercase(NULL, cases[0].in, cases[0].len), "null distination should fail")
+    JSVERIFY(FAIL, H5FD_s3comms_nlowercase(NULL, cases[0].in, cases[0].len), "null destination should fail")
 
     PASSED();
     return 0;
diff --git a/test/swmr_common.c b/test/swmr_common.c
index 9063ddd..159f068 100644
--- a/test/swmr_common.c
+++ b/test/swmr_common.c
@@ -190,7 +190,7 @@ generate_name(char *name_buf, size_t name_buf_length, unsigned level, unsigned c
 /*-------------------------------------------------------------------------
  * Function:    generate_symbols
  *
- * Purpose:     Initializes the global dataset infomration arrays.
+ * Purpose:     Initializes the global dataset information arrays.
  *
  * Parameters:  N/A
  *
diff --git a/test/swmr_sparse_writer.c b/test/swmr_sparse_writer.c
index 96eff6c..0c8e391 100644
--- a/test/swmr_sparse_writer.c
+++ b/test/swmr_sparse_writer.c
@@ -149,7 +149,7 @@ add_records(hid_t fid, unsigned verbose, unsigned long nrecords, unsigned long f
     symbol_t      record;            /* The record to add to the dataset */
     unsigned long rec_to_flush;      /* # of records left to write before flush */
 #ifdef OUT
-    volatile int dummy;            /* Dummy varialbe for busy sleep */
+    volatile int dummy;            /* Dummy variable for busy sleep */
 #endif                             /* OUT */
     hsize_t       dim[2] = {1, 0}; /* Dataspace dimensions */
     unsigned long u, v;            /* Local index variables */
diff --git a/test/tfile.c b/test/tfile.c
index f8e0a0a..f122f68 100644
--- a/test/tfile.c
+++ b/test/tfile.c
@@ -1153,7 +1153,7 @@ test_get_obj_ids(void)
     CHECK(oid_count, FAIL, "H5Fget_obj_count");
     VERIFY(oid_count, NGROUPS + 2, "H5Fget_obj_count");
 
-    /* Get the IDs of the left opend objects */
+    /* Get the IDs of the left opened objects */
     ret_count = H5Fget_obj_ids(fid, H5F_OBJ_ALL, (size_t)oid_list_size, oid_list);
     CHECK(ret_count, FAIL, "H5Fget_obj_ids");
     VERIFY(ret_count, oid_list_size, "H5Fget_obj_count");
diff --git a/test/tselect.c b/test/tselect.c
index cc5f96d..c402f3f 100644
--- a/test/tselect.c
+++ b/test/tselect.c
@@ -6785,7 +6785,7 @@ test_select_point_chunk(void)
 
     /* Verify data (later) */
 
-    /* Close everything (inclusing selections) */
+    /* Close everything (including selections) */
     ret = H5Sclose(pnt1_space);
     CHECK(ret, FAIL, "H5Sclose");
     ret = H5Sclose(pnt2_space);
@@ -10957,7 +10957,7 @@ test_shape_same_dr__full_space_vs_slice(int test_num, int small_rank, int large_
 **         m-cube (m >= n).
 **
 **    2) The dimensions selected in the slice through the m-cube
-**       are the dimesnions with the most quickly changing
+**       are the dimensions with the most quickly changing
 **       indices.
 **
 ****************************************************************/
diff --git a/test/twriteorder.c b/test/twriteorder.c
index c58b030..c7984e7 100644
--- a/test/twriteorder.c
+++ b/test/twriteorder.c
@@ -294,7 +294,7 @@ write_wo_file(void)
         return -1;
     }
 
-    /* all writes done. return succeess. */
+    /* all writes done. return success. */
 #ifdef DEBUG
     HDprintf("wrote %d blocks\n", nlinkedblock_g);
 #endif