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

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.