Changeset 30115 for branches/eam_branches/ipp-20101205/Ohana/src/opihi/lib.shell/parse.c

Timestamp:

Dec 20, 2010, 8:36:52 AM (16 years ago)

Author:

eugene

Message:

handle syntax errors in ohana scripts as errors

File:

• branches/eam_branches/ipp-20101205/Ohana/src/opihi/lib.shell/parse.c (modified) (9 diffs)

branches/eam_branches/ipp-20101205/Ohana/src/opihi/lib.shell/parse.c

@@ -1 +1 @@
 # include "opihi.h"
 
-char *parse (char *line) {
+char *parse (int *status, char *line) {
 
   double fval;
   float *fptr;
   char *newline, *N, *L, *val, *B, *V, *V0, *V1, *c1, *c2, *p;
-  int Nx, Ny, Nbytes, status, size, NLINE, isBuffer, inRange;
+  int Nx, Ny, Nbytes, filestatus, size, NLINE, isBuffer, inRange;
   FILE *f;
   Vector *vec;
   Buffer *buf;
+
+  *status = TRUE; 
 
   Ny = 0;
@@ -73 +75 @@
       Nbytes = fread (val, 1, 1023, f);
       val[Nbytes] = 0;
-      status = pclose (f);
+      filestatus = pclose (f);
       free (B);
 
-      if (status) gprint (GP_ERR, "warning: exit status of command %d\n", status);
+      if (filestatus) gprint (GP_ERR, "warning: exit status of command %d\n", filestatus);
 
       /* convert all but last return to ' '.  drop last return */
@@ -91 +93 @@
 
     /* simple variable assignment */
-    if (*V1 != '`') {
-      /* dvomath returns a new string, or NULL, with the result of the expression */
-      val = dvomath (1, &V1, &size, 0);
-      if (val == NULL) { 
-        while (OHANA_WHITESPACE (*V1)) V1++;
-        val = strcreate (V1);
-      } 
-    }
+    /* dvomath returns a new string, or NULL, with the result of the expression */
+    val = dvomath (1, &V1, &size, 0);
+    if (val == NULL) { 
+      while (OHANA_WHITESPACE (*V1)) V1++;
+      val = strcreate (V1);
+    } 
     /* both dvomath and command replacement create (char *) val */
     set_str_variable (V0, val);
@@ -172 +172 @@
       if (!inRange) {
         gprint (GP_ERR, "no element %d,%d\n", Nx, Ny);
-        goto escape;
+        goto error;
       }
       if (Nx < 0) Nx += buf[0].header.Naxis[0];
@@ -188 +188 @@
       if (!inRange) {
         gprint (GP_ERR, "no element %d\n", Nx);
-        goto escape;
+        goto error;
       }
       if (Nx < 0) Nx += vec[0].Nelements;
@@ -197 +197 @@
       }
     }
-
     goto escape;
   }
@@ -226 +225 @@
     if (c1 == NULL) {
       gprint (GP_ERR, "no close brackets!\n");
-      goto escape;
+      goto error;
     }
     c2 = strchr (L, '{');
     if ((c2 != NULL) && (c2 < c1)) {
       gprint (GP_ERR, "can't nest brackets!\n");
-      goto escape;
+      goto error;
     }
     *c1 = 0;
@@ -240 +239 @@
     if (val == NULL) {
       print_error ();
-      goto escape;
+      goto error;
     } 
 
@@ -257 +256 @@
   
   REALLOCATE (newline, char, strlen (newline) + 1);
+  *status = TRUE;
   return (newline); 
 
 error:
   gprint (GP_ERR, "syntax error\n");
-
-escape:
+  // assignment or increment operation: free line and return NULL
   if (line != (char *) NULL) free (line);
   if (val != (char *) NULL) free (val);
   if (V0 != (char *) NULL) free (V0);
+  *status = FALSE;
+  return NULL;
+
+escape:
+  // assignment or increment operation: free line and return NULL
+  if (line != (char *) NULL) free (line);
+  if (val != (char *) NULL) free (val);
+  if (V0 != (char *) NULL) free (V0);
+  *status = TRUE;
   return (NULL);
 }