allow DEBUGFILES to be overridden via getenv()

If getenv("DEBUGFILES") yields a value then it takes precedence over
sysconf.DEBUGFILES or sys.c's #define DEBUGFILES.  (It probably should
only be controlled via environment since it is not a system-wide
attribute, but I haven't taken out the SYSCF handling for it.)

finally remember to add the 'nh-' versions

dungeoneers update March 16, 2015

fix #H3724 - gold pile stacking bug

     Bug report included a pointer to a fix; this patch is a superset.
Gold pieces dropped on an altar by the player got their bknown flag set,
which is incorrect since bless/curse doesn't apply to coins.  If a
monster (in reported case, a slain temple priest) dropped gold there too
then the two stacks wouldn't merge.  For the normal !GOLDOBJ config, the
problem goes away as soon as the gold gets picked up.  I didn't test for
GOLDOBJ but think two inventory slots containing gold can result.

     The superset part is to not break agnostic conduct by dropping gold
on an altar since no information is revealed when doing that.

[This was one of the very last patches checked into the old cvs repository,
where the somewhat out of date message above was accidentally omitted.]

free 'sysopt' strings upon termination

DEBUG/debugpline() fixup

 * Replace variadic debugpline() with fixed argument debugpline0(str),
   debugpline1(fmt,arg), and so on so that C99 support isn't required;
 * showdebug() becomes a function rather than a macro and handles a
   bit more;
 * two debugpline() calls in light.c have been changed to impossible();
 * DEBUGFILES macro (in sys.c) can substitute for SYSCF's DEBUGFILES
   setting in !SYSCF configuration (I hope that's temporary).

Some few updates/additions; clean up formatting

withdraw fixes35.0 part of prior commit

There was actually a fixes35.0 entry further down that covered off
that particular entry already which I missed first time around.

> many instances of physical damage were not taking Half_physical_damage
> into account when reducing your hitpoints

catch up on half-physical damage docs

This is catching up on some things that were changed
in development years ago that Dave C. suggested be
documented.

For the record:
-The things that were evaluated and ruled out
are now documented in include/youprop.h so they don't
come up again.

- The things that were evaluated and deemed to be susceptible
to the intrinsic and thus led to a modification in the code
are listed below in this commit message.

Modifications:
- A crystal ball exploding on being applied
- Artifacts' blasting
- Being a fish out of water
- Being hit by Mjollnir on the return
- Being thwacked by an iron ball chained to you
- Boiling/freezing potions
- Broken wands
- Bumping head on ceiling by cursed levitation
- Burning (un)holy water
- Chest/door/tin traps
- Dipping a lit lamp into a potion of oil
- Exploding rings and wands (under all circumstances)
- Exploding spellbooks
- Falling downstairs
- Falling into a (spiked) pit
- Falling off or failing to mount a steed
- Falling on a sink while levitating
- Getting squished in a pit under a boulder
- Hitting your foot with a bullwhip
- Hitting yourself with your pick-axe
- Hooking yourself with a grappling hook
- iron-ball-pulling yourself out of a bear trap
- Jumping/Newton's-Thirding into something solid
- Kicking something that makes you go "Ouch!"
- Land mine explosion
- Sitting in a spiked pit
- Stinking cloud damage
- Thrown potion (bottle)
- Zapping yourself with a wand, horn or spell
- Jumping yourself out of a bear trap

Death talks in CAPITAL LETTERS

Additional obstruction messaging fixup

fix messaging in obstructed() in lock.c

From ais523's recent list of bugs:
If a long worm tail is blocking the door, and you're blind and not
telepathic, attempting to close the door marks the position of its head.

From an email received in late September 2014 before the git conversion:
> I was trying to close a door, not noticing that there was a garter
> snake there, and this message resulted:
> The garter snake stands in the way!
> I haven't tried it with any other monsters without feet, but
> "stands in the way" appears to be the wrong way to describe
> this situation...

Both of the above were found in the same function in lock.c

"swallowed whole" was preceded by "You bite into"

Another reported by ais523:
The death message "swallowed %s whole" is preceded by
"You bite into %s."; these contradict.

abuse wisdom in keeping with Rider eating msg

The death message for eating a Rider is "unwisely tried to eat...", but
it doesn't abuse Wisdom (potentially relevant if you lifesave)

Initial version of simple recipes file

Fix warnings

charge for a boulder that fills a pit in shop

 Changes to be committed:
modified:   src/do.c

From the email sent by ais523 earlier:
> You aren't charged for digging a pit below an unpaid boulder
> (causing the boulder to fill the pit).

from ais523 list: charge for boulder filling pit

TTY: Implement the menu_search command

When in a menu that allows selecting entries, press ':'
to enter any text. Entries that match the text (via pmatch)
will be toggled.

Add MAX_STATUENAME_RANK sysconf setting

MAX_STATUENAME_RANK tells how many score file entries to use
for random statue names.

getpos improvements: automatic description

Pressing '@' will move the cursor on top of the hero.
Pressing '#' will toggle automatic description mode, where
the feature under the cursor is automatically described
when the cursor is moved.

catch up on some fixes entries

Add boolean option "bones"

Setting bones off prevents saving and loading bones files.

Fix GENERIC_USERNAMES thinko

do_look() in post-3.4.3 code fixes entry

One VS Makefile for Nethack that works with VS2010-VS2015

typo in format string on previous commit

more do_look() glyph usage without it being set properly
 Changes to be committed:
modified:   src/pager.c

When the previous occurrence that triggered a segfault
was fixed, I didn't check for more of them. It turns out there
were more instances.

This also removes some dead code.

Change command X to twoweapon toggle

Explore mode is now an extended command #exploremode.
There's no sense that a command used max. once per game, and
in normal games not at all, takes up a key. So, analogous to
the 'x' command (swap weapons), 'X' now toggles two-weapon
combat.

Comment typofix

Use a menu to loot multiple containers

When looting a location with multiple containers, show a menu for user
to pick the containers to loot instead of asking a yes/no question for
each container.

Define for generic usernames which prompt for name

Instead of hard-coding the "play", "player", etc. usernames
which will prompt the user for their name, allow defining
the names in config.h

fix a segfault in mapglyph reported by Pasi earlier today
 Changes to be committed:
modified:   src/pager.c

Steps to reproduce the reported segfault:
Use / command, answer n, press space and enter

In do_look() variable glyph is only set to a proper
value if using the screen (from_screen) or the
mouse (clicklook).

On the code path that triggered the segfault,
glyph was being passed to mapglyph with a random
arbitrary value.

If glyph had been initialized at the start of
do_look(), it would have prevented the segfault,
but would have always displayed a giant ant or
something silly instead of the correct thing.

Don't use glyph except under
from_screen || clicklook.

TTY: Prevent accidental escapes from string entries

On NAO, one of the major complaints was accidental escaping
from wishing prompt when using cursor keys.  The users were
trying to go "back" on the entry to fix a typo, but lost
the wish instead.

This prevents escaping out of a text prompt if there is any
text entered into the prompt; pressing escape clears the prompt.

TTY: Do not cut off statuslines at 80 characters

Show as much of the status line as possible, instead of chopping
it at COLNO - this prevents possible game-influencing status
effects (Ill, Burdened, etc) from being hidden.

Implement fcntl(2) file locking

If you run a server, then you know of the somewhat annoying perm_lock
errors that creep up, requiring your attention before anyone else can
start a game.

This patch properly implements fcntl(2) locking on systems that can
handle it (*nix systems), that results in the lock being automatically
released on program termination, whether abnormal or not.

Original patch by Drew Streib, update by Edoardo Spadolini

You_hear already checks deafness

Add database entry for land mine

Add database entry for fedora

Make shuriken entry also match throwing star.

Fix shuriken attribution

Add database entry for shuriken

Fix data.base entries

 -"bat" overrode later "combat" entry.
 -"gelatinous cube" and "jack boot" have their own entries.
 -"vampire bat" matched twice; use the bat entry.
 -dagger attribution started with spaces instead of tabs.

Fix some README mistakes that Janet spotted

Updated README

On 3/1/2015 10:37 AM, coppro wrote:
> >>          Sun SPARC based machine running SunOS 4.x, Solaris 2.x, or
> Solaris 7
>
> By release time, I should have access to a BSD flavour installed on a SPARC
> machine. Would out be helpful for me to do cursory tests?

On 3/1/2015 10:41 AM, lorimer wrote:
> I have access to a Borland environment too, so I'll be
> borrowing that at some point.  [Just not soon.]

I didn't include the above, just in case they don't happen.
The README can, of course, be updated again once they have
been tried.

Fix NHtext when GIT_DIR is not defined.

Fix TTY crashing when a location has more than 32k items

Make a few fixes to code that was confusing indent

Fix segfault when MD termcap is not defined

Sanitize petnames and fruit to prevent escape codes

Print lava item destructions only if surviving.

This partially reverts commit 72c2ae233d35076adece2e8d02a7f8839e4a5aa8
in order to prevent players from using hangups during the destruction
messages to save themselves.

Merge fire-based erosion to common codepaths.

This finally eliminates all direct increases of `oeroded` and `oeroded2`
and moves them all to go via `erode_obj()`. They are still manipulated
directly in a few places, but not to erode objects.

This now merges the `fire_damage()` function to a common codepath, used
for items on lava and burning oil, but fire needs more work. There is
still a duplication between `destroy_item()` and `fire_damage()`; the
two codepaths should eventually be merged in some manner so that there
is only one codepath to say "an object was affected by fire". This path
might require some parameters, such as whether the fire will just erode
objects or burn them outright, but that can happen another day.

Fold `get_wet()` into `water_damage()`.

This now ensures that dipping into water works like other sources of
water damage. There is a potentially significant gameplay change here:
dipping a container into uncursed water will wet all its contents. If
this is a problem, then we should add another parameter to water_damage
which will suppress this behaviour for dipping.

Rename `rust_dmg()` to `erode_obj()`.

Eliminate `erode_obj()` and other erosion.

Now all erosion that isn't fire-based goes through `rust_dmg()`

A lot more cleanup to rust and water.

Cover a couple of corner cases with `rust_dmg()`.

Remove the `victim` argument to `rust_dmg()`.

The argument can be calculated from `otmp`.

Factor out a new `is_moat` function.

The fact that Juiblex's swamp is MOAT but not moat is weird and should
probably be looked at at some point.

Manually fix botched NHDT-Branch expansions.

Switch NHtext from "git branch" to "git symbolic-ref" to avoid corner cases.

catch up fixes with some recent changes; also add 'community patch' section

Remove useless dungeon.def mangling

Remove mention of compile-time AUTOPICKUP_EXCEPTIONS

Remove last mention of EXP_ON_BOTL

Remove last mention of DUNGEON_OVERVIEW

Change NHtext to double the speed of "git checkout"

Add new parameters to makedefs: --debug and --make

"makedefs --debug --make q" is equivalent to "makedefs -q" with
DEBUG defined.

remove and transfer INVISIBLE_OBJECTS to branch

Remove unused size from questpgr.c

Fix some warnings on the unconditionals branch.

Remove experimental Unicode support.

This reverts commit 7f0f43e6f98bccab20db7ab3cb509fdbe60a7b2b and some related
subsequent commits.

This compiles, but I have not done extensive testing.

Conflicts:
include/config.h
include/decl.h
include/extern.h
include/global.h
include/tradstdc.h
include/wintty.h
src/drawing.c
src/files.c
src/hacklib.c
src/mapglyph.c
src/options.c
sys/winnt/nttty.c
win/tty/getline.c
win/tty/topl.c
win/tty/wintty.c

Make EXP_ON_BOTL unconditional.

Make REDO unconditional.

Make LOADSYMSETS unconditional.

There are some bugs here, but this functionality should make it into the release
in some form.

Make AUTOPICKUP_EXCEPTIONS unconditional.

Make BARGETHROUGH unconditional.

Make SEDUCE unconditional.

Apply paxed's DEBUG patch to remove DEBUG/D_DEBUG.

    Move debugging output into couple preprocessor defines, which
    are no-op without DEBUG.  To show debugging output from a
    certain source files, use sysconf:

    DEBUGFILES=dungeon.c questpgr.c

    Also fix couple debug lines which did not compile.

This also includes fixes due to Derek Ray to depugpline to work better
on other platforms.

Make GOLDOBJ unconditional.

Make WIZARD unconditional.

Make DUNGEON_OVERVIEW unconditional.

Make REINCARNATION unconditional.

There is a lot of code affected by this, and Pat Rankin correctly
observes that it would be better to store roguelike as a level flag
rather than just using Is_rogue_level. A note for the future.

Make TOURIST unconditional.

Make SINKS unconditional.

Make KOPS unconditional.

Make STEED unconditional.

Make ELBERETH unconditional.

Another batch of manual header var additions

A batch of manual header var additions.

Update .gitignore files.

Fix lost mode bits.

Bulk recovery of file CVS headers and addition of NHDT- headers.

Add .gitattribute files.

Add a missing break

Stepping on a magic trap while riding could delete the trap and
make the deleted trap hit the steed.

Add fixes notes for the recent lava changes.

Remove the 'nul' variable.

Clean up lava messages a little.

This change has two parts.

First, the Book of the Dead now has a special message in lava. This was
originally added by Steve Melenchuk (aka Grunt) as a precaution against the Book
burning up. While I'm pretty sure that the Book can't burn up in the current
codepath, the message is a nice touch and it doesn't help to add a safety check.

Second, always print the inventory destruction messages. The reason for this is
fairly subtle, but relates to the timing. By not printing the messages when
dying, the visible effect to the player is that if they have lifesaving, the
items burn up before they die, but if they do not, they burn up after they
die---the fact that they do burn up is visible in the dump or bonesfile.

Clean up the lava code a little.

Destroy water walking boots on lava first.

This avoids a player being left standing on lava if their boots are burned up and
they survive the damage.

This change is originally due to Steve Melenchuk, fixed up by Derrick Sund.