Changeset 8374 for trunk/psLib/src/types

Timestamp:

Aug 15, 2006, 5:28:13 PM (20 years ago)

Author:

drobbin

Message:

removed error check for NULL vector in MetaItemCopy

File:

• trunk/psLib/src/types/psMetadata.c (modified) (3 diffs)

trunk/psLib/src/types/psMetadata.c

@@ -12 +12 @@
  *  @author Ross Harman, MHPCC
  *
- *  @version $Revision: 1.123 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2006-08-14 20:05:05 $
+ *  @version $Revision: 1.124 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2006-08-16 03:28:13 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -384 +384 @@
         PS_METADATA_ITEM_COPY_CASE(STRING,V); // This will copy the string, not point at it.
     case PS_DATA_VECTOR: {
+            PS_ASSERT_PTR_NON_NULL(in->data.V, NULL);
             psVector *vecCopy = psVectorCopy(NULL, (psVector*)(in->data.V),
                                              ((psVector*)(in->data.V))->type.type);
-            if (vecCopy == NULL) {
-                psError(PS_ERR_BAD_PARAMETER_NULL, false,
-                        "Error copying vector.  Vector skipped.\n");
-            } else {
-                newItem = psMetadataItemAlloc(in->name, PS_DATA_VECTOR, in->comment, vecCopy);
-                psFree(vecCopy);    // Drop reference
-            }
+            //Following condition was removed b/c it doesn't seem possible for a
+            //non-NULL vector to be returned for non-NULL input
+            //            if (vecCopy == NULL) {
+            //                psError(PS_ERR_BAD_PARAMETER_NULL, false,
+            //                        "Error copying vector.  Vector skipped.\n");
+            //            } else {
+            newItem = psMetadataItemAlloc(in->name, PS_DATA_VECTOR, in->comment, vecCopy);
+            psFree(vecCopy);    // Drop reference
+            //            }
             break;
         }
@@ -455 +458 @@
             psErrorStackPrint(stderr, "Error copying %s (%s) in the metadata\n", inItem->name,
                               inItem->comment);
+            printf("\nok we're here\n");
         }
         psFree(newItem);                // Drop reference