| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Broken by cb4b6e1; the stash title is now unwrapped again in Webtiles.
|
| |
|
|
|
|
|
|
|
| |
.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!
|
| |
|
|
|
|
|
| |
If there is no state, function objects are usually just extra verbosity
compared to a function pointer.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
If two corpses were sitting on a visible square, ctrl-x would never
show the rotting of the visible top corpse; and the stash tracker
itself would show it as "rotten by now" even though we can actually
see the corpse rotting. This also affects recharging rods since
the previous commit.
Now if the seen item and the remembered top item are "the same"
but not identical (because of rotting, recharging, etc.), we update
the stash anyway.
|
| |
|
|
|
|
|
|
| |
Somebody else might want to review my vault changes - I mostly replaced
them with honeycombs, which might be a problem, for example, in the
Jiyva vaults.
|
|
|
|
| |
This fixes all the instances caught by unbrace.
|
|
|
|
|
|
|
|
|
| |
Currently, if the player searches for "shop", the stash screen shows
a long list of items in shops, and after Orc there will be a great
deal of junk on the screen. This commit checks whether a search
only returns shops, and nothing on the floor or in shops ("shop" for
example, or "gadget"). If so, it displays the shops in one stack
by default.
|
|
|
|
| |
Broken by 0.14-a0-2083-gb79a34a - also fix the spacing to align left.
|
|
|
|
|
|
| |
It was mostly just causing confusion when the only matching items
were useless, and it was rather annoying when items that were only
temporarily useless were being searched for.
|
|
|
|
|
|
| |
'Relevant base types' are wands, staves, scrolls, potions, rings, amulets.
(Various tweaks and cleanups by Medar)
|
|
|
|
|
|
|
|
|
| |
show_as_stacks used to hide some items behind rather unintuitive parens,
and this has come up as actual problem more than once.
filter_useless on the other hand used to spam you with many mostly
irrelevant items and similarly suffered from the weird help line.
The new settings are: show_as_stacks = false, filter_useless = true.
|
|
|
|
|
|
|
|
|
| |
The old one did really not help any player whatsoever, I tested with
several new ones. Nobody was able to find out what it tried to convey.
Hopefully this layout gets the important points across better now.
What should also be changed are some of the defaults themselves. This
might be addressed in a subsequent commit.
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
That is, unless you count the ones in ncurses' headers, which I can't
do anything except suppress.
|
|
|
|
| |
Has been gone since aa300c, 6 years ago.
|
|
|
|
| |
Previously it was simply broken.
|
| |
|
|
|
|
|
|
|
|
|
| |
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.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
This also fixes a bunch of "act through glass" or "doesn't obey nightstalker/
lantern of shadows" bugs.
|
|
|
|
|
| |
Can't test Android, MSVC or Mac, but a very brief glance at the diff suggests
it's unlikely they're affected.
|
| |
|
|
|
|
|
|
|
|
| |
This was supposed to match singulars when the item was plural, but
scrolls and potions did not include their subtype in DESC_BASENAME.
This still won't let you search for "arrow of flame", since
DESC_QUALNAME does not include brands.
|
| |
|
|
|
|
| |
It should now fit into 80 and even 79 columns with up to 999 items.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Pull 'you.religion [!=]= FOO' checks into a function: you_worship(FOO).
This change is part of a large plan to clean up religion.
|
| |
|
|
|
|
|
|
|
| |
Convert conjunctive assertions into separate assertions. This ought to be correctness preserving. I ran the stress tests and didn't notice anything unusual. While I have confidence in it, if you are the slightest bit suspicious of this, please roll it back.
Found instances with `ASSERT(\([^(|]*\) && \([^)|]*\))`
Manually inspected each instance.
|
|
|
|
|
| |
This is for identified items in shops that are not yet identified by the
player. Was already shown in actual shop menu.
|
| |
|
| |
|
|
|
|
| |
Search for "shop" and press - to quickly see all available goods on the market.
|
|
|
|
|
|
|
| |
Since there are only two actions (travel and examine), display the name
of the inactive action rather than just the word "action". This should
provide a bit more of an on-screen hint as to how to view shop contents
(suggested by raistware).
|
|
|
|
|
| |
The only reason for this madness was so searching for "ident" doesn't
find identified items.
|
| |
|
| |
|
| |
|