Corrects implementation of cwevent "fate" fields.

The implementation of the cwevent runner/batter "fate" fields was simply not correct.  This fixes the problem.

This closes #45.

ChangeLog.md

@@ -14,6 +14,8 @@
   compatibility.
 - In `cwgame`, "unknown" in now an accepted value for `info,temp` and `info,windspeed`.
   "unknown" values are output as 0 for `BEVENT` compatibility.
+- In `cwevent`, the implementation of the batter/runner "fate" extended fields was
+  entirely incorrect.  This has been corrected.  (#45)
 
 
 # [0.10.0] - 2023-01-02

src/cwlib/gameiter.c

@@ -1037,19 +1037,20 @@ int cw_gameiter_runner_fate(CWGameIterator *orig_gameiter, int base)
       orig_gameiter->event_data->advance[base] >= 4) {
     return orig_gameiter->event_data->advance[base];
   }
-
   base = orig_gameiter->event_data->advance[base];
   gameiter = cw_gameiter_copy(orig_gameiter);
-  while (gameiter->event != NULL && 
-	 gameiter->state->inning == orig_gameiter->state->inning &&
-	 gameiter->state->batting_team == orig_gameiter->state->batting_team &&
-	 base >= 1 && base <= 3) {
-    cw_gameiter_next(gameiter);
-    if (gameiter->event && strcmp(gameiter->event->event_text, "NP") != 0) {
+  cw_gameiter_next(gameiter);
+  while (gameiter->event != NULL &&
+         gameiter->state->inning == orig_gameiter->state->inning &&
+         gameiter->state->batting_team == orig_gameiter->state->batting_team) {
+    if (strcmp(gameiter->event->event_text, "NP") != 0) {
       base = gameiter->event_data->advance[base];
+      if (base < 1 || base > 3) {
+        break;
+      }
     }
+    cw_gameiter_next(gameiter);
   }
-
   cw_gameiter_cleanup(gameiter);
   free(gameiter);
   return base;