aboutsummaryrefslogtreecommitdiff
path: root/sixteen.c
diff options
context:
space:
mode:
authorSimon Tatham <anakin@pobox.com>2013-04-12 17:11:49 +0000
committerSimon Tatham <anakin@pobox.com>2013-04-12 17:11:49 +0000
commit0b93de904a98f119b1a95d3a53029f1ed4bfb9b3 (patch)
treee1fef5c0a006feb7c2a631c7e5c81487221734d7 /sixteen.c
parent383c15794a517a4e6dc0b7364edac1f7056cee82 (diff)
downloadpuzzles-0b93de904a98f119b1a95d3a53029f1ed4bfb9b3.zip
puzzles-0b93de904a98f119b1a95d3a53029f1ed4bfb9b3.tar.gz
puzzles-0b93de904a98f119b1a95d3a53029f1ed4bfb9b3.tar.bz2
puzzles-0b93de904a98f119b1a95d3a53029f1ed4bfb9b3.tar.xz
Add 'const' to the game_params arguments in validate_desc and
new_desc. Oddities in the 'make test' output brought to my attention that a few puzzles have been modifying their input game_params for various reasons; they shouldn't do that, because that's the game_params held permanently by the midend and it will affect subsequent game generations if they modify it. So now those arguments are const, and all the games which previously modified their game_params now take a copy and modify that instead. [originally from svn r9830]
Diffstat (limited to 'sixteen.c')
-rw-r--r--sixteen.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/sixteen.c b/sixteen.c
index 88c7ef4..d5a2789 100644
--- a/sixteen.c
+++ b/sixteen.c
@@ -193,7 +193,7 @@ static int perm_parity(int *perm, int n)
return ret;
}
-static char *new_game_desc(game_params *params, random_state *rs,
+static char *new_game_desc(const game_params *params, random_state *rs,
char **aux, int interactive)
{
int stop, n, i, x;
@@ -398,7 +398,7 @@ static char *new_game_desc(game_params *params, random_state *rs,
}
-static char *validate_desc(game_params *params, char *desc)
+static char *validate_desc(const game_params *params, char *desc)
{
char *p, *err;
int i, area;