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.

1 files changed, 3 insertions, 3 deletions

diff --git a/flood.c b/flood.c
index e8ff167..bef45f3 100644
--- a/flood.c
+++ b/flood.c
@@ -845,9 +845,9 @@ static char *interpret_move(const game_state *state, game_ui *ui,
     } 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;
     }