Convert hbool_t --> bool in src (#3496)

* hbool_t --> bool in src

* Does not remove TRUE/FALSE
* Public header files are unchanged
* Public API calls are unchanged

* TRUE/FALSE --> true/false in src

* Add deprecation notice for hbool_t

1 files changed, 32 insertions, 32 deletions

diff --git a/src/H5Fquery.c b/src/H5Fquery.c
index b74884e..af120a4 100644
--- a/src/H5Fquery.c
+++ b/src/H5Fquery.c
@@ -236,10 +236,10 @@ H5F_get_shared(const H5F_t *f)
  *
  * Purpose:  Determine if two files have the same shared file pointer
  *
- * Return:   TRUE/FALSE on success/abort on failure (shouldn't fail)
+ * Return:   true/false on success/abort on failure (shouldn't fail)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_same_shared(const H5F_t *f1, const H5F_t *f2)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -277,10 +277,10 @@ H5F_get_nopen_objs(const H5F_t *f)
  *
  * Purpose:     Determines if a file ID exists for this file struct
  *
- * Return:      TRUE/FALSE
+ * Return:      true/false
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_file_id_exists(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -503,10 +503,10 @@ H5F_sym_leaf_k(const H5F_t *f)
  *
  * Purpose:  Get the setting flag for minimized dataset object headers
  *
- * Return:   TRUE/FALSE as set in file
+ * Return:   true/false as set in file
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_get_min_dset_ohdr(const H5F_t *f)
 {
     FUNC_ENTER_NOAPI_NOINIT_NOERR
@@ -761,7 +761,7 @@ H5F_get_fc_degree(const H5F_t *f)
  *              Failure:    (can't happen)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_get_evict_on_close(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -782,7 +782,7 @@ H5F_get_evict_on_close(const H5F_t *f)
  *           Failure:    (can't happen)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_store_msg_crt_idx(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -799,11 +799,11 @@ H5F_store_msg_crt_idx(const H5F_t *f)
  *
  * Purpose:  Check if a file has a particular feature enabled
  *
- * Return:   Success:    Non-negative - TRUE or FALSE
+ * Return:   Success:    Non-negative - true or false
  *           Failure:    Negative (should not happen)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_shared_has_feature(const H5F_shared_t *f_sh, unsigned feature)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -811,7 +811,7 @@ H5F_shared_has_feature(const H5F_shared_t *f_sh, unsigned feature)
 
     assert(f_sh);
 
-    FUNC_LEAVE_NOAPI((hbool_t)(f_sh->lf->feature_flags & feature))
+    FUNC_LEAVE_NOAPI((bool)(f_sh->lf->feature_flags & feature))
 } /* end H5F_shared_has_feature() */
 
 /*-------------------------------------------------------------------------
@@ -819,11 +819,11 @@ H5F_shared_has_feature(const H5F_shared_t *f_sh, unsigned feature)
  *
  * Purpose:  Check if a file has a particular feature enabled
  *
- * Return:   Success:    Non-negative - TRUE or FALSE
+ * Return:   Success:    Non-negative - true or false
  *           Failure:    Negative (should not happen)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_has_feature(const H5F_t *f, unsigned feature)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -832,7 +832,7 @@ H5F_has_feature(const H5F_t *f, unsigned feature)
     assert(f);
     assert(f->shared);
 
-    FUNC_LEAVE_NOAPI((hbool_t)(f->shared->lf->feature_flags & feature))
+    FUNC_LEAVE_NOAPI((bool)(f->shared->lf->feature_flags & feature))
 } /* end H5F_has_feature() */
 
 /*-------------------------------------------------------------------------
@@ -999,10 +999,10 @@ done:
  *           (Mainly added to stop non-file routines from poking about in the
  *           H5F_t data structure)
  *
- * Return:   TRUE/FALSE on success/abort on failure (shouldn't fail)
+ * Return:   true/false on success/abort on failure (shouldn't fail)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_is_tmp_addr(const H5F_t *f, haddr_t addr)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -1022,10 +1022,10 @@ H5F_is_tmp_addr(const H5F_t *f, haddr_t addr)
  *           (Mainly added to stop non-file routines from poking about in the
  *           H5F_t data structure)
  *
- * Return:   TRUE/FALSE on success/abort on failure (shouldn't fail)
+ * Return:   true/false on success/abort on failure (shouldn't fail)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_use_tmp_space(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -1070,7 +1070,7 @@ H5F_coll_md_read(const H5F_t *f)
  *-------------------------------------------------------------------------
  */
 herr_t
-H5F_shared_get_mpi_file_sync_required(const H5F_shared_t *f_sh, hbool_t *flag /*out*/)
+H5F_shared_get_mpi_file_sync_required(const H5F_shared_t *f_sh, bool *flag /*out*/)
 {
     herr_t ret_value = SUCCEED; /* Return value */
 
@@ -1096,10 +1096,10 @@ done:
  *           (Mainly added to stop non-file routines from poking about in the
  *           H5F_t data structure)
  *
- * Return:   TRUE/FALSE on success/abort on failure (shouldn't fail)
+ * Return:   true/false on success/abort on failure (shouldn't fail)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_use_mdc_logging(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -1119,10 +1119,10 @@ H5F_use_mdc_logging(const H5F_t *f)
  *           (Mainly added to stop non-file routines from poking about in the
  *           H5F_t data structure)
  *
- * Return:   TRUE/FALSE on success/abort on failure (shouldn't fail)
+ * Return:   true/false on success/abort on failure (shouldn't fail)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_start_mdc_log_on_access(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -1142,7 +1142,7 @@ H5F_start_mdc_log_on_access(const H5F_t *f)
  *           (Mainly added to stop non-file routines from poking about in the
  *           H5F_t data structure)
  *
- * Return:   TRUE/FALSE on success/abort on failure (shouldn't fail)
+ * Return:   true/false on success/abort on failure (shouldn't fail)
  *-------------------------------------------------------------------------
  */
 char *
@@ -1229,7 +1229,7 @@ H5F_get_pgend_meta_thres(const H5F_t *f)
  *           Failure:    (can't happen)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_get_point_of_no_return(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -1250,7 +1250,7 @@ H5F_get_point_of_no_return(const H5F_t *f)
  *           Failure:    (can't happen)
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_get_null_fsm_addr(const H5F_t *f)
 {
     /* Use FUNC_ENTER_NOAPI_NOINIT_NOERR here to avoid performance issues */
@@ -1340,11 +1340,11 @@ done:
  *
  * Purpose:  Get the file locking flag for the file
  *
- * Return:   TRUE/FALSE
+ * Return:   true/false
  *
  *-------------------------------------------------------------------------
  */
-hbool_t
+bool
 H5F_get_file_locking(const H5F_t *f)
 {
     FUNC_ENTER_NOAPI_NOINIT_NOERR
@@ -1360,14 +1360,14 @@ H5F_get_file_locking(const H5F_t *f)
  *
  * Purpose:  Determine if vector or selection I/O is supported by this file
  *
- * Return:   TRUE/FALSE
+ * Return:   true/false
  *
  *-------------------------------------------------------------------------
  */
-hbool_t
-H5F_has_vector_select_io(const H5F_t *f, hbool_t is_write)
+bool
+H5F_has_vector_select_io(const H5F_t *f, bool is_write)
 {
-    hbool_t ret_value; /* Return value */
+    bool ret_value; /* Return value */
 
     FUNC_ENTER_NOAPI_NOINIT_NOERR