| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
.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!
|
| |
|
|
|
|
|
| |
This is currently predicated on DEBUG_MONSPEAK, since all the relevant
messages are.
|
| |
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
| |
You can still set them to any value that fits in an int8_t [-128, 127].
The *modified* stat can never exceed 72, but there is no such cap on the
low end.
|
| |
|
|
|
|
| |
For added insult, nothing used that list anymore.
|
|
|
|
| |
Also, drop the caps.
|
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
Also in a couple of cases add other messages. The goal here is to always
print something when a prompt is dealt with.
|
|
|
|
|
|
| |
The default, substring match, was totally unintuitive and broken.
Only one place is actually changed, something about shapeshifters.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
This should be a std::string probably; for now, 1024 bytes should be enough
for anyone.
|
|
|
|
|
|
|
|
|
|
| |
For most header files, this only saves on having to recompile a
small number of source files, but there are also a few headers
where small changes would now take significantly less time.
This is most obvious for the Tiles build for which the dependencies
have been greatly reduced, so that the only additional includes
when compared to console are strictly library or tile related.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
_you_vault_list would overflow once you found 256 vaults that got shifted
away. env.props["level_{vaults,extra}_key"] could potentially overflow as
well, although the latter would require running with an amulet of statis on
as abyssal "new area" teleports cleared that prop.
This commit also fixes Pan vault info keeping only the last level generated;
Pan char dumps get quite spammy now. Perhaps refcounting them would be
better?
|
|
|
|
|
|
|
|
|
|
|
| |
This is a misdesign in the standard, the committee is debating whether it's
a defect or not, we'll know the result in ten years or so. In the meantime,
gcc-4.8 and clang(version?) added a fix so formerly legal code compiles (but
with a warning...). This fix is not yet in gcc-4.7 nor clang-3.1, and since
failing to build on compilers that happen in the wild is a bad idea, let's
kludge it around with spaces.
More info: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52538
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I had to rename distance() (in coord.h) to distance2() because it conflicts
with the STL function to compare 2 iterators. Not a bad change given how it
returns the square of the distance anyway.
I also had to rename the message global variable (in message.cc) to buffer.
I tried to fix and improve the coding style has much as I could, but I
probably missed a few given how huge and tedious it is.
I also didn't touch crawl-gdb.py, and the stuff in prebuilt, rltiles/tool
and util/levcomp.*, because I have no clue about those.
|
|
|
|
| |
For way too paranoid and underinclusive values of "simple".
|
| |
|
|
|
|
|
|
|
|
| |
For technical reasons (vault_placement), subvaults use renaming data of the
top-level vault, this might be changed one day; that'd require a bit more
writing but would allow different renames in different subvaults.
TODO: "line of fire blocked by XXX".
|
|
|
|
|
|
|
|
|
|
|
|
| |
actor::constricting is now a pointer to a map from mids to durations;
if the actor is not constricting anything, we use a NULL pointer rather
than an empty map to save memory.
Save compatibility: Because constrictees might be loaded after
constrictors (and vice versa), we cannot convert the old constriction
arrays (which use mindex) until all the monsters have been loaded.
Instead, save the constriction data temporarily, and create the maps
at the end of tag_read_level_monsters().
|
|
|
|
|
|
|
|
|
|
| |
dur_been_constricted was used only for a single debugging message.
Remove it and the associated accum_been_constricted(). Do not yet
change the save file format, as more significant changes that will
require a minor version bump are coming soon
Also move accum_has_constricted() from player and monster to actor; the
code was the same, modulo some bugs in the monster version.
|
| |
|
|
|
|
|
| |
It's a big header, hardly ever needed, yet it was pulled even for low-level
files that don't deal with the game itself like lib${UI}.cc
|
|
|
|
|
|
|
|
|
| |
These accumulate but never get removed; no wonder compilation times keep
rising.
The includes.sh script has lots of false negatives (and positives...), and
can't check .h files which cause the biggest slowdown, it'd be nice to run
multidelta on those somehow.
|
| |
|
|
|
|
| |
Better than hijacking debug_prompt_for_skill.
|
|
|
|
| |
by name and changes skill levels and base stats to the morgue file's levels
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
It's still a terrible mess. I didn't manage to understand it well enough
so all[1] functionality is preserved, at the cost of clarity. I unified
some values that are functionally identical, but not yet all.
[1]. The context is reported only as a number in crash dumps, a minor
regression but 1. saves speed, 2. it'd be a matter of copying the enum
names if someone actually needs this.
|
|
|
|
|
| |
There's no point in ever having gaps in this enum -- even if a skill is
removed, it is still valid until a save compat break.
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
crawl-ref/source/fight.cc
crawl-ref/source/food.cc
crawl-ref/source/libutil.cc
crawl-ref/source/melee_attack.h
crawl-ref/source/mon-stuff.cc
crawl-ref/source/spl-damage.cc
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
crawl-ref/source/Makefile.obj
crawl-ref/source/dbg-scan.cc
crawl-ref/source/decks.cc
crawl-ref/source/describe.cc
crawl-ref/source/directn.cc
crawl-ref/source/evoke.cc
crawl-ref/source/fight.cc
crawl-ref/source/fight.h
crawl-ref/source/ghost.cc
crawl-ref/source/ghost.h
crawl-ref/source/item_use.cc
crawl-ref/source/items.cc
crawl-ref/source/map_knowledge.cc
crawl-ref/source/melee_attack.h
crawl-ref/source/mon-abil.cc
crawl-ref/source/mon-act.cc
crawl-ref/source/mon-stuff.cc
crawl-ref/source/monster.cc
crawl-ref/source/monster.h
crawl-ref/source/player.cc
crawl-ref/source/player.h
crawl-ref/source/shopping.cc
crawl-ref/source/spl-damage.cc
crawl-ref/source/spl-summoning.cc
crawl-ref/source/spl-transloc.cc
crawl-ref/source/stairs.cc
crawl-ref/source/stuff.cc
crawl-ref/source/tags.cc
crawl-ref/source/traps.cc
crawl-ref/source/xom.cc
|
| |
| |
| |
| |
| |
| | |
Besides being non-standard and present only on some platforms, it cannot be
extended to Unicode as some characters expand or contract while being
upper/lowercased.
|
| |
| |
| |
| |
| | |
On some STL structures size() is slow. I did not bother limitting sed,
so innocent bystanders got pulled into the machine, but there's no loss.
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
crawl-ref/source/actor.cc
crawl-ref/source/delay.cc
crawl-ref/source/directn.cc
crawl-ref/source/directn.h
crawl-ref/source/fight.cc
crawl-ref/source/files.cc
crawl-ref/source/mon-act.cc
crawl-ref/source/monster.cc
crawl-ref/source/mpr.h
crawl-ref/source/player.cc
crawl-ref/source/shopping.cc
|
| |
| |
| |
| |
| |
| |
| | |
"File:" is shown in your editor's status bar.
"Written by:" was used only for the first person who changed a file. We got
git for that now, and pre-DCSS history is so woefully inaccurate it doesn't
really matter.
|
|/
|
|
|
|
| |
It's an ugly hack currently, to 1. not make the combat merge even nastier
than it already will be, 2. not do the work twice when we'll probably
overhaul mprf().
|
|
|
|
|
|
| |
env.properties[LEVEL_EXTRAS_KEY] is now only used for extra vaults, which are
saved into a hash table rather than a vector. Modify dbg-util.cc accordingly so
it dumps out the right data.
|
|
|
|
|
| |
This lets one layout specify multiple layout types, which can be useful in
combination with layout_* vault tags. Nothing uses this as of yet.
|
|
|
|
| |
I decided to reuse debug_prompt_for_int() for this purpose.
|
|
|
|
|
|
|
| |
I use a similar include, but there's no reason to not have this properly
as a part of the project -- as others can benefit from it as well.
Something like this is needed for times such as debugging the save browser
or the text display itself.
|
| |
|
|
|
|
|
|
| |
I did review it manually to find places where they made sense (like some
tables), but for a massive sed job like this there might be places that
I missed.
|