Fix mnearto return value confusion

This caused appeased shopkeepers returning to their shops being killed.

Silence Yet Another Valgrind complaint

Silence some Valgrind errors

Need to have the allocated memory explicitly cleared for Valgrind
to know it.  We should probably start using calloc or something...

Modified player name list to include new team members.

Fix valgrind complaint in wizmode terrain

The code was writing characters into row[x-1] in the loop, but putting the
string terminator at row[x], leaving one character between uninitialized.

tty's compress_str(), core's dat/keyhelp

Rewrite 3.6.1's compress_str() to avoid peeking past the end of the
input string.  This should eliminate the reported valgrind complaint.
The problem was noticed for post-3.6.0 code introduced [by me...] last
June, but it looks like it was present in the old code too.

Also, fix the wording in the paragraph about NUL in the keyhelp text.

tty_putstr() always passes non-message window text through compress_str(),
clobbering usage of two spaces to separate sentences.  putstr()'s caller
ought to have more control over that (possibly via its hardly ever used
'attribute' arg?).

Fix wishing for tins of specific type

I believe this bug already existed, but was only exposed by
my wishing parser change post-3.6.0

Set max players in sysconf, not in the shell script

Impossible instead of segfault in cursed

While fuzz testing, I've seen segfault a handful of times in here,
coming from do_takeoff(). Looks like context.takeoff.what is stale,
having WORN_BLINDF, but we're not wearing the blindfold anymore.

Haven't been able to trace it down yet, so guard it with impossible.

fix #H4597 - sitting on level teleport trap

During #sit:  "You sit down.  You step on a level teleporter."
Switch to alternate phrasing for #sit.

Webs and polymorph traps had similar issues.

fix #H4706 - non-zero hit points for poison death

For "the poison was deadly" against hero, hit points were set to -1
(which gets displayed as 0 when shown) but the status lines weren't
being updated, so stale positive HP value was visible during final
disclosure.

fix some recent lint

Fix segfault when attacked while polyed into disenchanter

The pointer keeping tabs of monster's current weapon was
not cleared.

How to trigger: Get hit by a monster wielding a weapon,
teleport to another level, poly into disenchanter, get hit
by any monster with AD_PHYS attack and not wielding any weapon.

Fix some issues found with fuzz testing

Mostly to do with relocating monsters when the level is already full,
and unsticking a monster if it gets relocated.

Add missing symbols to Blank symset and Guidebook

Fix segfault when looking at detected monster hiding under unknown armor

This segfault happened when a detected monster such as a garter snake was
hiding under an unknown/unseen armor of a type that has no description
field (eg. a leather armor), and you farlooked at the monster.

At the same time, simplifies the code a tiny bit.

Merge branch 'ais523-movespeed' into NetHack-3.6.0

Make Ogresmasher grant 25 constitution

Make Dragonbane confer reflection when wielded

Fix wrong material for novels

HI_PAPER is color, PAPER is material - the novel object definition
was using the wrong one. The caused the novel material to be gold.

Changelog for movement speed changes

Make free actions easier to predict, nonfree actions more predictable

This fixes melee kiting more comprehensively (it now doesn't work
against slower monsters either), and prevents you doing things like
opening up a gap when running from an imp (you couldn't do that in
3.4.3).

Sanity check for returning Mjollnir animation

Make older corpses not taste okay

Post-3.6.0 change made practically all corpses taste okay.
Change it so there's always a 10% chance for the corpse
to taste terrible, and increase the chance if the corpse
is slightly old.

Merge branch 'ais523-elbereth' into NetHack-3.6.0

Accept custom level annotations at the level teleport prompt

via UnNetHack, original implementation by aaxelb

With fragile wands, use snap instead of break

Changelog for the Elbereth changes

Wielding Demonbane prevents demons summoning friends

Ugh, why does NH3's commit process have to differ from NH4's?

Change the rules for Elbereth erosion

If you attack a monster under Elbereth protection, and it wasn't
scuffed by the attack itself, then it'll be automatically removed
with an alignment penalty. It no longer fades from scaring monsters;
only from being abused to attack monsters while protected.

Differentiate between monster anger from attacks and from other causes

setmangry() and wakeup() were being used for multiple purposes. Add an
extra parameter to track which. This fixes several minor bugs (e.g.
whether monsters with no eyes were angered by (useless) gaze attacks
against them previously depended on the state of a UI option, and
the Minetown guards would be annoyed if you used a cursed scroll of
tame monster on a shopkeeper). It's also a prerequisite for the
Elbereth changes I'm working on.

Fix engrave.c; I used the wrong commit command

Make the Elbereth restrictions more flavour-consistent

Elbereth now has to be on a square by itself; it's hard to justify
why text before it would prevent it working if text after it fails
to prevent it working.

Wielding Trollsbane prevents trolls from reviving

Put throne room gold in the chest

X11: Handle define-lines in NetHack.ad

xrdb can load a X recource file with #define-lines, but NetHack
cannot. Substitute the defines in Makefile during compile.

Requires grep

more lint and formatting

Fix a C89/C90 complaint about a declaration following executable code.
The rest is formatting (of stuff I saw rather than systematic fixups).

Allow toggling menu for any travel target list

Occasionally remove maze dead ends, creating loops

lint fix

Show correct keybind in ext cmd list

Free allocated memory

Fix valgrind complaint

Add wizmode command to recreate current level

Sometimes create mazes with wide corridors

...and/or thick walls.

Use even more enums

Use more enums

We're already using enums, so convert some already existing defines.

Don't clobber up and down ext cmd binds

Give quest guardians some equipment

Typofix

Hero polyed into ghoul can eat only non-veggy corpses or eggs

Change via dNetHack.  Restrict heroes polymorphed into ghouls
so they can only eat non-vegan corpses and eggs.  This matches
more closely what pet ghouls prefer.

Add couple hallu monsters

Make kicking activate statue trap

Change via dNetHack

Pet starting apport equals your current charisma

keybinding lint and formatting

Use symbolic name STONE for levl typ 0

win32: Save and load map colors from registry

Fixes entry for cursor positioning via menu

Iron bars, water and lava are not interesting for travel

Accessibility: Pick travel/cursor targets from a menu

Adds two new configurable keys to the cursor targeting: 'A' (getpos.menu)
and 'a' (getpos.menu.cansee). First one shows a menu of all interesting
glyphs on the map, second one shows only those in sight.

Travel command also now obeys the "request menu" -prefix, showing
the menu with interesting targets in sight, and then traveling there.

Idea via the NetHack accessibility research by Alexei Pepers.

Show menu control keys in help menu

Add key rebinding

This is a modified version of Jason Dorje Short's key rebinding
patch, and allows also binding special keys, such as the ones
used in getloc and getpos.

One of the ways to play NetHack on nethack.alt.org is via a HTML
terminal in browser. Unfortunately this means several ctrl-key
combinations cannot be entered, because the browser intercepts
those. Similar thing applies to some international keyboard layouts
on Windows. With this patch, the user can just rebind the command
to a key that works best for them.

I've tested this on Linux TTY, X11, and Windows TTY and GUI.

tribute: The Shepherd's Crown

Add configurable symbol entry for strange object

Fix Blank symset

Show some timed properties in #timeout

Also simplify the levitation_dialogue a bit

Fix a bug binding menu command to a default object class character

Due to a wrong looping variable, it was possible to bind a menu accelerator key
to a default object class.

Give feedback just before timed levitation runs out

Add missing dark room symbol to the Guidebook

Add missing glyphs to Blank symset

Separate function to check for prefix command keys

Move getpos return values to header

Show keys in travel help wrapped in single quotes

Make it easier to add entries to the help-menu

Add completely blank symbol set

The blank symbol set can be used with screen reader software
to prevent it from reading the map symbols.

Prevent a segfault when looking at the map and many symbols
share the same character. Don't list too many symbols when
looking at those, if many share the same character.

Add full compass coordinate mode for screen readers

fixes entry

Horses have a tiny chance of being generated with a saddle

Saddles are rare. Make them less so by giving domestic saddleable
creatures 1% chance of being generated with a saddle.

Fixes entries

Show mon vs mon attack messages only when target is visible

The visibility check in mattackm doesn't guarantee both the
attacker and defender can be seen by hero. Before giving
messages, check more strictly whether we could see the
message happening - either really seeing the monster or
sensing the monster by some other means, depending on
the message.  This should remove most of the "It" messages.

Also unhide mimics who get gazed by umber hulks.  We could
keep the mimic hidden and make the messages reflect the
hulk gazing at the thing the mimic is mimicing, but this
is much easier.  This fixes bz631 / H4500

Differentiate the symset and roguesymset menu prompts

Update X11 style, define button rounding

Better default style for X11

This tries to mimic the terminal, and adds some defines making it easier
to configure.

fix #H4521 - distant squeak

Vary the message about hearing a squeak when it is close.

Fix wrong parameter order

fix bz623 - 'D' when not carrying anything

With menustyle set to "full" or "partial", using 'D' when not
carrying anything gave no feedback.  (Modes "traditional" and
"combination" give "you have nothing to drop" via ggetobj().)

Also, there's no need to reset in-progress armor removal, lock
picking, or trap setting if you don't actually drop anything.
The inventory they're set to operate on or with stays intact.

Expand blind/accessibility section in the Guidebook

corpse eating feedback

Eating a corpse and being told that it "tastes okay" (relatively
recent change so that omnivores don't find the taste of everything
to be "terrible") doesn't warrant an exclamation point.

For travel target location, report if no known travel path

Yet another accessibility feature.  When asked for a location
to travel, and autodescribe is on, the location description
has "(no travel path)" appended, if there is no known path
to that location.

Accessibility messages when run or travel is interrupted

These were inspired by the blind accessibility changes by
Alexei Pepers.

Also pass raw printed texts to the msghandler

Wait for the message handler process to finish

fix #H4492 - ing_suffix("throw") gave "throwwing"

The bug report assumed "you mime throwwing something" feedback
from 't-' was a typo, but 'throwwing' gets generated from 'throw'.
Change ing_suffix() not to double final 'w'.  Presumeably 'w' and 'y'
are exceptions because they're sometimes used as vowels.

Change 'strrchr()' to 'rindex()' like the rest of nethack.  Someday
those will need to be switched the other way around.

Add some missing bounds checking, although since ing_suffix() isn't
used for user-supplied strings, that's probably superfluous.

Fix Guidebook typos

Fix tribute typos

Ran the tribute file through a spellchecker, and double-checked with Pat as he
added most of the tributes.

fix #H4482 - "a some <foo> corpses"

farlook was changed (end of December) to use doname instead of xname
to yield more info for items which had already been seen up close,
but it gave away info about ones which hadn't.  So doname was changed
(end of April) to use "some" instead of precise quantity (when the
quantity is greater than 1) for the latter, but that doesn't work
well with corpse_xname() when the hero is blind, yielding "a some
<foo> corpses".  While testing the first fix attempt, I noticed that
pickup gave "you can only lift some of the some <foo> corpses".

This fix is far from perfect.  farlook can still say "some <item>s"
but lookhere and pickup always say "N <item>s".  Picking up a stack
while blind will show "N <item>s" in inventory display, but dropping
it while still blind will revert to "some <item>s" for farlook.

fix #H4476 - monster class via member of class

Blessed genocide of "titans" wiped out all quadrupeds because
"titan" is a prefix of "titanothere".  After class letters and
class descriptions have been rejected, Have name_to_monclass()
resort to name_to_mon() instead of doing its own less detailed
name matching.

fix #H4475 - shop message about disenchanted item

drain_item() always assumed player was responsible, so called
costly_alteration() to adjust shop price of disenchanted item.
If it was unpaid and the effect was caused by a disenchanter
attack rather than by the hero, the feedback was nonsensical.

This also lets a disenchanter hit worn rings, amulet, or blindfold
if no armor gets targetted.  Amulets, blindfolds, and most rings
have no charge to be drained, but several types of rings do.

fix #H4472 & #H4473 - incorrect total_tiles_used

The value calculated for total_tiles_used never got adjusted for
displaying statues-as-monsters.  The most common configuration(s)
using tiles don't care, but the combination of X11 plus USE_XPM
needs an accurate value there.