| 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!
|
| |
|
|
|
|
|
|
|
|
|
| |
The crash was trivially triggerable by any user by calling
crawl.mpr("</red>") from Lua. It could also trigger, for example, when
printing an error message about Lua code that itself contains a colour
tag.
Now display the unbalanced tag in lightred so it stands out.
|
|
|
|
|
| |
Sometimes, they're there to emphasize a break between two sections of code,
which is good. In a majority of cases, though, they're just inconsistent.
|
| |
|
| |
|
|
|
|
|
| |
Can't test Android, MSVC or Mac, but a very brief glance at the diff suggests
it's unlikely they're affected.
|
|
|
|
|
| |
The perl one-liner I use for this had a bug where it didn't match "else"
at the end of a line (ie, most of the time).
|
|
|
|
| |
Hard to come up with something less greppable...
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
|
| |
This avoids unnecessary non-inlined copies.
Also, we had a few large functions that had no reason for inlining, let's
have them be regular ones.
I also made "static inline" always use the same order, for easier grepping.
|
|
|
|
|
| |
This is incomplete, partially because of me getting bored, partially because
of doubts about the point of leaving simple addition/etc in parentheses.
|
| |
|
| |
|
| |
|
|
|
|
| |
I intend to name actual per-language C++ files lang-en.cc, lang-pl.cc
|
| |
|
|
|
|
|
| |
Looks like get_XXX_path and the like in files.cc could use some drastic
simplification, or perhaps even a nuking and rewrite.
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| | |
Having different prototypes for different ports without a good reason is bad.
After unification, it's easier to have, for example, two ports at once.
|
|\|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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
|
| |
| |
| |
| |
| | |
It was used on user-supplied string (inscribed items), so non-ASCII can happen
in 0.8.
|
| |
| |
| |
| | |
Used by, among others, message history, so user-supplied strings can happen.
|
| | |
|
| |
| |
| |
| |
| | |
tagged_string_tag_length() is used just once, too -- duplicating code that's
done in different ways elsewhere. A candidate for clean-up.
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
Puts a stop-gap measure to the frequent assertions for Elyvilonists.
The text still needs shortening, as it's simply cut off when the last
line is reached (which is then overwritten with the ! toggle
explanation, anyway).
|
|/
|
|
|
| |
It's going to be the default for mprf() since 99% uses are capitalized, but
it's safer to do this piecemeal then sed it back.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
|
|
| |
* single-use only
* causes corona-like glow
* makes all messages ALL CAPS
* does what it says on the tin to all damage you deal
* cause gibs if it brings someone to less than -50hp
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
We were only setting it to "\r\n" on DOS, but according to commens
in #1045, DOS will translate "\n" to "\r\n" automatically for
text output.
Also, the DOS EOL handling was likely broken anyway looking at how
a lot of the linebreaking code only considered "\n".
Fixes issue #1045.
|
|
|
|
|
|
|
|
| |
It's not used anymore, and a lot of code doesn't even deal with it
properly.
Also fix what seemed to be a bug in FSOP_COLOUR handling in
formatted_string::substr.
|
| |
|
|
|
|
|
|
|
|
| |
The latter invokes parse_string_to_multiple and displays directly.
This fixes line wrapping issues with full lines: on an 80 column
terminal, an 80 character line could cause the cursor to wrap
to the next line, causing extra blank lines due to the manual
line feed in formatted_string::parse_block.
|
|
|
|
|
| |
(you don't need to cast an X* to a void* and you don't need to cast
arguments to math.h functions such as sqrt.)
|
| |
|
|
|
|
|
|
|
| |
Also extract unwind_var template to unwind.h. The latter is now
included from AppHdr.h, though it needn't really be.
This means it's now possible to use coord_def in libutil.h.
|
|
|
|
| |
You can't pass a va_list to a variadic function.
|
|
|
|
|
|
| |
immediately cast
to std::string.
|
| |
|
| |
|
|
|
|
| |
Also don't use get_symbol.
|
|
|
|
| |
Also add a few previously indirect includes.
|