touchstone again
     Recent patches broke rubbing gold coins on touchstones; for the
!GOLDOBJ configuration, the character's money would be lost and the
program leaked memory.  That problem was already present for rubbing
gold on other gray stones.

     This also gives a gem advantage back to archeologists:  they
can comprehend touchstone results when the stone is uncursed rather
than require it to be blessed.  (I gave gnome characters that benefit
too.  Why gnomes and not dwarves?  I don't have a reasonable answer
for that....)  To go along with that, make A's initial touchstone
start uncursed rather than blessed, so that other characters finding
them in bones won't get an immediate benefit from them for the 20%
of the time that they're not cursed when saving bones.

     Much of this is whitespace cleanup.  I reformatted use_stone()
completely.

Follow-up
I don't want defaults.nh to override EVERYTHING,
so leave the new section in there, but commented.

(from Yitzhak)
This is the part of Yitzhak's patch not currently under discussion.

Keep in sync with Guidebook.mn.

porthelp modifications
- keep boolean and valued option sections together
- be more consistent in use of terms "graphical port"
  and "graphical interface" in the document (use the
  latter where noticed)
- don't mention options available in other ports so
  much since this document is not a general
  document.
- move the numeric keypad stuff to the end. It seems
  to assume number_pad on. So I stated that it does.
  (Many players never play with number_pad on, and
   I'm one of them. I'm not sure of the merit in having
   the section in here at all, since it exists in the
   Guidebook doesn't it?)  How accurate is the text
   that is there if you aren't in number_pad mode?

touchstone follow-up
- Handle gold materials a littel better.

>Rubbing a clove of garlic on a gray stone to see if it went "scritch,
>scritch": "You make scratch marks on the stone".

minor Guidebook.mn update
- remove a strange \! in the section for Blind users
- update the wording in the subsequent paragraph to be consistent with
the wording in the preceeding IBMgraphics paragraph

<Someone> pointed out a possible infinite loop here.

It was pointed out that we didn't add names quite everywhere.

djgpp parts to general gcc parts

Move tile2bmp.c to win/share

This is so it can be shared with Qt.
If, for some reason, we have to cut another source tarball for
3.4.0, then this will be included.  Otherwise, this will
turn out to be a post-3.4.0 patch.

fix wizard mode crash (SPLEVTYPE)
     Set SPLEVLTYPE to "soko1-1"; visit soko4-* (or minetown)
on the way there; oops:  access violation from deferencing a
null pointer.

split long line
     A source line being 190 characters wide is rather excessive.

A couple of patches are required unfortunately

Unfortunately a couple of glitches were encountered during the build of
the DOS binary.

Also, somewhere along the way the ability to separate the debug info
for the win32 tty binary into its own PDB file was removed from
Makefile.msc.  I wasn't aware of that until I went to build the binary,
and discovered that I had no PDB file to stow away for bug
reports.

Workaround bug in SHARP SL5500 ROM.
(not essential for 3.4.0, since the SL5500 port still needs more work anyway)

Mark sources non-beta and update patchlevel

nethack.txt update
- created on RedHat Linux 7.2 using standard tools

Guidebook.txt update
- created on RedHat Linux 7.2 using standard tools

more credits
- update dat/history to match Guidebook
- use my name consistently :-)

Guidebook credits
- remove several people from the 3.4 list who haven't contributed in a while

(from <Someone>)

- fixed bug as From a bug report.
  validxxxx functions cannot handle ROLE_RANDOM and ROLE_NONE
  parameters like ok_xxxx functions do

undef WIZARD
- quiet a gcc warning if WIZARD is undefined

Another pre-ANSI bit.

more help file clarifications
- A pcwin team member reported that the ^<dir> notation used in a couple
help files was confusing

missing porthelp band

This band was accidentally left out of the patch for porthelp the other day.

document set_option_mod_status in window.doc

a statement in window.doc didn't reflect code

win32 menus must accept all of the accelerators

documentation updates
- various commands were missing from various help files
- fix a few inconsistencies between similar help files
- M-2 doesn't do #twoweapon when number_pad is enabled
- M-? usually displays #? help info
- weasel-word the reason the screen shown in the Guidebook looks little like
  several windowports
- document the playersuffix syntax in the nethack.6 man page

Add fixes* to the manifest.

beta feedback - add more wc options
- splash_screen (boolean for whether to display splash screen at startup)
- player_selection:dialog|prompts

Also moves the font and window manipulation stuff in defaults.nh
further down the file, so that a tty users doesn't have to wade
through it all to find the character adjustment samples.

(from Yitzhak)

(from Yitzhak)
more porthelp and documentation

(from Yitzhak)
patch: rollback alt-key changes

(from <Someone>)
Correct Makefile.gcc for win32

Add "experimental section" to config.h

- Move GOLDOBJ into the experimental section.

(from Yitzhak)
rollback alt-b

unleashing untame monsters
     Fix several cases that earlier patches didn't address.
If a leashed monster becomes untame, detach the leash.

Sometimes there is no main widget.

leash groundwork
     Add another parameter to m_unleash.  This should not introduce
any change in behavior.

steed/leash bit
     It's possible to leashed a saddled pet and them ride it,
but it wasn't possible to remove the leash while mounted.  This
fixes that; it also lets you put the leash on your steed while
mounted, but there's nothing wrong with that.

doc bit

Summarize past version 3.3 in Guidebook and dat/history.

(from <Someone>)

Fix crash.
WIN_MAP was used without before initialization in mswin_select_map_mode

(from <Someone>)
- changed splash screen fonts to measure in logical pixels rather that
points.
Points are 1/72 inch regardless of screen resolution. If screen resolution
changes so does visible image size. If font is measured in points then its
visible size remains the same. The text is getting cut off (or becomes to
small for the image) as a result.

Remove high-bit characters in recent comment

Get the god name right in comments.

various things
- a couple of README updates
- some color changes for windowcolors
- prevent crash From a bug report, but better than a crash. It
probably renders the map_mode:fit_to_screen ignored in defaults.nh.

fix some porthelp nits
- also add some windowcolors documentation

add new sys/unix file to Files

Add win32 port help support.

Some win32 updates
(There will be a couple more patches to follow in a bit)

(from Yitzhak) part2
part 2 touches only port files

Makes Borland happy with current sources.
Fixes bug in WM_PAINT handling in rip/splash code.
Fixes getline() bug (H0009)
Tidies up source.
Fixes up copyright notices.
Adds TODO docs for Borland. (Removes "TODO" note)
Small changes to defaults formatting.
Fixes ALT key on Graphical Port non NetHack mode.
Hilites pet in Graphical Port text mode.

Also Implements windowcolors for status and message window in graphical port.
On graphical port,
tested to see it actually works.

Tested on platforms Makefile MSC/BCC Graphical/Console, MSC IDE for
compile and run and performing hilite pet in graphical text, not
displaying rawio, alt key.

(from Yitzhak) part1
part 1 touches core files
ntconf.h changes differ slightly from Yitzhak's original.

Makes Borland happy with current sources.
Hides "rawio" on Graphical Port which doesn't do anything with rawio

Makefile fix following depend. changes

Update the dependencies in win32

Lint part 2, mostly parameters.

wizkit
This fixes the wizkit so that if someone wishes for a non-object such as
a trap, not only is no message printed, but we don't try to create anything.

Incidentally, if you have two bad wizkit items in a row, you get told to hit
space to continue, then you can hit space, then the error message for the
second bad item is printed on the same line as the first hit space message.
(I haven't tried to fix that.)

Lint part 1, unused variables, routines, and return codes.

dipping into burning potions
- candles, et al, light via catch_lit
- non-weapons can be damaged
- only flammable items can be damaged, previously SILVER objects, for example,
  would get a message, but add_erosion_words wouldn't display a damage word
- can't track burnt food, put this in the "seems" case too
- PLASTIC items are is_flammable, which is appropriate for all current uses
- paper gets destroyed (special artifacts excepted by earlier check)
- a cursed potion now spills even if not dipping weapons
- charge for damaging unpaid objects this way
- still very hard to destroy PYEC this way

more dead code

(from <Someone>)

writing in fog
- if you engrave in a cloud, the surface type should only be "air" if you're
on the air level

more update_inventory calls
in passive_obj and erode_armor

parallel unix make bits
- Pat noticed that makedefs -z makes both vis_tab.[ch], but they could be
built by two makes via make -j, causing corruption
- make -j at top level failed for similar reasons, added several
dependencies to ensure a valid ordering
- these changes will cause extra things to be built if you "make"
individual targets at the top level, but have little effect it you're a
real power user and "make" the actual target in the right directory

MPW #pragma unused updates

vms update: sync Makefile.src with `make depend'

lava effects while riding
- a case missed by the earlier lava patch, if you enter the lava while
mounted, your steed would be unaffected if you were lifesaved
- fix by making more code shared with the is_pool case

get rid of nori

B10003 fixed.

Run 'make depend', after fixing up output.

Something Janet pointed out.

fixing bad luck via sacrifice
- From a bug report: if sacrifice doesn't clear the timeout until prayer,
your bad luck is cleared, but if it does, your luck could remain negative

(from <Someone>)
- fixed tab-stop order for player selection dialog
- minor changes to the dialog layout

There's no point in mailing this out, but there doesn't seem to be a way
to stop it.

Make pre-ANSI compiler happy.

Remove a fixes34.0 entry that corresponded to removed code.

bit

winhack.c mods
- change sprintf -> Sprintf macro
- fix sprintf syntax
- point to a www.nethack.org URL, rather than a Microsoft one

provide linux build instructions in the source distro

Update dungeoneers list (<Someone>, Teemu).

(from <Someone>)
This patch implements Michael's idea to have a "NetHack mode" for
keypresses. It only does the Alt-keys now, but it should be easy
to extend to pressing space in dialogs; I left that out as we still
have to agree on a new dialog look.

The setting that you choose from the menu is stored in the registry.
Reading and writing the registry is done on init_nhwindows and
exit_nhwindows; there also is a menu item to delete the registry
settings (which makes sure they are not stored again this session.)
This provides a framework for future registry settings.

Also,
Alt-tab should show the app title when a menu or text window is up.

I left the app title as it was, although I still favour "Graphical
Version" or "With Graphics"; Anyway, that's only one resource string.

slap against floor while riding

> Greetings! <Someone> suggested I report this bug to you:
> in 3.3.1, riding a horse while blind, I ate a bad carrot (to
> unblind myself) and got the message, "Blecch! Rotten food! The
> world spins and you slap against the floor." Upon regaining
> consciousness, I found I was still safely mounted on my steed,
> raising the question of what part of me could have "slapped
> against the floor." I thought I would have fallen off my
> horse, or at least have slumped against it. <Someone> said in his
> reply to me on rgrn: "Looks like Yet Another message that
> fails to take riding into account; should be easy enough for
> the DevTeam to fix, though, since it's one that's already
> being modified for levitation and the like." Hope this helps.
> Thanks very much! <email deleted>
>

As Kevin pointed out.

follow-up to <Someone>'s patch.
I left a couple of prepanic() calls in there accidentally
when modifying the patch.

Michael

(from <Someone>)

Here's a patch that checks whether the common control
library is available and up-to-date

[slightly modified from <Someone>'s version to reduce
 indentation one level, and to fix wrap-around
 in the patch]

Something <Someone> reported.  Ensure that an early panic() doesn't
cause an access violation in win32 graphical port.

(from <Someone>)
Here's the gcc makefile.

(from <Someone>)
Here's the gcc makefile. I also cleaned up the batch file a
little and added instructions to install.nt.

I copied most of the items from the existing Makefiles,
including the 32x32 tile support, even though that is not
available. This way comparing the makefiles is easiest :-)

win32 shknam updates

win32: Consistent Makefile naming now that we have at least 2

Makefile.nt -> Makefile.msc

Support iflags.menu_tab_sep in the #enhance menu.

Ensure that vision is not used during done()
(and panic() which calls done())

This uses the flag that <Someone> recently added.

At least one crash in the past was caused by

? -> panic() -> done(11) -> * vision_recalc(2) -> newsym() -> crash!
if  u.ux and u.uy are 0

Ensure that each win/win32 source file has a copyright statement
and mentions the NetHack license.

(from Yitzhak)

Addresses <Someone>'s point that ALT-F4 in game-over, still brings up the
save/quit/cancel dialog box.  This makes ALT-F4 equivalent to quitting
without any further disclosure/windows, but makes sure the game exits
properly so it will save the bones and the record file.  I also cleaned
up the mhrip.c to use macros for all magic numbers.

another "your corpse" case
- switch aobjnam() to use cxname, not xname
- update Tobjnam() comment, since it can still use xname w/o confusion

A specific case that's fixed: You begin bashing monsters with your corpse.

(from <Someone>)
- added support for iflags.randomall (-@)
- echo yn_function selection to the message window (as From a bug report

weapon shattering follow-up

more weapon shattering
The Kop Kaptain's rubber hose shatters from the force of your blow!
Rubber hoses are an individual exception that would need to be dealt
with explicitly, since the PLASTIC they're made of is floppy rather
than rigid as for other PLASTIC items.