Several things that break savefile compatibility
- Version change from 3.4.x
- timed_delay feature ignore in makedefs
- several flags from iflags to flags
- use offsets from mons array entries in save file rather than storing
  the ptr and calculating the distance from beginning of array

typo in <email deleted>
#ifdef HOLD_LOCKFILE_OPEN
"exlusive lock on level 0 file",
#endif

U313 - crash applying figurine, cursed bell or candles
Reported for applying a figurine that was used up, but I found the same
problem could affect cursed bells and candles.  Modified all three helper
functions to indicate when the object is gone, and modified doapply to
deal with this before doing the artifact check.

win32 stuff held until after bugfix release (from <Someone>)
-better handling of "more" prompt for messages that would have      scrolled off the window
-support perm_invent
-menu option to add/remove windows captions

U58 - Targetting problem with infravision/ESP
A skilled/expert caster of fireball/cone of cold was not able to target
a location with a monster seen only by infravision/ESP.  Since you can
focus on the monster there, targetting shouldn't fail in this case.
Attempting to lock onto a monster inside stone still won't work.

B20008 - seeing pet move
The cansee() checks are not really correct for seeing your pet move.
Changed them to a pair of canseemon() checks, one before the move, one after.
I can see an argument for canspotmon(), but decided to keep it based on sight.
If your pet is unseen in both locations, you won't get any messages, which
I think is more correct.  If you do get the message, use noit_Monnam to
ensure no more "it" message.

remove duplicated fixes
Un-list fixes also listed in fixes34.2.  This causes the fixes files to
still reflect all the changes since the last one, from the point of view of
someone seeing a release tarball.

W341-2 Finnish and international keyboard arrangements

W341-2 Finnish and international keyboard arrangements

opthelp for use_inverse
<Someone> (beta) noticed while 3.4.1 was in the process of releasing
that use_inverse wasn't documented in the opthelp.

Yname2 vs yname
<Someone> noticed a capitalization mistake in a new message.

data.base typo
<Someone> noticed that we didn't thoroughly spellcheck data.base.

U287 - how old is nethack
Every release, we get at least one message asking when the first version
of nethack was released.  Add the year of 1.4 to the history file.

Building src objects via Makefile.utl
<Someone> noticed that if one builds something in util and the required .o
files aren't already built in src, the .o gets placed in the wrong place.
Added the missing '-o $@' to the compilation command.

U277 - Gnome compilation on Redhat 7.2 & 8.0
Reported on RH 7.2 and 8.0.  Compilation failed because system headers that
needed _GNU_SOURCE on these Redhat versions got included before it was
defined.  To ensure _GNU_SOURCE is defined, added an autodetect for it to
config1.h and removed the need to set it in unixres.c.  __linux__ is also
checked elsewhere.

U295 - bones file creation cursing secondary weapon caused panic
As reported, if you're twoweaponing and die, your secondary weapon may
become cursed and drop.  But, the bones file code is dropping everything
and tries to drop it again, causing a panic.  drop_upon_death just clears
things out, so follow suit for uswapwep.

C341-1 - don't make monsters angry if they enter non-hero regions
this currently affects only regions created created by deads heros
found in bones files

new 3.5.0 fixes file, for trunk changes until further notice

README file for 3.5.0

The branching process is nearly complete.  The changes below
ought to be applied only to the 3.5.0 trunk.

Work on more UNIX systems.

fixes34.2

even more missing newlines

more newlines at end of files

newlines at end of files

Update contact info

README updates for review

h/pc (from <Someone>)

Fixed output file name for H/PC Pro - SH3 configuration.

minor Unix Makefile mod
I just noticed that the installed owner/group of "recover" was not being
set correctly.  This has no important effect, but does look funny.

updating official Linux binary
I'm updating the official binary to be built on a Redhat 8.0 system.  This
should give the binary some legs if we take a while to get the next release out.

more win32 more (from <Someone>)
<Someone> and <Someone> both agreed that this was:
- useful
- unlikely to break anything.

Windows CE bits (from <Someone>)

(keni, for jhsa)
here are two minor patches for amigaport (ppc/gcc related, compiletime).
Teemu noticed these during a testcompile for 341 tarball today.

-jhsa

*** sys/amiga/amidos.p 2003/02/20 15:36:28 1.1
--- sys/amiga/amidos.p 2003/02/20 14:30:12
***************
*** 37,42 ****
  #ifndef strcmpi
  int FDECL(strcmpi, (char * , char *));
  #endif
! #if !defined(memcmp) && !defined(AZTEC_C) && !defined(_DCC)
  int FDECL(memcmp, (unsigned char * , unsigned char * , int ));
  #endif
--- 37,42 ----
  #ifndef strcmpi
  int FDECL(strcmpi, (char * , char *));
  #endif
! #if !defined(memcmp) && !defined(AZTEC_C) && !defined(_DCC) && !defined(__GNUC__)
  int FDECL(memcmp, (unsigned char * , unsigned char * , int ));
  #endif
*** sys/amiga/amiwind.c 2000/11/30 16:39:25 1.3
--- sys/amiga/amiwind.c 2003/02/20 14:31:06
***************
*** 21,31 ****

  #define BufferQueueChar(ch) (KbdBuffer[KbdBuffered++] = (ch))

- #ifdef __GNUC__ /* Conflicting includefiles ... */
- struct Device *ConsoleDevice;
- #else
  struct Library *ConsoleDevice;
- #endif

  #include "NH:sys/amiga/amimenu.c"

--- 21,27 ----

Roll back the last-minute patch after <Someone> found problems with it

win32 "more" handling (from <Someone>)

> There is a slight problem with "--more--" prompt being handled in
> WM_PAINT. WM_PAINT can be sent while you're inside
> mswin_nhgetch(). This causes some weirdness if window is resized
> while waiting for --more-- prompt.

get ready for tarball

fix B20003 - hallucination and warning

> Why does warning work normally while hallucinating?  I'd like to see
> random numbers.

     It's just a one-liner, although it has pointed out problems in the
symbol lookup code.  "0" won't work at present, so this doesn't use it,
but warning symbols "1" through "5" also have trouble--regardless of
hallucination--if someone overloads them to represent any feature aside
from warning.

Dig up an actual Kop quote.

more win32 Makefiles
update depend substitution instructions.

another win32 Makefile.gcc (from <Someone>)
another error in the makefile.gcc, not fatal, just a leftover
from the MS makefile. This patch includes both makefile fixes.

Makefile bcc (from Yitzhak)
1) Update documentation for using bison/flex
2) Fix use of $(CC).  In the makefile, it's supposed to be $(cc).
Otherwise it won't compile. [caused during recent dependency
updates]

building under MinGW (from <Someone>)
didn't work. Makefile.gcc had a space instead of a tab somewhere.

confuse monster bit

     '+' is not a valid argument to resist(), although it still works
since the default case catches it.  Make the spell class explicit,
although I don't think having the effectiveness of scrolls of confuse
monster depend on the character's level makes much sense.

runtime port identification
The CE ports use makedefs hosted on another platform,
so the version string generated at build time isn't really
appropriate.

Add a way to add information to the version string
at runtime for such ports.

ensure that shade_aware is only set for shades.

teleporting monster strategy

     Reported last spring and again last week:  monsters who teleport to
your location (named demons, master liches, and so forth) continue to do
that even if they've just used a scroll or wand of teleportation to get
away from you.  This doesn't prevent that situation but does make it be
much less likely to occur.

wizkit revisited

     The previous changed ended up discarding the begining portion of
excessively long lines and keeping the end.  It's unlikely that either
part is going to be valid, but reporting the ending portion as a failed
wish would make tracking down and fixing the situation trickier.

wizkit fgets() buffer size
also ignore lines where we didn't read a newline within BUFSZ characters

lint follow up
Janet wrote:
>There is one more new complaint that might cause problems:
>explode.c(545): warning: conversion from long may lose accuracy

undead turning of ghost on bones level

<email deleted>
Newsgroups: rec.games.roguelike.nethack
Subject: Re: Other ways to destroy the Amulet

<Someone> wrote:
>>> Zapped undead turning at his _corpse_, which brought the corpse
>>> back to life and also destroyed the ghost. The revived
>>> character was not carrying anything.
>>
>> Hmm, didn't the Amulet fall to the floor like when you genocide
>> something that's carrying something?
>
> Nope.
>
> That's because undead turning does mongone(ghost). The mongone
> function destroys the inventory of the monster:
> discard_minvent(mdef); /* release monster's inventory */
>
> Whereas genocided monsters are removed with mondead, which calls
> m_detach, which calls relobj, which drops the monster's inventory
> on the floor.

Lint cleanup, nothing significant.

There is one more new complaint that might cause problems:
explode.c(545): warning: conversion from long may lose accuracy

windows PORT_SUB_ID

allow PORT_SUB_ID to be defined in ntconf.h or ceconf.h

Update against what's in the directories.

Allow use of old compiler.

B20001 - shopkeeper angry at unnamed customer
The "following" flag could get set in several places where it was not
obvious that the customer name would be remembered.  Since the shopkeeper
should always get angry at the current player, set the name at the same
place that the flag is set.

fix warning without STEED defined

win32 and msdos recover

Since the ports have SELF_RECOVER defined,
remove the build of recover.exe from the default steps.
You must explicitly use 'make recover' now.

win32 and msdos recover

Since the ports have SELF_RECOVER defined,
remove the build of recover.exe from the default steps.
You must explicitly use 'make recover' now.

CEmore prompt (from <Someone>)

Prevent "--more--" prompt from appearing when several
questions being asked in the same loop (like selling
something in the shop

more MSDOS build

OS/2 patch (from <Someone>)

macro arrangement bit

win32 bits

win32 error message glitch noticed

fixes update SELF_RECOVER on winCE

fix djgpp Makefile

beta 20

typo fix on recent Makefile.gcc patch

more CE (from <Someone>)

win32 and DOS dependency update for 3.4.1

fix B18012 - Werebane

     Make wielding Werebane confer defense against catching lychanthropy
from monster bites.  It doesn't protect against catching that from eating
lycanthrope corpses and might blast the character if wielded at such time.

     Also fix artifact handling to recognize the character as lycanthrope
while in normal human/elf/whatever form--rather than only when in beast
form--just like it does for monsters.

fix U206 - cursed leash/pacifist conduct

      Treat strangling a pet with a cursed leash like a regular monster
kill; you'll get experience, lose pacifism conduct, and suffer various
pet-killer effects.  Also do some minor damage and reduce tameness for
the case where the pet is described as being choked but isn't killed.

vms Makefile

note buffer overflow fix

README update

no message

tombstone engraving with fingers
<email deleted> wrote:
> comments: When I try to write in the dust, with my finger, on a
> tombstone, it works the first time, but after that I'm getting a
> message that I can not erase what's written here.  Is that an
> intentional way of preventing us gaining a bunch of levels fighting

5 days, no objections to proposed code change.

U162 - killing shopkeeper with unpaid thrown objects
While an object is being thrown, it isn't on any list.  This means that
killing a shopkeeper with an unpaid object wouldn't be able to clear the
unpaid bit.  By the time the object lands, the shopkeeper is gone, and then
it's too late.  Added a new global to track a thrown object, set it and later
clear it in throwit(), also clear it as needed in dealloc_obj(), and check
it in setpaid().  It should be possible to use this global to avoid losing
thrown objects during hangup saves as well.  But that can wait.

steeds don't remember traps
<Someone> reported to the list that steeds didn't remember traps
encountered while mounted.  When not mounted, a monster will remember
traps, even when they don't damage the monster.  To that end, added code to
set the steed's mtrapseen mask.

CE patch (from <Someone>)

B17007 - ctrl/p documentation
mention msg_window in the Guidebook and long online help.  I didn't update
hh since that only has one-liners.  Also made the msg_window default clear
in the Guidebook and opthelp.  Plus, an update to Guidebook.txt in hopes
that it won't need another before release.

pline bits
a few plines that were without punctuation.  There may be more non-DEBUG
pline or pline-like things that are still missing punctuation.  They are
almost impossible to find after the fact, since they could be anywhere,
including in various dat files and functions that pass strings and formats
into other functions that call pline.

line length bit

unix dependency update for 3.4.1
should now be up to date unless someone makes a material change to includes

U260 - waiting times out for no reason while in were form
The initial report thought this was related to summoning help.  It's not.
moveloop would attempt to call you_were() even when you_were() would not
actually change your form.  Certainly there's a layering problem here, but
for now, just put in the same check peffects() uses to avoid calling
you_were() unnecessarily.

falling into water while asleep

     Try to fix the reported bug of not waking up if sleeping on ice
that gets melted out from under you.  This fixes the straightforward
case but I suspect there are other permutations that it doesn't cover.
Teleporting out of water is now blocked if asleep; waking up occurs
after the chance for that has passed.

vms update

magicbane tweaks

     Update the comments to [try to] more accurately describe the behavior.
Also add hallucination handling and make the probe effect be more likely
to occur (or to put it another way, make the stun effect be less likely
since one or the other always takes place--except when stun is superseded
by something more severe).

nethack -s buffer overflow fix
When printing invalid player names in -s mode, it was possible to overflow
the output buffer due to a missing buffer size check.  On shared Unix-like
systems with executable stacks, this could be used as a security exploit,
eg to obtain a shell running as user or group games.
While I was at it, removed a dead block of "#if 0" code

magicbane fixes

     Revamp the Magicbane code so that it won't result in "<monster>
turns to flee" followed by "the magic-absorbing blade scares <monster>".
In the process I noticed that resistance checks for its scare and purge
effects were based on the character's experience level regardless of who
was wielding it or who its target was.

     I didn't try to retain the exact behavior it had before, but the
new behavior is pretty close.  The main differences are that the "purge"
effect is now called "cancel" and that the stun effect will be less
common now and always gives feedback when it occurs.  It used to combine
stun with scare and/or purge in some cases, now it won't; and it used to
always scare when purging, now it will pick one or the other.

magicbane groundwork

     To fix Magicbane's message sequencing, its code needs to be redone
substantially.  These changes make that easier.

cancel_monst()  let caller know whether cancellation succeeds
resist()  give artifact weapons a resistance attack rating
vtense()  handle monster names and "you" as subjects; checking against
          object names and descriptions pointed out a couple of other
          words that would have ended up being miscategorized.

grammar bit

<email deleted> wrote:
> On Sun, 12 Jan 2003 15:40:33 +0100, <Someone>
> <email deleted> wrote:
>
> >You begin praying to Anu. You are surrounded by a shimmering light.
> >You finish your prayer. You feel that Anu is well-pleased.
> >Your stomach feels content.
> >You are feeling mildly nauseous.
>
> Huh.
>
> sh-2.03$ grep nauseous *.c
> pline.c:        if (Vomiting)           Strcat(info, ", nauseated"); /*
> !"nauseous" */
> potion.c:           if(talk) You_feel("much less nauseous now.");
> timeout.c:      "are feeling mildly nauseous.",         /* 14 */
>
> Well, pline.c has it right.  Nauseated means feeling sick.  Nauseous
> means sickening to contemplate.

Second opinion (dictionary.com):
> Usage Note: Traditional critics have insisted that nauseous is
> properly used only to mean causing nausea and that it is
> incorrect to use it to mean affected with nausea, as in Roller
> coasters make me nauseous. In this example, nauseated is
> preferred by 72 percent of the Usage Panel. Curiously, though, 88
> percent of the Panelists prefer using nauseating in the sentence
> The children looked a little green from too many candy apples and
> nauseating (not nauseous) rides. Since there is a lot of evidence
> to show that nauseous is widely used to mean feeling sick, it
> appears that people use nauseous mainly in the sense in which it
> is considered incorrect. In its correct sense it is being
> supplanted by nauseating.

win32tty  fixes34.1 catch up

beta 19

B18009 animate figurine over water

>More worrying is the fact that applying a figurine over water lets
>the monster wait until its next move before it drowns (giving
>you time to teleport it to safety, or whatever) [...]
>Should there be a minliquid() check as part of make_familiar()?

Applying at the water location next to you was easy. But
applying it at your own location (triggering BY_YOU)  could
end up placing the figurine at the far side of the level if
there was lots of water.

Correcting that required the ability to pass a flag from
make_familiar to makemon() telling it to not rule out
water locations as good positions.  The flag had to
be passed on down to goodpos() and enexto().

The bulk of this patch is just adding an additional
argument to goodpos() in all of the callers.

Cloning monsters doesn't count for extinction as comments imply
Another from <Someone>'s list.

Adjust the comment.

xorns sink follow-up
|

xorns sink when drawbridge is raised
From <Someone>'s list: Xorns don't sink if the drawbridge they're standing on is
raised.

fix B18011 warning loss

> If you remove a ring of warning, see_monsters() is called to make
> sure that the numbers get removed from the display. However, if
> your only source of warning is experience level and you get
> drained and feel "less sensitive", it isn't, and they're not.

fix B18010 fatal exploding chest at ball & chain

fix B18007 - travel cmd forces extra keystroke

     I don't know whether there were any other odd situations besides moving
onto known traps being caused by the out of date value in flags.run; several
places do check to see whether it's 8.