shithub: puzzles

Info  •  Files  •  Log  •  Branches

Flood: fix interpret_move() return value for CURSOR_SELECT2

If there's no solution in progress, it should return MOVE_NO_EFFECT,
not MOVE_UNUSED.

--- a/flood.c

+++ b/flood.c

@@ -845,9 +845,9 @@

     } else if (button == CURSOR_SELECT) {

         tx = ui->cx;

         ty = ui->cy;

-    } else if (button == CURSOR_SELECT2 &&

-               state->soln && state->solnpos < state->soln->nmoves) {

-	move = state->soln->moves[state->solnpos];

+    } else if (button == CURSOR_SELECT2) {

+        if (state->soln && state->solnpos < state->soln->nmoves)

+            move = state->soln->moves[state->solnpos];

     } else {

         return MOVE_UNUSED;

     }