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

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

win32-gui: parse value of gold correctly for status hilite

Guidebook updates (misc).

mondata.c formatting cleanup, ancient AIX PS/2 bug

When looking to see whether the monsndx() panic could provide any more
useful information [if a pointer that's supposed to point into the
mons[] array doesn't, I don't think that there's a whole lot of other
information available aside from whether it is null or not, and that's
implicitly provided already], I went through the whole file cleaning up
the formatting and making sure every routine was preceded by a short
(usually one line) comment.

There were a few bits of code reorganization.  I changed little_to_big
to have a single point of return.  The 25 year old workaround for a
compiler bug on a defunct platform may or may not still be applicable;
I took that out.  If we get segfault reports for AIX on PS/2, this is
the first place to look.  (big_to_little is nearly identical and didn't
have the same workaround.  Not needed, or not called often enough for
any AIX PS/2 user to be affected?)

more work on the tty status_hilite code
 Changes to be committed:
modified:   include/wintty.h
modified:   src/botl.c
modified:   win/tty/wintty.c

yet more updates
 Changes to be committed:
modified:   include/botl.h
modified:   include/extern.h
modified:   include/wintty.h
modified:   src/botl.c
modified:   src/options.c
modified:   src/windows.c
modified:   win/tty/wintty.c

get the tty versions started

more updates
 Changes to be committed:
modified:   include/botl.h
modified:   src/botl.c
modified:   src/windows.c
modified:   win/tty/wintty.c

some code rearrangement/cleanup in botl.c

notify windowport of hilites when status is initialized
 Changes to be committed:
modified:   src/botl.c

more adjustments
 Changes to be committed:
modified:   src/botl.c
modified:   src/options.c

more status_hilite work
 Changes to be committed:
modified:   include/extern.h
modified:   src/botl.c
modified:   src/options.c
modified:   src/windows.c

defer notification of the window port until after
proper initialization. Options are processed very
early in 3.6.0

interim updates
 Changes to be committed:
modified:   include/botl.h
modified:   src/botl.c
modified:   src/windows.c
modified:   win/tty/wintty.c

Move the windowport stuff out of botl.c and into windows.c
where it belongs.

last? Sting

If Sting is glowing when blindness gets toggled, give a new "glowing"
message.

So instead of
  Sting glows blue! [...] You can't see! [...] Sting stops quivering.
if you're still blind when the last orc goes away, or
  Sting quivers slightly. [...] You can see again. [...] Sting stops
  glowing.
if you were blind when the first orc arrived, now you'll get an
intermediate message between the second and third ones.  'Sting is
quivering' for the first case, 'Sting is glowing' for the second.
No matter how many times blindness toggles back and forth, the final
"stops glowing" or "stops quivering" will be consistent with the most
recent "is glowing" or "is quivering".

Sting again

When formatting uwep, verify that warning comes from wielded weapon
before appending "(glowing <color>)".

Use a menu to pick a container to tip

If there is more than one container, the #tip command will show a menu;
if there's just one container, prompt for tipping.

As per Boudewijn's suggestion, remove the superfluous
"There is a container here" message.

more Sting,&c

Add "(glowing light blue)" to the formatted object description when
Sting or Orcrist is glowing due to presence of orcs or "(glowing red)"
if Grimtooth is glowing due to elves.  Use "(glowing)" if blind;
assumes that some aspect of the glow (perhaps warmth or vibration) can
be noticed via touch.

Make enlightenment's "you are warned about <monster class> because of
<artifact>" catch up with Orcrist and Grimtooth.  It was attributing
Orcrist's warning against orcs to Sting, and Grimtooth's warning was
against "something" rather than elves.

The glow color is now a new field in artilist[], so the biggest part
of this patch is adding an extra value to each artifact's definition.

Merge branch 'master' into status_hilite

some updates
 Changes to be committed:
modified:   include/botl.h
modified:   src/botl.c

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

some documentation updates
 Changes to be committed:
modified:   dat/history
modified:   doc/Guidebook.mn
modified:   doc/Guidebook.tex

- include new 3.6.0 beta testers in dungeoneers list

get_cost_from_price

I couldn't figure out why walking over a container in a shop might
give the wrong price; the code looks correct.  But I've reorganized
get_cost_from_price to perform the cheapest tests first.  The u.ushops
check should probably be done in doname to avoid calling this routine
at all 99.99% of the time.

disoriented teleport attempt

Reported for pre-beta, getting "you feel disoriented" when attempting
to teleport within a level while carrying the Amulet, you still ended
up teleporting.  Wizard mode allows the disorientation to be overridden
but the logic was wrong.  It worked as intended when in wizard mode but
unintentionally always overrode disorientation when not in that mode.

Merge branch 'master' into status_hilite

formatting fix

menucolors parsing

Revise the menucolor parsing (color and attribute portion, not the
regexp part) to switch to the string matching used for wishing in
order to allow space in the "light <foo>" entries instead of forcing
the two words to be run together.  Having them be run together still
works, as does use of dash or underscore to separate the two words.
So the canonical form for light blue is now "light blue" instead of
"lightblue", but all of "light blue", "lightblue", "light-blue", and
"light_blue" match it.  (So do weird things like "--li-gh_-_tbl ue _"
but I won't lose any sleep over that.)

Almost all of this if formatting; mostly blank line after declarations
but also there was new stuff that didn't match the recent reformat.

add warning glow for Orcrist & Grimtooth

Make Orcrist glow light blue when orcs are present, just like Sting.
(Sting supposedly glowed because it was made by the elves of Gondolin
rather than any particular attribute built into it, and Orcrist was
made there too.  I think it also glowed in the Hobbit; that was how
Bilbo recognized what the situation was when he first saw Sting glow.
Maybe it was the other sword rather than Orcrist, but they were treated
as being functionally equivalent.)

Also make Grimtooth glow red when elves are present.  That's from thin
air, to give it some novelty.  Unlike Sting, whose double-damage bonus
is restricted to orc targets, Grimtooth's weak 1d6 bonus still applies
to all targets.

'Du' bug with non-empty containers

ckunpaid() had the same coding error as allow_category().  A hero-owned
container holding hero-owned contents followed in invent an any unpaid
object was mis-classified unpaid.

Adjust the getpos() hint for naming a floor object if the hero happens
to polymorphed into something which is hiding under an object.

Also, make the attempt to name a floor object while hallucinating give
a more interesting result.

Add vibrasquare symbol to Guidebooks

Renumber glyphs for vibrasquare trap conversion

Fix typo in potion dilution message, found by Raisse

using #name to call a floor object

Implement Boudewijn's suggestion that #name be extended to allow naming
something of the floor.  I'm sure he wants this so that he can avoid
picking up gray stones, but it's something I started to implement years
ago (probably at an earlier suggestion from him...) and then forgot all
about.

This changes the #name menu to be
   m - a monster
   i - a particular object in inventory
   o - the type of an object in inventory
   f - the type of an object upon the floor
   d - the type of an object on discoveries list
   a - record an annotation for the current level
   What do you want to name?
with the i and o choices omitted when inventory is empty.  If the
'lootabc' option is set it will use a through f instead, but then the
last three entries change letters when inventory is empty.  'y' and 'n'
are still accelerators (effectively hidden choices) for the i and o
entries, corresponding to the answers for the 3.4.3 and earlier "name
an individual object?" prompt.

The floor choice asks you to pick a location.  If you pick yourself,
then the top object of the pile underneath you is targetted.  Otherwise,
the target must be an object glyph, and the object must have its dknown
bit set, so have previously been seen up close or revealed via blessed
potion of object detection.  To make it be more useful, targetting an
object on an adjacent square will set the dknown bit.  (Just the top
object if there is a pile there.)  There's no cockatrice corpse touch
check since you aren't actually touching anything, just looking.

The setting of dknown bit for an adjacent object has been extended to
the '/' and ';' commands for examining things on the screen as well.
It's only done for adjacent spots you actively select, not all 8 spots
around you.

update the name of the fixes file in Files
 Changes to be committed:
modified:   Files

a note about the reformatting
 Changes to be committed:
modified:   doc/fixes36.0

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

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

Conflicts:
src/artifact.c

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

Resync prior to pushing tribute changes.

don't see Sting glowing light blue if you're blind
 Changes to be committed:
modified:   doc/fixes35.0
renamed:    doc/fixes35.0 -> doc/fixes36.0
modified:   src/artifact.c

Add a quote for The Fifth Elephant

Sting's glow while blind

Give an alternate message if Sting starts or stops glowing while the
hero can't see.  It probably ought to give an immediate message when
blindness toggles but that looks like it could get messy.

Having an 'o' die or migrate off level should probably also redo the
Sting message immediately, otherwise we see things like:
  The little dog kills the goblin.
  The little dog eats a goblin corpse.
  Sting stops glowing.
(There could be lots of additional intervening messages depending on
other monster activity at the time.)  Calling see_monsters() in the
relevant places--probably m_detach() and migrate_to_level()--would
address this but won't do because that could result in hallucinating
monsters changing appearance mid-turn.

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

Terry Pratchett quotes from Raisse
 Changes to be committed:
modified:   dat/tribute

MSGTYPE lint

Fix a couple of bits of lint in options.c, plus do some formatting
cleanup (in the vicinity only, not the whole file).

fix [pre-]beta 'Du' bug with containers in shops

'Du' in a shop was listing hero-owned containers that didn't contain
any unpaid items.  At least one unpaid item must be carried; bug
manifested iff one or more unpaid items followed the container in
the invent list.

Recently revised allow_category() was using count_unpaid() for
container contents incorrectly, inadvertently checking the rest of
inventory after the container in addition to its contents.

strncmpi, not strcasecmp

no, really, use the right header

Show cost of merchandise when walking over it

This is another feature the betatesters seemed to miss a lot.

Make options-menu more uniform

Rename omit_buc option to implicit_uncursed

... so the name is less cryptic.

Add MSGTYPE config option

MSGTYPE allows the user to define how messages in the message
area behave. For example:

  MSGTYPE=stop "You swap places with "

would always make that message prompt for -more-.  Allowed types
are "show" (normal message), "hide" (do not show), "stop" (wait
for user), and "norep" (do not repeat message).

Adding this, because it's relatively simple, proven to work, and
it seemed to be the major thing betatesters felt was lacking when
compared to NAO.

Remember novel titles in bones.

hacklib.c formatting

hacklib.c took a beating in the reformatting, so clean it up.
A tweak to the anti-predictability hack in setrandom() is the only
change in the actual code.

omit_buc tweak

gcc complained about mixing && with || without parantheses.  After
scratching my head a bit, I think this change yields the intended
result.

'omit_buc' is a bad option name.  It's cryptic and it doesn't even
describe the function.  At a minimum it ought to be changed to
'omit_uncursed' to accurately describe what it does.
'implicit_uncursed' or 'explicit_uncursed' (with opposite boolean
value) would be even more precise but probably not any clearer.

Add omit_buc -option

The option defaults to on, which is the old-style behaviour.
Turning the option off will never omit the "uncursed" -status
from inventory lines. This is pretty much required if users
want to use menucolors based on the BUC state.

New precompiled versions of these

mimics mimicking strange objects

The code that intended to have mimics occasionally take on the form
of "strange object" always produced downstairs instead because
S_MIMIC_DEF is greater than MAXOCLASSES.

This problem was present in 3.4.3.  I didn't try to go back to see
how long it's been there, but strange objects used to occur once
upon a time.  Either nobody noticed that they'd gone away or there's
an alternate way to produce them.

More wishing nits.

Count only stacks when listing container contents.

By popular demand of the beta testers.

Provide feedback when one of your items gets wet.

questpgr.c's in_line[], out_line[]

Replace static in_line[] and out_line[] with local variables that are
released when the quest pager code returns to caller.  QTEXT_IN_SIZ
was already removed from makedefs; now QTEXT_OUTSIZ is removed from
nethack.  Use regular BUFSZ for them instead of trying to maintain a
separate size for quest text.

wishing bit - royal jelly

Fix squeaky board creation segfault

If a trap is created on top of another trap, maketrap reuses
the trap struct in place, instead of deleting and recreating it.

If a squeaky trap was created on top of another trap, maketrap
first set the trap type to squeaky board, and then tried to
look through all squeaky boards on the level, to determine
what note the new trap should play. Unfortunately, the union
with the trap note most likely contained a rolling boulder
coordinate or something else outside the 12 note range, so
then the tavail-array lookup would cause a segfault.

Fix dark room glyphs on level re-entry

trickier lint cleanup

Suppress some mostly longstanding "unused parameter" warnings where
the usage was generally conditional.

restlevl() had a conditional closing brace that confused the recent
reformat, resulting in some code inside a funciton ending up flush
against the left border (first column, that is, as if outside of the
function).

makedefs lint

makedefs.c: In function 'fgetline':
makedefs.c:2630: warning: comparison between signed and unsigned
makedefs.c: At top level:
makedefs.c:2638: warning: ISO C does not allow extra ';' outside of a function

fgetline() makes my recent check for quest line truncation become
redundant, so strip that out.  I haven't taken the corresponding
macro out of qtext.h though.

[I didn't see any patch introducing 'fgetline'.  Was it hidden as
part of some big merge?]

add missing tile and correct clobbered header file

Add clang-formatting file.

This should be in the repository, even though nobody can use it yet.

Remove stale version control lines.

Reformat .h files.

I did my best to exempt some of the bigger aligned blocks from the reformatting
using the /* clang-format off */ and /* clang-format on */ tags. Probably some
that shouldn't have been formatted were anyway; if you encounter them, please
fix.

The clang-format tags were left in on the basis that it's much easier to prune
those out later than to put them back in, and it means that, modulo my custom
version of clang-format, I should be able to run clang-format on the source tree
again without changing anything, now that Pat has fixed the VA_DECL issues.

Convert the vibrating square to a trap

Patch due to ais523 in NetHack 4.

This is not ready to be merged yet; the vibrating square needs a tile
image for tiles builds.

remove #include of file that doesn't exist

fix a mistake on patch merge
Changes to be committed:
modified:   util/makedefs.c

Warning cleanup.

Make the wand of wishing chest immune to trapping

This eliminates a source of very random, unpreventable frustration to
players.

Make long worms unleashable.

Fixes a bug reported by ais523. Rather than account for individual
segments, I opted just to make them unleashable, because it's not very
useful behaviour anyhow.

Clean up ceiling nits.

Fixes C343-102.

Files update

merge two sets of patches to makedefs.c,qtext.h
 Changes to be committed:
modified:   include/qtext.h
modified:   util/makedefs.c

Provide documentation for the regex engine.

The guidebook entries could do with more detailed descriptions, but this
will do for now.

Use the common regex engine in more places.

In particular, in autopickup_exceptions and user sounds.

Modify makedefs to use dynamic buffers for reading.

This will prevent crashing due to excessively long lines.

prep for sean's emailed patch

prep for sean's emailed patch

Add pmatch-based regex engine.

As mentioned, this is not tested. It should be simple though.

Audit rloc()

Most of the time, rloc() is used for teleporting monsters and it's not a
big deal if they can't find somewhere to go. In a few cases, it is. I
went through all the callsites and made calls to rloc() not cause
impossible()s if they don't need to.

Fixes a bug/suite of bugs reported by ais523.

Clean up a rare corner case with mnearto().

With this fix, if the second mnearto fails, we don't accidentally strand
othermon at (0,0).

Fix sleep explosion causing impossible().

Also clean up come ternaries while I'm here.

My first attempt to fix was to add AD_SLEE to explode(), but that failed
because do_break_wand() already does the sleeping portion. I don't
generally like the duplication between explode() and do_break_wand as a
result, but I consider that issue a project for another day.

minor comment typo spotted
 Changes to be committed:
modified:   src/objnam.c

makedefs -q fix

Make the input buffer for quest messages bigger so that the expanded
header line from nhsub won't be too long.  Also, makedefs will notice
and report too long lines ('makedefs -q' only) and sanely proceed with
the rest of the file instead of treating the excess part of a long
line as a separate line.

quest summary messages for B and C

Add '%E [summary line]' to '%Cc' messages for Barbarian and Caveman.
Archeologist was done several years ago; the other roles still need
them.  Creating them is fairly tedious, but DEBUGFILES=questpgr.c
allows them all to be checked on turn 1 via ^P, a great improvement
since that first set.

msghistory bandage

Have genl_putmsghistory() pass the message to pline() for the !restoring
case, so that quest summary lines are delivered as ordinary messages.
No effect on tty or win32, which have their own putmsghistory routines.
But for X11, which has a multi-line message window but no save/restore
implementation for its contents, this makes the quest summary lines
actually show up somewhere.  (I looked at maybe implementing
X11_getmsghistory() and X11_putmsghistory() but don't have the energy to
tackle it.)

Other interfaces which lack their own history save/restore will see the
quest summary messages too.  Presumeably they'll all have multi-line
history windows so the extra line won't be displacing the most recent
message.  If not, they'll essentially get the long quest messages twice,
once in full via popup window, then the one-line summary via pline.

qtdump() overhaul

When dumping quest messages at startup via DEBUGFILES=questpgr.c,
give a single message for each one, instead of a pline showing the
message number and delivery protocol followed by a popup message
window containing the text.  This puts the number and protocol info
at the start/top of the popup window, bypassing the pline (and the
extra --More-- given for tty).

beta info
 Changes to be committed:
modified:   src/version.c