Changeset 9848 for trunk/psLib/src/types

Timestamp:

Nov 3, 2006, 2:20:10 PM (20 years ago)

Author:

drobbin

Message:

Removed a couple unreachable spots from psMetadataConfig in parseMetadataItem. Added error check to parseVector for invalid values (was looping indefinitely for ie, % or NAN). Updated tests accordingly

File:

• trunk/psLib/src/types/psMetadataConfig.c (modified) (6 diffs)

trunk/psLib/src/types/psMetadataConfig.c

@@ -10 +10 @@
 *  @author Eric Van Alst, MHPCC
 *
-*  @version $Revision: 1.105 $ $Name: not supported by cvs2svn $
-*  @date $Date: 2006-11-03 20:20:38 $
+*  @version $Revision: 1.106 $ $Name: not supported by cvs2svn $
+*  @date $Date: 2006-11-04 00:20:09 $
 *
 *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -365 +365 @@
     saveValue = inString;
     while(*inString!='\0') {
-        strtod(inString, &end);
+        double check = strtod(inString, &end);
+        if (isnan(check) || check < DBL_EPSILON) {  //make sure inString contains valid #'s
+            if ( strncmp(inString, "0", 1) != 0 || strncmp(inString, " 0", 2) != 0
+                    || isnan(check)) {
+                *status = 1;
+                return NULL;
+            }
+        }
         /*        if(inString==end) {
                     *status = 1;
@@ -625 +632 @@
         // Point to first item in template
         iter = psListIteratorAlloc(template->
-                                   list,PS_LIST_HEAD,true);
+                                   list, PS_LIST_HEAD, true);
         // For each item in template parse line string for values
         for(psS32 i = 0;
@@ -633 +640 @@
             templateItem = psListGetAndIncrement(iter)
                            ;
-            if(templateItem == NULL) {
-                psFree(md);
-                psFree(iter);
-                md = NULL;
-                break;
-            }
+            /*            if(templateItem == NULL) {
+                            psFree(md);
+                            psFree(iter);
+                            md = NULL;
+                            break;
+                        }
+            */
             // Get the next token on the line
-            token = getToken(&linePtr," ",&status,false);
+            token = getToken(&linePtr, " ", &status, false);
             if(token != NULL) {
                 // Allocate metadata item
-                mdItem = psMetadataItemAllocStr(templateItem->name,templateItem->comment,token);
-                if(mdItem == NULL) {
-                    psFree(md);
-                    md = NULL;
-                    psFree(token);
-                    psFree(iter);
-                    break;
-                }
-                // Add item to metadata
-                if(!psMetadataAddItem(md, mdItem, PS_LIST_TAIL, PS_META_DEFAULT)) {
-                    psFree(md);
-                    md = NULL;
-                    psFree(mdItem);
-                    psFree(token);
-                    psFree(iter);
-                    break;
-                }
+                mdItem = psMetadataItemAllocStr(templateItem->name,
+                                                templateItem->comment, token);
+                /*                if(mdItem == NULL) {
+                                    psFree(md);
+                                    md = NULL;
+                                    psFree(token);
+                                    psFree(iter);
+                                    break;
+                                }
+                 
+                                // Add item to metadata
+                                if(!psMetadataAddItem(md, mdItem, PS_LIST_TAIL, PS_META_DEFAULT)) {
+                                    psFree(md);
+                                    md = NULL;
+                                    psFree(mdItem);
+                                    psFree(token);
+                                    psFree(iter);
+                                    break;
+                                }
+                */
+                psMetadataAddItem(md, mdItem, PS_LIST_TAIL, PS_META_DEFAULT);
                 psFree(mdItem);
             } else {
@@ -833 +845 @@
         status = 0;
         strValue = getToken(&linePtr, "#", &status,true);
-        if(status) {
-            psError(PS_ERR_IO, true,
-                    _("Failed to read a metadata %s on line %u."), "value", lineCount);
-            psFree(strType);
-            psFree(strValue);
-            return false;
-        }
+        /*
+                if(status) {
+                    psError(PS_ERR_IO, true,
+                            _("Failed to read a metadata %s on line %u."), "value", lineCount);
+                    psFree(strType);
+                    psFree(strValue);
+                    return false;
+                }
+        */
         if(strValue==NULL) {
             psError(PS_ERR_IO, true,
@@ -850 +864 @@
         status = 0;
         strComment = getToken(&linePtr,"~", &status,true);
-        if(status) {
-            psError(PS_ERR_IO, true,
-                    _("Failed to read a metadata %s on line %u."), "comment", lineCount);
-            psFree(strType);
-            psFree(strValue);
-            psFree(strComment);
-        }
+        /*
+                if(status) {
+                    psError(PS_ERR_IO, true,
+                            _("Failed to read a metadata %s on line %u."), "comment", lineCount);
+                    psFree(strType);
+                    psFree(strValue);
+                    psFree(strComment);
+                }
+        */
     }