puzzles/tree234.c, branch rockbox My sgt-puzzles tree 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.
Adopt C99 bool in the tree234 API. 2018-11-13T21:48:24+00:00 Simon Tatham anakin@pobox.com 2018-11-13T21:41:45+00:00 33b55db48e9573c6c825e3a222ff968f6a0bd4f4 The only affected function here is splitpos234, which I don't think these puzzles are even using at the moment. 
The only affected function here is splitpos234, which I don't think
these puzzles are even using at the moment.
Patch from Ben Hutchings: explicitly initialise something which some 2010-04-18T15:07:52+00:00 Simon Tatham anakin@pobox.com 2010-04-18T15:07:52+00:00 b7e0d854fe39016589fd73bfa2b98eb0e9a72228 gccs complained about. [originally from svn r8923] 
gccs complained about.

[originally from svn r8923]
Add a precautionary pair of initialisations to placate optimisers, 2009-11-10T19:11:03+00:00 Simon Tatham anakin@pobox.com 2009-11-10T19:11:03+00:00 9c95ea261980860cf74bc583417981960f91648b and asserts to check that the preinitialised values are never actually used. [originally from svn r8745] 
and asserts to check that the preinitialised values are never
actually used.

[originally from svn r8745]
Rather to my surprise given the amount of testing this code has had, 2007-01-11T09:26:09+00:00 Simon Tatham anakin@pobox.com 2007-01-11T09:26:09+00:00 c06e371f55b97576421588d21d809c02c637584b 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] 
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]
Yikes! I've only just noticed that this copy of tree234.c was still 2005-10-12T17:57:40+00:00 Simon Tatham anakin@pobox.com 2005-10-12T17:57:40+00:00 2bef4dfb50ec69cff1bc0569b76a7eb10115f576 using unwrappered malloc/free, leaving plenty of openings for out- of-memory segfaults. Switch to using Puzzles's memory management, which I should have done right at the start but can only assume I forgot about. [originally from svn r6388] 
using unwrappered malloc/free, leaving plenty of openings for out-
of-memory segfaults. Switch to using Puzzles's memory management,
which I should have done right at the start but can only assume I
forgot about.

[originally from svn r6388]
Initial checkin of a portable framework for writing small GUI puzzle 2004-04-25T14:27:58+00:00 Simon Tatham anakin@pobox.com 2004-04-25T14:27:58+00:00 96dbb537ee14f992459a8359a113a78414addf9d games. [originally from svn r4138] 
games.

[originally from svn r4138]