[svn-r7434] Purpose:

    Bug Fix and code cleanup

Description:
    Correct error in H5T_detect_class that was causing nested compound datatypes
with to not detect the datatype class of fields correctly, which caused errors
with fill-values, variable-length datatypes and chunks later on.

    Return the rank of the array datatype from H5Tget_array_dims(), like
H5Sget_dims().

    Lots of cleanups to datatype code, to make the handling of arrays, compound
types, variable-length strings and sequences and enumerated types more
consistent and robust.

Platforms tested:
    FreeBSD 4.9 (sleipnir)
    h5committest

1 files changed, 2 insertions, 7 deletions

diff --git a/src/H5Tcompound.c b/src/H5Tcompound.c
index b13788f..790de46 100644
--- a/src/H5Tcompound.c
+++ b/src/H5Tcompound.c
@@ -447,16 +447,11 @@ H5T_insert(H5T_t *parent, const char *name, size_t offset, const H5T_t *member)
     parent->u.compnd.nmembs++;
 
     /*
-     * Set the "force conversion" flag if VL datatype fields exist in this type
-     * or any component types
+     * Set the "force conversion" flag if the field's datatype indicates
      */
-    if(member->type==H5T_VLEN || member->force_conv==TRUE)
+    if(member->force_conv==TRUE)
         parent->force_conv=TRUE;
 
-    /* Set the flag for this compound type, if the field is an array */
-    if(member->type==H5T_ARRAY)
-        parent->u.compnd.has_array=TRUE;
-
 done:
     FUNC_LEAVE_NOAPI(ret_value);
 }