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.

found another popd command.

Fix B2001
B2001 <Someone> [reported] win32?: giant ant is black square

Where is the open doorway with the giant ant standing in it?
There used to be a closed door there, if I recall correctly.
[now there is just a black square]
I'm saving the level files at this point, and I will be saving
the game after I did that, in case you're interested in any save
files.

Allow the options specific to the win32 graphical port
to be silently ignored by the tty port, so that the
config file can be conveniently shared between the two.

Unix port bits
don't require setup.sh to be executable, just tell to use sh explicitly.
Fix funny DOS-ish comment

One of the beta testers asked if the win32 scrolling
offset could be configured.  This allows it to be
set from the config file using:
win32_map_cliparound_margin.

unused variables

another follow-up for mhmap.c
Oops, it should have been VERSION_MAJOR < 4,
VERSION_MINOR < 4 and PATCHLEVEL < 2

follow-up for mhmap.c

Change mhmap to use mapglyph()

update initial wizard quest message
The initial wizard quest message still used the word "master", which
can be interpretted as implying "male"; changed to teacher.

Makefile.nt fixes from Yitzhak.

>B1014 <Someone> [reported] change request - disclosure default
>
> I'd like the default for "Would you like to see your <whatever>"
> at the end of a game to be "y" instead of "n". I haven't asked
> for full disclosure in order to have it skipped if I press the
> space bar once too often by mistake.

This changes the way the flags.end_disclose array is used to
allow what this request is asking for.  It should be backward
compatible with previous "disclose" options.

The order that the end_disclore options are stored:
inventory, attribs, vanquished, genocided, conduct
There is an array in flags:
end_disclose[NUM_DISCLOSURE_OPT];
with option settings for the each of the following:
iagvc [see disclosure_options in decl.c]:
Legal setting values in that array are:
DISCLOSE_PROMPT_DEFAULT_YES  ask with default answer yes
DISCLOSE_PROMPT_DEFAULT_NO   ask with default answer no
DISCLOSE_YES_WITHOUT_PROMPT  always disclose and don't ask
DISCLOSE_NO_WITHOUT_PROMPT   never disclose and don't ask

Those setting values can be used in the option
string as a prefix to each disclosure option
to get the desired behaviour for that option.

For backward compatibility, no prefix is actually required,
and the presence of a i,a,g,v, or c without a prefix sets
the corresponding value to DISCLOSE_YES_WITHOUT_PROMPT;

The actual prefixes used are controlled by the following in flag.h:
#define DISCLOSE_PROMPT_DEFAULT_YES 'y'
#define DISCLOSE_PROMPT_DEFAULT_NO 'n'
#define DISCLOSE_YES_WITHOUT_PROMPT '+'
#define DISCLOSE_NO_WITHOUT_PROMPT '-'

As far as the docs go, I don't know if I've got the *roff
stuff right.   The TeX stuff looks okay when I converted it to .pdf.

This increments EDITLEVEL.  If that is a problem, I can
add a routine to restore.c to perform a conversion of the old
values in flags. Let me know.

scalpel and corrosion
make scalpel type METAL (for stainless steel), also increase cost to
reflect it's higher quality

getobj using counts from inventory display
getobj used display_inventory when "?" was selected.  However, any count
entered via the menu interface was lost.  Provide a new internal function
that can return both a letter and a count

touchstone ugly check didn't take GOLDOBJ into consideration.

Add a macro to provide universal consistency around the check
for what sort of creatures tend to revive.

 some special code
was added, specific to menustyle:combination in
menu_drop(), which short-circuited a redundant
prompt that <Someone> had complained about by returning
immediately if a special code came back from
ggetobj() in ggoresults.

The top of menu_drop, however, had created a
u_gold object and inserted it into the invent
chain, so you can't just return.  You have
to use a "goto" to get to drop_done, so that
the object is cleaned up.

fixes tid

secret door detection
There was an old bug where a wand of secret door detection was identified
even though it didn't find anything.

It was unconfirmed, but I had no trouble confirming it.  It happened because
the wand worked on squares that were couldsee() but which (because they weren't
lit) you couldn't actually see.  The wand would detect the secret corridor and
then not display it because it was out of sight.

I fixed it to display the corridor, in sight or not (I could have had it not
detect anything, but the wand is fairly weak already.)

more Makefile.nt
- Fix the tty build so it wasn't trying to link with GUI switches.
- Don't make graphical the default for the Makefile. It is for the
  IDE projects.
- Be explicit about compiler versions required/tested.

Makefile.nt bit (from Yitzhak)

Explicitly state the service pack of VC supported in Install.nt
Yitzhak discovered that there is a problem if you have the original
Visual Studio 6 with no service packs.

inappropriate very difficult reading message
add missing set of parentheses around lense test in read_ability calculation

Makefile.nt (from Yitzhak)

FIXES:

- added counting to the menu window. I changed the way it works in the TTY
port though  ("you hear the rumble of distant thunder..." :)  The count
applies to the currently selected item instead of being aplied to the item
to be selected  (in other words a12 instead of 12a). It works better this
way in the graphical port since there is a notion of current menu item.
 I also notes that the count is ignored for PICK_ONE menu - is this by
design?

- somebode mention that text is not scrollable by default - this is fixed.

- fixed player selection dialog to use initXXXX options

-----------------------
B1028 [reported] win32: popup window is too short
The text in the popup window's top line "What do you want to
call the ..." is cut off. Could the window be bigger, so you
can read the entire line?

Fixed.

---------------------
B1027 [reported] win32: select via traditional class char
In a shop (and perhaps elsewhere), I can autoselect a class of
items to drop: a for all types, b for coins, c for weapons, ...,
A for every item, B for blessed items, ... I like the new
possibilities, but I would like to be able to use the old way of
using ! for potions, ) for weapons, etc., as well.

Fixed.

---------------------
B1026 [reported] win32: setting multiple options

I want to set packorder and pickup_types. The first comes up
first, and I type in the order. Then comes the second, without
any hint what is asked of me. A little explanation here would
be appropriate, like "pick up what things automatically?" or
somesuch. When more than one option is set where extra input
is required, some explanation is useful.

Looking into. Apparently something  end_menu() call is ignored by nethackw.

---------------------
B1025 [reported] win32: menu navigation
I press O. Now, when I press "a" to turn on "autodig", not only
is the mark set at "a", but there also appears a dotted
rectangle around option "A" (capitalized). Shouldn't this
rectangle also appear at "a"? Pressing "a" repeatedly toggles
the checkmark, but the rectangle switches between "A" and "a".
My CAPS LOCK is not set. The same goes for B, C, D, ... N,
by the way. Pressing "A" works the other way around: the first
press checks "A" and "rectangles" "a", the next unchecks "A"
and rectangles "A" as well.

Fixed.

---------------------
B1024 [reported] win32: no text in final window
After I clicked away the last screen with information
(challenges, I believe it was), I got an empty text screen, as
big as my screen, without any content. It looked like the text
screens that appear on the right, only it was larger and had
only a large OK button over the full width of the window, and
no cancel button. Why this window?

Fixed. The program did not set window text for RIP window.

---------------------
B1010 [known issue] win32: cosmetic
When I get my list of spells, the columns aren't properly
aligned: if the spell name is longer, the rest of the line
shifts right.

Fixed. Fixed-pitch font is used for menu window.

---------------------
B1004 [reported] win32: numeric keyboard Del/.-key
The Del/.-key on my numeric keyboard doesn't make me rest a turn

Fixed. Added rest on Del.

---------------------
B1003 [reported] win32: focus
When the window has the focus, the window bar isn't highlighted.

This is by design. Menu and text widnows are popup and the main
window is disabled when they are up.

landmines and boulders
Make pushing a boulder onto a landmine share code with the trap case,
resulting in pits, waking sleepers, et al.
Don't leave a boulder suspended over the new pit, fill it.
Make sure any remaining boulder is placed on top of the pile.
If player sets off landmine, monsters killed are credited to/blamed on player.

Windows Me and Windows XP.

Windows Me and Windows XP.

code sync

Remove extraneous line from petmark.uu

(from Steve VanDevender)

Not using FDECL to declare the prototype for dogmove.c:can_reach_food()
causes the Digital UNIX C compiler to throw a prototype mismatch error
when compiling dogmove.c.

no message

Fix a crash-causing null pointer dereference.
(Thanks to Yitzhak)