Solidify the astral plane

...to prevent xorns appearing in the inaccessible map fringes

mongone vs vault guard

mongone() would send away a vault guard, whether alive or already
dead and parked at <0,0>, losing the information needed to remove
the temporary vault corridor once whatever obstacle (ie, the hero)
that's keeping it in place has been cleared.  Have mongone() call
grddead() like mondead() does, so that a guard will just be moved
to <0,0> with 0 HP and kept there rather than being purged during
the next dead monster removal.

Move polearm context reset to m_detach() [and, unfortunately,
duplicated in grddead()] rather than in having it in dmonsfree()
since execution of the latter is deferred.  I'm not sure offhand
whether a fast hero could get a second move before end-of-turn
purging of dead monsters.

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

Developer.txt: add notes about branches

Make sure can't chat out of the map

tribute in-game bonus

Give 20 experience points the first time the hero reads a passage
from a tribute novel.  It's enough to go from level 1 to 2 or from
2 to 3.  By the time a book store is found, that's too trivial for
most to care about, but it's potentially useful to a pacifist.

invalid escape
 Changes to be committed:
modified:   win/share/tiletext.c

Update the tile RGB values and mappings
 Changes to be committed:
modified:   win/share/monsters.txt
modified:   win/share/objects.txt
modified:   win/share/other.txt
modified:   win/share/tile2bmp.c
modified:   win/share/tiletext.c

extend the tty statue patch to tiles
 Changes to be committed:
modified:   doc/fixes35.0
modified:   win/share/gifread.c
modified:   win/share/monsters.txt
modified:   win/share/objects.txt
modified:   win/share/other.txt
modified:   win/share/tile2bmp.c
modified:   win/share/tilemap.c

The tty code already had the statue patch included, where
statues are represented by stone versions similar in
appearance to their monster likeness.

This extends it to tiles.

A new pass through the monsters.txt file is done
in tile2bmp to include new modified tiles to the output
file that are gray-scaled versions for mapping to the
NetHack statue glyphs.

Fix dark room glyphs when searching

Searching while blind showed room glyphs as lit even when
dark_room option was on.

fix "bad fruit #0"

When minimal_xname() set up a dummy object containing as few details
as possible, it wasn't setting up the fruit id field, so xname()
couldn't figure out what type of fruit it had and issued a warning.

I haven't managed a test case that uses minimal_xname so testing of
the fix is less than comprehensive.  [Pasi got it through dopay, but
that only resorts to minimal_xname if the formatted name is really
long and would otherwise cause the shopkeeper's prompt to overflow.
Long fruit name combined with long individual object name wasn't
long enough to trigger that.  Maybe uncursed, greased, rustproof
the like, or possibly just a longer shopkeeper name than I had?]

revisit the forcefight crash fix

The previous fix prevents the crash from 'the()' when NO_GLYPH was
used as an index into the defsyms array, but it resulted in giving
feedback of "you attack thin air" regardless of what was at the
target location, reverting to the situation that the buggy code was
attempting to address in the first place.  Handle that differently
by removing the unseen monster glyph sooner.  Also, the underwater
handling wasn't working as intended.

I blamed Derek's pudding farming patch for introducing the problem,
but all that did was replace the offending line(s) with different
indentation.  The older post-3.4.3 patch which produced the problem
was mine.  Sorry, Derek.

revert unneeded lookat() change

The enclosing switch statement is only reached when `glyph_is_cmap' is
known to be true, so remove the redundant extra test.

suggested booktribute mods
 Changes to be committed:
modified:   include/extern.h
modified:   src/files.c
modified:   src/objects.c
modified:   src/spell.c

- charge a little more.
- no free read in the bookstore.

prevent a couple of reported array index segfaults
 Changes to be committed:
modified:   src/hack.c
modified:   src/pager.c

Don't use glyph_to_cmap as an array index into
the defsyms[] array unless it really is a cmap.

Recent situation: glyph_to_cmap will return
NO_GLYPH for the unknown monster glyph 'I', which
is not a valid index for the defsyms[] array.

Allow wishing for polearm or hammer

which gives a random weapon with the matching skill,
so a random polearm, or a war hammer, respectively.

Fix saving xlogfile realtime

add an initializer

protect against bad dates

A recent fault on mingw32 revealed that faulty
code which passes a bad or out-of-range date
value could have game-fatal consequences.
Add some protection.

date verification

Prevent divide by zero in newman

...which happened when your max power was 0

Prevent illegal positions for tmp_at

It seems like m_throw() can call tmp_at() with !isok coordinates,
so prevent those in both functions

Prevent infinite loop

...if the map is filled with monsters, and for some reason
the drowning just won't kill you.

Infinite looping cannot currently happen, because no-one who
can drown can keep surviving the drowning once their amulet
of life saving is used up.

Prevent out-of-bounds coords when closing a door

Prevent segfault when looking around inside monster

A recent change by me introduced a segfault when
counting swallowed hero as a monster inventory item.

Move triplicate defines into single place

YAFM for magic flute

Unify description of hero's visage into single function

Sinks may be teleported or polymorphed by rings

Original patches by Leon Arnott and me.

Guard against merge segfaults

Trying to throw a quivered object resulted in a segfault, when
the code tried to merge the quivered object into itself.

YAFM for dragon sitting on gold

lint bits

regex lint

For the POSIX regexp interface, move local declaration to beginning
of block to avoid requiring C99.  Also switch to alloc() from bare
malloc() so that MONITOR_HEAP won't log a free which doesn't match
up to any allocation.  This results in a change in behavior:  if
the allocation fails, nethack will panic rather than report an
option parsing error.  In practice there will be no difference
because nethack is not going to run out of dynamic memory during
initial options processing.

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

nhsub: handle git add/commit options that take an argument

Add Auto open doors -patch

Inaccessible niches occasionally have iron bars

Some monsters can eat through iron bars

Any monster with rusting or corrosion attack can eat through
the bars. This includes rust monsters, grey oozes, and black puddings.

Original patch by Malcolm Ryan

A camera may contain a picture-painting demon

Original patch by Leon Arnott

Add litmus test

Original patch by Dylan O'Donnell

Use our dupstr instead of strdup

Fix gitignored

more regex integration bits

Makefile bit following merge

Add some new files to top level Files

add cppregex.cpp to NetHackW so it'll link properly

Add POSIX implementation of regex.

This also includes documentation of the regex engine in posixregex.c,
because I couldn't think of anywhere better to put it.

Catch regex matching errors to avoid crashing.

Use extern.h for regexes.

I was planning to do this anyway, but it created an include loop that
was breaking it on Windows.

Implement a new system-based matching harness.

The intent is to look for platform-specific facilities for regex
matching, to provide portable MENUCOLORS configuration files.

This is a prototype implementation being committed to see if Windows can
use the POSIX regex implementation provided with the C++11 standard
library. If this works, I will write a harness for POSIX regexes and for
pmatch(), and those can be linked in by platforms as appropriate.

pmatch() should be used only as a very last resort, because it breaks
compatibility between platforms.

Document wallification changes.

tribute post-merge smoothing
 Changes to be committed:
modified:   src/mon.c
modified:   win/share/objects.txt

Merge branch 'master' into nhmall-booktribute

Conflicts:
doc/fixes35.0
include/extern.h
src/mkobj.c
src/mon.c
src/objnam.c
win/share/objects.txt

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

More Guidebook
 Changes to be committed:
modified:   doc/Guidebook.mn
modified:   doc/Guidebook.tex

revised menu searching

Convert search in tty menus from pmatch to case-insensitive pmatchi;
convert search in X11 menus from substring strstri to wildcard pmatchi.

tty bug:  if the menu is full screen, the search prompt and subsequent
user input clobbers the menu header.

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

windows team update

new pmatch variations

Add pmatchi() to perform case-insensitive wildcard matching, and
pmatchz() which is also case-insensitive and ignores spaces, dashes,
and underscores like the type of matching done during wish parsing.
At the moment, neither is being used, although DEBUGFILES handling
uses pmatch and needs to be taught to distinguish between case-
sensitive and case-insensitive filenames so will eventually use
pmatchi when appropriate.

color of mithril armor

From a bug report two years ago, mithril armor should have color
HI_SILVER rather than HI_METAL.  Tolkien describes mithril as "like
silver which never tarnishes".  (The bug report had a more precise
quote, and a github URL for a patch which I've ignored.)  The Amulet
of Yendor has material set to MITHRIL but I haven't changed its color.
All amulets are HI_METAL, so if any modification is needed for it,
change the material rather than the color.

Mithril-coats are displayed as cyan both before and after this patch,
so it's hard to tell whether any change actually happened.

Files correction

text file error corrections
 Changes to be committed:
modified:   DEVEL/code_features.txt
modified:   Files
modified:   dat/history

On 4/11/2015 6:45 PM, Dion Nicolaas wrote:
> Errors in text files:
> - DEVEL/code_features.txt, line 10: 'alterting' should be 'alerting'

fixed

> - dat/history: Is it intentional that I'm no longer mentioned as
> maintaining the Windows port? I am for 3.4, but not for 3.6.

fixed

> - Files: The Files file is incomplete. Notably, the new files in dat/
> are missing (bogusmon.txt, engrave.txt, epitaph.txt),

fixed

> the DEVEL directory, doc/Guidebook.*

> - Some files in Files have the wrong case: NetHack.sln,
> NetHackW.vcxproj. I know it doesn't matter on windows but it looks wrong.

fixed

Allow disabling savefile UID checking in sysconf

Fix comment typo

obj_nexto_xy lint

restore accidentally consumed tmp_config

turn off annoying warnings in Visual Studio

lock MENU_COLOR_REGEX away since it isn't Win32 yet

Merge branch 'master' into derek-farming

Reverse merge before final reintegration.

* master: (40 commits)
  Fix magic mapped dark room symbols
  Disclose extinct species alongside genocided ones
  ...

Conflicts:
doc/fixes35.0
src/do.c
src/files.c
src/invent.c
src/objnam.c
sys/share/pcmain.c

Fix magic mapped dark room symbols

Disclose extinct species alongside genocided ones

Fix warnings, unused and shadowed variables

Cast time_t into unsigned long

Update precompiled lex and yacc files

Fix attribution of sortloot patch

Merge branch 'master' into nhmall-booktribute

Minor special level and compiler tweakage

Fix allowed map characters.
Make some predefined MAPs blend in better with randomly
generated parts.

Fix Sanctum phasing bug.

Restore the insect legs on Baalzebub's level.

revisit variable arguments in new lev_comp

The previous USE_OLDARGS worked with gcc on Intel, but was inherently
unsafe.  This method is completely safe, just obnoxiously intrusive.
It you disliked debugpline*(), you're bound to hate this....

Make WALLIFIED_MAZE into a level flag.

It should now be randomly disabled for a 3rd of Gehennom, to make things
a tad more interesting there. It's also disabled in Baalzebub's lair,
to make things a little more interesting.

Still don't know why the beetle is disappearing.

Add sortloot -patch

Adds the "sortloot" compound option, with possible values
of "none", "loot", or "full".  It controls the sorting of
item pickup lists for inventory and looting.

support pre-ansi varargs in new lev_comp

Remove the requirement for <stdarg.h> that was introduced to lev_comp.
USE_STDARG still works.  USE_OLDARGS required hackery but has been
tested and actually works, although I wouldn't trust it on platforms
where 'long' and 'char *' aren't the same size.  USE_VARARGS didn't
require any hackery--aside from the conversion to core's pline code--
but has not been tested:  <varargs.h> supplied with OSX won't compile,
with an #error directive that basically says "switch to <stdarg.h>".

I changed several printf formats of %i and %li to %d and %ld because
I'm not sure how widespread the 'i' variant was back in days of yore.
[TODO:  avoid use of snprintf since pre-ANSI systems won't have it.]

Add 2 new Medusa variants

Add 10 new bigroom variants

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

nhsub docs and fixes

Do not clean tribute file on Unix.

Renumber other.txt tiles

Clive Crous' Dark Room patch

This patch adds a new glyph that represents dark parts of room,
splitting the dual functionality of the "unexplored/dark area" glyph.

Add quotes for Eric and Moving Pictures.

Indicator glyph for good target positions

Add a new glyph, S_goodpos, to indicate valid locations
instead of reusing the camera flash beam glyph.

tty fix for bad curs positioning (-1,n)

DEBUGFILES set to "wintty.c" reported a bad cursor positioning attempt
at the end of the RIP tombstone, and when set to "questpgr.c wintty.c"
reported a whole bunch right at the start of the game when enumerating
all the quest messages for the chosen role.  Both were triggered by
this x==0 call to tty_curs() near the end of process_text_window().
    if (i == cw->maxrow) {
        if(cw->type == NHW_TEXT){
            tty_curs(BASE_WINDOW, 0, (int)ttyDisplay->cury+1);
            cl_eos();
        }
    ...
The x value is always decremented in tty_curs, so passing in 0 yields
a bad value of -1.  The bad call returns without doing anything, and
when DEBUG is disabled, it does so silently.

No fixes entry; it was caused by a post-3.4.3 fix for something else.

attempt to fix Makefile issue with tribute

Merge branch 'nhmall-booktribute' of https://rodney.nethack.org:20040/git/NHsource into nhmall-booktribute

function definitition to match prototype const

Adding quotes for The Light fantastic

Reformat and passage for The Truth

nhsub: handle some more cases

Merge branch 'master' into nhmall-booktribute