ref: f7e755628b681804e4dd82cd8bcb501a5a6dca57
parent: 4212ef4bd2ce85c594d28f7e737fe280e827d1b3
author: Simon Tatham <anakin@pobox.com>
date: Sun Mar 11 06:12:45 EDT 2007
Gary Wong observes that solo's decode_params() is overenthusiastic about eating the letter `d' (for `diagonal') when it appears in a symmtery description: it should only be used after `m', because mirror symmetry is the only type that can be diagonal. This was causing parsing of the parameter description `3x3adu' to produce the wrong answer: the d would be swallowed, then the u ignored for being incomprehensible, and you'd get default Trivial difficulty. [originally from svn r7386]
--- a/solo.c
+++ b/solo.c
@@ -207,7 +207,7 @@
if (*string == 'r' || *string == 'm' || *string == 'a') {int sn, sc, sd;
sc = *string++;
- if (*string == 'd') {+ if (sc == 'm' && *string == 'd') {sd = TRUE;
string++;
} else {