Backport change from 3.0.0: GCU: use waddnstr() in the text hook rather than using a fixed size temporary buffer to insert a terminating null character

Backport change from 3.0.0: GCU: remove assumed window width of 80 columns in wipe hook

English: remove duplicated letter in birth screen reroll prompt.

mac workflow:  for universal builds, include what configure would normally set for the compiler flags

Mac workflow: also don't override the default SDK.

Mac workflow: update comment about the macOS SDK.

Mac:  update notes in src/cocoa/AppDelegate.m for working with Xcode 13

Better handle if the font recorded n the preferences is not available.  Likely resolves the underlying problem in https://github.com/backwardsEric/hengband/issues/19 .

Test for -fno-strength-reduce support before including it in the compiler flags; avoids warnings from clang about an unsupported optimization flag.

macOS:  use an uninstall-local rule rather than a distclean-local rule as the counterpart to the actions from an install-data-local rule.

Simply use a Boolean directly rather than do an explicit comparison with YES or NO.

macOS:  avoid unused variable warning when compiled with -DNDEBUG.

Remove handling of open_when_ready - never set to true and appears to be vestigial.

MacOS:  rationalize handling of Booleans.  Use bool/TRUE/FALSE for data interchanged with Hengband's core and BOOL/YES/NO for stuff that's only touched by the Objective C code.  Remove use of Boolean (historical Mac usage).

Use AC_CONFIG_FILES() to set the files written by configure. Call AC_OUTPUT() with no arguments to match its current form and avoid a warning message from autotools.

Drop use of AC_TYPE_SIGNAL since it is obsolete when assuming C89 or better.

Drop use of AC_HEADER_TIME since recent versions of autotools complain about it being obsolescent.

Drop AC_HEADER_STDC since recent versions of autotools complain about it being obsolescent. Drop check for strings.h since no source files currently use it.

Backport change from 3.0.0: replace "the" with "this" in Robin Hood's description.

Backport change from 3.0.0: fix misuse of "it's".

Declare internal routine as static to avoid warning about missing prototype.

Rename local variable in playSound to avoid warning about it shadowing a function parameter.

Adopt change to mouse event handling (currently unused in Hengband) from Angband: check for isKeyWindow set to true on [event window] before processing mouse events. That way the initial click in the main window when some other window has focus only changes focus which is the typical Mac behavior.

Adjust double-height tile handling (not currently used in Hengband) to better match up with the current main-cocoa.m in Angband.

Tweak the default window layout since rendering changes and constraining window sizes to be a multiple o fthe tile size cause overlap with the old layout (at least on an early 2015 13" MacBook Pro; native resolution is 2560 x 1600; using the English interface).  Set the sizes so they should work without overlap using the default fonts (Menlo/Hiragino Maru Gothic 13 pt for the main window; 10 pt for the rest) on the smallest native resolution for currently supported Retina Macs (MacBook; 2304 x 1440).

When using the median advance to compute the tile size, round the width to the nearest integer (unless it would be zero) rather than round up. That can give a more compact display (for the default font in subwindows the width of each tile will be 6 pixels rather than 7 in English and Japanese; there's no effect on the width per tile in the main window with the default font).

Use 32 as the minimum for maxBits to match the declared dimension for window_flag_desc. Update to use unsigned arithmetic for the bits placed in the mask to match it's declaration.

Adopt optimization from Angband: only call makeKeyAnOrderFront in AngbandUpdateWindowVisibility() if there was a mismatch between which subwindows the core had flagged as active and which subwindows the Mac front end was displaying.

Add workflows to test that Mac and Linux builds work.

Had renamed the mask file for the Adam Bolt's tileset to 16x16-mask.bmp (to make it clearer what the mask was for).  Adjust the path in main-win.c to match that.

No longer use Angband 4's version of the tiles (with additional preference files to patch up the differences with Hengband's version).  Instead, convert Hengband's versions to PNG with lib/xtra/graf/osx_bmp2png.py script so they're usable with the Mac front-end.  Store the results of the conversion in the repository (so the prerequisites for osx_bmp2png.py aren't needed for ordinary builds; only when the tiles have to be reconverted).  Copy over the converted results during installation.

Modified to handle Hengband's conversion requirements: allow use of a separate bmp input file to specify the transparency mask (as is used for Adam Bolt's tile set) and adjust the defaults for the --terrain option so the conversion of the original tile set is easy.

Add the 1.0 version of Adam Bolt's tile set. Downloaded as heng-graf-16x16.tar.gz (file dated 2003/8/6) from Hengband's osdn page.

Copy over the bmp to png conversion script from sil-q (lib/xtra/graf/osx_bmp2png.py as of 2020/8/24) so can adapt it to convert Hengband's versions of the tiles rather than reusing Angband 4's versions.

Correct sanity check for a truncated Kanji character at the end of the changed sequence passed to Term_text_cocoa(). Resolves the report of uncleared character in the Japanese version when changing between pages in teh character description (No3.png in https://osdn.net/projects/hengbandforosx/ticket/41228 ). Also fixes at least some of the cases where the Japanese version garbles item names displayed in the shops (No4.png in ticket 41228).

Since Cataline no longer includes the Osaka font (it does download it on demand if requested), use the Hiragino Maru Gothic ProN ("HiraMaruProN-W4") as the default font when compiled for Japanese.

In the default preferences, use the keys, FontName-0 and FontSize-0, since those, rather than FontName and FontSize, are what are examined to get the name and sie of the font for the main window.

In drawWChar(), allow for the unlikely possibility that the character to be rendered requires two UniChars (as a surrogate pair). Previously had assumed that only one UniChar was needed and cast the whcar_t to a UniChar.

Bring over Term_mbcs_cocoa() (currently unused in Hengband) with more checks for bad UTF-8 from Angband.

In case conversion from EUC-JP to Unicode requires surrogate pairs for some characters, use CFStringIsSurrogateHighCharacter() and CFStringGetLongCharacterForSurrogatePair() in the conversion. Both require 10.6.

Fixed up another multi-line comment.

Fixed up the formatting of more multi-line comments.

Be more consistent about the format of comments.

Aligned misaligned indent.

Removed extraneous semicolon.

Corrected two misspellings in comments. Added an article in a comment for clarity.

Since the contents of Term aren't used, don't bother to call Term_activate() from drawRect().

Moved some commented out code and adjusted the comment to match up with what's in Angband.

Changed to use VERSION_NAME rather than Hengband in a log message.

Use VERSION_NAME rather than Hengband when setting name of the directory with the user's files.

Changed some whitespace to better match Angband's Mac OS X front end.

Removed synthesize statements that are redundant with the property declarations.

Moved comment so that it makes more sense.

Fix some whitespace from a recent change.

Update the instructions about rebuilding the nib file for Xcode 12.

Corrected an incorrect verb tense and made some wording changes in teh comments about how to rebuild the nib file.

In the comments about rebuilding the nib file, corrected the paths given for MainMenu.xib and MainMenu.nib in the source directories.

Moved loadPrefs(), initWindows(), and linkTermData() from the application delegate to file-level static functions: that's where they were in the past and they weren't gaining anything by being in the delegate.

Rearranged whitespace in some comments

Group Term_mbcs_cocoa() with the other Term_* functions.

Wasn't considering the full GLYPH_COUNT glyphs in updateGlyphInfo() (regression introduced with the change to dynamically allocate working space). Fixed.

Changed to remove the empty .deps directory in 'make distclean'.

Updated comment for create_angband_image() since it no longer looks up the path in the bundle directory.

In Term_pict_cocoa(), step through the passed in arrays in increments of the tile width scaling (i.e. 2 in big tile mode; 1 otherwise). Does not matter in the current configuration, with always_pict off, but would matter if always_pict was on or if the core terminal code was modified to pass more than one tile at a time when higher_pict is on.

Changed to update the inventory window after eating a staff or wand.

In init_saved_floors(), only check for out-of-date save floor files if savefile is not an empty string.  That prevents spurious messages about old temporary files when "New" is selected in the Mac OS X front-end.

Extended internal buffer to avoid overflow in English diary entry about player's death.

The setting for big tiles initially shown in the menu wasn't being initialized from the stored value in the settings and was always shown as big tiles enabled. Fixed.

Changed the Macintosh portion of the English readme with the instructions for the Mac OS X version.

Provide splash screen prompt in the EUC-JP encoding since the source files aren't preprocessed by nkf in 1.6.2.

The transfer of Mac-specific stuff to src/Makefile.am was incomplete.  Added what was missing.

Added trigger names for the keycodes sent by the Mac OS X interface.  Have the macros use the trigger names so most macros can be the same for the classic Mac interface and the Mac OS X one.

Except for specific tile set workarounds, treat ANGBAND_SYS=coc (the Mac OS X front end) the same as ANGBAND_SYS=mac (the classic Mac front end).

Went back to using "coc" for ANGBAND_SYS in the Mac OS X interface to avoid conflicts with the classic Mac interface.

Brought over .gitignore from the master branch.

To avoid a fatal "cannot set permissions correctly" error, set player_egid (and to be safe, player_euid) before calling init_angband().  Did not determine why setting player_egid after the init_angband() call happens to work in Angband and 2.2.1.7.

Changed configure.ac and the files for automake to handle building and installing the Mac OS X version.  Mostly matches what's in 2.2.1.7 with the notable excetion that set USE_TRANSPARENCY, SAFE_DIRECTORY, and VERSION_STRING in configure.ac for the Mac version (USE_TRANSPARENCY isn't used in 2.2.1.7; the others could be but currently aren't used there).

Since 1.6.2 doesn't set TERM_LEN, TERM_COLOR, and concptr, use the matching types for the terminal hooks are compatible.

Brought over the Mac OS X interface from 2.2.1.7.  Changed ANGBAND_SYS to "mac" (no older Mac OS X interface unlike in 2.2.1.7 and not worrying about conflict with the classic Mac interface).  Did not bring over all the changelog history; see the main branch for that.

Angband 4.2's version of the original tiles, 8x8.png, does not always line up with Hengband's 8x8.bmp.  Since 8x8.png is used on the Mac, added workarounds in lib/pref/graf-ori.prf which is conditionally included in graf-xxx.prf.

Angband's version of Adam Bolt's tiles does not always line up with Hengband's version.  Since use Angband's version on the Mac, added workarounds in lib/pref/graf-abt.prf which is conditionally included in graf-new.prf.

Added graf field since Hengband does not ANGBAND_GRAF to store the preference file name.

Added comment about how Hengband handles the directory for the tile sets and where it puts the prf files.

Commented out the tile sets which don't already have .prf files in Hengband.

Brought over the tile set configuration file lib/tiles/list.txt from Angband and the two tile sets, lib/tiles/adam-bolt/16x16.png and lib/tiles/old/8x8.png, that Angband uses that are also used by Hengband. For placement in the Hengband directory, replace "lib/tiles" in the path names with "lib/xtra/graf". Did not bring over Angband's prf files for the two tile sets: use Hengband's versions in lib/pref.

Replaced "tiles set" with "tile set" in error message.

Since ANGBAND_GRAF in Hengband is not the preference file name, added an extra field in the configuration to be what ANGBAND_GRAF will be assigned for a tile set. Largely replaced the implementation in grafmode.c with something that will work with Hengband's routines. If Hengband chooses to adopt the datafile.h/datafile.c parsing from Angband, then it would make sense to go back to Angband's version of grafmode.c.

For now, keep the GRAPHICS_NONE preprocessor macro in defines.h: comment out the equivalent definition in grafmode.h.

Added comment to explain relationship with the rest of Hengband.

Imported from Angband 4 to support an OS X main program, main-cocoa.m, which expected that style of tile set configuration. These are the upstream versions, commit 6cbfed5 for grafmode.c and commit 6583275 for grafmode.h in the angband/angband respository, https://github.com/angband/angband .

Added create_needed_dirs() for use on platforms that do not store everything in lib.  Modified main.c to make use of create_needed_dirs() in the PRIVATE_USER_PATH case rather than the private function it had used.

In preparation for Mac OS X version, skip the contents of main.c if MACH_O_COCOA is set.

In preparation for a Mac OS X version, allow both Unix and Mac line termination in my_fgets() if MACH_O_COCOA is set.

In preparation for a Mac OS X version, disable PRIVATE_USER_PATH and SAVEFILE_USE_UID if MACH_O_COCOA is set.

Replaced instances of fprintf(fd, s) with fputs(s, fd) as has been done in 2.2.1.7; that avoids potential problems if s contains something that looks like a format specifier and gets rid of compiler warnings.

Added casts for cases where msg_format/format had %ld specifiers but the argument passed wasn't equivalent to a long int.

Avoided compiler warnings about a 32-bit integer being passed to a %ld or %lu format specifier by adding casts, changing the specifier, or changing the type of a local variable.

Bring over habu's 2012 November 18 change, autoconfから--enable-c99を削除し、stdint.hの有無で判定するよう変更 , to use stdint.h as the basis for configuring fixed-size integer types.  Avoids a problem with the initialization of the random number generator when compiling on Mac OS X.

Changed init_file_paths() to treat the passed arguments as readonly in preparation for bringing in a Mac OS X front end.

Updated comment for init_file_paths() to reflect PHO's patch.  Most of the comment text has been copied from 2.2.1.7 (there hasn't been any change to the behavior besides PHO's patch between 1.6.2 and 2.2.1.7).