Embarrasing mistake. Always return 1 on error

progund · Nov 15, 2016 · 8aff990 · 8aff990
1 parent 8a67088
commit 8aff990
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/hand-in/calculator/expression.c b/hand-in/calculator/expression.c
@@ -20,15 +20,15 @@ comp_str(char* a, char* b)
 {
   if(a==NULL || b==NULL)
     {
-      return -1;
+      return 0;
     }
 
   /* Perhaps a bit overly optimised, but we like to make you think about this one
    * If two strings are of different length they can't be "equal"
    */
   if (strlen(a)!=strlen(b))
     {
-      return -1;
+      return 0;
     }
   return strcmp(a,b)==0;
 }
@@ -44,8 +44,8 @@ string_to_op(char *str)
   */
   while (op_strings[(int)++i]!=NULL)
     {
-      /* printf ("i: %d  %s %s \n", */
-      /*         i, op_strings[i],str); */
+      printf ("i: %d  %s '%s' \n", 
+              i, op_strings[(int)i],str); 
       if (comp_str(op_strings[(int)i],str))
         {
           return i;