From b3243d75043cf1d70beb88d2a36eaebfe85c2c3f Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Sun, 1 Oct 2017 13:53:24 +0100 Subject: Return error messages as 'const char *', not 'char *'. They're never dynamically allocated, and are almost always string literals, so const is more appropriate. --- tracks.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'tracks.c') diff --git a/tracks.c b/tracks.c index 78355cb..b05fbd2 100644 --- a/tracks.c +++ b/tracks.c @@ -182,7 +182,7 @@ static game_params *custom_params(const config_item *cfg) return ret; } -static char *validate_params(const game_params *params, int full) +static const char *validate_params(const game_params *params, int full) { /* * Generating anything under 4x4 runs into trouble of one kind @@ -782,7 +782,7 @@ newpath: return desc; } -static char *validate_desc(const game_params *params, const char *desc) +static const char *validate_desc(const game_params *params, const char *desc) { int i = 0, w = params->w, h = params->h, in = 0, out = 0; @@ -1364,7 +1364,7 @@ static char *move_string_diff(const game_state *before, const game_state *after, } static char *solve_game(const game_state *state, const game_state *currstate, - const char *aux, char **error) + const char *aux, const char **error) { game_state *solved; int ret; -- cgit v1.1