update fixes with a few things

cherry-pick the YACC/LEX upgrades from elsewhere

Merge branch 'master' into win32-x64-working

Melded pudding object may be gone

more "deleting worn obj"

Lit candles and burning potions of oil can be on the migrating objects
list and get deleted before arrival, so they need the same cleanup as
rotting corpses to prevent obfree from complaining that they're worn.

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

Pull to resync.

Made sysconf define contiditional to ensure no double inclusion.

Merge branch 'master' into win32-x64-working

Conflicts:
src/dig.c
src/mkobj.c

 Changes to be committed:
modified:   src/dig.c
modified:   src/mkobj.c

owornmask sanity_check fix and enhancement

Thinko fix:  sanity checking for owornmask was mis-treating
OBJ_MIGRATING as OBJ_MINVENT of migrating monsters rather than as
unattended objects and would have had problems similar to obfree's
inappropriate impossible check.

Sanity checking for objects worn in invalid slots (amulet worn in
a ring slot and so forth) is extended to items worn by monsters.
Also add a check for wielded coins since the loophole that let them
become wielded has been closed.

fix "deleting worn object" impossibility

Migrating objects overload obj->owornmask with a destination code,
so rot_corpse needs to clear that before deleting corpses.  (Buried
objects don't touch owornmask, so rot_organic, which does the actual
object deletion, shouldn't need any similar change.)

The corpses with owornmask 3 that have been observed recently were
slated to arrive on the up stairs, so presumeably fell down the down
stairs of the current level and rotted before the hero went down.
Put plainly, it was the [post-3.4.3] impossible() check which was in
error, not the active game data.

Merge branch 'master' into win32-x64-working

fix "gold wield inconsistency"

'w$' reported "you can't wield gold" but
'w*$', choosing from inventory and picking gold, let you wield gold.

The old code checked whether gold had been picked before checking
whether '?' or '*' had been picked to request selection from inventory.
This wasn't an issue with 3.4.3's !GOLDINV configuration (but probably
was for anyone who explicitly switched to GOLDINV) because getobj()'s
callers only inserted gold into inventory when they intended to accept
it as a valid choice.

Fix is just to swap two adjacent 'if' blocks in getobj() so that '*'
is processed before the test of whether '$' has been chosen.  Most of
the diff is indentation and other minor reformatting.

tiles lint

Several instances of "comparison between signed and unsigned".

  win/share/tiletext.c
  win/X11/tile2x11.c

tiles-related build stuff

* Add missing entry for include/tile.h to (top)/Files; also
  add new entry for generated file util/tiletxt.c (Unix only);
* Add several missing entries for tile utility programs that
  can be built by sys/unix/Makefile.utl to util/.gitignore;
* Update sys/unix/Makefile.utl to build 'tilemap' differently so
  that it won't leave behind an unwanted subdirectory tree under OSX:
    util/tilemap.dSYM/
    util/tilemap.dSYM/Contents/
    util/tilemap.dSYM/Contents/Info.plist
    util/tilemap.dSYM/Contents/Resources/
    util/tilemap.dSYM/Contents/Resources/DWARF/
    util/tilemap.dSYM/Contents/Resources/DWARF/tilemap
  It now generates util/tiletxt.c on the fly, to be compiled into
  tiletxt.o, so that tilemap.c can be compiled in the ordinary
  manner and tilemap.o can be kept around for dependency checking.
  (Creating real source file win/share/tiletxt.c would be a little
  bit cleaner, but it's effectively two lines long so seems silly
  to be in the source distribution.)  I looked to see whether I
  could find a linker or compiler option to suppress that stuff but
  failed.  I'm sure something of the sort must exist but didn't
  pursue it.  Someday I might actually learn about how OSX works....

Randomize secret door locations

... in the Wizard quest locate level and Rodney tower entrance

Barb quest home has a forest

tiletxt lint

Use 'const' for string literals when compiling win/share/tilemap.c
with '-DTILETEXT' to generate util/tiletxt.o.

Merge branch 'master' into win32-x64-working

commit the latest update for wiz_identify

git add mistake on my part

permanent wiz_identify correction

wiz_identify (^I) is supposed to bring up the inventory list
with the items identified, in contrast to the regular (I)
inventory, but only for that inventory display.

It is intended that if you then hit ^I again while at that
wiz_identify inventory menu, you'll trigger permanent
identification of itemsand any other key is supposed to
leave them as they were.

In tty however, <space> was doing the same thing as the
second ^I, and permanently identifying everything.
So, for those that like to <space> through menus (like
me) this makes it work as originally intended and
requires a deliberate second ^I at the menu to permanently
identify. (It might also allow trigger permanent ID if you hit
underscore, but nobody "underscores" through menus,
so that's ok.)

Make the slanted bigrm have more variance

Occasionally fog maze bigrm is not made of fog

More variations for bigrm-1

Minor variation for Earth and Astral planes

Replace some pools in Hea quest home with floor

Merge branch 'master' into win32-x64-working

More minetown and mine end variation

Merge branch 'master' into win32-x64-working

Make Monk quest home match Priest one

Add random trees to Ranger quest home

Merge branch 'master' into win32-x64-working

Random holes in the walls of Sam quest goal

Merge branch 'master' into win32-x64-working

Resolved Conflicts:
src/cmd.c
src/mkobj.c

 All conflicts fixed but you are still merging.

 Changes to be committed:
modified:   dat/Priest.des
modified:   dat/Rogue.des
modified:   dat/Valkyrie.des
modified:   dat/Wizard.des
modified:   dat/gehennom.des
modified:   src/cmd.c
modified:   src/mkobj.c
modified:   src/steal.c
modified:   sys/share/lev_yacc.c
modified:   util/lev_comp.y

Random melted spots in Valk quest home

Add some clouds to the Wiz quest

Add trees to Priest quest home level

Put Rogue quest home monsters on the "streets"

Update pregenerated levcomp lex and yacc files

Make the Valley have slight variations

Some work on the BCC make file.

This is still not complete; the linker is complaining for both builds (NetHack and NetHackW.)
Also, the change to make it build both versions wasn't done yet.

BCC is more strict about not statements before parameter declarations

more owornmask sanity_check

Fix the problem with erroneously detecting wielded or quivered
chained ball which legitimately has more than one mask bit set.

Add an additional check for valid wornmask of an item somehow worn
in an invalid slot (such as an amulet in one of the ring slots).
Only lightly tested; it really needs debugger assistance to force
various invalid situations but I don't know gdb well enough for that.

enlightenment for polymorph & lycanthropy

Fine-tune lycanthropy feedback by combining "you are a werecritter"
and "you are in beast form" into one message.  Also, add some new
feedback when lycanthropy and intrinsic polymorph are blocked by
intrinsic unchanging.

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

Adding sysconf makefile changes to allow builds under both old and new methods (part 1).

sysconf fixes

stabilize loss of gold

When gold is stolen by a leprechaun or lost when being "overwhelmed
by an urge to take a bath" while dipping in a fountain, if you had
99 gold pieces or less, you'd lose all of it (in the bath case, only
if it was at least 10 to start with), but if you had 100 or more,
you would lose a random amount which could be as little as 1.  And
in the bath case, if the random amount was less than 10, you would
lose nothing but be told that "you lost some of your money in the
fountain".  After this change, it is still possible to lose less
when starting with more, but not as likely and not as extreme a case
as maybe losing only 1 when starting with thousands.

The fountain-dip bath case has code to handle mutiple denominations
of coins, possibly the only place in the program where that exists.
I've left that alone although it should probably be taken out....

Merge branch 'master' into win32-x64-working

fixes entry for previous bubble fix
 Changes to be committed:
modified:   doc/fixes35.0

Merge branch 'AlexK-misc'

Handle boulder-option in config file

Setting boulder in config file did not work correctly in post-3.4.3
code, due to the symset changes.

Merge branch 'master' into win32-x64-working

Resolved Conflicts:
src/mkobj.c
src/mon.c
src/options.c

 Changes to be committed:
modified:   DEVEL/hooksdir/nhsub
modified:   src/invent.c
modified:   src/mkobj.c
modified:   src/mon.c
modified:   src/options.c

Uninitialized variable

another ball & chain tweak

checking if uball is OBJ_FREE was inappropriate, because
sometimes it is in inventory.

sanity check owornmask

Extend the processing done by the wizard mode 'sanity_check' option
to look for anomalies with obj->owornmask since there seem to have
been a few lately.  I haven't actually triggered any so this code
isn't very well exercized yet.

sanity_check uses pline() rather than impossible() or debugpline()
to deliver messages so might not be very useful with keymasking.

A sizeable chunk of this diff is just cleaning up indentation so
that I could see what I was working with....

Initialize variable

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

nhsub: add -m flag for fixing commit dates

lint: use const for string literal

limit obj manipulation by handless creatures

something that is M1_NOHANDS ought not to be able to pick up a pile of
rocks, daggers, or gold pieces unless it has other features that would
permit such a thing.

Merge branch 'master' into win32-x64-working

can't use BALL_IN_MON after u.uswallow set to 0

u.uswallow is set to 0 a couple of lines above so
the test will never be TRUE.

Merge branch 'master' into win32-x64-working

Conflicts:
src/mon.c
src/options.c
src/pickup.c
src/zap.c

 Changes to be committed:
modified:   doc/Guidebook.mn
modified:   doc/Guidebook.tex
modified:   doc/fixes35.0
modified:   include/color.h
modified:   include/extern.h
modified:   include/mondata.h
modified:   src/ball.c
modified:   src/dbridge.c
modified:   src/dig.c
modified:   src/display.c
modified:   src/dokick.c
modified:   src/dungeon.c
modified:   src/fountain.c
modified:   src/invent.c
modified:   src/mkmaze.c
modified:   src/mkobj.c
modified:   src/mon.c
modified:   src/monmove.c
modified:   src/mthrowu.c
modified:   src/options.c
modified:   src/pickup.c
modified:   src/sit.c

fix a panic and infinite recursion
 Changes to be committed:
modified:   src/ball.c
modified:   src/display.c
modified:   src/mon.c

comment rearranged

Requested by Sean for some work he's doing for reindent.

win32_gui: fix text window background color

Extract ball and chain before flooreffects

Check for valid coord in obj_nexto_xy

Prevent monsters throwing items out of map

If a monster was in the projectile's path, the projectile stopping
checks would not be reached.  The thrown object could fly up to
the maximum range, through walls, or even outside the map.

Unquiver gold before making it vanish

This fixes a complaint for "deleting worn obj":  If reading a cursed
spellbook took your money, and the gold was quivered.

Make spell list title obey menu_headings

Exclude "grey" from color picker menu

trade a segfault for a panic
 Changes to be committed:
modified:   src/invent.c
modified:   src/options.c

Apparently we need this based on Pasi's segfault. We just
don't yet know why we need them.

Also fix a warning:
..\src\options.c(1282) : warning C4101: 'tmp' : unreferenced local variable

Unify getting terrain under drawbridge

Unify guards yelling

Add is_watch define for watchmen

Allow all text attributes for menu_headings

Allow changing menucolors in-game via Options

Show only beginning of annotation when redoing it

more BUCX: union vs intersection

Fix filtering used by the 'D' command, and a few other activities that
allow both object class filtering and bless/curse state filtering, so
that when both class(es) and state(s) are specified, objects need to
match both rather than either.  D?C will present the player with cursed
scrolls to drop rather than all scrolls plus all other cursed objects.

This also fixes another instance when gold could end up with its bknown
flag set.

Merge branch 'AlexK-misc' into win32-x64-working

bubble structure contains pointer into static array. Bubble objects
are being flat-dumped into save file and this causes segfault in restore()
whenever data segment layout changes (e.g. global variables added/removed).
bmask should either be stored with the objects.

Make nhsetup switch to the right directory always, and improve detection of double-click

Merge branch 'win32-x64-working' of https://rodney.nethack.org:20040/git/NHsource into win32-x64-working

Fix a Borland compile error. Now fix the linking...

Prevent possible buffer overflow

getlin() gets at most a BUFSZ string from user; make the buf big
enough to hold that _and_ the query itself.

Merge branch 'master' into win32-x64-working

'I' command support for BUCX

Allow the 'I' command to show inventory of known blessed items via
pseudo object classes B, C, U, and X.  That's instead of an showing
inventory of specific object class.  The two can't be combined
because 'I' operates on single character input.

I had to modify tty_yn_function to prevent it from forcing a BUCX
character into lower case (simply using lower case would cause a
conflict with 'u' and 'x' for inventory of shopping bill), and did
that by checking whether any of the acceptable response characters
are upper case.  Pretty straightforward and shouldn't impact any
other uses that don't specify upper case choices.

I did the same thing for X11.  Other interfaces most likely need
to do something similar.  If they don't, a response of 'B' or 'C'
(for menustyle:traditional or menustyle:combination) will simply
not work, without causing any problems, same as typing an invalid
choice, and 'U' or 'X' will give shop feedback instead of the
requested subset of inventory.

The Guidebook revisions are untested.

Add level annotating to the Call menu

Via UnNetHack

Show level annotation when overwriting existing one

Idea by Chris Smith, via UnNetHack

Mung annotation spaces

Show level annotation on level entry

Idea by Chris Smith, via Unnethack

comment termination bit

checkfile protection from caller
 Changes to be committed:
modified:   src/pager.c

Merge branch 'master' into win32-x64-working

Conflicts:
src/end.c
src/files.c
src/options.c

Show only the short automatic glyph description

When you turn on the automatic description of a glyph under cursor,
we want to show the short description of what glyph it actually is.
The long full description of all possibilities is far too long, so
may cause more-prompts, and is awkward for blind players.

Also allow q at the Itemized billing -prompt

Original change via AceHack by Alex Smith

isspace() usage

Replace most uses of isspace() with a simple test for ' ' after
processing the string buffer with mungspaces (which replaces tab
with space, converts instances of consecutive whitespace into a
single space, and removes leading and trailing spaces).  The uses
where this wasn't done now cast their argument to (uchar) so that
platforms with signed chars will never pass negative values to it.

I didn't mess with the menu coloring code (except for casts to the
isspace() argument); it almost certainly could benefit from using
mungspaces.  I did mess with the symset processing quite a bit,
and hope I haven't accidentally broken anything.  Default symbols
and DECgraphics symbols still parse and display ok, so the rest of
dat/symbols should be ok too.  I didn't test symbols in the user's
config file because I don't remember how that's supposed to work.

Fix adding doors to levels

Fix out-of-map condition in buzz

Merge branch 'master' into win32-x64-working