(from Yitzhak)
-Yitzhak provided this prior to release
 but it never made it in.

track Revision in Guidebook's
- Keni mentioned on 3/20 that it would be nice to have $Revision$
in Guidebook.mn.  Added to both .mn and .tex versions.

win32tty: use_inverse option

respect use_inverse and default to ATR_BOLD if disabled

bad wizkit items
Format multiple bad wizkit items a little better.  It will scroll off the screen
if there are more than a screen of bad items, but that's probably not too
likely.

more explode panic()
You could still get a panic. To reproduce:
- wizwish for a statue, and drop it
- wizwish for a landmine, and apply it at same location as statue
- move one square and wish for a boulder, drop it (automatically likely)
- push the boulder on the landmine to trigger it.
- the panic came from the statue this time.

The spellbook fadefades.

<email deleted>
<email deleted>
Sent: Saturday, March 23, 2002 6:01 AM
Subject: Bug in 3.4.0: spellbook fading

> Dip a spellbook in a fountain:
> "The spellbook fadefades."
>
> In get_wet (potion.c), otense is used, which returns "fades",
> redundantly, as a suffix to "fade".

conduct bit
     Document the way to decline a wish or genocide, and allow both
actions to accept both "none" and "nothing" so that players don't
have to remember which response goes with which prompt.

update ./Files with new fixes34.1

annoying shop bug
     The shop billing code assumes that food marked partly eaten
is worthless, but the eating code was changed to make sure that any
eating attempt will never leave food marked as not partly eaten.
The end result is that non-corpse food which is consumed in a
single byte can be eaten off shop floors for free, and if eaten
from inventory--so already on shop bill--the "Ix" display of used
up goods lists the item as partly eaten (with right price though).

     This fix makes single-byte food be handled the same as other
food:  the first byte taken causes the food to be auto-purchased
immediately instead of waiting until it's used up to add it to the
shopping bill.

Mjollnir fix
     Fix one of the problems From a bug report:  Mjollnir is only
giving the intended feedback when it is thrown.  Against ordinary
monsters the problem isn't too bad--you get a lightning message
without the preceding hit message--but against resistant ones there
is no message at all when you hit hand-to-hand.  (There is a similar
bug for artifacts which add magic missile damage, but since no such
artifacts exist that isn't much of a problem. :-)

#R650: Game crash (Panic - obj not free)

> <email deleted>
> Oops! [...]
> Suddenly, the dungeon collapses.
> place_object: obj not free
>
> The crash is reproducible in wizmode by wishing for a landmine, arming it and
> pushing a wished-for boulder onto it.

Remove files Pat noticed.
- they are now included in the oldfiles.tgz archive
available in the usual place old/oldfiles.tgz

clean up Files

final? 3.4 Guidebook.txt update

Mac documentation tidbit

Fix a documentation inaccuracy pointed out by Michael.  Intended for
the 3.4.0 source tarball, even though it was submitted after the
code freeze.  Does not affect compiled binaries.

(final patch from <Someone>)
This patch implements two things:
- space now dismisses a menu, text or menutext window when you are
  at the bottom of that window
- Page scrolling in menus now works as follows:
  pressing PgDn (or Space in NH mode) moves the focus down one page,
  and moves the scroll position down one page. The effect of this
  is that you always get a full new page of items (which is what
  NetHack players expect), and that the focus moved down one page
  (what Windows users expect.)
  The same (revers) goes for scrolling up a page.

A Windows user will still be a bit surprised when the focus is on
the top item, and he presses '>': he gets a new page of things instead
of the focus moving to the bottom of the page.
However, the PgUp/PgDn keys (which he probably uses) still have the
old Windows behaviour.

Add the fixes disclaimer to README.

Guidebook dates to match release date

(from Yitzhak)
This band adds handling for the "5" key in the numeric keypad.  This is
called VK_CLEAR in the VK_ constants.  Using this band, and with
NumLock off, doing 5 will send NetHack the letter 'g'.  Doing SHIFT-5 will
send 'G'.  Without this band and with NumLock off, 5 does nothing.  With
NumLock on, 5 sends 5.

date change again

probing of empty containers--right diff this time

probing of empty containers

Mac port update

* Update Mac port to new options system.
* Update Mac-specific documentation.
* Remove hardcoded version strings.
* Remove obsolete popup_dialog.

Mac resource file update

Changes to the resource file used for the Macintosh port.
Specifically, remove hardcoded version string and popup dialogs.

This is formatted with BinHex so it can be stored on non-Mac
systems without losing the resource fork.  Though it is text,
it is not intended to be human-readable.

List <Someone>'s new win/gem files.

<Someone>'s new files.

<Someone>'s gem changes.

<Someone>'s changes for the makefiles.

release date adjustment

(from <Someone>)
This implements some items on the ToDo list:
- H0005: Space does the same as PgDn in text, menu and 'menutext'
        windows if NetHack mode is on.
- M0004: The cursor (caret) is hidden in text windows and menu
         windows.
         PgUp/PgDn/Up/Dn don't move the cursor, but scroll the
         window.

Gnome player selection
- provide full 3.4-style player selection, based on tty_player_selection
- generalize selection dialog so it can be used for all 4 prompts

final(?) touchstone
     Put back the "better gold handling" that was inadvertently
dropped from the previous rewrite.  Prevent gems rubbed on cursed
gray stones other than touchstones from being shattered.  Fix
several pluralization buglets, including allowing the player to
rub a stone against itself if the quantity is more than one (just
like potion dipping is handled).  Overall, streamline the rather
convoluted logic, eliminating the `goto's.

/tmp/msg

By "Rush commands", I meant `G' (though I've only ever pressed Alt).

(from Yitzhak)
accelerators

Spotted a missing % in Guidebook.tex

<Someone>'s suggested changes needed to go in history too.

Fix most of the errors <Someone> found.  The perpetrator of the "Rush commands"
will need to either straighten it out themselves or let people assume it's
the capitalized ones.

Guidebook suggestions by <Someone>
- Also, I just noticed that an entire devteam paragraph
was duplicated in the .tex version.

better trap searching behavior
- if a trap is hidden by clutter, cls() before delay, remove windowport
specific behavior, makes trap more obvious in all cases

kelp fronds are really not gems
- finish un-confusing the kelp frond case

kelp fronds are not gems
- mksobj had a misplaced test for KELP_FROND in the GEM_CLASS code, moved
to the FOOD_CLASS code

disclosure bug

From <Someone>:
>And finally a bug-report:
>The new disclose-option didn't fully work.
>While the -ny prefixes works like explained the + prefix
>(show it don't ask) acts like -.
>A possible fix is, change the return-value of
>should_query_disclose_option for case DISCLOSE_YES_WITHOUT_PROMPT
>to TRUE and handle the case defquery=='q' in list_*-function
>as "don't ask". Or add a new boolean ask parameter to the
>list_*-functions.

(from Yitzhak) mhmain, ntty

New file: fixes34.1
Note that this is not tagged as part of 3.4.0.

One leftover height use in Qt tile support

Gnome basic functionality
- Makefile.src update for new dependency

As pointed out by <Someone>
> while testing the new wc_-Features for Gem I noticed
> in nh340b11 options.c parse_options there is a bug.
> If a optionname starts with "font" a opts+=4; is
> performed on it, but if via string_for_opt an
> error occurs badoption is reporting i.E.
>    Bad Syntax: _menu use "?g" ...
> instead of font_menu.
>
> Possible fixes include shift the pointer back (bad)
> or use another pointer.
>
> <Someone>

Gnome basic functionality
- let the "#" key work as per Guidebook
- role selection didn't work if you had gender specified in your .nethackrc
  similar problems would occur for other .nethackrc selections
- fix an obvious memory leak
- fix one crash bug from accessing a freed pointer (M-? dismiss M-?)
- note some invalid behavior in comments for a real Gnome developer to fix
- reformatted some code so I could follow it

X11 wide tilemap support - Gnome patch
- update Gnome code to support a 40 tile wide XPM file

X11 wide tilemap support
- support X11 tile files (with or without XPM) that are 40 tiles wide
- rearrange some X11 code to share more code between XPM & non-XPM options
- clean out some deprecated X11/winmap.c #ifdefs
- update Qt code minimally to handle such an XPM file

(from Yitzhak) ALT-?
This patch allows ALT-? to do #? in NetHack mode by not doing
TranslateAccelerator in NetHack mode.  This relies on the short circuit
logic of || (much of NetHack code relies on it, it's just an explanatory note).

(from Yitzhak) ALT-H
This patch disables ALT-H in NetHack mode.  Before patch: ALT-H in NetHack
mode brings down the Help menu in the menu bar.  After patch: ALT-H does
nothing in NetHack mode.  In both cases, in Windows mode, ALT-H brings
down the Help menu in the menu bar.   I can easily write a patch to have
ALT-H in NetHack mode act the same as '?'.

Explanation:
return 0: this window proc has processed this event.
return DefWindowProc(): process using default window proc.

ALT-H is the only key for which DefWindowProc() does something special AND
reaches the else.

<Someone>'s trap detection complaint

To address <Someone>'s complaint, make the cluttered
trap detection tty only. Also put in some dead
code that could make it work elsewhere in future
when there is time to test.

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

Change timestamp on Guidebook since it has changed anyway.

change timestamps; tweak a couple of port things

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