--- capa/capa51/pProj/capaCommon.c	2000/08/07 20:47:29	1.14
+++ capa/capa51/pProj/capaCommon.c	2000/10/24 22:17:19	1.17
@@ -48,6 +48,8 @@ int     yyparse();
 extern  FILE *yyin;
 extern  void yyrestart();
 
+extern  FILE *dfp;
+
 /*----------------------------------------------------------*/
 /*  RETURN: -1 file error                                   */
 /*           0 success                                      */
@@ -3017,10 +3019,10 @@ capa_check_ans(ai,ans, error) AnswerInfo
               for(idx=0;idx<ANSWER_STRING_LENG;idx++) choice[idx] = 0;
               result = EXACT_ANS;
               for(ii=0;ii<corr_len; ii++) {
-                idx = toupper(correct[ii]) - 'A'; choice[idx] =  1;
+                idx = toupper(s[ii]) - 'A'; choice[idx] =  1;
               }
               for(ii=0;ii<input_len;ii++) {
-                idx = toupper(answer[ii]) - 'A';
+                idx = toupper(ans[ii]) - 'A';
                 if(choice[idx] != 1 )  result = INCORRECT;
               }
             } else { result = INCORRECT; }
@@ -3200,6 +3202,10 @@ capa_check_answer(p, answer, error) Prob
          /* and the result will be given back as 0 or 1 to indicate the */
          /* given answer is correct or not */
          /* arguments are given to the program as */
+         /* before running the program, check its existance first */
+         /* should we specify a time out period in capa.config file? */
+         /* set up a timer for this purpose */
+         /* FILE *popen (const char *command,const char *type ); */
 
           break;
     
@@ -3207,7 +3213,7 @@ capa_check_answer(p, answer, error) Prob
   return (result);
 }
 
-/* ----------------------------------------------------------------------------------- */
+/* -------------------------------------------------------------------------- */
 /*   assumming the formula is *fml_str and the student input is *input_str             */
 /*   according to the type of tolerance, we form the final formula as                  */
 /*      absolute tolerance:  (*fml_str) - (*input_str)                                 */
@@ -3367,8 +3373,15 @@ Problem_t *p; char **answers; int cnt; c
     outcomes = (int *)capa_malloc(sizeof(int),cnt);
     for(ii=0;ii<cnt;ii++) outcomes[ii]=0;  /* initialize the outcomes array */
     outcomes[0] = capa_check_answer(p, answers[0], &errormsg[0]);
+#ifdef    COMMON_DBUG
+    printf("CAPA_CHECK_ANSWER(%s,%s):: outcome[0]=%d\n", 
+       p->answer,answers[0],outcomes[0]); 
+#endif
     for(ii=1, ai = p->ans_list; ai; ii++,ai = ai->ans_next  ) {
        outcomes[ii] =  capa_check_ans(ai,answers[ii],&(errormsg[ii]));
+#ifdef    COMMON_DBUG
+    printf("CAPA_CHECK_ANS(%s,%s): outcome[%d]=%d\n", ai->ans_str,answers[ii],ii,outcomes[ii]); 
+#endif
     }
     done = ii = 0;
     result = 0;
@@ -3383,10 +3396,12 @@ Problem_t *p; char **answers; int cnt; c
       ii++;
       if(ii==cnt) done = 1;
     }
-    if( result == 0 ) { /* check if any of the outcome has failed to be a numeric */
+    if( result == 0 ) { 
+    /* check if any of the outcome has failed to be a numeric 
+       or was a malformed equation */
       done = ii = 0;
       while( !done ) {
-        if( outcomes[ii] == WANTED_NUMERIC ) {
+        if( outcomes[ii] == WANTED_NUMERIC || outcomes[ii] == BAD_FORMULA ) {
           result = outcomes[ii];
           done = 1;
         }
@@ -3394,7 +3409,7 @@ Problem_t *p; char **answers; int cnt; c
         if(ii==cnt) done = 1;
       }
     }
-    if( result == 0 ) { /* check if any of the outcome has failed on sig figs */
+    if( result == 0 ) {/*check if any of the outcome has failed on sig figs*/
       done = ii = 0;
       while( !done ) {
         if( outcomes[ii] == SIG_FAIL ) {