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.

[originally from svn r6161]
This commit is contained in:
Simon Tatham
2005-08-04 17:07:51 +00:00
parent e55838bc9b
commit 2e214d740c

View File

@ -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;
}