Some win32 updates from <Someone>;
also corrects a cut-and-paste error that I made in options.c

The following options can be set in the game now:

  WC_HILITE_PET
  WC_ALIGN_MESSAGE
  WC_ALIGN_STATUS
  WC_SCROLL_MARGIN
  WC_MAP_MODE
  WC_FONT_MESSAGE
  WC_FONT_STATUS
  WC_FONT_MENU
  WC_FONT_TEXT
  WC_FONTSIZ_MESSAGE
  WC_FONTSIZ_STATUS
  WC_FONTSIZ_MENU
  WC_FONTSIZ_TEXT

fix B3032 -- wand of speed monster id

     Make wands of speed or slow monster known if their effect
on monsters is observed; likewise for speed boots.  Also, avoid
giving odd "the bat is moving faster" when seeing a bat created
in gehennom and inaccurate "the monster is moving slower" when
a monster puts on speed boots.

win32 argc fix
fix a problem when NetHack directory contains spaces.

fix some problems with option modification settings in the
new options processing.

fix B1022 -- bare handed to-hit bonus

     Make the code match the comments.  I'm sure I checked that
forwards and backwards when I put it in; I don't know how it ended
up mismatched.

Files update

     Remove entries for the dropped sys/*/old/* files.

This just brings the version number references in line.
An actual history blurb summarizing the past 3.3
release is still missing from guidebook and history.

Remove experimental option from defaults.nh.
sys/winnt/defaults.nh to be specific.

display "default" in options list
Have the options list display the word "default" for some
of the window-port options when the preference hasn't been
set by the player.

More rename follow-up

rename follow-up
This patch assumes that the file rename of
sys/winnt/winnt.cnf to sys/winnt/defaults.nh has been done.

file rename winnt.cnf to defaults.nh

(from <Someone>)
The patch that deals with tabs is attached. I also fixed bug when the
program won't quit if Quit option is selected in the player selection
dialog.

modifying 'O' command behavour

I was asked how a window-port controls which options are
set to SET_IN_FILE, DISP_IN_GAME, or SET_IN_GAME.

This provides a run-time way to change an option's SET_IN_FILE,
DISP_IN_GAME, or SET_IN_GAME status through code, rather
than clog up options.c with a lot of compile-time #ifdefs
for different ports to offer different default option settings.
Update the documentation to reflect this.

data.base entries
Master Kaen and the Grand Master were the only quest leaders/nemeses
for which /n printed nothing at all, fall back on human entry

autodig bit

demon summoning using wrong alignment
make msummon pass along A_NONE, allowing summoning of any demon, not just
chaotic ones

discovering identify and fire scrolls
reading one of these for the first time should give experience, like
other scrolls

untrapping traps in unreachable locations
don't allow untrapping of an adjacent trap you cannot reach by movement

salamanders
If salamanders are made immune to slime, then all the other places in the
code which check for fire creatures should be modified too.

salamanders immune to sliming
Since salamanders are plane-of-fire creatures, they too should be immune to
fire like fire elementals & vortexes

msdos build would still be broken.

compilation bit
prototype should come before actual declaration of give_may_advance_msg

options fixes

     Several boolean options were mis-categorized and became
unsettable during play.  Also, the eight_bit_tty option got
dropped into limbo.  Fixing that revealed a typo which would
have broken compilation for MSDOS.

     `eight_bit_tty' has been renamed `wc_eight_bit_input',
but its only use anywhere in the code is for output.

Fixes from <Someone>
- added support for new options
- fixed couple of typos in options.c
- added sunken edge around message window
- added capability to replace font in the font table (if we will want to
  change font sizes dynamically later)

resync generated files

Fixes (from <Someone>)

- visual feedback after a prompt: append an empty line to the message
  window when clear_nhwindow(WIN_MESSAGE) is called. Filter
  out empty lines in the buffer except for the active slot.

- append ellipses to the status line text if the text is truncated

- get rid of message boxes at the end of the game (ignore empty lines in
  raw_print as Yitzhak suggested)

Use tabs for option menu separation if iflags.menu_tab_sep is set.
Simplifies some ports option menu displays.

proto bit

from Yitzhak

Add absent prototypes to some core routines.
Also add some port function() to function(void) in some win32 routines.
Also updates the Borland C Makefile for win32.

fixes34.0 updates

touchstones
win32 gui
wish word order
window-port preferences
disclosure prompts (or not)

touchstone bit

grammar in README

more inventory selection
the !allowcnt case wasn't being handled correctly, cuasing counts to
be set to 0 inappropriately.

fix cl_eos() in NT tty port.

crash fix
don't try to clear the window if windows haven't been inited yet

Update to 3.4; missed in first go-around.

Documentation updates
Document the change to the window options and
correct a typo that Yitzhak pointed out in window.doc

3.3.2 to 3.4.0

3.3.2 to 3.4.0

Add fixes34.0, send fixes33.2 to the attic

Please update fixes34.0 from this point, rather than fixes33.2.

wincap field updates

As Warwick suggested, instead of having fixed tile
sizes as options, allow specification of the size
explicitly.

Also, at Yitzhak's suggestion, provide a hook for
overriding the port's tile file name.  That name,
and the contents of the file it points to, will
be window-port specific of course.

test_move() fixes, for travel.

Turns lots of duplicated "flags.move = 0; nomul(0);" code into
a single instance, fixing the ones that weren't complete.

Doesn't feel_locations when only testing.

force tingling nose
<Someone> wrote:
>I also note that it's weird for your nose to stop tingling and your
>sense of smell to return to normal, if flags.beginner prevented your
>realising that your nose was tingling in the first place.

Qt: Fix default dialog button
In Qt 3, default button is always set to something... it was set to "Cancel"
in this case - not a good thing when you are granted a wish :-(

tty: ^C while prompted re-displays last message
- attempt to determine if tty_wait_synch() is called during the ending
 dialogs due to an interrupt, and not re-display the previous message
 at the wrong time
- also, "msg_window display anomaly" fix was missing code to still treat ^P
 properly while in a prompt

unseen monster messages
- watch messages if you can't see the watch now start "You hear"
- unseen spellcaster messages are shown as "Something", to remove
 some silly message pairs.
- unseen spellcaster that casts at itself causes no message
- recalc vision before spoteffects messsages, to avoid invalid "It"

artifact.c comment

From the beta-testers; the ability to detect bad food should detect slime as
well as cockatrice meat.

reflection
Found in the beta-test.

makedefs.c

Fix some misplaced logic in makedefs.c, and improve the surrounding
comments.

<email deleted>

The following fixes several bugs:
1) Mismatch between docs and game in definition of what '+' resolved in
favor of docs...
2) When game needs to be recovered a message box is shown.  This is a very
deprecated fix.  It pretty much answers just the conditions that require
this (a yn question to an erroneous winid), and is not useful for other
purposes.
3) The score file is written.

Add final slash to URL
in README, Guidebook, dat/history. (from <Someone>)

update window.doc document.

Fix problem with cloth scratching the stone From a bug report.

tty: msg_window display anomaly
If you interrupt nethack (^C) in tty windowport mode while a pompt is
displayed and and msg_window=true, it can display the full message history
at an unexpected time.  Detect this and display only the topl, as expected.

address compilation warnings from gcc

Adjust window-port related option processing
to allow common parsing in the core, and direct access to the
results by the window port.

Notes:

o Adds a new field, wincap, to the window_procs
structure for setting bits related to the preference
features that the window port supports.  This allows
run-time determination of whether a particular option
setting is applicable to the running window port.  A
window-port is free to support as many, or as few,
of the available options as it wants.  Ensure that
only the ones supported have their corresponding bit
set in window_proc.wincap. [see chart in
doc/window.doc for help with that.]

o The settings I stuck into wincap for each window
port are almost certainly not accurate, so each port
team should review them.  You should only include
the ones that you will actually react to and make
adjustments for if the user changes that option.
Without the setting in wincap, the option won't even
show up in the 'O'ptions menu.

o preference_update() added to the window-port
interface, so that the window-port can be notified
if an option of interest (an option with its
corresponding bit set in wincap field) is
changed.

o provided a genl_preference_update() routine in
windows.c and used it for all the existing
window ports since they don't have a functional
one of their own yet.

o this messes around heavily with iflags and the options
arrays in options.c

o I hope I didn't break any port's existing code. I
tried not to.  The Mac however, in particular, should
be looked at because it suffered a namespace collision
with what I was working on around fontname.  It had
Mac specific font stuff in options.c. Please test
the Mac.

fixes33.2 tuneup

Log recent additions

<Someone>'s OS2 updates.

Artifact prices

Each artifact has its own individual price, rather than being solely
based on the cost of its base item.

samurai quest bit

     From one of <Someone>'s ascension posts:  the introductory message
when you enter the Samurai's quest is slightly odd if you can't see at
the time.  This rewording acknowledges the difference between being
blind or sighted, but it is still a bit odd to able to know which flag
is flying when unable to actually see it.

blessed +2 fireproof speed boots
The inability to wish for a blessed +2 fireproof speed boots really was a bug.
Prefixes are checked for inside a loop which should allow them to be in any
order; for some reason +nnn and empty were outside that loop.

Also adding the secret door detection fix to betabugs 3.3.0.  (Is it safe to
hand-modify betabugs 3.3.2?)

unix: default to groff for generating .ps from .mn
also make it more obvious how to switch commands to create a .ps file

teleporting monster while swallowed
if player teleports a monster while swallowed on a noteleport level, the
player should not teleport along with the monster

fixes updates
update fixes to include updates sent via patches

Mac pragma tuneup

The Macintosh CodeWarrior port does not need nor support
the undefined pragmas.

Knights start with a lance

Make Knights start with a lance.
Eliminate "bashing" message if jousting.
Remove obsolete code from u_init.c.

Macintosh getlin and BUFSZ

Ensure getline() returns no more than BUFSZ characters
on the Macintosh port.

dragons have scales, not fur
body_part should return "scales" for the HAIR part of a dragon

vortexes are not really alive
add vortexes to the set of nonliving() monsters

hilite_pet Guidebook entry
update hilite_pet entry so it allows for variations possible in current ports
and be less Unix-specific

SPLEVTYPE debug mode environment variable
SPLEVTYPE can have a value like "minetn-3,minend-2,soko2-2" and allows
someone in debug mode to select specific instances of random levels to use
in a test game.  Invalid values found in SPLEVTYPE are silently ignored.

grappling hook followup
fix a compilation bug

At the request of <Someone>:
I'm surprised that you credit me with the ninja yumi-ya thing as a
"patch" in README; I didn't submit any code for it, it's trivial, and
it's vanishingly unlikely ever to come into play :-) If I'm to be
credited with anything on that list, I'd have thought it'd be the
no starting pet option.

dead key fix
Apparently, " is part of an accented character in US(international) keyboard
layout (additional character specific to the language of an origin - for
example, left and right double quotation marks). The code did not handle it
too well since it maps to 2 ASCII characters instead of one (one of them is
so called "dead" character). We can ignore the dead character as a
workaround for this problem. The patch is attached.

<Someone>.

Rumors tuning

Changes to rumors, From a bug report.

Gnomish Mines tuning

Give names to remaining town levels.
Change elements in College Town that don't follow theme.
Remove isolated niche and helm of brilliance from end #3.

grappling hook ignores spot effects
Using a grappling hook can land you in water, lava, et al, but you were
unaffected.

fix crash during Drop unpaid
The patch is attached. Array bounds went unchecked in
the menu page operations (, \ ~)  This resulted in
memory corruption.

The actual crash depends on your luck actually. It will
only crash if  heap headers are corrupted, otherwise it
can go unnoticed.  When you do "Du," the list page size
is 18 (on my screen) with only 2 items in the menu. The
program assigned count of -1 to 18 items in the array
of 2. Ka-boom.  I put bounds checking code in several
places.  The window size does not have anything to do
with it.

<Someone>.

tty pet hiliting may turn off on color changes
check to turn on hilite must come after check to change color, since
changing color uses term_end_color which may turn off inverse video too

<email deleted>

bugfix: map position was calculated incorrectly when switching into
fit-to-screen mode.

medium and large flyers do not flutter

remove debug message

leather cloak should be made of leather

typo
"splatt" should be "splat"

correct updated cost of scalpel
rev 1.6 included the wrong updated cost

remove left over debugging message

various Guidebook updates
+ attributes may exceed 18 for non-humans
+ update spell casting paragraph to loosely describe 3.3 style spell casting
+ correct description of the output of the '+' command
+ note default value for 'mail' option

eggs and other breakables break falling down stairs
<Someone> questioned why eggs and potions don't break when falling downstairs.
For that matter, other breakables should as well.

/tmp/msg

disclose refinements:
- allow spaces between the different possibilities
- add a missing null at the end of the list of prefixes to prevent index()
  from going crazy
- slight re-wording of the Guidebook

Fix "gems in apply menu" From a bug report.

B2014 (comment change)

egg breaking
This fixes beta bug 2012.

autodig chattiness
The autodig code still requires lots of interaction, because of all the
digging messages.  Make these messages behave more like boulder pushing
messages, where they only display if the previous move wasn't a push.

Fixes:
- B1026  [confirmed] win32: setting multiple options
- bugfix: menu: set focus to the item and make it visible when it is
selected  with accelerator key.
- increased menu font size by 1 point
- layout the main window on WM_MOVE message so the menu windows stays with
the main window  (it was not changing the location before)
- got rid of WinCE specific define (ARM)
- handle all keyboard input via WM_KEYDOWN instead WM_CHAR

Files affected: config1.h mhmsg.h mhmain.c mhmenu.c mswproc.c resource.h
winhack.rc

Notes:
 That overrides Yitzhak's mhmenu patch - I don't like the menu prompt on the
title bar since it can be null and in most cases it is. It looks much better
as a header column in the list and is displayed only if present.

I moved ARM-related (processor) defines to wceconf.h  You cannot undef ARM
before windows.h is processed - ARM version won't compile.

.rc files are generated by IDE - you cannot edit them directly or your
changes will be gone next time the file is saved. If anything needs to be
added to winhack.rc file manually it has to be marked by the following
defines or via menu option View->Resource Includes:
#define APSTUDIO_READONLY_SYMBOLS
...
#undef APSTUDIO_READONLY_SYMBOLS

win32gui patches (from Yitzhak)
[...]
One of them deals with the patch you just sent, that changes the name but
does so more extensively, particularly, also in the Save dialog in
mhmain.c.  Another is one of the changes to winhack.rc to make it compile
under Borland.

This also (beyond patches I sent): fixes no Options prompt bug, and fixes
up the menu bar (which I broke in one of my patches).

Tested on msc/bcc tty/win makefiles and vc ide.

invisible things due to black colour
Patch to fix invisible orcish helmets, orcish daggers,
mimic (]), ants etc.

Change title to prevent Beta tester complaint.