shithub: puzzles

Download patch

ref: 789e11f8f802889904c7eb137d354877f647c88d
parent: ccd579e72ee8ea2de46786a6b296f37a0ecf5867
author: Ben Harris <bjh21@bjh21.me.uk>
date: Tue Jan 31 16:05:47 EST 2023

Remove various unused game functions

If can_configure is false, then the game's configure() and
custom_params() functions will never be called.  If can_solve is false,
solve() will never be called.  If can_format_as_text_ever is false,
can_format_as_text_now() and text_format() will never be called.  If
can_print is false, print_size() and print() will never be called.  If
is_timed is false, timing_state() will never be called.

In each case, almost all puzzles provided a function nonetheless.  I
think this is because in Puzzles' early history there was no "game"
structure, so the functions had to be present for linking to work.  But
now that everything indirects through the "game" structure, unused
functions can be left unimplemented and the corresponding pointers set
to NULL.

So now where the flags mentioned above are false, the corresponding
functions are omitted and the function pointers in the "game" structures
are NULL.