Minor cleanups (mostly \ns in diagnostic printfs) from James H.

[originally from svn r7363]

My favourite kind of patch, from James H: one which decreases the
amount of code. James has ripped out the solver's version of
check_complete(), in favour of using the one I wrote for the
game-playing UI. My one checks connectedness, which means that the
solver will now not believe non-solutions to puzzles where
connectedness becomes a difficult issue. Examples of game IDs which
are now solved correctly but were previously not are 5x3:ubb and
7x7:ajfzmfqgtdzgt.

[originally from svn r7362]

About time I got round to documenting the new
REQUIRE_{RBUTTON,NUMPAD} flags.

[originally from svn r7356]

Fix problems with arrow UI with non-square grid.

[originally from svn r7355]

Silliness! Here's a somewhat hacky patch which builds an additional
binary from the Galaxies source file. The function of the new
`galaxiespicture' is to take a .xbm bitmap on standard input and
convert it into a Galaxies game ID using both black and white dots,
such that when solved the puzzle displays the input bitmap.

In the process of this I've implemented a post-processing pass after
the main game generation, to prevent clusters of adjacent
singletons. James H already solved that problem for unconstrained
game generation, but for some reason it came back when I did this.
However, the post-processing pass is still turned off for normal
usage, on the basis that (a) if it ain't broke don't fix it, and (b)
it's rather slow and best avoided if not necessary.

[originally from svn r7354]

Amend README so that it makes reasonable sense no matter whether
you've got your source code from a tarball (with makefiles) or from
svn (without makefiles). While I'm here, mention Makefile.wce.

[originally from svn r7352]

Cleanup patch from James H: disable the `s' key, which was only in
there by accident.

[originally from svn r7351]

Adam D. Lopresto and Phil Bordelon independently point out a
signedness mismatch.

[originally from svn r7350]

Bound edge thicknesses below so that they're always thicker than the
grid lines.

[originally from svn r7349]

Fix some border drawing issues.

[originally from svn r7347]

MinGW windres doesn't like an empty resource file, so invent a resource to
keep it happy. (Hopefully this hack will be temporary.)

[originally from svn r7346]

General cleanups patch from James H:
 - missing static in filling.c
 - better robustness in execute_move() in filling.c
 - remove side effects in assert statements
 - remove rogue diagnostic in galaxies.c
 - remove // comment in map.c
 - add more stylus-friendly UI to Pattern
 - bias Unequal towards generating inequality clues rather than numeric

[originally from svn r7344]

Gary Wong points out a couple of minor errors in the setting of
`used_solve'.

[originally from svn r7343]

Provide my old drag-based interface to Net as an ifdef-enabled
option, and turn it on by default on stylus-based platforms (i.e.
currently PocketPC).

[originally from svn r7341]

Another thing to watch out for when adding new puzzles.

[originally from svn r7340]

Create a blank .rc file to be used in the absence of icons/foo.rc.
This means that puzzles.rc2 is always included in all Windows and
PocketPC builds, which in turn means that I should be able to start
filling it full of VERSIONINFO and have that reliably included as
well.

[originally from svn r7339]

This game requires the numpad.

[originally from svn r7338]

Dariusz Olszewski's changes to support compiling for PocketPC. This
is mostly done with ifdefs in windows.c; so mkfiles.pl generates a
new makefile (Makefile.wce) and Recipe enables it, but it's hardly
any different from Makefile.vc apart from a few definitions at the
top of the files.

Currently the PocketPC build is not enabled in the build script, but
with any luck I'll be able to do so reasonably soon.

[originally from svn r7337]

After it confused Verity, clarify in the Unequal docs that the
Trivial and Recursive difficulty levels are available for custom
selection even though no preset uses them.

[originally from svn r7336]

Hardwiring the grid line width to 1 is really bad for printing. Use
a slightly more conventional method of drawing the grid lines, and
thereby fix printing.

[originally from svn r7335]

Galaxies doesn't print in colour.

[originally from svn r7334]

Don't create an undo-chain entry for a move with no effect.

[originally from svn r7333]

Fix a UI glitch where dragging an existing arrow could change what it
referred to.

[originally from svn r7331]

Tweak the cropping of the Filling icon very slightly.

[originally from svn r7330]

Update the checklist (yet again) to try to prevent a recurrence of
the error I corrected in r7328.

[originally from svn r7329]
[r7328 == b1679913f68675806f88f4682423c99f14a47b02]

Forgot to enable the Windows icon.

[originally from svn r7328]

More updates to the new-puzzle checklist.

[originally from svn r7327]

New puzzle: `Filling', a Fillomino implementation by Jonas Koelker.

[originally from svn r7326]

Oops. Having built the installer, we need a redirect for it in
.htaccess.

[originally from svn r7324]

Build an installer for Puzzles.

[originally from svn r7319]

Somehow I let a couple of // comments get past me when I accepted
this file. Remove them.

[originally from svn r7318]

Phil Bordelon points out that when I renamed `Recursive' I missed a
bit.

[originally from svn r7314]

Remove spurious semicolons from macros; their upset compilers that don't like
mixing declarations and code (I accidentally tried to compile it with one
such).

[originally from svn r7313]

And fix the error _I_ made in r7311. I should stop coding now,
because I'm clearly too sleepy to get it right.

[originally from svn r7312]
[r7311 == 8c0890a331d93c9d12ef782601b9953463872fc6]

Part of r7310 completely changed the careful semantics of my
coordinate-rounding, causing picking up arrows from a dot anywhere
other than at the centre of a square to break.

[originally from svn r7311]
[r7310 == a428c5899de86897fe4e92fa6585b3c3908ce2ad]

Patch from James H to make the Palm compiler stop objecting to my
(as far as I could tell) perfectly legal use of floats, and also
remove some VC warnings.

[originally from svn r7310]

We may reluctantly permit the returning of a too-easy puzzle if an
adequately hard one couldn't be found, but we must never return a
puzzle harder than the user asked for, and we _certainly_ mustn't
return one that we aren't even sure has a unique solution.

However, when I attempted to implement this rule, it turned out that
an attempt to generate a 15x15 Easy puzzle didn't terminate before I
got bored and killed it. I'm therefore retiring the Easy difficulty
level on the grounds that it's not a wide enough band to be useful.
I've renamed Hard to Normal, and (while I was there) renamed
Recursive to Unreasonable in line with my normal practice.

[originally from svn r7309]

James H forgot to credit himself for Galaxies!

[originally from svn r7308]

Update the new-puzzle checklist for the post-bob world.

[originally from svn r7307]

Missing uses of \q{}.

[originally from svn r7305]

Add James Harvey's excellent new puzzle, `Galaxies'.

[originally from svn r7304]

Forgot to shuffle the num[] array! That was probably introducing
some really subtle probabilistic bias in the generated latin squares.

[originally from svn r7302]

Replicate r7285 from PuTTY: make keyboard input work in HTML Help.

[originally from svn r7286]
[r7285 == 3d78bf9b261b4bfd30ae00d5ba43744c4144c934 in putty repository]

Now we have the new `module' command, use it.

[originally from svn r7224]

Reinstate the icons in the Windows binaries.

[originally from svn r7217]

Forgot to tag the Mac and Windows binaries with the revision number.

[originally from svn r7215]

Looking for puzzles*.tar.gz in the right directory is liable to make
the .htaccess build work better :-)

[originally from svn r7214]

Build script for Puzzles using bob. I've also added a piece of extra
infrastructure to the mkfiles.pl framework for the convenience of
the build script: it generates `wingames.lst', a list of the Windows
binaries which are ship-worthy games as opposed to nullgame or
command-line auxiliary programs.

[originally from svn r7206]

Misdirected cross-reference spotted by Jonas Koelker.

[originally from svn r7172]

Marcin Wojdyr points out that the use of `>&' to redirect both
stdout and stderr is non-standard. Switch to a POSIX-blessed
alternative.

[originally from svn r7116]

Phil Bordelon points out an off-by-one error: since Solo doesn't use
zero as a valid puzzle symbol, it can support at most 35 symbols,
not 36. (This is largely academic since IME anything above about 25
is impractical to generate, but there we go.)

[originally from svn r7115]

Patch from James H which shouldn't change the functionality of
Loopy, but makes it build more sensibly on the Palm.

[originally from svn r7114]

Patch from James H to fix the occasional generation of puzzles
harder than requested.

[originally from svn r7113]

Phil Bordelon points out that the Unequal difficulty settings
documentation is a bit odd, and also offers a signedness fix in
latin.c.

[originally from svn r7112]

Give all the intermediate files distinct file names, so that a
multi-CPU machine can speed up the icon build using `make -j' and
not have it break.

[originally from svn r7106]

Formatting tweaks / index terms in Unequal docs.

[originally from svn r7105]

First time I've added a new puzzle since the icons-and-screenshots
upheaval. Modify the checklist accordingly.

[originally from svn r7103]

Icon for Unequal.

[originally from svn r7101]

Add James H's new puzzle, `Unequal' (otherwise known as the
Guardian's `Futoshiki').

[originally from svn r7100]

Remove a couple of rogue make targets for `osx.icns.o' and
`osx-info.plist.o' which had crept into most of the Makefiles.

[originally from svn r7099]

Rather to my surprise given the amount of testing this code has had,
Kevin Lyles spotted a tree234 bug! copytree234() segfaulted when
asked to copy a tree containing no elements, due to failing to allow
for the case that t->root might be NULL. Fixed, and added a
regression test.

[originally from svn r7092]

In Windows/Gtk front-ends, consistently use the ellipsis convention for naming
menu items which bring up dialogs.

[originally from svn r7058]

Phil Bordelon's patch yesterday should have initialised the GError
to NULL.

[originally from svn r7052]

Phil Bordelon points out that my invocation of gdk_pixbuf_save()
isn't quite right.

[originally from svn r7050]

It's a new year.

[originally from svn r7048]
[this svn revision also touched putty,putty-website]

Document the new --game and --load options on Unix.

[originally from svn r7040]

Make errors in option parsing actually _do_ something in interactive mode.

[originally from svn r7039]

Bring .R files for nullgame and unfinished puzzles up to date with the rest of
the world (by adding icon support).

[originally from svn r7038]

Update manual copyright to 2006.

[originally from svn r7036]

Add NO_HTMLHELP and turn it on by default in Makefile.cyg.

[originally from svn r7033]

Enhance icon.pl to be a fully general Windows icon builder: it now
supports monochrome icons, can deal with any size of image you're
mad enough to put in there, and will construct icons with whatever
combination of sizes and resolutions you feel like specifying. This
has involved a change in the command-line syntax, hence the
adjustment to Makefile.

(I don't imagine that the changes described here will be critical to
Puzzles any time soon, but I might reuse this script elsewhere and
then I won't want it to have arbitrary limitations.)

[originally from svn r7031]

`make all' targets are usually useful.

[originally from svn r7026]

Er, further makedist fixes. Helps if you test _before_ checkin.

[originally from svn r7025]

And update makedist.sh to put all the new stuff into the source
tarball.

[originally from svn r7024]

Fix the Cygwin RCFLAGS in the light of recent changes.

[originally from svn r7023]

Aha, this seems to be a more sensible way of getting screenshots:
instead of having the puzzle binary export its window ID to a script
which then runs xwd, we can use the gdk-pixbuf library to have the
puzzle binary _itself_ read its own internal pixmap and save it
straight to a PNG. How handy. And faster, and less timing-sensitive.

[originally from svn r7022]

Support for run-time icons in the GTK puzzles. This involved another
mkfiles.pl change (I don't seem to be planning ahead very well this
week), this time to provide a list of fallback options for an object
file. That way, I have a no-icon.c which quietly replaces
icons/foo-icon.c if the latter doesn't exist, and so again people
checking straight out from Subversion shouldn't have trouble.

[originally from svn r7021]

Actually introduce the ability to build the Windows icons into the
Windows puzzle binaries. This checkin involves several distinct
changes:
 - mkfiles.pl now has an extra feature: if an object file is listed
   in Recipe with a trailing question mark, it will be considered
   optional, and silently dropped from the makefile if its primary
   source file isn't present at the time mkfiles.pl runs. This means
   people who check out the puzzles from Subversion and just run
   mkfiles.pl shouldn't get build failures; they just won't get the
   icons.
 - all the .R files now use this feature to include an optional
   Windows resource file.
 - the .rc resource source files are built by icons/Makefile.
 - windows.c finds the icon if present and uses it in place of the
   standard Windows application icon.

[originally from svn r7020]

Improve the icon images by cropping selected pieces out of most of
the main screenshots. (A few, like Map, were perfect already.)

In the process I've vertically reflected the puzzle shown in the
Pattern save file, to bring a more interesting piece of it into the
top left corner :-)

[originally from svn r7019]

Code to construct Windows icon files for the puzzles, by munging the
screenshots into appropriate sizes and colour depths. This is all
done with a nasty Perl script, because ImageMagick does not output
correct .ICO format. Not sure why; it isn't _that_ hard.

I intend at some point to link the resulting icons into the actual
Windows puzzle binaries, but before then I have to make them
prettier: most of them would benefit from being derived from a
smaller crop of the puzzle screenshot instead of trying to fit the
whole thing in.

[originally from svn r7017]

New mechanism for automatic generation of the puzzle screenshots on
the web, which I hope will also end up being extended to generate
both Windows and X icons for each individual puzzle. The mechanism
is: for each puzzle there's a save file in the `icons' subdirectory
showing a game state which I think is a decent illustration of the
puzzle, and then there's a nasty set of scripts which runs each
puzzle binary, loads that save file, grabs a screenshot using xwd,
and munges it into shape.

In order to support this I've added two new options (--redo and
--windowid) to all the GTK puzzles, which I don't expect ever to be
used outside the icons makefile. I've also added two more options
(--load and --id) which force a GTK puzzle to treat its command-line
option as a save file or as a game ID respectively (the previous
behaviour was always to guess, and sometimes it guessed wrong).

[originally from svn r7014]

Minor const fix.

[originally from svn r7013]

HTML Help support for Puzzles, with the same kind of automatic
fallback behaviour as PuTTY's support.

[originally from svn r7009]

Ability to build a .CHM for Puzzles. I haven't yet arranged for it
to be used by the Windows binaries; that's the next step.

[originally from svn r7006]

Small printing fix from Gary Wong. Shows up when printing with
solutions: the rightmost and lowermost lines of the grid never get
solution lines drawn on them.

[originally from svn r7003]

Fix a couple of rounding errors in Inertia's gem-drawing code which
didn't happen to show up at the default resolution but made the gem
slightly skewed at larger sizes.

[originally from svn r6919]

Optionally label the colours in Guess with letters of the alphabet.
Labels are toggled on and off by pressing L (just like Map). Might
help colour-blind users, and might also make it easier to describe
game positions to other people because `abbc' has fewer syllables
than `red, yellow, yellow, green', and hugely fewer letters if
you're typing it.

[originally from svn r6916]

Patch I've had lurking around for over a year and not remembered to
commit: arrange that midend_set_timer(), hence game_timing_state(),
is called when the game_ui is changed. This allows timed games to
work by obscuring the initial layout until an initial click causes
it to be revealed, without requiring that they store that reveal
operation as a move in the undo chain. Not that any games actually
do this, but it's clearly a sensible thing to want to do: since
game_timing_state() _receives_ a game_ui as a parameter, obviously
it should be consulted when the game_ui changes.

[originally from svn r6914]

James H's Palm-compatibility updates to the latest Loopy changes,
working around bugs in the Palm compiler and removing Palm-
incompatible diagnostics such as fprintf.

[originally from svn r6889]

Mike's changes to dsf.c alter the internal storage format of dsf
structures, meaning that ad-hoc initialisation now doesn't work.
Hence, this checkin converts all ad-hoc dsf initialisations into
calls to dsf_init() or snew_dsf(). At least, I _hope_ I've caught
all of them.

[originally from svn r6888]

Minimal (i.e., lame) update to Loopy documentation to match reality -- it's
been wrong since r6299.

[originally from svn r6884]
[r6299 == c389f623f66fe5296f7ef5c66d88884607b82cff]

I'm sick and tired of having unfinished puzzle code lying around on
several different systems in strange directories. So I'm creating an
`unfinished' directory within source control, and centralising all
my half-finished, half-baked or otherwise half-arsed puzzle
implementations into it. Herewith Sokoban (playable but rubbish
generation), Pearl (Masyu - rubbish generation and nothing else),
Path (Number Link - rubbish generation and nothing else) and NumGame
(the Countdown numbers game - currently just a solver and not even a
generator yet).

[originally from svn r6883]

r6880 accidentally backed out r6780. That's what I get for accepting
source files from Mike rather than patches, and not adequately
checking the result...

[originally from svn r6882]
[r6780 == f05c25347d66821d928668a7e87dffbf3ffed027]
[r6880 == b9547673c6462bf73e642328300479df6df71d7b]

Mike Pinna has done some major reworking of the Loopy solver, giving
rise to a new Hard difficulty level.

[originally from svn r6880]

Couple of minor errors.

[originally from svn r6872]

Forgot to put the new .R files into makedist.sh. Oops!

[originally from svn r6849]

Jesse W points out that I forgot to update the OS X front end in
r6420, so both drawing.c _and_ the front end were prepending the
time to the front of the Mines status bar, leading to a double
timestamp.

[originally from svn r6796]
[r6420 == 240b6cab8ce8729b7270a0411ab39038814058f1]

Always something. Mention `svn add'ing the .R file in the new-puzzle
checklist, and mention the now-not-version-controlled list.c in
svn:ignore.

[originally from svn r6782]

I'm sick of repeatedly adding and removing local changes to Recipe
when testing a new game, so here's a new architecture for the Recipe
file. mkfiles.pl now supports several new features:

 - an `!include' directive, which accepts wildcards
 - += to append to an existing object group definition
 - the ability to divert output to an arbitrary file.

So now each puzzle has a `.R' file containing a fragment of Recipe
code describing that puzzle, and the central Recipe does `!include
*.R' to construct the Makefiles. That way, I can keep as many
experimental half-finished puzzles lying around my working directory
as I like, and I won't have to keep reverting Recipe when I check in
any other changes.

As part of this change, list.c is no longer a version-controlled
file; it's now constructed by mkfiles.pl, so that it too can take
advantage of this mechanism.

[originally from svn r6781]

Extra utility function.

[originally from svn r6780]