| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
They were only used in one place, and the relevant code was moved to
that one place in the previous commit.
|
| |
|
|
|
|
|
|
|
| |
.cc, moving its contents into the new stepdown.cc and strings.cc.
(The latter also got many donations from libutil.h.)
Down with stuff! Up the new flesh!
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
A good deal of functions move to the two new files, mon-poly and
mon-message. Of the others, some go to where they are used, some to
mon-util, and a few are made member methods of monster.
This probably breaks Xcode compilation, and I'm not able to test
the changes I made to MSVC that will (hopefully) keep it working.
|
| |
|
|
|
|
|
|
|
| |
Chris Oelmueller made an excellent patch for this, but unfortunately it
was rather rotted by the time somebody decided to look at it. It was
easier to recreate than update. I've also added some tiles stuff which
was missed in the original patch.
|
|
|
|
|
|
|
|
|
|
| |
Phase out ARTP_ACCURACY, rename ARTP_DAMAGE to ARTP_SLAYING which now
combines both Acc+ and Dam+ bonuses.
Bracers of archery are +4 now instead of +5,+3.
[Committer's note: fixed a description and cleaned up various other
small issues.]
|
|
|
|
|
|
| |
The amulet of conservation and cloaks of preservation are no longer
necessary without item destruction. Maxwell's patent armour now has
the resistance ego and grants rF+ and rC+.
|
| |
|
| |
|
|
|
|
| |
This fixes all the instances caught by unbrace.
|
| |
|
|
|
|
|
| |
Adds a new artefact designed to encourage players to cast in
ultra-heavy armour.
|
| |
|
|
|
|
|
| |
Convert existing ones into a negative level of ARTP_STEALTH, or in the case of
Black Knight's barding, to rPois (mostly for flavor reasons).
|
|
|
|
|
|
|
| |
Touches a lot of files since their #includes have to be edited.
(Pushing now since it shouldn't break anything and keeping it updated
is nasty.)
|
|
|
|
|
|
|
|
| |
[It had much the same problems as racial weapons, with only very slightly more
significant effects. Some of the old effects of racial armour could potentially
be rethought and made into a new armour ego. Beogh still gives a (slightly
smaller than before) bonus for armour use, without the orcish requirement.
-MarvinPA]
|
|
|
|
|
|
|
|
|
|
| |
This binds the commands to & y/Y. We have & i/I, but that only affects
inventory items. For various testing purposes, it's nice to be able to
mass-identify/unidentify all item types. This command also
identifies/unidentifies all inventory items on the level, since items can refuse
to unidentify if they exist on the floor, and because level-item identification
is also a convenient thing to have. We don't change the identification status
of off-level items.
|
|
|
|
| |
See Mantis for discussion.
|
|
|
|
|
| |
No particular reason, other than consistency. And all but two used wasteful
double-conversion, so this is not a speed regression.
|
|
|
|
|
|
|
|
|
| |
Also simplify quite a few cases.
It turns out in >90% cases of non-literals the argument had .c_str(),
which meant it was pointlessly malloc()ed and converted from and to
std::string. I believe a sprintf is faster, so even the argument of
miniscule speed-up doesn't apply.
|
|
|
|
|
|
|
|
| |
It trades readability and consistency for an utterly negligible bit of
speed. With the amount of further processing mpr() does, a single sprintf
is nothing.
This reverts commit d9dfa8fc9755fb0a4e8954c7eb94f32fe97b82e0.
|
|
|
|
| |
Scripted, then manually reviewed.
|
|
|
|
|
|
|
|
|
| |
This was changed from bool (add 40 to rate or not) to a number in
180ebbfc23cde438bae5f78af1c326cab8ce24a5.
Not changing the property type (from bool) however meant this number
was either 0 or 1 up until now, which is rather different from 40.
Why does this property never generate "in the wild" by the way?
|
| |
|
|
|
|
|
| |
Also add a couple of asserts so these are less likely to go out of sync
in the future.
|
|
|
|
|
| |
Can't test Android, MSVC or Mac, but a very brief glance at the diff suggests
it's unlikely they're affected.
|
|
|
|
|
| |
It degenerated to a simple loop over menv, hardly worth any syntactic sugar.
I kept it for now, though.
|
|
|
|
|
|
|
|
|
|
|
|
| |
Does the "in view" part of functionality of monster_iterator, is simpler,
allows using los models other than LOS_DEFAULT, and gets rid of a lot of
uses of get_los().
The code is nearly identical as actor_near_iterator, but the old delegation
used more code than either of those. Still, perhaps templating could work?
This commit also fixes a buttload of ignoring invis / see invis / sense invis
(ie, visible_to()) and act-through-glass bugs.
|
|
|
|
| |
Such weapons retained the brand, yet it did nothing.
|
| |
|
| |
|
|
|
|
|
| |
It never had been fully implemented: art-data would break if you tried to
use it.
|
| |
|
|
|
|
|
|
|
| |
The jump enums (two abilities, arte prop, mutation) were previously not added to
the proper location, which would break save compat with trunk. Also add a bit
of save compat code to give Fe innate jump if loading from a save where they
don't have it.
|
|
|
|
|
|
| |
* Can be added to anything that's not a ranged weapon with same frequency code
as used for fly/blink/rage (occurs after rage)
* Shops appraise this for same value as +Rage.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
A warding amulet that provides you with another ring slot.
I rewrote some of the jewellery code, but never re-added the
TOUCH_UI code I removed in _prompt_ring_to_remove. Note that octopodes
never had TOUCH_UI code for their jewellery in the first place. This
should probably be looked at.
Also, the player dumps do not show the amulet
ring on the top, but I do not consider this a problem.
|
|
|
|
| |
Double troll armour regen branded with spirit shield.
|
|
|
|
|
|
|
|
|
|
|
|
| |
(Currently +Fog)
We are at 36, just over the number of letters plus nonzero digits.
Allow zero, and also allow keypresses for ASCII characters between the
numbers and uppercase letters (:;<=>?@). This will allow seven
more properties; after that, or perhaps before, we should move to
uppercase/lowercase. If we do exceed the limit, use '-' to indicate
that the later properties aren't selectable (instead of re-using
letters).
|
|
|
|
| |
Also, use "+Fog" instead of "Fog".
|
|
|
|
|
|
| |
Looks like, unlike "target[t]ing" where a single t is used by many brits and
even some aussies, "cancel[l]ing" has double l even for a good deal of
americans.
|
|
|
|
|
|
|
| |
Mostly modernizing and unifying the abbreviations to kind-of match
current inscription policies. Notable changes are adding 'r' in front
of a bunch of stuff, renaming 'Mag' to 'MR' and introducing the "-+*"
modifiers.
|
|
|
|
|
| |
Pull 'you.religion [!=]= FOO' checks into a function: you_worship(FOO).
This change is part of a large plan to clean up religion.
|
|
|
|
|
| |
Also in a couple of cases add other messages. The goal here is to always
print something when a prompt is dealt with.
|
|
|
|
|
| |
Again, a magic value of -1. It was also wrongly checked in a few error
handling routines, rendering them inoperative even with gcc.
|