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.

Set spread test to get height from $20 instead of $24.

    Per the Z-machine Standard 1.0 and probably earlier as well, location
    $24 in the Z-machine header contains the "screen hight in units" whereas
    $20 contains "screen height in characters".  Usually "units" is
    equivalent to "characters", but might not always be true.

    I also got rid of excessive blank lines in Main_spread() which was
    probably left over from when I pulled in Andrew Hunter's code.

Fixed mistake of "top == 0;" in bellcurve test.

Mentioned failure to do all processing in src/misc/snavig.pl.

Added mention of X11 (Xaw) for graphics support.

Added the Frotz icon, renamed existing to a banner.

Bumped "last updated" for webpage.

Update standard version in webpage

Originally submitted against 794a137dbfbd18e8369f6042f01c76c327583eb5
Reworked and committed again by David Griffith <dave@661.org>.

Only skip separators if the separator count is >0

Orginally submitted against 0c8af37b6a50cbad279e6ed9bc8aa1888574e73d.
Reworked and commited again by David Griffith <dave@661.org>.

Fix indentation in common/text.c

os_fatal() needs to call dumb_show_screen() first otherwise print_string() etc won't print.

For some yet unknown reason, the dumb interface needs to have its screen refreshed by a
call to dumb_show_screen() prior to anything happening in os_fatal().  If not done,
anything that was supposed to be printed through print_string(), print_num(), or
new_line() won't get printed at all.  os_warn() doesn't have this problem.

Updated the fix for Beyond Zork's rotating mirror bug by now correctly
detecting attempts to address an invalid object.

Beyond Zork release 60 contains a bug wherein the dictionary entry for
the circlet is incorrectly treated as an object when blown.  This turned
out to be object number 41460 (or similar).  Its entry in the object
table would reside far outside the expanse of memory the Z-machine can
access.  Infocom's interpreter simply returns zero when the zcode
checked what turns out to be an invalid object and the game continues
running as if nothing was wrong.  Back when Zip was current, this bug
was worked around by checking if the game is Beyond Zork and the object
number was over an arbitrarily large number (2000) and if so just return
zero.  I rewrote this code to check ALL accesses to the object table,
throw a fatal error if the access is invalid, and return zero if the
game happens to be Beyond Zork.

See these resources for more information:
https://ifarchive.org/if-archive/infocom/interpreters/zip/zip_bugs.txt
https://intfiction.org/t/beyond-zork-passing-invalid-values-to-get-prop-addr/48347
https://intfiction.org/t/z-machine-undefined-behavior/44990
https://intfiction.org/t/z-machine-standard-unclear-aspects-ambiguities/64128

Added commentary about failing to properly internalize the substitutions.

Made symbol shortening more versatile.

Be a bit clearer about which version of sed is being used.