puzzles/obfusc.c, branch master My sgt-puzzles tree Move other test main()s out of library source files. 2023-04-02T13:35:12+00:00 Simon Tatham anakin@pobox.com 2023-04-02T09:42:42+00:00 83244294f56cbf4c6ac03564489d1775054bbcfc Having stated the principle in the previous commit, I should apply it consistently. A source file linked into the Puzzles library of common support code should not also define a main() under ifdef. This commit only goes as far as the _library_ support modules. It would be a much bigger job to do the same for all the actual _puzzles_ that have test main()s or standalone-solver main()s. And it's not necessary, because modifying one of those source files only triggers a rebuild of _one_ puzzle, not absolutely everything. (Not to mention that it's quite likely the puzzle and the test main() will need to be modified in conjunction anyway.) As in the previous commit, this has required exposing a few internal API functions as global, and maybe editing them a bit. In particular, the one-shot internal function that divvy_rectangle() loops on until it succeeds is now exposed as divvy_rectangle_attempt(), which means the test program doesn't have to condition a failure counter into the real function. I've thrown away penrose-vector-test completely, because that didn't look like a test program with any ongoing use at all - it was surely vestigial, while James was getting the vector representation up and running in the first place. 
Having stated the principle in the previous commit, I should apply it
consistently. A source file linked into the Puzzles library of common
support code should not also define a main() under ifdef.

This commit only goes as far as the _library_ support modules. It
would be a much bigger job to do the same for all the actual _puzzles_
that have test main()s or standalone-solver main()s. And it's not
necessary, because modifying one of those source files only triggers a
rebuild of _one_ puzzle, not absolutely everything. (Not to mention
that it's quite likely the puzzle and the test main() will need to be
modified in conjunction anyway.)

As in the previous commit, this has required exposing a few internal
API functions as global, and maybe editing them a bit. In particular,
the one-shot internal function that divvy_rectangle() loops on until
it succeeds is now exposed as divvy_rectangle_attempt(), which means
the test program doesn't have to condition a failure counter into the
real function.

I've thrown away penrose-vector-test completely, because that didn't
look like a test program with any ongoing use at all - it was surely
vestigial, while James was getting the vector representation up and
running in the first place.
Use C99 bool within source modules. 2018-11-13T21:48:24+00:00 Simon Tatham anakin@pobox.com 2018-11-13T21:45:44+00:00 5f5b284c0bddbe67de14b2d2bfb596bc7ba1298a This is the main bulk of this boolification work, but although it's making the largest actual change, it should also be the least disruptive to anyone interacting with this code base downstream of me, because it doesn't modify any interface between modules: all the inter-module APIs were updated one by one in the previous commits. This just cleans up the code within each individual source file to use bool in place of int where I think that makes things clearer. 
This is the main bulk of this boolification work, but although it's
making the largest actual change, it should also be the least
disruptive to anyone interacting with this code base downstream of me,
because it doesn't modify any interface between modules: all the
inter-module APIs were updated one by one in the previous commits.
This just cleans up the code within each individual source file to use
bool in place of int where I think that makes things clearer.
Replace TRUE/FALSE with C99 true/false throughout. 2018-11-13T21:48:24+00:00 Simon Tatham anakin@pobox.com 2018-11-13T21:44:02+00:00 a550ea0a47374705a37f36b0f05ffe9e4c8161fb This commit removes the old #defines of TRUE and FALSE from puzzles.h, and does a mechanical search-and-replace throughout the code to replace them with the C99 standard lowercase spellings. 
This commit removes the old #defines of TRUE and FALSE from puzzles.h,
and does a mechanical search-and-replace throughout the code to
replace them with the C99 standard lowercase spellings.
Don't ignore the return code of fwrite(). (Slight modification of a 2008-12-12T19:49:05+00:00 Simon Tatham anakin@pobox.com 2008-12-12T19:49:05+00:00 17334d19452b8818865c31bf2e42d2cf334c1f02 patch from Phil Bordelon.) [originally from svn r8373] 
patch from Phil Bordelon.)

[originally from svn r8373]
Stand-alone command-line interface to the obfuscate_bitmap() 2008-11-29T13:51:36+00:00 Simon Tatham anakin@pobox.com 2008-11-29T13:51:36+00:00 ec38952c4c8e924c1c89ad5b934c6b8e70ce4102 function. Useful in conjunction with the new --save option to generate lots of games, extract the aux strings from the game generator, and de-obfuscate them in order to measure statistical properties of their solutions. [originally from svn r8352] 
function. Useful in conjunction with the new --save option to
generate lots of games, extract the aux strings from the game
generator, and de-obfuscate them in order to measure statistical
properties of their solutions.

[originally from svn r8352]