[svn-r19765] Purpose:

    fix a few minor issues related to flush/refresh code

Description:

    - Replaced the globality macros with an enum typedef, to
      better encapsulate of the values that can be assigned
      to the variable.
    - Combined 'tag' and 'globality' properties into a single
      property by creating a new structure housing both values,
      since they are always accessed at the same time anyways.
    - Added an extra parameter check to H5Tflush/H5Trefresh
      routines to make sure the supplied datatype is a committed
      datatype.
    - Renamed and reworked some code to move "metadata" (H5AC)
      specific things out of the generic cache (H5C) level of code.

Tested:

    h5committest

1 files changed, 4 insertions, 0 deletions

diff --git a/src/H5T.c b/src/H5T.c
index 130921c..b6ecc50 100644
--- a/src/H5T.c
+++ b/src/H5T.c
@@ -5298,6 +5298,8 @@ H5Tflush(hid_t type_id)
     /* Check args */
     if(NULL == (dt = (H5T_t *)H5I_object_verify(type_id, H5I_DATATYPE)))
         HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a datatype")
+    if(!H5T_is_named(dt))
+        HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a committed datatype")
 
     /* Call private function to flush datatype object */
     if (H5O_flush_metadata(&dt->oloc, H5AC_dxpl_id) < 0)
@@ -5332,6 +5334,8 @@ H5Trefresh(hid_t type_id)
     /* Check args */
     if(NULL == (dt = (H5T_t *)H5I_object_verify(type_id, H5I_DATATYPE)))
         HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a datatype")
+    if(!H5T_is_named(dt))
+        HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not a committed datatype")
 
     /* Call private function to refresh datatype object */
     if ((H5O_refresh_metadata(type_id, dt->oloc, H5AC_dxpl_id)) < 0)