]> granicus.if.org Git - nethack/log
nethack
2 years agoMerge branch 'unicode' of https://github.com/chasonr/nethack into unicode
Ray Chason [Mon, 3 Oct 2022 00:41:21 +0000 (20:41 -0400)]
Merge branch 'unicode' of https://github.com/chasonr/nethack into unicode

2 years agoUpdate the README for the fonts
Ray Chason [Mon, 3 Oct 2022 00:38:30 +0000 (20:38 -0400)]
Update the README for the fonts

2 years agoRewrite the makefont program in Lua
Ray Chason [Sun, 2 Oct 2022 17:29:47 +0000 (13:29 -0400)]
Rewrite the makefont program in Lua

2 years agoSupport the DOSVGA build
Ray Chason [Sun, 2 Oct 2022 07:12:03 +0000 (03:12 -0400)]
Support the DOSVGA build

2 years agoMore native compile fixes
Ray Chason [Sun, 2 Oct 2022 05:44:06 +0000 (01:44 -0400)]
More native compile fixes

* subst doesn't seem to work in variable assignments
* LUADLL isn't meaningful for MS-DOS

2 years agoUpdates to the native compile
Ray Chason [Sun, 2 Oct 2022 05:05:46 +0000 (01:05 -0400)]
Updates to the native compile

2 years agoSupport wide Curses on MS-DOS
Ray Chason [Sun, 2 Oct 2022 03:44:42 +0000 (23:44 -0400)]
Support wide Curses on MS-DOS

Also, fix IBMGraphics on Curses

2 years agoUpdate the README for the fonts
Ray Chason [Mon, 3 Oct 2022 00:38:30 +0000 (20:38 -0400)]
Update the README for the fonts

2 years agofix wishing help for X11
PatR [Sun, 2 Oct 2022 19:42:12 +0000 (12:42 -0700)]
fix wishing help for X11

Pass the wait-for-response arg when displaying the wishing help text
window.  tty, curses, and Qt waited regardless, but X11 honors the
no-wait request.  It was showing the text window then letting the
core immediately resume, resulting in reissuing the wish prompt on
top of the help window.  Entering a successful wish then dismissed
the prompt but left the help on the screen, possibly obscuring the
map depending on placement.

2 years agoRewrite the makefont program in Lua
Ray Chason [Sun, 2 Oct 2022 17:29:47 +0000 (13:29 -0400)]
Rewrite the makefont program in Lua

2 years agoSupport the DOSVGA build
Ray Chason [Sun, 2 Oct 2022 07:12:03 +0000 (03:12 -0400)]
Support the DOSVGA build

2 years agoMore native compile fixes
Ray Chason [Sun, 2 Oct 2022 05:44:06 +0000 (01:44 -0400)]
More native compile fixes

* subst doesn't seem to work in variable assignments
* LUADLL isn't meaningful for MS-DOS

2 years agoUpdates to the native compile
Ray Chason [Sun, 2 Oct 2022 05:05:46 +0000 (01:05 -0400)]
Updates to the native compile

2 years agoSupport wide Curses on MS-DOS
Ray Chason [Sun, 2 Oct 2022 03:44:42 +0000 (23:44 -0400)]
Support wide Curses on MS-DOS

Also, fix IBMGraphics on Curses

2 years agofix github issue #679 - orc strength
PatR [Sun, 2 Oct 2022 01:14:59 +0000 (18:14 -0700)]
fix github issue #679 - orc strength

Reported by eakaye:  orcish hero has maximum strength of 18/50 but
hero poly'd into an orc was given 18/100 strength.  Also, a comment
from vultur-cadens pointed out that orcish heroes start with poison
resistance while monster orcs lack it.

Even though the boost to 18/100 is only temporary until the poly
times out, make orcs a special case where strongmonst from poly'ing
into them only gives 18/50 strength instead of 18/100.  Adopt the
suggestion that Uruk-hai be an exception and continue to give hero
poly'd into them 18/100.

If any gnome becomes strongmonst (currently none are), treat them
as 18/50 too.  Elvenking and elf-lord are strongmonst; treat their
forms as plain 18 though, matching the limit of elf heroes.  Lesser
monster elves aren't strongmost.

While in there, add another special case so that hero poly'd into a
giant gets 19 strength.  Monster giants are still plain strongmonst
so might warrant some sort of adjustment.

Give orcs poison resistance, but eating their corpses doesn't provide
an opportunity to confer it.  Note goblins and hobgoblins still don't
have the resistance (to distinguish them from orcs a bit).

Take away strongmonst from orc shamans and give it to orc mummies.
Human mummies should have it too (at least according to movies) but
I didn't alter them becuase they're already pretty dangerous at the
point they start occurring.  Take away strongmonst from plain 'elf'
placeholder.

New:  when hero polymorphs into a form that lacks the strongmonst
attribute, take away any exceptional strength (drop 18/01 through
18/100 down to 18; as mentioned above, the drop is only temporary).
There's no attempt to set the maximum even lower for wimpy forms.

Fixes #679

2 years agouse Norep() for searching while engulfed
PatR [Sat, 1 Oct 2022 09:35:51 +0000 (02:35 -0700)]
use Norep() for searching while engulfed

Suggested by entrez:  when you search while engulfed the feedback asks
whether you're looking for the exit, but the joke about the exit isn't
funny when repeated over and over which happens if the player waits to
be expelled by using 's','s',... rather than '.','.',....

2 years agofix #K3739: engraving in dust mentions frost
PatR [Sat, 1 Oct 2022 09:34:28 +0000 (02:34 -0700)]
fix #K3739: engraving in dust mentions frost

[sic] should be "engraving in frost mentions dust"

Writing on ice with fingers is described as writing in frost, but if
you overwrite an existing engraving rather than add to it the game
said you wiped out the engraving in the dust (immediately followed
by writing in the frost).

Not mentioned in the report:  finishing a multi-turn engraving on
ice had the same problem.

2 years agofixes entry for PR #884 - X11 extended command
PatR [Fri, 30 Sep 2022 15:32:45 +0000 (08:32 -0700)]
fixes entry for PR #884 - X11 extended command

Pull request from entrez:  X11's get-extended-command widget
allocated 1 extra byte for a couple of end-or-array terminators
and then wrote more than 1 byte to them when initializing its list
of extended commands.

Fixes #884

2 years agoFix: X11 extcmd menu heap buffer overflow
Michael Meyer [Fri, 30 Sep 2022 00:22:00 +0000 (20:22 -0400)]
Fix: X11 extcmd menu heap buffer overflow

Instead of allocating space for ((n + 1) * size) (to make room for all
entries plus terminator), it allocated space for (n * size + 1).
init_extended_commands_popup would therefore write past the end of the
memory allocated for command_list and command_indx when trying to store
their respective terminator entries.  This meant the X11 windowport
would crash when accessing the extended command menu, if NetHack had
been compiled with -fsanitize=address.

I also did some minor cleanup/refactoring to eliminate variables and
lines that were made redundant or useless by 9d64d13, which changed the
way the function worked and removed the need for things like tracking
indices in the source and destination arrays with separate variables.

2 years agoa couple of warnings
nhmall [Fri, 30 Sep 2022 12:59:57 +0000 (08:59 -0400)]
a couple of warnings

2 years agocurses: scrollbars on clipped map
PatR [Thu, 29 Sep 2022 20:58:50 +0000 (13:58 -0700)]
curses: scrollbars on clipped map

A change to the curses interface from three years ago to make sure
that round-off didn't make the horizontal and vertical clipped map
indicators appear to not be clipped was using ROWNO for both instead
of COLNO for the horizontal one.  For modest clipping the mistake
was unnoticeable; I don't know whether that remained true for more
extreme clipping.

[Not fixed:  the curses scrollbar stuff ignores the fact that map
column 0 is unused.]

2 years agohuge chunk of meant name change not comprehensive
nhmall [Thu, 29 Sep 2022 16:48:03 +0000 (12:48 -0400)]
huge chunk of meant name change not comprehensive

warning: for tile 250 (numbered 250) of objects.txt,
        found 'huge chunk of meat' while expecting 'enormous meatball'
../win/share/objects.txt: 460 tiles

2 years agomore genocide prompts
PatR [Wed, 28 Sep 2022 22:19:41 +0000 (15:19 -0700)]
more genocide prompts

Make the prompt for single genocide be more precise
 from:  What monster do you want to genocide?
 _to_:  What type of monster do you want to genocide?
and the re-prompt for it be slightly more verbose
 from:  What... [type the name of a monster]
 _to_:  What... [enter the name of a type of monster]

Also, make the already verbose re-prompt for class genocide even
more verbose
 from: What class... [type the symbol representing a class]
 _to_: What class... [type the symbol or name representing a class]

Possibly should have changed 'type' to 'enter' on the last one to
keep them consistent but I left that as-is.

2 years agopull request #882 - prompting for genocide target
PatR [Wed, 28 Sep 2022 21:48:48 +0000 (14:48 -0700)]
pull request #882 - prompting for genocide target

Pull request from entrez:  genocide prompt has a short explanation
appended.  Skip that on first try, and on all tries if 'cmd_assist'
option is Off.

For class genocide, don't retry an unlimited number of times if
player supplies an empty response.  Any empty response will now
count toward the 5 chances available to supply a valid choice, as
it does with single genocide.

Fixes #882

2 years agoShow genocide prompt help iff 'cmdassist' enabled
Michael Meyer [Wed, 28 Sep 2022 01:08:30 +0000 (21:08 -0400)]
Show genocide prompt help iff 'cmdassist' enabled

The "[type the name]" prompt seems appropriate for handling via
cmdassist, so experienced players who don't need the help can hide it.
I also added a corresponding help note for the class genocide prompt.

2 years agoDon't screen out empty input in class genocide
Michael Meyer [Wed, 28 Sep 2022 00:47:57 +0000 (20:47 -0400)]
Don't screen out empty input in class genocide

So that a blank line wouldn't use up one of the player's "tries" for
class genocide, the game would continue to prompt until the input was
non-blank (or the user hit <esc>), with a tight loop around the getlin
call that only exited when it got some non-empty input.  This apparently
risked leaving the game endlessly looping under some worst-case-scenario
hangup conditions.  It was also inconsistent with normal genocide, which
doesn't have special handling of blank lines.  Make the class genocide
prompt behave like the normal genocide prompt by removing the "blank
input" loop (and consequently treating a blank line the same way as any
other attempt to write a name).

2 years agofixes entry for pull request #881 - curses exit
PatR [Wed, 28 Sep 2022 21:31:11 +0000 (14:31 -0700)]
fixes entry for pull request #881 - curses exit

Pull request from chasonr:  if the curses interface did a full screen
update while the game was ending, it could attempt to refresh windows
that had already been removed.

For #if PDCURSES, raw_print() could attempt to write to the message
window after it had been removed.

[Why is raw_print() using the message window at all?  Isn't the main
point of it to bypass that?]

Fixes #881

2 years agoFix undefined behavior when exiting Curses
Ray Chason [Wed, 28 Sep 2022 00:57:09 +0000 (20:57 -0400)]
Fix undefined behavior when exiting Curses

* When saving: curses_exit_nhwindows calls curses_uncurse_terminal,
  which calls endwin. curses_exit_nhwindows then calls raw_print,
  which calls more Curses functions after endwin has been called.
  Fix this by having curses_raw_print use puts if window_inited
  is false.

* When dying, quitting, etc.: really_done opens the "Goodbye" window,
  which refreshes the other windows when it closes. But the status
  window (and possibly the map and message windows) are gone by that
  point. The window pointers are properly NULLed, but the NULL is then
  passed to touchwin. Fix this by checking window pointers for NULL.

2 years agogithub pull request #859 - IS_SUBROOM_INDEX
PatR [Wed, 28 Sep 2022 21:11:57 +0000 (14:11 -0700)]
github pull request #859 - IS_SUBROOM_INDEX

Pull request from entrez:  fix an off by one error in the allowed
range of subroom indices.

I've never understood subrooms and am accepting this fix on faith.

Fixes #859

2 years agoFix: IS_SUBROOM_INDEX range
Michael Meyer [Thu, 25 Aug 2022 01:38:56 +0000 (21:38 -0400)]
Fix: IS_SUBROOM_INDEX range

The macro (currently unused, I think) for checking whether a particular
index designates a subroom was off by one on the maximum allowable
value.

Because of the dedicated extra space for the g.rooms array terminator
flag (hx == -1), subroom indices in g.rooms are set out in the range
[MAXNROFROOMS+1, MAXNROFROOMS*2], inclusive.

Also some minor formatting tweaks.

2 years agogithub pull request #733 - prices of unID'd gems
PatR [Wed, 28 Sep 2022 20:43:03 +0000 (13:43 -0700)]
github pull request #733 - prices of unID'd gems

Pull request from vultur-cadens:  don't change unID'd gem prices
when source changes insert some new object before them or the game
gets rebuilt with different featues enabled such as MAIL.

This is more about keeping the spoiler table in the wiki accurate
than anything to do with game play.  It definitely gives players
who use spoilers an advantage over ones who don't, but does so in
such an insignificant aspect of play that it likely doesn't matter.

I'm fairly sure that the remark that keeping the prices the same
is what the original implementor intended is not correct.  I don't
recall who implemented this and the commit log for the first cvs
repository is long gone so it's moot.

Fixes #733

2 years agoUnidentified gem selling prices
vultur-cadens [Fri, 15 Apr 2022 04:39:06 +0000 (21:39 -0700)]
Unidentified gem selling prices

Make the token selling prices for unidentified gems not depend on how
many items were defined before FIRST_GEM.  Now the unidentified gem
selling prices will depend only on the number and defined order of the
types of gems, and won't inexplicably change when objects are added,
or depend on compile-time options such as MAIL.

Also don't do the regular item price reduction for unidentified gems,
since they are already not based on the actual value.  This restores
the pre-3.6 behavior, allowing players to gain a bit more information
from the nominal selling prices of unidentified gems.

Whoever first introduced this special handling for gems probably
intended for players to be able to gain information from gem prices
this way, but probably nobody has been doing it since 3.6.

2 years agorename "huge chunk of meat" to "enormous meatball"
PatR [Tue, 27 Sep 2022 20:32:51 +0000 (13:32 -0700)]
rename "huge chunk of meat" to "enormous meatball"

Pull request #607 by Vivit-R proposed renaming "huge chunk of meat"
to "giant meatball" to better reflect the similarity to meatball.
But an object name that contains a monster name prefix requires extra
work in the wishing code.  I considered "huge meatball" which retains
more of the original name but decided to go with "enormous meatball"
becaues it seems more evocative.

Supersedes #607
Closes #607

2 years ago__attribute__(returns_nonnull)
PatR [Mon, 26 Sep 2022 22:49:42 +0000 (15:49 -0700)]
__attribute__(returns_nonnull)

Refine commit 4885653014e6651118e58315e95c7655539ca21b.

> I'm not sure whether gcc 3 is really the right test for whether the
> returns_nonnull attribute setting is available.

The gcc.gnu.org website only goes back to 5.1, and searching the
documentation of that version for returns_nonnull finds it.  I used
ftp to get gcc-core-3.0.0 and gcc-core-4.0.0 and their doc files don't
mention this attribute.  It might have been added for some later 4.x
but that really doesn't matter for nethack's purposes.

Use __GNUC__ >= 5 instead of __GNUC__ >= 3 when testing whether
__attribute__(returns_nonnull) is available.

2 years agowindowborders menu bit
PatR [Mon, 26 Sep 2022 22:05:22 +0000 (15:05 -0700)]
windowborders menu bit

The menu to interactively set the windowborders option for curses
uses 'a'..'e' for choosing 0..4.  Accept '0'..'4' (via unseen group
accellerator) too.

2 years agomore TTYINV=4 - lamps/leashes
PatR [Mon, 26 Sep 2022 21:50:10 +0000 (14:50 -0700)]
more TTYINV=4 - lamps/leashes

When persistent inventory is set to only show items-in-use, include
leashes attached to pets and lit lamps and candles, same as the '*'
command.

2 years agoTTYINV vs end-of-game disclosure
PatR [Mon, 26 Sep 2022 21:34:23 +0000 (14:34 -0700)]
TTYINV vs end-of-game disclosure

If environment had TTYINV=4 (perm_invent shows worn/wielded only),
disclosing inventory at end of game only showed worn and wielded
items instead of full inventory.  Didn't matter whether perm_invent
is On or which interface is in use or whether game was built with
TTY_PERM_INVENT enabled.

2 years agoobject name assignment vs persistent inventory
PatR [Mon, 26 Sep 2022 21:25:06 +0000 (14:25 -0700)]
object name assignment vs persistent inventory

This is an alternate way to deal with pull request #876, where
splitting a stack that has a name assigned updated perm_invent when
cloning the name and ran into trouble with shop billing when trying
to format for persistent inventory display.

The PR#876 fix has been left in place but wouldn't have been needed
if this had gone in first.

2 years agofix #H5459 - explosions and steeds
PatR [Sun, 25 Sep 2022 08:21:07 +0000 (01:21 -0700)]
fix #H5459 - explosions and steeds

Reported nearly five and half years ago:  mounted hero and steed
shared resistances if they got hit by an explosion.

2 years agofix "unique monster doppelganger and bones bug"
PatR [Sun, 25 Sep 2022 01:49:48 +0000 (18:49 -0700)]
fix "unique monster doppelganger and bones bug"

Reported by paxed 8 years ago:  if a bones file contains a
doppelganger imitating a unique monster, when it gets loaded
that monster ends up being marked as having been created.  The
doppelganger itself will shapechange to other forms, but the
unique monster won't be created when it should be because it has
become extinct.

Report involved creating a statue of a unique monster which
yields a doppelganger in that monster's shape, then using stone
to flesh to animate the statue, dying before it changes to some
other shape, and having bones be saved.

2 years agopull request #850 - level teleporter feedback
PatR [Sat, 24 Sep 2022 22:27:13 +0000 (15:27 -0700)]
pull request #850 - level teleporter feedback

Pull request from copperwater:  don't give "flash of light" feedback
when activating a level teleporter because it's too much like one of
the outcomes of a magic trap.

This doesn't use the suggested commit.  Getting the "you feel
disoriented" feedback before being asked for destination level (when
having teleport control) seemed contradictory.  This gives different
feedback and does so after the actual teleport.

Supersedes #850
and since nhcopier doesn't seem to understand "supersede"
Closes #850

2 years agoalloc() never returns Null
PatR [Sat, 24 Sep 2022 11:39:12 +0000 (04:39 -0700)]
alloc() never returns Null

Mark alloc()--also dupstr() and re_alloc()--for gcc and clang as
always returning non-Null.  This should silence some of the static
analysis complaints.

Almost all the monster and object naming functions (anything that
returns an mbuf or an obuf) should be marked this way too but I'll
leave that for somebody else to deal with.

I didn't attempt to mark alloc() with the 'malloc' attribute because
macro definitions could end up causing trouble.  Specifying its
deallocator would probably be useful but is at even bigger risk of
macro interference.

I'm not sure whether gcc 3 is really the right test for whether the
returns_nonnull attribute setting is available.

2 years agoFix curses cursor keys
Pasi Kallinen [Sat, 24 Sep 2022 07:40:34 +0000 (10:40 +0300)]
Fix curses cursor keys

My change to allow fuzzer run in curses caused the cursor keys
to not work (eg. paging through menus with left and right cursor),
so fix that.

2 years agoitalic attribute on tty
PatR [Sat, 24 Sep 2022 06:37:02 +0000 (23:37 -0700)]
italic attribute on tty

If switching to italic isn't supported for tty, switch to underline
instead.  Comparable to the curses interface.

2 years agofix github issue #581 - "You see no door there."
PatR [Fri, 23 Sep 2022 22:30:00 +0000 (15:30 -0700)]
fix github issue #581 - "You see no door there."

Year old issue from copperwater:  'open' directed at a non-door told
player that there isn't a door and took no time unless character was
blind and learned what type of terrain it is, applying a key gave
the same message but used a turn and didn't update map to reflect any
terrain discovery.

Attempting to open an adjacent door or applying a key to one while in
a pit had a similar issue:  they produced the same "you can't reach
it from here" but had different time vs no-time outcome.

There may be other actions in the same situation.

Closes #581

2 years agogithub PR #868 - fix replacement of 'w' terrain
PatR [Fri, 23 Sep 2022 06:56:31 +0000 (23:56 -0700)]
github PR #868 - fix replacement of 'w' terrain

Pull request from copperwater:  terrain replacement couldn't match
'w' because it only checked actual terrain types.

Fixes #868

2 years agoMake replace_terrain respect fromterrain='w'
copperwater [Sun, 4 Sep 2022 16:31:45 +0000 (12:31 -0400)]
Make replace_terrain respect fromterrain='w'

Noticed that an attempted terrain replacement wasn't taking hold even
though 'w' is supposed to mean "match any stone or wall"; this was
because w converts into non-terrain-type MATCH_WALL and replace_terrain
was doing a simple comparison on whether the potentially replaced
terrain matches that type. Add a special check here for w so it will
match the terrain types it's supposed to.

Note that using replace_terrain with 'w' now WILL match stone, since
this is the documented behavior of w, to match IS_STWALL rather than
just IS_WALL. If a level designer really wants to exclude stone, they
can work around this by either making a selection and filter out stone
terrain, or doing two replace_terrains with '-' and '|'.

2 years agofixes entry for PR #848 - no traps in Tou shops
PatR [Fri, 23 Sep 2022 06:43:36 +0000 (23:43 -0700)]
fixes entry for PR #848 - no traps in Tou shops

Pull request from copperwater:  the random traps specified by the
special level definitions of the tourist locate and goal levels could
be placed inside the shops present on those levels.

Fixes #848

2 years agoFix: prevent traps in shops in the Tourist quest
copperwater [Sun, 14 Aug 2022 00:04:25 +0000 (20:04 -0400)]
Fix: prevent traps in shops in the Tourist quest

The Tourist locate and goal levels have 2 shops each, and also have
various traps randomly placed on the level. Unfortunately, this does not
account for placing them in the shops, so it was possible to wind up
with a magic trap or falling rock trap or whatever inside the shop,
which the shopkeeper would not clean up.

This commit makes selections that exclude the shop areas, and picks the
traps from those selections, keeping the shop floors trap-free as their
customers expect.

2 years agodgn_bottom() fix
PatR [Fri, 23 Sep 2022 06:16:19 +0000 (23:16 -0700)]
dgn_bottom() fix

Use the level passed in instead of the hero's location when checking
the dungeon branch.

It probably doesn't make any difference, but use the argument that's
already being provided.

2 years agovibrating square
PatR [Thu, 22 Sep 2022 23:50:41 +0000 (16:50 -0700)]
vibrating square

Give a [probably pointless] hint if the player tries to move '>'
while on the vibrating square.

2 years agorefine bottom of Gehennom check
PatR [Thu, 22 Sep 2022 23:47:37 +0000 (16:47 -0700)]
refine bottom of Gehennom check

In the quest branch, dbg_bottom() returns a barrier level rather
than actual bottom.  Do things the same way for the gehennom branch.

2 years ago'fix' #K3716 - engraving with Fire Brand
PatR [Thu, 22 Sep 2022 23:09:22 +0000 (16:09 -0700)]
'fix' #K3716 - engraving with Fire Brand

Add a comment to the effect that engraving with Fire Brand doesn't
cause it to become dull.

[I'm not sure that is the behavior we really want.  It seems like an
unintended side-effect of changing Fire Brand's engrave type to BURN.]

2 years ago'fix' #K3676 - artifact_hit()'s magr
PatR [Thu, 22 Sep 2022 22:34:16 +0000 (15:34 -0700)]
'fix' #K3676 - artifact_hit()'s magr

Someone asked whether the 'magr' argument to artifact_hit() can be
Null or not since the code sometimes checks whether it is Null and
other times uses it unconditonally.  The answer is "it depends."
Can't reply to asker due to forced anonymity when the contact form
was submitted.

2 years agoCurses: enable debug fuzzer
Pasi Kallinen [Thu, 22 Sep 2022 10:10:13 +0000 (13:10 +0300)]
Curses: enable debug fuzzer

2 years agoPrevent impossible fall dmg if falling up
Michael Meyer [Thu, 22 Sep 2022 02:19:42 +0000 (22:19 -0400)]
Prevent impossible fall dmg if falling up

The way hole destinations work now theoretically allows for a
cross-branch hole or trap door to move you across branches in a way that
decreases your overall depth.  If this happened, it would cause an
impossible when the negative result of (depth(new) - depth(old)) was
used to calculate fall damage.  Limit fall damage to 1d6 if dist <= 0.

2 years agoApply trap door destination restrictions in dodown
Michael Meyer [Thu, 22 Sep 2022 01:25:25 +0000 (21:25 -0400)]
Apply trap door destination restrictions in dodown

Missed this way to use the trap door (in a block added in 05761ba) in
previous commits, though I'm a little confused about whether that block
in dodown is even reachable given how various trap scenarios are handled
with dotrap earlier in the function.

2 years agoApply dest. limit to monster trap door usage
Michael Meyer [Thu, 22 Sep 2022 00:19:33 +0000 (20:19 -0400)]
Apply dest. limit to monster trap door usage

To prevent monsters from falling past the bottom level or into the
sanctum early, and to maintain consistency between monster and hero hole
usage.

2 years agoRefine attempt to clamp trap door fall destination
Michael Meyer [Thu, 22 Sep 2022 00:06:35 +0000 (20:06 -0400)]
Refine attempt to clamp trap door fall destination

This should prevent anyone from exploiting falling into the sanctum (by
taking advantage of a trap door in a bones file from a differently
laid-out dungeon, as described in the previous commit) to bypass the
invocation, in addition to falling past the actual end of the dungeon.

2 years agoPrevent hero from falling past end of dungeon
Michael Meyer [Wed, 21 Sep 2022 23:55:04 +0000 (19:55 -0400)]
Prevent hero from falling past end of dungeon

Similar story with saved trap door destinations: if a bones file near
the end of the dungeon came from a longer dungeon (i.e. with a
lower-depth castle) than the one the bones file is loaded into, the
trap door destination could be past the dungeon end.  Clamp the
destination so it won't be lower than the bottom level of the dungeon.

2 years agoHave monsters' hole destination match the hero's
Michael Meyer [Wed, 21 Sep 2022 23:44:27 +0000 (19:44 -0400)]
Have monsters' hole destination match the hero's

The fixed destination of a hole or trap door was being used for the hero
but not for monsters.  Make everyone land in the same place, so you can
chase a monster into a hole and actually find it.

2 years agoFix: antigravity trap doors
Michael Meyer [Wed, 21 Sep 2022 23:41:45 +0000 (19:41 -0400)]
Fix: antigravity trap doors

Trap doors saved their destinations as an absolute level, rather than a
relative one, so if you loaded bones from a special level their
destinations would reflect the dungeon layout from the bones player's
game.  For example, die on the Oracle level, on dlvl5, with a trap door
that goes to dlvl6.  Another player gets those bones on their Oracle
level, which is dlvl8... the trap door would still go to dlvl6.  Pretty
amazing trap door -- something you might see in a funhouse!

Include relative rather than absolute destinations in save and bones
files, much like stairs do, to avoid this problem.

I bumped EDITLEVEL because although this won't break save files in an
obvious way, it will interpret the (absolute) destinations in existing
save and bones files as relative, leading to some crazy long falls. :)

2 years agogithub pull request #628 - duplicate code
PatR [Thu, 22 Sep 2022 00:02:45 +0000 (17:02 -0700)]
github pull request #628 - duplicate code

Pull request from argrath nearly a year ago: an 'if' and corresponding
'else' have the same code so there's no point in testing for if/else.

I'm still not convinced that simply removing the if/else is the right
fix here but nothing else is going on.  I've put part of the removed
code back inside '#if 0' in case it needs to be resurrected someday.

Closes #628

2 years agoremove duplicate code
SHIRAKATA Kentaro [Sun, 31 Oct 2021 10:04:45 +0000 (19:04 +0900)]
remove duplicate code

Here, `then` clause and `else` clause is identical.

2 years agogithub PR #879 - pets eating shapeshifter corpses
PatR [Wed, 21 Sep 2022 19:07:05 +0000 (12:07 -0700)]
github PR #879 - pets eating shapeshifter corpses

Pull request from entrez:  the check for whether a pet in desperate
straits will eat a corpse that will cause it to polymorph used bad
logic.  The suspect code was added post-3.6.

Fixes #879

2 years agoupdate tested versions of Visual Studio 2022-09-21
nhmall [Wed, 21 Sep 2022 18:55:34 +0000 (14:55 -0400)]
update tested versions of Visual Studio 2022-09-21

2 years agoFix: starving herbivore pet vs polymorphing corpse
Michael Meyer [Tue, 20 Sep 2022 01:59:34 +0000 (21:59 -0400)]
Fix: starving herbivore pet vs polymorphing corpse

The special handling of polymorphing corpses included an "is acceptable
food if starving" rule which ignored the pet's other food preferences,
so a starving herbivorous pet would become willing to eat a
non-vegetarian corpse iff the corpse would polymorph it when eaten.
Along the same lines but latent: if there were a vegan polymorphing
corpse, a carnivorous pet would eat it not only if starving, but also if
maltreated and on the verge of becoming feral.

Instead of trying to fix this by reimplementing all the herbi vs carni
rules specifically for polymorphing corpses, just have a "don't eat
polymorphing corpses if neither starving nor almost untame" rule, and
fall back to the normal palatable corpses tests once it's been
determined that the pet is willing to eat even a polymorphing corpse.

I rephrased the comment just to make it negative (about avoiding
polymorph, rather than polymorph being OK under certain circumstances)
to match the new form of the rule.

2 years agogithub PR #858 - feedback when poly'd hero engulfs
PatR [Tue, 20 Sep 2022 23:59:43 +0000 (16:59 -0700)]
github PR #858 - feedback when poly'd hero engulfs

Pull request from entrez:  the change to remove digestion damage
from trappers and lurkers above didn't handle engulfing by poly'd
hero adequately.

Fixes #858

2 years agoDescribe engulf attacks a bit more consistently
Michael Meyer [Wed, 24 Aug 2022 17:08:46 +0000 (13:08 -0400)]
Describe engulf attacks a bit more consistently

Use verbiage for mon vs mon and hero (mostly hero) engulf attacks that
matches recent changes to monster vs hero engulf attacks more closely
(e.g. "swallows whole" instead of "engulfs" for purple worm, other
changes in b07fe59...).  Also ensure non-AD_DGST engulf attacks
(e.g. from revamped trapper or lurker above polyforms) aren't treated as
"eating" (or as involving "debris").

Also change the enfolds and digests macros so they produce booleans
rather than attack pointers (I got a compiler warning about casting
struct attack * to boolean when I did 'boolean b = digests(ptr);').

2 years agofixes entry and tweaks for PR #871 - revive corpse
PatR [Tue, 20 Sep 2022 21:33:39 +0000 (14:33 -0700)]
fixes entry and tweaks for PR #871 - revive corpse

Pull requet from entrez:  give better feedback than "it" when hero
observes a corpse reviving into a monster that can't be seen.

Tweak reviving from a container which was coded as if the container
was optional.  That can lead to confusion when someone reads the
code so make the situation more explicit.

Fixes #871

2 years agoImprove description of invisible mon revival
Michael Meyer [Sat, 10 Sep 2022 03:07:57 +0000 (23:07 -0400)]
Improve description of invisible mon revival

An invisible monster reviving would be called "it" (as in, "It rises
from the dead!").  Improve on this a bit by instead saying that "the
troll corpse disappears!" (similar to the messaging used when undead
turning is used on an invisible monster's corpse), or calling the
revived monster "something" (as in "something escapes from a sack!")
instead of "it".  Distinguish between the original location of the
corpse and the location of the revived monster when describing seeing
things that have happened to the corpse, since revival may not place it
on the corpse's location.

2 years agogithub #877 - fix "disintegration resistance"
PatR [Tue, 20 Sep 2022 20:24:54 +0000 (13:24 -0700)]
github #877 - fix "disintegration resistance"

Pull requet from vultur-cadens:  use space instead of hyphen in
enlightenment and end-of-game feedback.

All the other resistances already use space.  The inappropriate
hyphen in "disintegration-resistance" seems to have been present
since enlightenment was added (in 3.0; back then the relevant code
was in cmd.c; current insight.c didn't exist until 3.6).

Fixes #877

2 years agoUse a space instead of a hyphen in "disintegration-resistant"
vultur-cadens [Sat, 17 Sep 2022 06:40:56 +0000 (23:40 -0700)]
Use a space instead of a hyphen in "disintegration-resistant"

This is for consistency with the other resistance insight messages,
which use spaces instead of hyphens.

2 years agoPrevent trapdoors and holes dropping you into Sanctum
Pasi Kallinen [Tue, 20 Sep 2022 09:07:33 +0000 (12:07 +0300)]
Prevent trapdoors and holes dropping you into Sanctum

2 years agofixes entry for PR #876 - shop impossible \
PatR [Tue, 20 Sep 2022 02:01:35 +0000 (19:01 -0700)]
fixes entry for PR #876 - shop impossible \

when splitting a stack of named, shop-owned objects

Pull request from entrez:  when perm_invent is on, splitting an unpaid
stack would issue impossible "unpaid_cost: object wasn't on any bill"
while cloning the new stack's name from the old stack.

Fixes #876

2 years agoFix: impossible from splitting named stack on bill
Michael Meyer [Sat, 17 Sep 2022 02:50:39 +0000 (22:50 -0400)]
Fix: impossible from splitting named stack on bill

Trying to split an unpaid stack of named items in a shop, with
perm_invent enabled, would cause an impossible 'unpaid_cost: object
wasn't on any bill' because copy_oextra -> oname triggered an inventory
update while the newly created split stack was marked unpaid but before
the billing information had been split to match.  Defer the copy_oextra
call until the billing info has already been split.

2 years agoremove unused math.h in isaac64.c
nhmall [Mon, 19 Sep 2022 21:32:42 +0000 (17:32 -0400)]
remove unused math.h in isaac64.c

Close  #878

2 years agounix Makefile.src and hints
PatR [Sun, 18 Sep 2022 21:25:33 +0000 (14:25 -0700)]
unix Makefile.src and hints

Change the handling for windowing system specific files so that
when building for more than one set, each gets compiled as a set
instead of some being interspersed among rival window systems.
Put differently, handle tile.o specially so that there's no need
for the hints to sort the WINOBJ list in order to avoid tile.o
duplication.

So the order of compilation is
  common source files
  unix-specific files
  tty files
  curses files
  X11 files
  Qt files
  tile.c (if applicable), version.c, date.c

Previously, some of the X11 files were scattered around among the
others because of the spelling of their file names.

Only matters if you're watching the progress of a build.

2 years agoKnockback requires solid physical hit
Pasi Kallinen [Sun, 18 Sep 2022 14:56:01 +0000 (17:56 +0300)]
Knockback requires solid physical hit

Exclude unsolid monsters - air, water, and fire elementals.

2 years agoThemerooms: Engraving hints the location of buried treasure
Pasi Kallinen [Sun, 18 Sep 2022 09:35:30 +0000 (12:35 +0300)]
Themerooms: Engraving hints the location of buried treasure

Add two new themeroom functions that are called when generating
the level: pre_themerooms_generate and post_themerooms_generate,
calles before and after themerooms_generate.

Allow the buried treasure -themeroom to put down an engraving
anywhere on the level, hinting at the location of the treasure.

des.object contents function now gets the generated object passed
to it as a parameter.

2 years agoavoid a format-overflow warning
nhmall [Sat, 17 Sep 2022 23:29:09 +0000 (19:29 -0400)]
avoid a format-overflow warning

options.c: In function ‘option_help’:
options.c:8820:55: warning: ‘%s’ directive writing up to 255 bytes into a region of size 220 [-Wformat-overflow=]
 8820 |     Sprintf(buf, "Set options as OPTIONS=<options> in %s", configfile);
      |                                                       ^~   ~~~~~~~~~~

2 years agoRevert "NEED_VARARGS bit for end.c and files.c"
nhmall [Sat, 17 Sep 2022 19:58:05 +0000 (15:58 -0400)]
Revert "NEED_VARARGS bit for end.c and files.c"

This reverts commit af71163d99ab914c51ec3e3789cd4fb4f3f78c1f.

2 years agoRevert "NEED_VARARGS followup"
nhmall [Sat, 17 Sep 2022 19:57:52 +0000 (15:57 -0400)]
Revert "NEED_VARARGS followup"

This reverts commit 4d34e153e06b9d38588f0e856f5728ff8285df00.

2 years agoRevert "paste fix"
nhmall [Sat, 17 Sep 2022 19:57:31 +0000 (15:57 -0400)]
Revert "paste fix"

This reverts commit d1750e4abed25c0f34ecbca443a8cef46daf3605.

2 years agopaste fix
nhmall [Sat, 17 Sep 2022 18:48:02 +0000 (14:48 -0400)]
paste fix

2 years agoNEED_VARARGS followup
nhmall [Sat, 17 Sep 2022 18:22:20 +0000 (14:22 -0400)]
NEED_VARARGS followup

2 years agoNEED_VARARGS bit for end.c and files.c
nhmall [Sat, 17 Sep 2022 16:31:26 +0000 (12:31 -0400)]
NEED_VARARGS bit for end.c and files.c

avoid build failure in the event that NEED_VARARGS is already defined
on the compiler command line.

2 years agoRevert "onefile: isaac64.c <math.h> vs yn()"
PatR [Sat, 17 Sep 2022 00:45:31 +0000 (17:45 -0700)]
Revert "onefile: isaac64.c <math.h> vs yn()"

This reverts commit 94945a719afc0aa08eaff9aa1e0e2fbf159bf388.
It was too intrusive and can be handled by the 'onefile' script
by compiling isaac64.c before any source file that includes hack.h.

2 years agoinsults
PatR [Fri, 16 Sep 2022 23:38:44 +0000 (16:38 -0700)]
insults

random_insult[] and random_malediction[] are only used in wizard.c
so make them file-scope static instead of global.

2 years agofix issue #875 - contnr-to-contnr tip explosion
PatR [Fri, 16 Sep 2022 23:14:20 +0000 (16:14 -0700)]
fix issue #875 - contnr-to-contnr tip explosion

Issue reported by k2:  tipping a wand of cancellation, bag of
holding, or bag of tricks from a non-magic container into a bag of
holding causes the bag of holding to explode but it wasn't dealing
out explosion damage nor being logged for livelog/chronicle the way
putting the item directly into a bag of holding is handled.

This fixes the described issues but bones handling leaves a lot to
be desired.

Fixes #875

2 years agotradstdc.h, USE_STDARG
PatR [Fri, 16 Sep 2022 22:02:23 +0000 (15:02 -0700)]
tradstdc.h, USE_STDARG

Make USE_STDARG explicitly the default (it was implicitly the default
when NHSTDC is defined) and update some comments related to VA_DECL.

2 years agoPrevent impossible lighting a massive stack of candles
Pasi Kallinen [Fri, 16 Sep 2022 16:55:49 +0000 (19:55 +0300)]
Prevent impossible lighting a massive stack of candles

2 years agomore inconsistencies
nhmall [Fri, 16 Sep 2022 14:44:39 +0000 (10:44 -0400)]
more inconsistencies

2 years agomodernize added old-style function declarator
nhmall [Fri, 16 Sep 2022 04:55:08 +0000 (00:55 -0400)]
modernize added old-style function declarator

2 years agofix issue #843 - vampire revival sequencing
PatR [Fri, 16 Sep 2022 01:02:07 +0000 (18:02 -0700)]
fix issue #843 - vampire revival sequencing

Reported by Umbire:
|You kill SpaceMannSpiff!  SpaceMannSpiff puts on a dwarvish cloak.
|SpaceMannSpiff puts on a dwarvish iron helm.
|The seemingly dead SpaceMannSpiff suddenly transforms and rises as
| a Vampire.

This was tough to reproduce but I finally managed it.  The issue
text mentions that it was fixed by copperwater in xNetHack with
commit 8c4af50f0aa3e72522f3eb98df039ff25c2a1ea0 to the repository
for that variant.  My attempt to cherry-pick that failed--I'm not
even sure whether it should have been expected to work--and some of
the code has been impinged upon by changes, so I ended up applying
the contents of that commit manually.

The commit changes how/when monsters put on new armor rather than
anything directly related to vampires.  Circumstances similar to
the example above now yield:
|You kill SpaceMannSpiff!
|The seemingly dead SpaceMannSpiff suddenly transforms and rises as
| a Vampire.
on one turn, then on the next turn the revived vampire produces:
|SpaceMannSpiff puts on a dwarvish cloak.

My test case only had one item of interest; I assume that the second
item of armor gets worn on a subsequent turn rather than at the same
time as the first one.

Fixes #843

2 years agoobj.h mixup
PatR [Thu, 15 Sep 2022 21:17:43 +0000 (14:17 -0700)]
obj.h mixup

\#if 0 should have been #if 1.  Swap #then and #else instead.

2 years agoempty horn of plenty
PatR [Thu, 15 Sep 2022 21:14:12 +0000 (14:14 -0700)]
empty horn of plenty

Format a horn of plenty whose charge count is unknown but is known to
be empty as "empty horn of plenty" like is done for real containers.

This was too easy; I must have missed something....

2 years agomore #tip, mostly container-to-container
PatR [Thu, 15 Sep 2022 19:30:44 +0000 (12:30 -0700)]
more #tip, mostly container-to-container

Require a free hand when tipping a container into another container.
Presumeably you need to open the destination container and possibly
keep holding it open.

If you try to tip a carried container into an unknown bag of tricks,
apply the bag (once) instead of performing the tip.  (To 'open' the
destination as above.)  Possibly slightly confusing if bag is empty.

When tipping a container, always ask for the destination instead of
doing that only when carrying other containers.  Confirming floor
as destination can be annoying but having to do that sometimes and
skipping that sometimes is aggravating because it is error prone.
And floor is preselected so can be chosen with space or return.
(I wanted to change the selector letter for floor from '-' to '.'
and then keep '-' as an unseen group accelerator, but the latter
doesn't work for PICK_ONE so I've left '-' as-is.)

Don't display "monsters appear" after tipping a bag of tricks.
Monster creation gives feedback these days.  (Comparable to recent
"summon nasties" fix.)

2 years agoFix CI warning
Pasi Kallinen [Thu, 15 Sep 2022 15:55:15 +0000 (18:55 +0300)]
Fix CI warning

2 years agoSplit themeroom shape from themeroom contents
Pasi Kallinen [Thu, 15 Sep 2022 15:08:32 +0000 (18:08 +0300)]
Split themeroom shape from themeroom contents

Previously, the tetris-shaped rooms were always either
normal rooms, or turned into shops or other special rooms
in NetHack core. Now, the themed room lua code first picks
the themed room (which can be a themed or shaped), and some
of those will then pick a random filling (eg. ice floor,
traps, corpses, 3 altars).

Adds a new lua binding to create a selection picking locations
in current room.

The content-function in special level regions now get passed
the room data as a parameter.