Fixed a bug in unicode_tolower() - Unicode range detection was broken by some of the code size reduction changes

Rearranged help text to order options by functional groups rather than alphabetically. Dropped 'quiet' option from NO_SOUND builds. Removed 'Ignore Errors' option as it's unimplemented in the DOS build. Simplified basename logic in dosinit.c os_process_arguments(). (384 bytes reduction in EXE size)

Error counting is no longer supported if NO_DEBUG is set. Command line option for display mode setting is again allowed if TEXTMODE_ONLY is set, but only allows selecting modes 0 and 1 (104 bytes EXE size reduction)

"Seed random numbers" hotkey is now disabled if NO_DEBUG is set (180 byte reduction in EXE file)

Disabled in-game tab completion (EXE file size reduced by 336 bytes)

On non-unicode builds, disabled the unicode downcasing tables in unicode_tolower, and replaced the table for latin-1 with range test logic. EXE file reduction of 640 bytes

Implemented "Undo memstate bitmap" - a data structure which is updated with every Z-machine memory write and used to make creation of undo snapshots faster (approx +1kB exe size)

Initial load of game header now loads directly to the z_header struct. Byte swapping code for z_header struct now loops over most of the fields rather than loading each field individually from the program memory. Program memory no longer requires realloc. Approx 500 byte code size reduction

Quetzal save/restore and the undo system now use the same memory diff/undiff code (saving 1.2k in exe size). Added code to set the buffer size on file pointers opened by os_load_story (but using file descriptor read() is still faster for opening the story file)

More changes to reduce EXE size: "Expand abbrev text" feature can now be disabled by macro. Story file identification info is now ifdef'ed out for cases where identification of those story files is only relevant to certain features (sound/graphics). Factored out common portions of z_inc/z_inc_chk and z_dec/z_dec_chk

Error messages in quetzal code are made more generic (to save memory) if NO_DEBUG is set

Updates to DOS os_read_file_name: Corrected memory leak in "restricted path" code (strdup) as well as a "jailbreak" when the file path uses forward slashes and a failure to append the file extension if the path contains a dot. PATH_SEPARATOR is now defined in the OS directories (as OS_PATH_SEPARATOR) rather than in the common directory.

Changed CFLAGS for Borland build to use compact memory model and eliminate duplicate strings

Frotz: Pared down various pieces of functionality to make the code smaller and faster on an early DOS PC

Fixed potential out-of-bounds read in gen_blorb_init().

Thanks go to Petter Sjölund for pointing out that Address Sanitizer
flagged that line as problematic.

Modified Makefile to use "which" if "command -v" isn't available.

Replace `which` with POSIX-mandated portable `command -v`
One is a program that needs installing, other is part of the shell
See, eg. https://lwn.net/Articles/874049/

Fix "auxiliary" typo

Signed-off-by: Stephen Kitt <steve@sk2.org>

Added blurb to ux_audio.h

Fix odd indentation in snavig.pl.

Fixed nonstandard pkgconf variable names.

See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1095250

Implemented script-from-start for dumb interface.

This can be helpful when doing bugtesting runs.  Simply doing
dfrotz < input.txt > output.txt
will not put anything the player types into output.txt.  By using the
new -T flag, a transcript of the session is started immediately.  Using
the new -n flag, the name of the transcript file can be set at the
command line.

Removed useless $(DEFINES) in PHONY target.

Fixed potential SDL interface build failure when using "make -j".

Shifted to development and created a new release section in ChangeLog.

curses: Make ux_defines.h a prereq of object files

Add two missing includes, to fix builds on Fedora Rawhide (GCC 15.0.1)

Quelled some warnings from Turbo C GUI compilation.

Added Turbo C for 16-bit DOS fix for 64k boundary crossing.

The LOW_WORD() macro for TurboC can't handle a zword passed as the
address because that macro assumes the first parameter is a 16-bit
value.  Maybe someday I'll find a clean way to rewrite that macro.

Updated Makefiles for Turbo C and Open Watcom C.

Updated ChangeLog and Makefile in preparation for new release.

Updated dates in manpages again.

Fatal error instead of segfault when writing a string that crosses the 64k mark.

Static and dynamic memory is restricted to below $0ffff (64k).  Past
this mark is high memory.  Frotz previously stored the address for
printing a string in a zword variable (unsigned short).  An unsigned
short is two bytes and therefore loops back to zero after passing
0xffff.  When decode_text() was given a string that straddles the 64k
boundary, the address variable looped back to zero, which then provoked
a segfault.

Per the Z-machine Standard, the proper action to take when crossing that
boundary is undefined.  If the value in the address variable is copied
to a zlong variable (unsigned long) before proceeding, printing one of
these naughty strings will succeed without complaint.  Allowing that to
happen is in direct opposition to why segfaults happen in the first
place: when a program starts writing outside your allocated memory,
that's a clear indication that the program has aquired a faulty picture
of what should be happening. Therefore I have Frotz throw a fatal error
when encountering such a print attempt.

Add src/sdl/sf_defines.h to gitignore

Updated manpages.

Updated ChangeLog with filename extension stuff.

Reworked dumb interface to add appropriate filename extensions for all files.

This reworks 915e385.

Reworked curses interface to add appropriate filename extensions for all files.

This reworks 25d5cfc.

Reworked SDL interface to add appropriate filename extensions for all files.

This reworks 1e70ac9 and 40fc04b.

For X11 interface, add appropriate filename extensions if not already there.

For saving in dumb interface, append ".qzl" to filename if not already there.

For saving in curses interface, append ".qzl" to filename if not already there.

Fixed potential source of trouble when reading file names.

For saving in SDL interface, append ".qzl" to filename if not already there.

Updated ChangeLog.

Ifdef'ed code to check if a short > 16 bits for only TOPS20.

There's reason to call s16() other than for compilation on a PDP10.  It
was introduced originally by Andam Thornton to ensure that Frotz is
36-bit clean.

Fix stray $ that breaks the frotz filename when specifying LDFLAGS

Fixed spurious complaint of missing graphics when Z-code is in the Blorb file.

For curses, dumb, sdl, x11, and dos interfaces.
Note: The code I removed from os_load_story() wasn't doing anything.

Removed m_exitPause from SDL as it caused problems without being useful.

Ensured complaint about missing graphics if blorb file is garbage.

Moved X11 pkgconf calls to X11 Makefile and allow for no pkgconf present.

Fixed X11 interface build calling pkg-config directly.

Moved SDL pkgconf calls to SDL Makefile to avoid spurious complaints.

Otherwise the build process would throw warnings of SDL libraries not
being found when building some other interface on a machine without SDL
installed.

Removed commented out stuff left over from sdl_nosound implementation.

Implemented sdl_nosound build target.

Reworked "make help" output.

Redid declaration of "avail" to satisfy ANSI.  Open Watcom C for DOS complained.

Previously Open Watcom C for DOS didn't have a problem with this.

Added some commentary for the common defines.

Moved ux_defines.h generation to src/curses/Makefile.

Added dummy functions for no sound support.

Saved the computed Blorb filename for later use.

This ensures that if a V6 game is started and we're using one of the
default Blorb filenames, a complaint about a missing Blorb file won't be
printed.

Fixed potential NULL references in the curses interface.

Fixed potential NULL references in the curses interface.

These are the same places that caused segfaults for the X11 interface.
Who knows why they didn't cause trouble for the curses interface.

Fixed many NULL references when X11 interface loads V6 without resource.

These caused xfrotz to segfault instead of continue and complain if a V6
game is loaded but no Blorb file is provided.

Update Makefile to account for no pkgconf
Before this it would try to run libpng as a program, resulting in lots of "make: libpng: No such file or directory" messages

Redid warnings for missing graphics.

The previous attempt wasn't very good.  This time around, it's not as
kludgey and the warning will appear with all combinations of Infocom's
V6 games and Frotz interfaces.

Removed superfluous include of execinfo.h.

Added ifndef NO_EXECINFO_H around include of execinfo.h.

Fixed $(EXTENSION) applied twice to executables.

Added runtime warnings when sound or graphics are used but no Blorb is loaded.

Directed players to the IF Archive to get graphics and sound for Infocom games.

Fixed typo.

Put my_strdup() and my_strndup() into separate ifdefs.

Moved my_strrchr() to src/common/missing.c and made my_basename().

I'm not sure why I didn't implement my_basename() until now.

Removed CC declaration in SDL interface Makefile.

Ensured that either pkgconf or pkg-config can be used.

Lately pkgconf is functionally the same as pkg-config.  The latter name
is deprecated.  These changes ensure that pkgconf will be tried first,
then pkg-config.  If neither are found, some defaults will be tried, the
build may or may not succeed.

Simplified means of not including stdint.h for DOS targets.

Finalized (for now) Randomization Test Suite.

Reordered display_spread() to make a little more sense.

Getting ChangeLog a bit more ready for release.

Extended the name of the randomization test to randomization test SUITE.

Laid foundation for spread test to automatically increase granularity.

Removed useless parameters fed to _bellcurve() functions.

Parked cursor out of the way for the spread test.

Cleaned up printing of bar segments and removed @set_colour calls.

Consolidated the two bin and count arrays into a single Count array.

Only one was in use at any time: one for one test and the other for the
other test.

Spread test width is now the terminal width.

Put back the Lanes variable as a Global.

Added intro to spread test.

Committed a new build of the (now named) Z-machine Randomization Test.

Commented out Serial in random.inf since this is not yet ready for release.

Added optimzations to random.inf.  6656 bytes reduced to 4096 bytes.

Changed name of randomization test to Z-machine Randomization Test

Inform hasn't much to do with this program as it's intended to
test an interpreter's behavior.

Eliminated LANES from the global variables in random.inf.

Now setting Height and Width only in Main() in random.inf.

Corrected pausing goofs for the bellcurve test in random.inf.

Fixed problem with being unable to exit the spread test.

Updated copyright and signon stuff and put random.inf in public domain.

Moved printing of the last drop count in display_bellcurve() to before the loop.

Added release and serial number display to random.inf.