Bug fix from James H: solve_game() was returning error messages in

the return value rather than in *error. In the old days type-
checking would have caught this, but now of course they're the same
type.

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

diff --git a/slant.c b/slant.c
index 06c1f51..1310018 100644 (file)
--- a/slant.c
+++ b/slant.c
@@ -734,9 +734,10 @@ static char *solve_game(game_state *state, game_state *currstate,
        if (ret != 1) {
            sfree(soln);
            if (ret == 0)
-               return "This puzzle is not self-consistent";
+               *error = "This puzzle is not self-consistent";
            else
-               return "Unable to find a unique solution for this puzzle";
+               *error = "Unable to find a unique solution for this puzzle";
+            return NULL;
        }
        free_soln = TRUE;
     }