Merge branch 'derek-elbereth'

* derek-elbereth:
  ensure that the 'safe' objects remain safe
  finish up the changes to trigger erosion on use
  initial pass for toning down Elbereth

Conflicts:
dat/castle.des
dat/sokoban.des
include/extern.h
src/engrave.c
src/mklev.c
src/monmove.c
src/zap.c

don't break other ports use of tilemap.c

The modifications made for supporting monster-looking tiles
may not be supported on platforms other than windows yet.

Merge branch 'win32-x64-working'

Used invent letters didn't destroy the window

Merge branch 'master' into win32-x64-working

make internal numbering match updated headers

more header update
 Changes to be committed:
modified:   sys/winnt/nttty.c
modified:   sys/winnt/winnt.c

Merge branch 'master' into win32-x64-working

Restore lost executable bits.

Version number increment

couple bits omitted from lint.h patch

* list lint.h in Files
* add dependency to win/X11/dialogs.o since dialogs.c includes lint.h
  directly rather than via hack.h

makefile updates

updates forced from master
 Changes to be committed:
modified:   sys/winnt/Makefile.bcc
modified:   sys/winnt/Makefile.gcc
modified:   sys/winnt/Makefile.msc

put back Derek's _MSC_VER changes into the lint.h version

Merge branch 'master' into win32-x64-working

Conflicts:
include/hack.h
win/X11/winmenu.c
win/X11/winstat.c

Add a funny message for when a burrower eats food.

update fixes with a few things

cherry-pick the YACC/LEX upgrades from elsewhere

jonadab's patch for engraving on the drawbridge

Should be splinters, not gravel.
Addendum to C343-239

Hidden holes made by breaking a wand of digging

Holes created via applying a wand of digging were not
shown on the map, because holes are always marked as seen,
and seetrap/feeltrap then do nothing.

X11 lint suppression

Suppress close to 400 warnings generated by gcc on the win/X11/*.c code,
most due to -Wwrite-strings which makes string literals implicitly have
the 'const' attribute.  (Since modifying a string literal results in
undefined behavior, that is an appropriate check to have enabled, but
it can be troublesome since string literals have type 'char *' and code
that uses them that way is correct provided it avoids modifying them.)

 113  warning: initialization discards qualifiers from pointer target type
 127  warning: assignment discards qualifiers from pointer target type
  29  warning: passing argument discards qualifiers from pointer target type
 109  warning: unused parameter
  12  warning: comparison between signed and unsigned

The nhStr() hack casts to 'char *', explicitly removing 'const', for
situations where it isn't feasible to make code directly honor const.
The vast marjority of uses are for the second parameter to XtSetArg(),
which is a macro that actually performs an assignment with the second
argument rather than passing it in a function.  It takes values like
'XtNtop', which doesn't need to be altered (although in many places I
changed that to nhStr(XtNtop) for uniformity with the surrounding code,
and 'XtNbottom', which does need to have the extra const stripping to
avoid a warning.  Go figure.

The nhUse() hack actually uses its argument in a meaningless way if the
code is compiled with FORCE_ARG_USAGE defined.  When GCC_WARN is defined,
FORCE_ARG_USAGE will be enabled if it hasn't been already.  Example:

 /*ARGUSED*/
 int foo(arg)
   int arg;  /* not used */
 {
+  nhUse(arg);
   return 0;
 }

The extra line will expand to ';' when FORCE_ARG_USAGE is not defined
or too
   nhUse_dummy += (unsigned)arg;
when it is.  I figured direct assignment might lead to a different
warning by some compilers in a situation like
   nhUse(arg);
   nhUse(otherarg);
where the first assignment would be clobbered by the second, and using
bitwise operations or safer '+= (arg != 0)' would most likely generate
more non-useful code.  Some tweaking might turn out to be necessary.

new file: include/lint.h

modified files: include/hack.h, src/decl.c, sys/unix/Makefile.src

Groundwork for cleaning up the X11 sources, where gcc with the option
settings specified in the OSX hints file currently generates close to
400 warnings for win/X11/*.c.

lint.h is included by hack.h, and I've moved the debugpline stuff from
the latter to the former to hide it better.  (By rights it belongs in
debug.h or something of the sort, but I didn't want to go that far.)
Makefile and project dependencies need to catch up.

nhStr() hides a cast to char *, and is intended to by used on string
literals where it isn't feasible to maintain the 'const' attribute.
(A pernicious problem with X11 code, where the include situation can
become very convoluted, and many, MANY string literals are hidden
behind macros to look like keyword-type tokens.)

nhUse() can be used to force a fake usage on something which triggers
an unused parameter warning.  There are a 6 or 8 or 10 places in the
core code where that applies, but so far I have't touched any of them.
There's a tradeoff since it will result in some worthless code being
generated and executed, but is much simpler than tacking on compiler-
specific workarounds like '#pragma unused' or gcc's __attribute__ hack.

Comment for the pudding glob order

another pass at "gold wield inconsistency"

Handle !fixinv by forcing gold to have slot '$' all the time; that
particular type of object is 'fixed' regardless of user preference.
Also add a couple of checks for non-'$' gold when selecting from
inventory, just in case the issue of multiple gold stacks reappears.

Fix impossible when quivered gold was stolen from you

Revert previous, with a comment in code

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.

ensure that the 'safe' objects remain safe

since Elbereth doesn't work if you're not on the square anymore, we need
to make sure that critters are discouraged from grabbing the sokoban
prize or the castle wand.

also, fix up the level compiler makefiles so that uncommenting the
YACC/LEX definitions (presuming you have the right tools installed)
works properly.

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

finish up the changes to trigger erosion on use

Elbereth is a magical power, and like everything else, magic eventually
gets used up... even for burned engravings.

Unify getting terrain under drawbridge

Unify guards yelling