Made HGOTO_ERROR a do-while loop (#3308)

* Made HGOTO_ERROR a do-while loop

1 files changed, 14 insertions, 14 deletions

diff --git a/src/H5Torder.c b/src/H5Torder.c
index 36c2276..ccdf4bf 100644
--- a/src/H5Torder.c
+++ b/src/H5Torder.c
@@ -85,11 +85,11 @@ H5Tget_order(hid_t type_id)
 
     /* Check args */
     if (NULL == (dt = (H5T_t *)H5I_object_verify(type_id, H5I_DATATYPE)))
-        HGOTO_ERROR(H5E_DATATYPE, H5E_BADTYPE, H5T_ORDER_ERROR, "not a datatype")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_BADTYPE, H5T_ORDER_ERROR, "not a datatype");
 
     /* Get order */
     if (H5T_ORDER_ERROR == (ret_value = H5T_get_order(dt)))
-        HGOTO_ERROR(H5E_DATATYPE, H5E_UNSUPPORTED, H5T_ORDER_ERROR, "can't get order for specified datatype")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_UNSUPPORTED, H5T_ORDER_ERROR, "can't get order for specified datatype");
 
 done:
     FUNC_LEAVE_API(ret_value)
@@ -129,14 +129,14 @@ H5T_get_order(const H5T_t *dtype)
             /* Retrieve the number of members */
             if ((nmemb = H5T_get_nmembers(dtype)) < 0)
                 HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, H5T_ORDER_ERROR,
-                            "can't get number of members from compound data type")
+                            "can't get number of members from compound data type");
 
             /* Get order for each compound member type. */
             for (i = 0; i < nmemb; i++) {
                 /* Get order for member */
                 if ((memb_order = H5T_get_order(dtype->shared->u.compnd.memb[i].type)) == H5T_ORDER_ERROR)
                     HGOTO_ERROR(H5E_DATATYPE, H5E_CANTGET, H5T_ORDER_ERROR,
-                                "can't get order for compound member")
+                                "can't get order for compound member");
 
                 /* Ignore the H5T_ORDER_NONE, write down the first non H5T_ORDER_NONE order. */
                 if (memb_order != H5T_ORDER_NONE && ret_value == H5T_ORDER_NONE)
@@ -185,17 +185,17 @@ H5Tset_order(hid_t type_id, H5T_order_t order)
 
     /* Check args */
     if (NULL == (dt = (H5T_t *)H5I_object_verify(type_id, H5I_DATATYPE)))
-        HGOTO_ERROR(H5E_DATATYPE, H5E_BADTYPE, FAIL, "not a datatype")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_BADTYPE, FAIL, "not a datatype");
     if (order < H5T_ORDER_LE || order > H5T_ORDER_NONE || order == H5T_ORDER_MIXED)
-        HGOTO_ERROR(H5E_DATATYPE, H5E_BADVALUE, FAIL, "illegal byte order")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_BADVALUE, FAIL, "illegal byte order");
     if (NULL != dt->vol_obj)
-        HGOTO_ERROR(H5E_ARGS, H5E_CANTSET, FAIL, "datatype is already committed")
+        HGOTO_ERROR(H5E_ARGS, H5E_CANTSET, FAIL, "datatype is already committed");
     if (H5T_STATE_TRANSIENT != dt->shared->state)
-        HGOTO_ERROR(H5E_DATATYPE, H5E_CANTINIT, FAIL, "datatype is read-only")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_CANTINIT, FAIL, "datatype is read-only");
 
     /* Call internal routine to set the order */
     if (H5T__set_order(dt, order) < 0)
-        HGOTO_ERROR(H5E_DATATYPE, H5E_UNSUPPORTED, FAIL, "can't set order")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_UNSUPPORTED, FAIL, "can't set order");
 
 done:
     FUNC_LEAVE_API(ret_value)
@@ -218,7 +218,7 @@ H5T__set_order(H5T_t *dtype, H5T_order_t order)
     FUNC_ENTER_PACKAGE
 
     if (H5T_ENUM == dtype->shared->type && dtype->shared->u.enumer.nmembs > 0)
-        HGOTO_ERROR(H5E_DATATYPE, H5E_CANTSET, FAIL, "operation not allowed after enum members are defined")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_CANTSET, FAIL, "operation not allowed after enum members are defined");
 
     /* For derived data type, defer to parent */
     while (dtype->shared->parent)
@@ -227,7 +227,7 @@ H5T__set_order(H5T_t *dtype, H5T_order_t order)
     /* Check for setting order on inappropriate datatype */
     if (order == H5T_ORDER_NONE && !(H5T_REFERENCE == dtype->shared->type ||
                                      H5T_OPAQUE == dtype->shared->type || H5T_IS_FIXED_STRING(dtype->shared)))
-        HGOTO_ERROR(H5E_DATATYPE, H5E_BADVALUE, FAIL, "illegal byte order for type")
+        HGOTO_ERROR(H5E_DATATYPE, H5E_BADVALUE, FAIL, "illegal byte order for type");
 
     /* For atomic data type */
     if (H5T_IS_ATOMIC(dtype->shared))
@@ -241,16 +241,16 @@ H5T__set_order(H5T_t *dtype, H5T_order_t order)
             /* Retrieve the number of fields in the compound datatype */
             if ((nmemb = H5T_get_nmembers(dtype)) < 0)
                 HGOTO_ERROR(H5E_DATATYPE, H5E_CANTGET, FAIL,
-                            "can't get number of members from compound data type")
+                            "can't get number of members from compound data type");
 
             /* Check for uninitialized compound datatype */
             if (nmemb == 0)
-                HGOTO_ERROR(H5E_DATATYPE, H5E_UNINITIALIZED, FAIL, "no member is in the compound data type")
+                HGOTO_ERROR(H5E_DATATYPE, H5E_UNINITIALIZED, FAIL, "no member is in the compound data type");
 
             /* Loop through all fields of compound type, setting the order */
             for (i = 0; i < nmemb; i++)
                 if (H5T__set_order(dtype->shared->u.compnd.memb[i].type, order) < 0)
-                    HGOTO_ERROR(H5E_DATATYPE, H5E_CANTSET, FAIL, "can't set order for compound member")
+                    HGOTO_ERROR(H5E_DATATYPE, H5E_CANTSET, FAIL, "can't set order for compound member");
         } /* end if */
     }     /* end else */