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)

MacOSX/Qt system tweaks
These changes clean up build warnings and allow the resulting "NetHack"
Application icons to be dragged around freely in the Finder, as is expected
for Mac apps.

piranha
An extra space in the piranha entry was causing it not to work.

window.doc getlin clarification
Clarify the getlin interface, which is used with the implicit assumption
that an input buffer of size BUFSZ (including the nul) is sufficient.

more GOLDOBJ

     The earlier patch made sure that bribe() didn't pass an
invalid value to money2mon().  This one changes money2mon() so
that if some other code else does so, reporting the impossible
situation won't be followed by a splitobj panic.  Most of this
patch is reformatting though.

Updates to Makefile.bcc (from Yitzhak)

Fix several touchstone-related things:
1. The switch statement was using the material "GOLD"
   rather than GOLD_CLASS.
2. If getobj() had been working for gold when it
   came to touchstones, there would have been a
   memory leak here because the object returned
   would have been from mkgoldobj().  The goldobj
   was not being freed anywhere, nor was it being
   put on a chain. You also would have had zero
   gold after rubbing it on the stone. The intent
   was clearly to allow gold since there was a
   case in the switch statement.
3. getobj() wasn't working properly for gold
   selection here anyway, so this was
   not the cause of <Someone>'s gold obj in inventory.
   You ended up dropping through to code that
   was supposed to print "You cannot verb object."
   For touchstones that came out as:
   "You cannot rub on the stone gold."

another vision-related message
don't just say "the knife misses" when you can actually see the monster
with infravision by testing using the correct function

bribe fix for GOLDOBJ

bit

another WIZKIT fix
previous rev put the new "else" in the wrong place

another WIZKIT fix
if $HOME is not set, don't use uninitialized buffer as the filename to open

wizkit fix

typo fix
with -> which

These are the patches.  In general, they seem
to fix general problems that MSC didn't catch, but I can't tell.

Yitzhak Sapir

Enough MacOS X documentation to answer FAQs.

micro fopenp() fix

     From a bug report.  I can't test this fix, but
inspection of the code shows that his suggested fix is clearly
necessary.  Once `bp' gets incremented, storing via `bp[BUFSZ-1]'
writes beyond the bounds of `buf' and clobbers something.

fixes bit
Now that there are two different NT ports,
distinguish between them for fixes entries.

Mac docs update

This is mostly to test the commit scripts...

Qt/MacOSX: change working directory
chdir to resource fork when running from finder (find leaves us in "/", we
want to be in nethack playground).

X11 buffer overflow avoidance
Don't overflow the buffer passed to getlin().

update unused pragmas\r\rUpdate some of the mac pragmas for unused variables\rfor the MPW compilers.

Fix some Guidebook mistakes (tex dungeoneer table for one)
Also shorten up levcomp.dsp lines

Always use maximized window in Qt windowport.

MacOSX: no lrand48()
No lrand48() on MacOSX, since MacOSX is really BSD.

minor things
Two unrelated tids.  I wouldn't bother putting them into Beta 1. :-)

Extraneous file removed as I head out the door of the office.

Qt GUI cleanup
Use "Alt+X", not "Alt-X".
This is actually *necessary* on MacOSX.

It was ignoring the command line parameters
because there was an extraneous argv[1] that was
a repeat of the module name, but in quotation
marks.  The processing in pcmain stops on the first
argument that doesn't start with '-' so my other
arguments got ignored.

 argv[0] 0x00b40800 "C:\test\binary\nethackw.exe"
 argv[1] 0x00b40878 ""C:\test\binary\nethackw.exe""
 argv[2] 0x00b408f0 "-uwizard"
 argv[3] 0x00b40938 "-D"

Remove all uses of pushd/popd
in case someone  wants to build on Windows98 where they don't exist.
They are available in NT or Windows 2000 only (and possibly XP).

Update ./Files to match file rename for win32.

rename winhack.dsp to nethackw.dsp
Adjust Install.nt to match.
Modify Makefile.nt to use ../binary directory for output by default,
just like the windows version does.

rename winhack.dsp to nethackw.dsp
Actually removes winhack.dsp and adds nethackw.dsp.

generated lex output

Fix some crashes in unused tile editor.
tileedit is the program I use to edit tile files (directly).

Remove win/Qt/nethack.png
Obsoleted by nhsplash.xpm (a text format). The binary still has more detail,
but until someone has a use for it, I'll keep it aside.

Doc fix for Install.Qt
(USER_SOUNDS now undeffed automaticaaly for old versions)

airplane/taxi transportation

     The player can teleport objects and monsters on no-teleport
levels, a strange quirk which I think has become entranched as
a feature.  When swallowed or engulfed, teleporting the monster
from inside ends up teleporting the character along with that
monster.  Some players have been exploting this on Plane of Air
to avoid facing elementals and dragons and whatnot by repeatedly
teleporting any vortex that engulfs them until they land somewhere
in the vicinity of the portal leading to Plane of Fire.

     This patch divides the Plane of Air into three zones that
teleportation can't cross.  You'll arrive in the left-hand 30% of
the level, as before, but no longer at a specific spot.  The exit
portal is in the right-hand 30% as before (although it used to
have more range, perhaps 40%).  Teleporting within the left 30%
always arrives in that same area; within the central 40% always
remains within that same area; and teleporting within the right
30% always sticks in that area.  So it's still possible to get
around quite a bit via multiple teleports, but you'll need to walk
at least across the two unmarked boundaries to actually traverse
the whole level.

     A moderately long description for a very short patch....

From <Someone>,
Fixes:
- menu shortcuts implemented
- most windows close on space (except for menus with
  PICK_ANY style)
- "hilite_pet" option is implemented
- map scrolling is improved somewhat (it now scrolls if
  the char is within 5 spaces from the edge of the map -
  configurable by #define CLIPAROUND_MARGIN)
- added 3  winhack-specific options:

 win32_map_mode:[tiles|ascii4x6|ascii6x8|ascii8x8
                 |ascii16x8|ascii7x12|ascii8x12|ascii15x12
                 |ascii12x16|ascii10x18|fit_to_screen]=20
 win32_align_status:[left|top|right|bottom]
 win32_align_message:[left|top|right|bottom]

Note: aligning status window to left or right edge of the screen does
not look good.

Add petmark.uu

swallowing zombies/mummies

polymorphed quest leader
Duuuh.  Of course adding objects already changed the editlevel.

Anyway, here's the fix I was working on.  It only matters in a very obscure
situation.  (Also, the quest leader still speaks no matter what he's
polymorphed into.)

polymorphed quest leader
Duuuh.  Of course adding objects already changed the editlevel.

Anyway, here's the fix I was working on.  It only matters in a very obscure
situation.  (Also, the quest leader still speaks no matter what he's
polymorphed into.)