Comment typofixes

Comment typofix

fountain.c formatting

Add missing thrownobj clears when the obj is destroyed

'v' vs '%'

Guard against the possibility of BETA_INFO (or other fields used to
construct the one-line version string) having a percent sign that would
be misinterpreted during pline processing.

sp_lev.c warnings

Fix a couple of warnings about prototype not matching function definition.
Prototype specified a narrow scalar ('boolean') but function uses old-style
definition which implicitly promotes narrow types (char,short,'boolean' to
int, float to double).  Switch prototypes to BOOLEAN_P.

A whole bunch of prototypes specify STATIC_DCL ('static') and then leave
the definition without STATIC_OVL (also 'static').  I only fixed up a
couple of those, but we risk getting somebody using an old compiler that
doesn't like
  static int foo();
  int foo() { return 42; }
Even if every comiler accepts that (I don't recall whether that was kosher
in the ancient K&R days), you can't tell by looking at the function
defintion (in the usual case where the forward declarations are at the top
of the file rather than immediately before the defintiion) whether it's
local to that file or global, so the missing STATIC_OVL qualifiers ought
to be added.

fix inappropriate merge during aborted throw

Fix the situation reported by Pasi where cancelling a throw via ESC at
the "which direction?" prompt resulted in "object null after merge"
panic from addinv().  I wasn't able to reproduce the panic, but I could
see the throw-nowhere stack getting merged into the quiver stack when
it shouldn't.  This fixes that.

Also a couple of formatting bits.  I wish I'd quit coming across those.

Fix segfault when the Wiz fails to reappear due to makemon failing

Fix infinite loop if god cannot fix your trouble

For example, if safe_teleds cannot land you in a safe spot,
because level is full of monsters.

Fix C343-341: Stuck in minetown room

Bustling Town can be generated with inaccessible areas outside the
top edge of the fixed town map.  If you end up in one of those area
without any way to dig or teleport, you're stuck.

This adds a new level flag "inaccessibles" to force checking for
such inaccessible areas, and add secret doors, holes/trapdoors,
or some random escape item into the areas.

Wet towels deal more damage

Dipping a towel into a potion, fountain, or some other water source
makes the towel wet.  Hitting with a wet towel deals up to 6 points
of damage, but every hit reduces wetness, as does throwing or applying
the towel.  You can also wish for a moist or wet towel.

tribute: The Truth

Avoid referencing a real mental disorder.

Add at least one quote from the final novel now that it is published

Fix the makemon random coord picking again

This time try random locations up to 50 times, then start going
through the map in order to find a good position. First round
tries to pick a location not in sight, if that fails, it might
try stair or ladder location. If that fails, then it will pick
any good position, whether in sight or not.

Outside field of vision restriction and trying stair or ladder
locations does not happen when the monster is placed from special
level code.

wishing for novels

Make novels be wishable in normal and explore modes in addition to
wizard mode.  I don't think this weakens the tribute and it prevents
someone who attempts such a wish from getting misleading feedback of
"Nothing fitting that description exists in the game."

Wishing for "novel" will yield "novel named Foo" where "Foo" is a
randomly chosen Discworld title.  Wishing for "novel named Bar" will
yield "novel named Bar" or "novel named The Bar" if "Bar" or "The Bar"
is a valid Discworld title, or else override "Bar" and pick random
Discworld "novel named Foo" if it isn't.

Since first read of a novel bestows some experience (once per game, no
matter how many novels become available), a pacifist with an early
wish can get a head start.  I don't think that's a big deal.  And it
will require an awful lot of wishes for any player who wants to acquire
all 41 titles in one game.  I imagine someone will manage it.

Fix segfault when tripping over something

If you tripped over something, and there were no rocks on
that place, the otmp object variable was set to null but used
later for checking if it was a corpse.

Extra checks when freeing objects and monsters

Just in case, so we don't leave objects or monsters inaccessible
by accident.

Add monster data sanity checking

No need to sanity check objects of a dead monster

Ensure null nobj pointer when object is extracted from list

Put monsters into limbo if they cannot be placed

Whenever mnearto tries to displace a monster from underneath
another, and the displaced one cannot be placed anywhere,
make it drop special objects and put it into migration, with
the current level as the target.

This should be a good enough stopgap measure - it's not going
to happen unless the level is (nearly) full of monsters.

And it seems to cure a near-impossible-to-track data corruption,
with monster list pointing to garbage.

Fix infinite loop in makemon if no goodpos available

If the level was full of monsters, goodpos never returns TRUE,
and tryct would not get incremented.

tribute: The Fifth Elephant

New true rumors, one new hallu mon

Fix cursed scroll of light on the rogue level

Being blind and engulfed, reading cursed scroll of light did still darken
the room on the Rogue level.  This also changes the logic a bit, so that
cursed scroll of light will douse lamps in your inventory, even when
swallowed.

litroom() vs Blind

An item from the "A few bugs" mail was that reading a scroll of light
when swallowed didn't light the surrounding area--which is intended--
but that doing so while blind *did*.  The logic in litroom()--which
the report was based on--was wrong, but do_clear_area() prevented the
light was escaping the engulfer.  So there was no bug from the player's
perspective, but only because the vision code has special handling for
being swallowed.  This fixes litroom()'s logic and does some formatting
cleanup.

spell of protection

Another item from the "A few bugs" mail.  Casting spell of protection
when previous casting(s) hadn't timed out yet miscalculated the new AC
boost.  At low levels--when this spell probably gets its most use--the
bug wasn't noticeable.  (At high levels when someone might cast it a
whole bunch of times in succession, the effect could be noticed but
was probably just assumed to be working as intended.  Its behavior is
somewhat convoluted.)

MUSE for branch stairs

Fix another item in the "A few bugs" mail.  Monsters who wanted to flee
weren't able to use 'sstairs' (extra stairway leading to different branch
of dungeon) due to a logic error in the find_defensive() choices.
  if (terrain==STAIR) {
  } else if (terrain==LADDER) {
  } else if (x==sstairs.sx && y==sstairs.sy) {
  } else { /* check traps */
  }
wouldn't find 'sstairs' because they have terrain type STAIRS.  (Also,
the sstairs check wasn't screening out immobile monsters, but that bug
didn't have a chance to manifest.)

There's a bunch of reformatting, and some code re-organization to improve
other formatting, and some additional logic fixes.

some mthrowu.c formatting cleanup

setmnotwielded should always MON_NOWEP

Instead of making the caller remember to use MON_NOWEP, make
setmnotwielded handle that automatically.  This fixes the
"bad monster weapon restore" errors I've been seeing.

Also adds sanity checks for this.

flesh golem healing

Fix one of the entries in the "A few bugs" mail.  Flesh golems hit by
electric damage calculated a healing amount of dam/6, then ignored it
and used dam instead.  Probably never noticed in actual play....

sanity check for embedded dragon scales

The problem discovered with sanity checking of embedded dragon scales
was with the checking, not with the object flagged "embedded in skin".
I thought W_ARM got cleared when switching 'uarm' object to 'uskin',
but it isn't.

tribute: Capre Jugulum

Document persistent options

Some options are applied from config file only to new games,
so document those.

tribute: The Last Continent

Refer to relevant config options in the Guidebook

Give the reader a hint what to look for if they want to configure
the message line.

Add paranoid option for breaking a wand

Add OPTION=paranoid_confirm:wand to prevent accidental
wand breaking - suggested by scorchgeek

tribute typos

tribute: Jingo

tribute: Hogfather

I changed the first quote to use the American spelling (snowplow vs
snowplough) found in the edition I have on hand.

Passages for several more books are still queued up.  Transcribing them
is very tedious.

One more scroll name, a tongue twister

Jump-and-bump should wake up monsters

It was possible to wake up single monsters by jumping at the monster.
Make such bumping somewhat noisy.

novel bit

Feedback phrasing:  don't refer to a novel as a "spellbook" when
rejecting the attempt to write on one with a magic marker.

tribute: Small Gods

Improve wand of nothing randomizing

As per Sean's suggestion, just use rn2. This means the line
can also be moved to better place, with the other object
init stuff.

Make wand of nothing zap ID harder

Just to mess with zap identification, make wands of nothing
either ask for direction or not, determined at game start.

VMS update from KevinS (2 of 2: sys/vms)

Update the command procedure that can be used to compile and link
everything.  The sys/vms Makefiles still need updating.

I added 'tribute' to the command procedure used to create and populate
the plaground directory but am not able to test it.

VMS updates from KevinS (1 of 2: core)

His changes add file_exists() to sys/vms/vmsfiles.c, which might be
needed in the future but is not useful here.  Checking for 'gbd' and
'grep' for PANICTRACE support (which I had working on Alpha--and
ought to work on VAX, no idea about IA64--before losing access to vms)
is pointless since neither is used.

src/files.c
  SYSCF support, parse_config_line():
    #if VMS, ignore GDBPATH and GREPPATH if present in sysconf;

  SYSCF support, assure_syscf_file():
    #if VMS, force open()'s optional third argument since the macro
    which redirects open to vms_open requires it.

src/sp_lev.c
  selection_do_randline():
    use configuration-specified 'Rand()' rather than raw 'rand()'.

Add new scroll names

via UnNetHack, with some slight changes:

1) Folded the two Portal references into one
2) Removed "ACHAT SHTAYIM SHALOSH" ("One Two Three" in Hebrew and apparently
   Uri Geller's catchphrase) - I know nothing about Hebrew nor Geller, or
   whether this would be appropriate to add.
3) Added "XOR OTA" ("Atorox", reference to Finnish fandom and early scifi)

Fix typo "promt"

unicorn horn vs deafness

Implement the suggestion that applying a non-cursed unicorn horn can
cure deafness like other similar troubles.  Also, applying a cursed one
can cause deafness, although I made the chance be half of what it is
for the other troubles since they tend to be more significant.

This is entry #2 on the bugzilla list, but I haven't figured out how to
update that yet.

branch traversal via W quest artifact

Require the hero to pass the next_to_u() check when using the Eye of the
Aethiopica to portal to another dungeon branch.  Even though the reported
exploit of having a steed which is carrying the Amulet bring it along had
already been prevented, this changes the temporary portal behavior to be
like level teleport.  A steed carrying the Amulet or a non-adjacent pet
on a cursed leash will inhibit the attempted change of location.

couple of tribute typos

Add a chroot install hints file for linux

This makes installing NetHack on a public server much easier.
Required some minor changes to the unix top-level Makefile, and
the other hints files; Makefile variable HACKDIR may not be a
full path anymore, use INSTDIR instead.

tribute: Feet of Clay

fix up Guidebook.mn's preface

Make the preface have an unnumbered header and force a new paragraph
  for its text;
remove the unterminated switch to bold font;
change "Terry Pratchett" from bold to italics to match Guidebook.tex's
  setting.

wear/remove commands, P & R vs W & T

Allow 'P' and 'R' commands to accept armor and wear/take-off the chosen
item, and 'W' and 'T' commands to accept accessories and put-on/remove
the item.  The which-object prompt only lists the type(s) of items that
traditionally go with each command, as does an inventory menu if the
user picks '?', but items of the alternate type(s) can be chosen, by
unshown letter or by the inventory menu given for '*'.

There shouldn't be much difference if you continue picking items that
go with the original commands, although you will somestimes get
"which object? [*]" when the only choices are for alternate command.
And you won't see the all-four-accessories-are-already-worn message
for 'P' unless you also have something worn in all seven armor slots.

The Guidebook.mn changes have been tested (that's how/why I noticed
the preface glitch) but the corresponding Guidebook.tex ones haven't.

Revert "Add tombstone tribute to S.T.P. for 3.6.0"

This reverts commit 64dfb4fcc871d77901f50104594aa79088e71cec.

Revert "Properly define NH360_DEDICATION in Makefiles"

This reverts commit 3db39ca221f34d6f606f7b2854f7f9be7744fce1.

obj bypass: move vs turn

Reported by ais; clearing object bypass bits once per turn isn't often
enough.  Clear them after the hero moves (which might be more than once
in a turn) and before each monster moves (ditto) and after last monster
moves.  This might not be optimal but that shouldn't matter since it's
usually a no-op.

Properly define NH360_DEDICATION in Makefiles

tribute: Raising Steam

Plus a couple miscellaneous typo fixes.

tribute typos

tribute: Maskerade

W_WEAPON, W_ACCESSORY

Add macros W_WEAPON and W_ACCESSORY, similar to existing W_ARMOR, bitmask
of all the relevant worn bits.  Just for code readability; there should
be no change in behavior.

Also, reformat the "ugly checks" portion of getobj().  Slightly better
readability and fewer continuation lines, but only a modest improvement.

Add tombstone tribute to S.T.P. for 3.6.0

I've added build files for unices, but other platforms will need to define
NH360_DEDICATION on their own.

A little bit more tribute.

Also clean up a pline->pline1 in the tribute code.

tribute: Interesting Times

I have several psssages for Maskerade too, but after the time and effort
spent fixing up the ones already present for that book, they'll have to
wait until some other occasion.

Makefile typo

tribute: Soul Music

formatting fixup (1 of 2)

Replace instances of strings split across lines which rely on C89/C90
implicit concatenation of string literals to splice them together
with single strings that are outdented relative to the code that uses
them.  It's uglier but it won't break compile for pre-ANSI compilers.

This covers many files in src/ that only have one or two such split
strings.  There are several more files which have three or more.  Those
will eventually be '(2 of 2)'.

Noticed along the way:  the fake mail message/subject
  Report bugs to devteam@nethack.org.
wasn't using its format string of "Report bugs to %s.", so would have
just shown our email address.  Doesn't anybody enable fake mail anymore?

I modified that format to enclose the address within angle brackets and
made a similar change for the 'contact' choice of the '?' command.

Fix dark_room glyphs when restored from save

If color or dark_room options were toggled during gameplay, and then
the game is saved and restored with different options, the dark room
glyphs were wrong.

Reported by both Boudewijn and Pat.

Fix more dark_room and nocolor

Fixes for example the case of searching while blind, as
reported by Boudewijn.

Fix nocolor messing with dark_room on levelchange

As reported by Pat:
> After leaving a level and then returning, I'm seeing lit
> room squares as blank
> Options all have their default settings

OPTIONS=nocolor is the default; we should probably change this
sometime, because nearly everyone plays with color.

quest message summaries: knight and monk

Six roles down, seven to go....

questpgr: plural neuter pronoun

%o[hij] relied on makesingular() converting "the Eyes of the Overworld"
into "the Eye of the Overworld" to recognize when it should use
they/them/their instead of it/it/its, but makesingular() was changed to
keep "eyes" intract instead of stripping the 's'.  So qtext_pronoun()
needs to check for "Eyes" itself.

healer quest messsage summaries

Add summary lines for the deliver-by-window messages of the Healer
quest.  Also, replace a bunch of hard-coded pronouns for leader,
nemesis, and in one case, patron deity in the Healer messages plus
the three roles which precede it (Arc, Bar, Cav).

tribute: Men at Arms

The breakfast+coffee+doughnut one is by the far the longest we've had,
but I decided not to try trimming any of it out.  It was at the start
of a 10 page stretch that has six good passages, but I've only included
three of them.

I've got passages for two more books queued up, but transcribing and
proofreading is a chore....

'/' enhancement

Add new entries to the menu used for the '/' command:  describe nearby
monsters, describe all shown monsters, describe nearby objects, and
describe all shown objects.  It makes a text window listing monsters--
or objects--currently displayed on the map, showing lines of
X  r,c  monster-or-object description
where 'X' is symbol (monster or object class letter for tty), 'r,c' is
row and column separated by comma, and description is similar to what
using '/y' or ';' manually would provide (how-seen info is omitted
when listing monsters).

Originally intended for blind players using screen readers to describe
what is displayed, but will probably get used by other players too.

The map doesn't use a separate set of glyphs for objects which are the
tops of piles, so the information that there are additional objects
beneath the ones shown isn't available to '/' and ';'.  That feels like
a bug to me....

vampshifter messages

If you've just received the "dead bat revives as a vampire" message,
suppress the rather obscure "Maybe not..." message given when an unseen
creature gets life-saved since it ends up being out of sequence.  You
could get them both when the death/revival was sensed via telepathy
without being in sight.

Also, some formatting cleanup and a couple miscellaneous code tweaks.

\#overview quest feedback

Implement the requested feature to have an automatic annotation on the
dungeon level with the quest entry portal where you sense the leader
summoning you.  It stays even after entering the portal (which results
in another automatic annotation of "portal to quest"), up until you
return to that level after having completed the quest.

Add a second one for the quest home level once the leader has given
you the go ahead to start the quest.  After completing the quest that
one remains but its wording is changed.

This ought to haved incremented EDITLEVEL but I decided to risk leaving
current save files viable.  That should work ok for anyone who isn't
overriding the default definition of Bitfield(), although odd behavior
could conceivably occur.  New games have nothing to worry about.

another do.c formatting bit

do.c formatting

Fix a couple of instances of a mis-indented block comment which happens
to be immediately preceded by an end-of-line comment.  Change a couple of
|  if (condition)
|    something;
|  else {
|    other_stuff;
|  }
to have braces around the 'then' part.  Remove some gratuitous 'register'
declarations.

boot weights

Make both pairs of iron boots weigh the same (kicking boots become
heavier).  Make boots of water walking weigh the same as levitation
and elven boots (slightly lighter; 3/4 as much as speed, fumbling,
and mundane high boots).

epitaph punctuation

Clean up some of the epitaphs.  Mostly to always use two spaces when
separating sentences.

A lot of them have terminating punctuation, and a lot of them don't.
I mostly left those alone; should they be made consistent?

Comment typofix

TTY: Separate hilite_pile option

relocate a couple of options in the Guidebook

separate option to toggle hilite_pile

Beta tester report.

The use of use_inverse meant that you couldn't selectively
choose to hilite pets but not piles. Add its own option.

fix windows console bug reveals monster (console tty)

Merge branch 'master' of https://rodney.nethack.org:20040/git/NHsource

guidebook

Force full-level wallify for orctown

Betatesters noted the "Orc town" variant of minetown had
"black" (solid stone) walls, so fix those.

Merge branch 'master' of https://rodney.nethack.org:20040/git/NHsource

we left one beta tester off the list somehow
 Changes to be committed:
modified:   dat/history
modified:   doc/Guidebook.mn
modified:   doc/Guidebook.tex

try not to miss ! marks

update beta #

I cut a new windows binary for one of the beta testers.

Slight improvement to handless pets' pickup changes

Allow one item to be taken out of a pile, and leave framework in place
for partial splits so that all monsters will take up to their capacity,
rather than leaving the whole pile if it's too big to take all at once.

tiles.bmp doesn't need to be in the binary folder