simon-git: puzzles (main): Ben Harris

[Commits to Tartarus hosted VCS]

TL;DR:
  7364ce8 Make sure that moves in Flood use only valid colours

Repository:     https://git.tartarus.org/simon/puzzles.git
On the web:     https://git.tartarus.org/?p=simon/puzzles.git
Branch updated: main
Committer:      Ben Harris <bjh21 at bjh21.me.uk>
Date:           2023-02-14 22:13:28

commit 7364ce8e266d947be146d635958a7b282752aac6
web diff https://git.tartarus.org/?p=simon/puzzles.git;a=commitdiff;h=7364ce8e266d947be146d635958a7b282752aac6;hp=5a0a2b9166a144b1775411a47060efa483e61971
Author: Ben Harris <bjh21 at bjh21.me.uk>
Date:   Tue Feb 14 22:02:35 2023 +0000

    Make sure that moves in Flood use only valid colours
    
    If execute_move() receieves a move that uses a colour beyond the range
    for the current game, it now rejects it.  Without this a solve string
    containing an invalid colour would cause an assertion failure: "fill:
    Assertion `oldcolour != newcolour' failed."  While I was in the area I
    put a range check on colours for normal moves as well.  To demonstrate
    the problem, load this save file:
    
    SAVEFILE:41:Simon Tatham's Portable Puzzle Collection
    VERSION :1:1
    GAME    :5:Flood
    PARAMS  :7:6x6c6m5
    CPARAMS :7:6x6c6m3
    DESC    :39:432242034203340350204502505323231342,17
    NSTATES :1:2
    STATEPOS:1:2
    MOVE    :2:S6

 flood.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)