Phil Bordelon points out an off-by-one error: since Solo doesn't use

zero as a valid puzzle symbol, it can support at most 35 symbols,
not 36. (This is largely academic since IME anything above about 25
is impractical to generate, but there we go.)

git-svn-id: svn://svn.tartarus.org/sgt/puzzles@7115 cda61777-01e9-0310-a592-d414129be87e

diff --git a/solo.c b/solo.c
index a2f40eb..0a4e852 100644 (file)
--- a/solo.c
+++ b/solo.c
@@ -331,8 +331,8 @@ static char *validate_params(game_params *params, int full)
        return "Both dimensions must be at least 2";
     if (params->c > ORDER_MAX || params->r > ORDER_MAX)
        return "Dimensions greater than "STR(ORDER_MAX)" are not supported";
-    if ((params->c * params->r) > 36)
-        return "Unable to support more than 36 distinct symbols in a puzzle";
+    if ((params->c * params->r) > 35)
+        return "Unable to support more than 35 distinct symbols in a puzzle";
     return NULL;
 }