From f50327a38c3162a1a6caf7394e5997e992bc33b8 Mon Sep 17 00:00:00 2001 From: simon Date: Mon, 23 Jan 2012 18:56:04 +0000 Subject: [PATCH] Fix default parameter assignment in Light Up when validating an incomplete parameter string: if the user hand-types a game ID along the lines of '18x10:stuff', we should not assume SYMM_ROT4 in the resulting game_params, since it'll be failed by validate_params. git-svn-id: svn://svn.tartarus.org/sgt/puzzles@9386 cda61777-01e9-0310-a592-d414129be87e --- lightup.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/lightup.c b/lightup.c index 5fe95b0..3747ec1 100644 --- a/lightup.c +++ b/lightup.c @@ -215,6 +215,11 @@ static void decode_params(game_params *params, char const *string) if (*string == 's') { string++; EATNUM(params->symm); + } else { + /* cope with user input such as '18x10' by ensuring symmetry + * is not selected by default to be incompatible with dimensions */ + if (params->symm == SYMM_ROT4 && params->w != params->h) + params->symm = SYMM_ROT2; } params->difficulty = 0; /* cope with old params */ -- 2.11.0