simon-svn: puzzles: simon

Thu Apr 30 18:56:56 BST 2009

SVN root:       svn://svn.tartarus.org/sgt
Changes by:     simon
Revision:       8541
Date:           2009-04-30 18:56:56 +0100 (Thu, 30 Apr 2009)

Log message (13 lines):
Fix a misdesign I must have missed when I reviewed the Killer patch:
merge_some_cages() was written in the assumption that it would
always be able to do something, in that it returned void on success
and if it couldn't find anything to do it would just loop round
forever trying the same things over and over again.

Now it makes a methodical list of the pairs of cages which are merge
candidates, goes through them in a random order until it finds a
viable one, and returns a boolean indicating whether it succeeded or
ran out of candidates.

A test case which previously hung and now does not is "solo
--generate 1 7jxkdt#12345-10".

Modified files:
U   puzzles/solo.c

Links:
http://svn.tartarus.org/sgt/?rev=8541&view=rev
http://svn.tartarus.org/sgt/puzzles/solo.c?rev=8541&r1=8540&r2=8541