[svn-r9583] Purpose:

    Code optimization

Description:
    Revise mechanism for calling H5open/H5check to better avoid calling them
from internal libray code.

    Only copy a datatype's group entry information if it has some.

Platforms tested:
    FreeBSD 4.10 (sleipnir) w/parallel
    Solaris 2.7 (arabica)
    Too minor to require h5committest

2 files changed, 6 insertions, 5 deletions

diff --git a/src/H5Ppublic.h b/src/H5Ppublic.h
index eedcda6..1c5fc0c 100644
--- a/src/H5Ppublic.h
+++ b/src/H5Ppublic.h
@@ -83,9 +83,9 @@ typedef herr_t (*H5P_iterate_t)(hid_t id, const char *name, void *iter_data);
  *
  */
 
-/* When this header is included from H5Pprivate.h, don't make calls to H5open() */
+/* When this header is included from a private header, don't make calls to H5open() */
 #undef H5OPEN
-#ifndef _H5Pprivate_H
+#ifndef _H5private_H
 #define H5OPEN        H5open(),
 #else   /* _H5Pprivate_H */
 #define H5OPEN
diff --git a/src/H5T.c b/src/H5T.c
index 905d89d..20eb663 100644
--- a/src/H5T.c
+++ b/src/H5T.c
@@ -3058,9 +3058,10 @@ H5T_copy(const H5T_t *old_dt, H5T_copy_t method)
             break;
     } /* end switch */
 
-    /* Deep copy of the symbol table entry */
-    if (H5G_ent_copy(&(new_dt->ent), &(old_dt->ent),H5G_COPY_DEEP)<0)
-        HGOTO_ERROR(H5E_SYM, H5E_CANTOPENOBJ, NULL, "unable to copy entry");
+    /* Deep copy of the symbol table entry, if there was one */
+    if (H5F_addr_defined(old_dt->ent.header))
+        if (H5G_ent_copy(&(new_dt->ent), &(old_dt->ent), H5G_COPY_DEEP)<0)
+            HGOTO_ERROR(H5E_SYM, H5E_CANTOPENOBJ, NULL, "unable to copy entry");
 
     /* Set return value */
     ret_value=new_dt;