Changeset 7933 for trunk/psLib/src/types

Timestamp:

Jul 18, 2006, 11:53:25 AM (20 years ago)

Author:

drobbin

Message:

Edited parseBool to only accept valid boolean values. Edited psMetadataItemTransfer and added test. Not finished testing yet...

Location:

trunk/psLib/src/types

Files:

• psMetadata.c (modified) (2 diffs)
• psMetadata.h (modified) (2 diffs)
• psMetadataConfig.c (modified) (2 diffs)

trunk/psLib/src/types/psMetadata.c

@@ -12 +12 @@
  *  @author Ross Harman, MHPCC
  *
- *  @version $Revision: 1.118 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2006-07-17 22:41:38 $
+ *  @version $Revision: 1.119 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2006-07-18 21:53:25 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -465 +465 @@
 
 // may need to extend this to change the keyname in the copy
-bool psMetadataItemTransfer(psMetadata *out, const psMetadata *in, const char *key)
-{
+psMetadata *psMetadataItemTransfer(psMetadata *out, const psMetadata *in, const char *key)
+{
+    PS_ASSERT_PTR_NON_NULL(in, NULL);
+    if (key == NULL) {
+        psError(PS_ERR_BAD_PARAMETER_NULL, true,
+                "psMetadataItemTransfer:  'key' cannot be NULL.\n");
+        return NULL;
+    }
 
     psMetadataItem *item = psMetadataLookup(in, key);
     if (!item) {
-        return false;
-    }
-
-    return psMetadataAddItem(out, item, PS_LIST_TAIL, PS_META_REPLACE);
+        return NULL;
+    }
+    if (out == NULL) {
+        out = psMetadataAlloc();
+    }
+    if (!psMetadataAddItem(out, item, PS_LIST_HEAD, PS_META_DEFAULT) ) {
+        psFree(out);
+        return NULL;
+    }
+
+    return out;
 }
 

trunk/psLib/src/types/psMetadata.h

@@ -11 +11 @@
 *  @author Ross Harman, MHPCC
 *
-*  @version $Revision: 1.81 $ $Name: not supported by cvs2svn $
-*  @date $Date: 2006-06-27 03:42:01 $
+*  @version $Revision: 1.82 $ $Name: not supported by cvs2svn $
+*  @date $Date: 2006-07-18 21:53:25 $
 *
 *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -400 +400 @@
  *  @return bool:       True if successful, otherwise false.
  */
-bool psMetadataItemTransfer(
+psMetadata *psMetadataItemTransfer(
     psMetadata *out,                   ///< output Metadata container for copying.
     const psMetadata *in,              ///< Metadata collection from which to copy.

trunk/psLib/src/types/psMetadataConfig.c

@@ -10 +10 @@
 *  @author Eric Van Alst, MHPCC
 *
-*  @version $Revision: 1.65 $ $Name: not supported by cvs2svn $
-*  @date $Date: 2006-07-15 03:29:05 $
+*  @version $Revision: 1.66 $ $Name: not supported by cvs2svn $
+*  @date $Date: 2006-07-18 21:53:25 $
 *
 *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -279 +279 @@
 
 
-    if(*inString=='T' || *inString=='t' || *inString=='1') {
+    /*    if(*inString=='T' || *inString=='t' || *inString=='1') {
+            value = true;
+        } else if(*inString=='F' || *inString=='f' || *inString=='0') {
+            value = false;
+        } else {
+            *status = 1;
+        }
+    */
+    if ( !strncmp(inString, "T", 2) || !strncmp(inString, "t", 2) ||
+            !strncmp(inString, "1", 2) || !strncmp(inString, "TRUE", 5) ||
+            !strncmp(inString, "true", 5) || !strncmp(inString, "True", 5)) {
         value = true;
-    } else if(*inString=='F' || *inString=='f' || *inString=='0') {
+    } else if ( !strncmp(inString, "F", 2) || !strncmp(inString, "f", 2) ||
+                !strncmp(inString, "0", 2) || !strncmp(inString, "FALSE", 6) ||
+                !strncmp(inString, "false", 6) || !strncmp(inString, "False", 6) ) {
         value = false;
     } else {