From 339329449f4db72e4754f21c14295303ffe0ea5d Mon Sep 17 00:00:00 2001 From: Simon Tatham Date: Sat, 13 Apr 2013 10:37:27 +0000 Subject: Fix a memory management bug in Filling: in some situations its solve_game() was returning its aux parameter un-dupstr()ed, which is wrong. Also clarified the developer docs on that function to make it clearer that the returned string should be dynamic. [originally from svn r9831] --- devel.but | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'devel.but') diff --git a/devel.but b/devel.but index e95e1d2..75ddeb7 100644 --- a/devel.but +++ b/devel.but @@ -1020,9 +1020,11 @@ it may return \cw{NULL}. If it does this, it must also set \q{Solution not known for this puzzle}); that error message is not expected to be dynamically allocated. -If this function \e{does} produce a solution, it returns a move -string suitable for feeding to \cw{execute_move()} -(\k{backend-execute-move}). +If this function \e{does} produce a solution, it returns a move string +suitable for feeding to \cw{execute_move()} +(\k{backend-execute-move}). Like a (non-empty) string returned from +\cw{interpret_move()}, the returned string should be dynamically +allocated. \H{backend-drawing} Drawing the game graphics -- cgit v1.1