finish implementing pmatchregex

I started out cleaning up a bit of lint in the recent run-time options
handling and discovered that pmatchregex wasn't finished.  Finish it and
also deal with the version lint.  Argument declarations for function
definitions in pmatchregex.c have been switched to K&R style.  (The ones
in posixregex.c have been left in ANSI style.)

There wasn't any build rule for pmatchregex.o; now there is (for Unix).
posixregex.o is still the default.

There isn't any build rule for cppregex.o (again, for Unix); the change
to cppregex.cpp is untested.

fix another logic error for nowin_buf read_tribute

logic bit

Fix a logic mistake in the nowin_buf scenario
of read_tribute.

a couple more tribute easter eggs
 Changes to be committed:
modified:   include/context.h
modified:   include/extern.h
modified:   src/files.c
modified:   src/invent.c
modified:   src/sounds.c
modified:   src/spell.c

Add a couple more tribute easter eggs.

 - can lead to a remark by Death if you happen to have a pratchett book on
   your person, as suggested by M. Stephenson (fat chance you will, or
   think to #chat if you do, but it could be a tournament novelty or something
   obscure to strive for).
 - can draw some additional Death quotes from the tribute file. (There's two
   in there right now. If anyone wants to add or suggest some more, please go
   ahead. The Death quotes are at the end of the tribute file. One-liners
   only please or the code will only pull the last line.

tribute: Lords and Ladies

It's a lot of passages, even when leaving out ones about Cresote and
King Midas (or whatever name his Discworld analog has), Nanny Ogg's cat,
Ridcully's custom made wizard hat, and the mention of a law which
requires that bystanders be warned before commencing to play folk music.

rising from the dead during panic

Prevent rising from the dead as a giant ant if the game panics during
initialization.

#overview, level difficulty

Three fixes, the first leading to the need to fix the second, and that
fix making dealing with the third be straightforward.

First, make the furthest level reached in any given branch be considered
interesting by #overview, even if no interesting features have been
encountered.  This will result in listing Gnomish Mines and their first
level when someone goes down the stairs and immediately back up.  It will
also produce a reminder of how far you've been--in each branch--after
retreating for any reason, without the need to manually add an annotation.

Second, #overview was suppressing the range of level numbers for Sokoban
because the author realized that the values were wrong.  The record of
the furthest level reached was incorrect for builds-up branches, always
sticking with the deepest level even though it was the entrance.  The
overview patch neglected to do the same suppression for Vlad's Tower and
the level range ("36 to 38" or similar) there was wrong.  This fixes the
furthest level reached problem and also fixes #overview's level range
handling for builds-up branches.

Third and last, a long-standing issue which I don't think has ever been
formally reported:  the level difficulty calculation used for monster
creation treated the upper (harder to get to) levels of builds-up branches
as if they were easier since they're closer to the surface as the gopher
burrows.  So sokoban generated easier monsters on its final level than on
the ones leading up to that.  Make depth for difficulty purposes account
for descent to the entrance and then ascent to the level of interest.

There was a distressing amount of trial and error involved.  The dungeon
layout structures are not exactly easy to work with, and I never managed
to get builds_up() based on branch data to work correctly.  Basing it on
dungeon data works as intended provided the branch has more than one
level, but it will yield incorrect result if we ever add a single-level
branch reached via stairs up rather than stairs down.

some typos spotted in tribute
 Changes to be committed:
modified:   dat/tribute

There are probably more.

tribute formatting cleanup and additions
 Changes to be committed:
modified:   dat/tribute

I've also added some of Boudewijn's suggestions and
added his name in a comment for easy reference
when validating the content agains the book.

Fix the shapechanger fix again

Ensure there are candles in Vlad's tower

Shapechangers got a completely random form

mtmp->cham was NON_PM, which select_newcham_form interpreted as a completely
random form.  This also resulted Vlad getting a random shape, and not getting
the Candelabrum, making the game unwinnable.

rolling back after criticism of approach

more tribute fixup
One book had the top and bottom credits.

typo on quote count for Reaper Man
 Changes to be committed:
modified:   dat/tribute

avoid error

The presence of [...] in middle of tribute quote
wasn't handle correctly in the new formatting code.

more quotes - Reaper Man
 Changes to be committed:
modified:   dat/tribute

Ensure more consistent formatting of tribute passages
 Changes to be committed:
modified:   src/files.c

start new tribute content handling

support for link-time option in #version
 Changes to be committed:
modified:   src/version.c
modified:   sys/share/cppregex.cpp
modified:   sys/share/pmatchregex.c
modified:   sys/share/posixregex.c
modified:   util/makedefs.c

Some options in 3.6.0 are determined by what you link with.
The choice of regex support is one.
Let #version show that linked option along with the compile-time options.

Makefile typos in sys/winnt/Makfile.msc

Copypaste mistake: Message types, not menu colors.

beta complaint 2 (again)

beta complaint 2

beta complaints 1

role selection: priest/priestess' prompt

Change "Shall I pick your priest/priestess' race, gender, ..." prompt
to "... priest/priestess's ...".

Unfortunately that makes it long enough to push the cursor onto the
next line for basic 80 column display.  If caveman/cavewoman's wasn't
already longer I think I would have left the clumsier phrasing/spelling.

characteristics disclosure

Change end of game disclosure's display for strength, dexterity, &c to
always show the maximum possible value instead of only when the final
value was less than maximum.  For both end of game and ^X, change the
description from "limit" to "innate limit" if the current value exceeds
limit due to worn items (gauntlets of power, +N ring of adornment, &c).

punctuation in failed messages

more menustyle

Make the list of menustyle values in the game match their order in the
revised documentation:  traditional, combination, full, partial.

menustyle documentation

Reorder the description of the values for the menustyle option, swapping
'full' and 'partial' since 'full' is closer in operation to 'combination'.

Remove unused variable

Fix unused variable thinko

alter beta identifier

menucolors

Error reported in display.c in current master code is gone,
but the menucolors stuff generated the following warnings when TEXTCOLOR wasn't defined.

..\win\tty\wintty.c(1688) : warning C4013: 'term_start_color' undefined; assuming extern returning int
..\win\tty\wintty.c(1711) : warning C4013: 'term_end_color' undefined; assuming extern returning int

punctuation in dat/tribute

actual field is always around anyway
 Changes to be committed:
modified:   include/flag.h

more tribute quotes
 Changes to be committed:
modified:   dat/tribute

tribute passage 'summary'

When reading a passage from a tribute novel, put the final attribution
line "[$TITLE, by Terry Pratchett]" into message history, comparable to
the summary line for deliver-by-window quest messages.

Files didn't match sys/unix/hints
 Changes to be committed:
modified:   Files

Merge branch 'macosx-build'

another punctuation bit

punctuation bit

more quotes from books without quotes

tribute: The Last Hero

Flesh out _The_Last_Hero_.  Due to its publishing history (no ordinary
paperback, some lists of "other books by this author" miscategorize it
as being co-written by the illustrator) it's most likely not as widely
read as the other Discworld books.

use NO_GLYPH as an indicator to ignore background glyph
 Changes to be committed:
modified:   doc/window.doc
modified:   src/display.c

default to original background
 Changes to be committed:
modified:   src/display.c

The work for getting this working fully is now moving to the background_tiles branch.
In master, we just return the standard lit room tile for now, no change in behavior.
No ports utilize the new parameter as yet.

X11: Hilight piles of objects

Add the mapglyph special flags to window doc

TTY: Hilight piles of objects

Add MG_OBJPILE flag, which windowports can use to check if a location
has more than one object stack. If use_inverse is on, TTY will use
inverse to show such piles.  If a boulder is the topmost item on a pile,
then the object pile flag is not used; mainly because boulders are "solid",
boulders dropped by monsters are nearly always over other objects, and so
that special levels such a Sokoban can "hide" items under the boulders.

TODO: a "pilemark", analogous to "petmark", perhaps a green plus sign,
which can be used by windowports with tiles.

many contributors to this please
 Changes to be committed:
modified:   doc/Guidebook.mn
modified:   doc/Guidebook.tex

Merge branch 'master' into AlexK-misc

provide bkglyphs
 Changes to be committed:
modified:   src/display.c

Modify the window interface for print_glyph

 Changes to be committed:
modified:   doc/window.doc
modified:   include/qt_win.h
modified:   include/trampoli.h
modified:   include/winX.h
modified:   include/wingem.h
modified:   include/winprocs.h
modified:   include/wintty.h
modified:   src/display.c
modified:   src/windows.c
modified:   sys/amiga/winami.p
modified:   sys/amiga/winfuncs.c
modified:   sys/amiga/winproto.h
modified:   sys/wince/mswproc.c
modified:   sys/wince/winMS.h
modified:   win/Qt/qt_win.cpp
modified:   win/X11/winmap.c
modified:   win/chain/wc_chainin.c
modified:   win/chain/wc_chainout.c
modified:   win/chain/wc_trace.c
modified:   win/gem/wingem.c
modified:   win/gem/wingem1.c
modified:   win/gnome/gnbind.c
modified:   win/tty/wintty.c
modified:   win/win32/mswproc.c
modified:   win/win32/winMS.h

print_glyph now takes a second parameter.

Tiles on tiled ports always looked odd on places like the plane of air
where the background color of the tile didn't match the general background
of the surrounding area.

3.6 made that even worse and more glaringly noticeable with the introduction
of darkened room tiles.

The code to actually send something useful through the new parameter
for window ports to take advantage if they want will follow.

Use MAKE variable instead of hardcoding make

Chatting with priest converted to minion crashes the game (few code paths in priest_talk() assumed the priest has an altar)

lookat again

Remove second 'alt_i' initialization, which was first in implementation.
Superseded by the preceding line, which came later.  Works either way,
but the conditional initalization avoids the two extra loop iterations
when they're not useful.

lookat issues for unknown/solid rock/dark room

Fix two things with the ';' and '/' commands, both for looking at blank
space.  The list of possibilies included "a dark part of a room or the
dark part of a room" even though the code involved goes out of its way
to avoid redundant clauses.  S_stone let dark part be prefixed by 'a',
S_room and S_darkroom forced it to be 'the' which is better phrasing
but outsmarted the redundancy check.  Make S_stone's use of "dark part
of a room" force 'the' too.

That's trivial; this is more complicated:  the new maze variations
exposed/aggravated an issue that's been there all along.  In a non-
WALLIFIED maze, doing look-at on the solid stone in-place-of-wall
next to you reported "dark part of a room" which is clearly wrong when
you can tell it's not a room.  (The same thing happens in any ordinary
corridor, but players rarely try to identify blank space next to them
it that circumstance so it hasn't mattered very much.)  This change
results in look-at listing "unexplored" and "stone" as additional
possibilities when looking at blank spots.  Final description will be
"unexplored" instead of dark room if you haven't seen the spot, "stone"
if you have and that's what it is, or "dark part of a room" otherwise.

remove use of __FUNCTION__

The special level loader has been using __FUNCTION__ in error messages
for a few months now, but that is a gcc extension (evidently picked up
by other compilers since only Borland had an issue so far).  The
standard way to do the same thing is with __func__, but that's C99 so
we should avoid it.  (__FUNCTION__ came earlier; gcc supports both.)

This switches to convential C code to achieve the same effect, using
the name 'nhFunc' rather than __FUNCTION__:
  void foo()
  {
    static const char nhFunc[] = "foo";
    ... code that might report problem in nhFunc ...
    return;
  }
This has only been added to the functions which actually reference it,
not a blanket intrusion into every routine.  In special level loader's
case, the reference is hidden in the opvar_free() macro which is used
quite a lot.

At first I used a macro:
  void foo()
  {
  #define nhFunc "foo"
    ... code that might report problem in nhFunc ...
    return;
  #undef nhFunc
  }
but using an actual variable avoids duplicate copies of the function
name string when used more than once inside a given function, and it
can't accidentally carry over into the next function due to missing or
misspelled #undef.

If we someday switch alloc() to give more specific information than
__FILE__, the macro variation would be better since the function name
won't be used most of the time (ie, when MONITOR_HEAP isn't defined).

hint file for macosx10.10

add /usr/X11R6/include to the include paths

Remove extraneous line from linux-X11 hints file

Prevent rays from bouncing off Plane of Air edges

more dat/tribute updates
 Changes to be committed:
modified:   dat/tribute

Merge branch 'master' into win32-x64-working

ensure dummy obj owornmask is clear (from Alex)

Another minor oddity (did not have time to trace it). Charges for damaged
weapon refer to it as "weapon in hand":

--
As you read the scroll, it disappears.  Being confused, you mispronounce
the magic words...  Demirci's long sword is covered by a mottled purple
glow!  "You degrade that long sword, you pay for it!"
Call a scroll labeled VERR YED HORRE:
What do you want to wield? [- ajrw or ?*] j
j - a rustproof athame named Magicbane (weapon in hand) (10 aum).
What do you want to drop? [$a-df-rtwxM or ?*] r
You drop a long sword (40 aum).
Demirci offers 8 gold pieces for your long sword.  Sell it? [ynaq] (y) y
You sold a long sword (40 aum) for 8 gold pieces.
You see here a scale mail (250 aum).
You see here a ring mail (250 aum).
A rustproof long sword (weapon in hand) (40 aum) for 15 zorkmids.  Pay?
[yn] (n)
You paid for a rustproof long sword (weapon in hand) (40 aum) at a cost of
15 gold pieces.  "Thank you for shopping in Demirci's used armor
dealership!"
--

win32_gui: use menu keys (<, >, ^, |) to scroll text windows

shapeshifting on rogue level

Limit vampire shapeshifting on rogue level to vampire bats (only
choice represented by uppercase letter) and have other shapeshifting
try for uppercase.  The latter isn't rigorous because shapeshifters
(chameleon=':', doppelganger='@', sandestin='&') aren't uppercase
themselves, so won't be created there under ordinary circumstances.
It applies to the "summon nasties" monster spell and post-invocation/
post-Wizard's-death harassment effect too.

Make priest door opening check match dog_move

Dwarves may be generated with dwarvish spears

Show detected pets with correct glyph

When detecting monsters with blessed potion of monster detection,
show pets with the correct glyph; highlights them if hilite_pet is on.

Monster detection shows pets with the correct glyph

Make pets hilited when detected with uncursed potion of monster detection
and hilite_pet is on.  Noticed by Raisse.

Fix looking around when swallowed

From Boudewijn:

> I am currently swallowed by an ice vortex, and used the ; command
> to identify the \ on my top right.
>
> It said: "\       an opulent throne (interior of a monster)"

Now, when you're swallowed, and look at anything else than yourself,
you'll get "\        the interior of a monster (interior of an ice vortex)".
Based on the comment in the code, it seems this was the original
intention anyway.

Minor Guidebook fixage

Move struct istat_s above prototypes using it

Fix Rogue-level lighting bugs caused by dark room

Found by Boudewijn

Merge branch 'win32-x64-working'

offer perm_invent as "personal preference" option

enlightenment for unarmored, also blindness

Augment the existing enlightenment feedback for blindness:  "innately"
blind if poly'd into something without eyes, "permanently" blind if
using the blind-from-birth option, "deliberately" blind if blindness
is  solely due to a blindfold, or "temporarily" blind otherwise.

Add status of "not wearing any armor" when applicable, with slightly
different phrasing if it's due to adhering the OPTIONS:nudist conduct.

blind-from-birth fix

The Blindfolded_only macro didn't track u.uroleplay.blind so would
give a false positive if wearing a blindfold, not able to see due to
the blind option, and not afflicted with any other blindness factors.
Dipping a worn blindfold into holy or unholy water is supposed to
reveal a glow if that blindfold is the only reason for blindness but
the glow was described even when blind-from-birth.

There's no feedback at all when the glow isn't seen.  I'm punting on
that one.  (This change didn't introduce that, just added one extra
situation where it happens.)

breaking blind-from-birth conduct

OPTIONS:blind starts the hero off blind, but putting on the Eyes of the
Overworld confers sight.  Make that break the blind-from-birth conduct.

Sight persists after removing the Eyes even though they aren't intended
to cure anything.  It doesn't make sense to restore the blind-from-birth
flag when taking the Eyes off, but we may want to add another flag, or
make u.uroleplay.blind into a bit mask that can track both can't-see-now
for play and could-never-see for conduct.  (Actually, u.uroleplay.blind
should track only the conduct, and starting the game with it enabled
should set one of the extra bits in u.uprops[BLINDED].intrinsic.  The
Eyes already override that, and taking them off would restore blindness
since the bit would still be set.  As a bonus, the expression in the
macro 'Blind' could be simplified.)

make option show as disabled
 Changes to be committed:
modified:   src/options.c

make option show as disabled if compile time options aren't defined.

defaults.nh update

win32 raw_printf needs better formatting to read
 Changes to be committed:
modified:   src/options.c

Noticed while testing status bits, the meaningful part of
the message is truncated on WIN32.

status conditional code commented oout for merge

runtime enable statushilites if compiled in
 Changes to be committed:
modified:   src/options.c

default to runtime enabled of statushilites if compile time option set

win32_gui: honor use_status_hilites flag

guidebook status_hilite adjustment

option name change to statushilites
 Changes to be committed:
modified:   doc/Guidebook.mn
modified:   doc/Guidebook.tex
modified:   include/flag.h
modified:   src/options.c
modified:   win/tty/wintty.c

example:
OPTIONS=statushilites
OPTIONS=hilite_status:hitpoints/30%/bright-magenta/normal

Merge branch 'master' into status_hilite

Conflicts:
include/extern.h
win/tty/wintty.c

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

oops, Guidebook.tex paste error

ready for testing
 Changes to be committed:
modified:   doc/Guidebook.mn
modified:   doc/Guidebook.tex
modified:   include/config.h
modified:   win/tty/wintty.c

tty role selection when filter by options

Honor things like OPTIONS:role=!tourist and NETHACKOPTIONS='race=!orc'
when performing interactive role selection.  I don't think it was
completely correct when players let the program choose, but it must
have been close enough because we haven't gotten any complaints.
The post-3.4.3 interactive selection was ignoring options-base filtering
entirely and did get complaints for the pre-beta.

Role selection has a ton of code which bloats the program without doing
anything useful for actual game play.  It ought to be split off into a
separate front end.

functional state achieved
 Changes to be committed:
modified:   win/tty/wintty.c

hilite_status: spaces in color names break option parsing

Merge branch 'master' into status_hilite

Conflicts:
include/extern.h
src/options.c

Reduce the number of gnomes with candles

Gnomes in mines during level generation have 1/20 chance of getting a candle
(should give approximately 4 candles in all of the mines total), and every
randomly generated gnome has 1/60 chance.

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

more updates and on/off option

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