diff options
| author | Ben Harris <bjh21@bjh21.me.uk> | 2023-01-08 10:42:45 +0000 |
|---|---|---|
| committer | Ben Harris <bjh21@bjh21.me.uk> | 2023-01-15 16:24:27 +0000 |
| commit | ba944f9f95238d0b820e63d9b6c2d83efc09b8a2 (patch) | |
| tree | 454b1c76d688ee838bcf6c8fdf42419bfc593430 /bridges.c | |
| parent | 8a06ff26fc6abd77a4b4a08b839943f588d92dcf (diff) | |
| download | puzzles-ba944f9f95238d0b820e63d9b6c2d83efc09b8a2.zip puzzles-ba944f9f95238d0b820e63d9b6c2d83efc09b8a2.tar.gz puzzles-ba944f9f95238d0b820e63d9b6c2d83efc09b8a2.tar.bz2 puzzles-ba944f9f95238d0b820e63d9b6c2d83efc09b8a2.tar.xz | |
Tighten Bridges' validate_desc()
It allowed V, W, X, Y, H, I, J, and K to appear in game descriptions
even though new_game() didn't ascribe any meaning to those letters and
would fail an assertion if they ever occurred. As far as I can tell,
those letters have never done anything, so I've just removed the
checks for them from validate_desc().
Diffstat (limited to 'bridges.c')
| -rw-r--r-- | bridges.c | 5 |
1 files changed, 0 insertions, 5 deletions
@@ -2016,11 +2016,6 @@ static const char *validate_desc(const game_params *params, const char *desc) i += *desc - 'a'; /* plus the i++ */ else if (*desc >= 'A' && *desc <= 'G') /* OK */; - else if (*desc == 'V' || *desc == 'W' || - *desc == 'X' || *desc == 'Y' || - *desc == 'H' || *desc == 'I' || - *desc == 'J' || *desc == 'K') - /* OK */; else if (!*desc) return "Game description shorter than expected"; else |