Revamp Item Destruction

Replace outright scroll/potion destruction with smoldering/freezing statuses.
Smoldering conveys a 50% chance of scroll destruction on *use*, while freezing
does the same to your potions.

Make Spectral Weapon damage sharing use a fineff.

This way the order of targets in an AoE shouldn't matter for the damage
sharing leaving you with at least 1hp.

Fixed issue with merging deferred_damage_fineff.

Merging now checks that both have the same state for attacker_effects.

Halve Spectral Weapon damage sharing.

Fixed some effects being applied twice to damage.

When a deferred_damage_fineff is used, ::hurt is called twice.  Once for
the original target, and a second time for the shared target.  Both
calls would apply effects such as weakening from wretched and QUAD
DAMAGE.  While SIXTEEN TIMES damage is funny, the attacker's effects
should only be applied once.

Now ouch/hurt functions and the deferred_damage_fineff have an
'attacker_effects' parameter, which defaults to true.  If false, then
damage effects from the attacker are assumed to have already been
applied.  All existing uses of deferred_damage_fineff have been
appropriately updated.  Also applied to spectral weapon damage sharing.

Modify Spectral Weapon damage sharing.

Moved the code from monster::hurt into monster::react_to_damage.  This
has the effect of using the actual damage received, after such things as
capping at the weapon's current health and applying any wretched status
the attacker may have.

Additionally, less properties of the attack are passed to the owner.
How the caller intended to use the weapon (cleanup_dead) and the attack
flavour could've only caused bugs or misleading effects.

Buff Spectral Weapon defenses (AC, EV)

Instead of simply having 2-10AC, 2-10EV linearly over the range of 100
power, it now has 2-12AC, 10-20EV over that range.

In particular, giving it hopefully reasonable EV should make it take
less hits to share with the player.  These numbers are probably still
not enough, but it should be a start.

Also, now it's hit dice are bounded below by 1.  A monster with HD0 has
a number of special cases, such as dying to any hit.  Some of these
might be appropriate for a no-weapon-skill spectral weapon, but I don't
want to figure out all the other side effects it might have.

Buff Spectral Weapon (Gain immunity to allied beams)

Like battlesphere, one can now throw/cast/evoke past allied spectral
weapons, as if they weren't there.

Buff Spectral Weapon damage (scales with weapon skill)

Spectral Weapon damage now scales with weapon skill similarly to how
player and player ghost damage does.  However, the effective skill is
reduced by spell power.

At 0 power, weapon skill is 1/3 as effective.
At max (100) power, weapon skill is fully as effective as on the player.
In between, the effect on skill scales linearly with power.

There's a table in the comments of the code, showing some values.

The numbers might need tweaking, since getting comparable damage at max
power is likely desirable.  However, encouraging players to train more
charms is a good thing.

Don't crash on ia=<up>

Eliminate an unused variable.

Change Polymoth behavior.

Remove same genus polymorph.

Introduce polymorph tougher: morph a monster into something with one more HD.
Track what the moth has polymorphed and veto high HD monsters that have
already been polymorphed to prevent runaway morphing.

Let the temporarily foodless still acquire food (Mantis #7283)

Adds an is_foodless_normally() call for other such uses.

Don't spam fail messages when checking form validity.

Any dry runs, including but not limited to the previous commit, would
produce "You would burn in your new form.", etc.

Roll Zin's protection against polymorph only once.

Pepper "you" and level chunks of save files with canaries.

They should make debugging marshall/unmarshall errors a good deal easier:
baring a lucky failed assert, subsequent reads will accept garbage until
the end of file.

Axe a stray comma.

Reformat some perl.

Mostly so I don't get spammed when forgetting to restrict a search to *.cc
*.h files only.  There's too much idiosyncratic formatting to force a
standard without thinking about it first.

We seem to have a bunch of unused scripts, perhaps some should get pruned
away?

Reformat perlin.cc

I left it aside in the past, but it stands in the way of automated checks.

Silver wall tile.

Clean up layout_forest

There were some WIP versions intended for different branches
which are pretty redundant now this layout has a proper home.
Stopped it spawning at all in D/Lair. Left in some still
unused code for a possibly more interesting Crypt version.

Rename layout_woods -> layout_forest for clarity

Spirit wolf tile

Revert "Demonic Taxonomy"

This reverts commit f2d916f53646c573646c5c00db4cb96d3c2cc20f.

Fix tiles rendering issue changing between levels

When in level map mode (X) items/monsters/clouds from another level
could show up in your LOS when switching back to your level.

Demonic Taxonomy

Give most demons a general genus, one of:
imp, devil, demon, fiend.

Fix the minibar bug on WebTiles too

Minibars showing on levels when player isn't there.

Fix minibars showing on levels without the player

Fixed for local tiles only.

Allow logging in while watching someone

Ask login if necessary when starting WebTiles game

Previous behaviour was redirecting to lobby if anonymous user navigated
to #play... URL directly.

Add blackcustard, elliott, qoala, infiniplex, and lainiw to CREDITS.txt.

If you want to be listed differently, let us know; real names or handles
are fine, but "Remove cTele MarvinPA" is not :P

Mark unknown scrolls as useless for hot lava orcs (SamB)

A desc for the book of battle.

Brace fixes.

Fix formatting.

Correct Grinder's genus.

This was a holdover from when crimson imps were less crimson, and all
imps shared a genus.

Add Polymoth to the Lair

Disable Polymoth Downgrades

Downgrades pose a problem with monsters that are at the peak of their genus.

Further restrict polymoth's polymorph.

Introduce a level restriction on polymoth.
-2 HD to +6 HD, with peak weight on +2 HD.
Don't polymorph things that the moth can't see, or things that are
out of the player's view.

Polymo{th,rph} tweaks.

1. Don't polymorph godless monsters into priests.
2. Add a flag to monsters that have been polymorphed.
3. Don't allow the polymoth to upgrade a monster more than once.

Note Flayed and Engulfed in the logfile

Show hp/mp minibars in WebTiles while transformed

Also remove two function calls that did nothing.

Clarify wording of changelog for summons cap

The nerf affects temporary summons; I did not mean to
imply that the nerfs were intended to be temporary.

RANDOM_SAME_GENUS polymorph fixes.

_valid_morph was silently redefining the monster it was checking for,
allowing uniques to slip through the cracks; the second _valid_morph
call in monster_polymorph was also losing valid RANDOM_SAME_GENUS
targets to this kind of check.

Polymorphs using this target should operate fine now.

Display spectral sacred scourges correctly.

Make spell list indentation more consistent.

Make Spectral Weapon Charms/Hexes, not Conj.

No more Vehumet support or gifting.

Allow vampires to Cure Poison whenever they have blood (#7261).

Still prevent it when they are bloodless and therefore poison-immune,
for consistency with liches.  There may be residual (grey) poison,
but Cure Poison won't touch it.

Don't crash when a monster instakills another with LRD.

Let monster_die take a const killer.

Fix display of HP costs (#7267).

Broken by 0.13-a0-548-g832c960.

Increase cap and timeout of demonic horde

Fix spelling (elliott)

Multiply before division, not after.

Revert "Fix compilation with old (4.1) GCC and libstdc++"

Since we have removed the ambiguous calls; and since there are other
toolchains where pow(int,int) is a problem (see #7265).

This reverts commit cae46e358690cf86f43a5743b5b048be0652072e.

Conflicts:
crawl-ref/source/skills.cc

Revert "Reduce giant mites' range a mite."

So here you go, you still have a chance to summon one at 10 Evoc, once in
a blue moon.

This reverts commit de8e8e0bc01e85ea6c1f5c3a9b9f243371f7bf1e.

Scale the calculations in mon-pick by 2520.

The algorithm was designed with values based at 10000 in mind so numbers
small enough to zero off were not a concern, they could be avoided by scaling
up the data in other users of that code, but it's probably better to get rid
of gotchas rather than to document them.

Reduce giant mites' range a mite.

They had so small a chance that with 10 skill their effective chance was 0
because of round-offs.  It's debatable what should be done; it was not an
issue for mon-pick because the scale there is based on 10000 so rounding
is not noticeable (and was present in the old code too).

Revert "Don't fail random pick on 0 rarity"

If this assertion triggers, the data is invalid (a monster with a chance of 0),
which is exactly what it was supposed to guard against.

This reverts commit 2b2fa887ae752b55d2232950b11acaf50caf1d26.

Write pow(x, 0.5) as sqrt(x), for readability.

Don't use floating point pow() for squaring an int.

It's put back into an int variable right after, so range is not an issue.

Leash Spectral Weapon to a range of 2, let it attack retreating targets.

Instead of only going after targets that could still be attacked by the
owner, it now chases after retreating/batty targets as long as it can
reach them while staying within 2 squares of its owner.

This makes it more likely to attack such targets, but also runs the risk
of putting itself (and thus the owner) at more risk.

On reset of a Spectral Weapon, remove the target if we're not tracking.

Prefix spectral weapon properties with "sw_" and use #define for them.

This allows easier understanding of what code uses the properties,
reduces chances of conflicts, and helps catch mispellings.

Make Spectral Weapon reset on player's next turn for real this time.

If the Spectral Weapon hasn't attacked by the player's next turn, it
gets reset.  This occurs either right before the next order is given to
the weapon, or (if the player did something else) after the player's
turn.

Unlike 04540f5, this reset mechanism doesn't prevent the spectral weapon
from ever attacking.

Don't fail random pick on 0 rarity

It can happen, seen at the lower bound of "DOWN" distribution
types, but can probably for others too.

Revert "Fix previously broken but moot polymorph validity check for M_UNIQUE"

This reverts commit 6e1df92eca0744543f6cbea42e50e11d4b57f691.

Fix a build failure on OS X.

Unlike Grandiloquent Gentleman (the bug report submitter)'s patch, let's
just use integer multiplication instead of mucking with pow() and casts.

std::cide.

Sync manual from the wiki.

Fix previously broken but moot polymorph validity check for M_UNIQUE

Normal random polymorph only worked on base classes anyway so this
was irrelevant; but since with RANDOM_SAME_GENUS we're actually working
with proper monster types, checking M_UNIQUE became important,
revealing this flaw.

Don't let random polymorph rarely produce unvalidated monster types

Prohibit polymothing firewood.

Revert "Make Spectral Weapon reset on player's next turn if it hasn't attacked."

It was preventing them from attacking at all.  qoala is working on a
better solution to the original problem ("saving up" a SW attack).

This reverts commit 04540f58001281626c71620fdd56ef4cf216de79.

Nerf Polymoth

Merge branch 'madness'

Merge branch 'madness'

Conflicts:
crawl-ref/source/enum.h

Reference the same mc variable for the valid check and the actual poly

Formatting fixes

Fix overflow in curse skull monster energy pickup_percent.

Curse skulls got an action energy of 15, meaning their pickup_percent
was 150, outside the range of the int8_t that stores it.  Since none
of the energy usage numbers should be negative, make that a uint8_t
instead.  Also change the other members to uint8_t so we don't need
duplicate overloaded versions of combine().

This wasn't actually relevant to curse skulls, since they had
MONUSE_NOTHING, but it was producing a compiler warning and more
importantly could turn up with some future monster.

Add a polymoth quote.

Moth of Madness -> Polymoth

Fix compilation with old (4.1) GCC and libstdc++

We use GCC version <= 4.1 as the criterion for the fix (which is the
same as the similar Solaris fix).  Probably we could use __GLIBCXX__
and/or __GLIBCPP__ to detect this better, but I haven't been able
to test that (in particular, old g++ with a different libstdc++).

This reverts commit 0e1e71dfe8e254a2d925fcb0a7ee6905c4175dd1.  We still
have to do some kind of casting for that particular case, though,
because the first argument is unsigned.

Adjust lost soul sacrifice behavior

Instead of always sacrificing themselves for the first valid ally
killed within range of them, lost souls will prefer to ignore the
deaths of low HD allies, so long as at least one other valid ally
is near them. If several lost souls are clustered together, they
will be more likely to sacrifice themselves for weaker things.

I am somewhat less sure of this commit than the last one, as I
wonder if it might be unintuitive for souls to sometimes trigger
and sometimes not, but the hope is that they will waste their
ability on junk like wights and flying skulls less often, where
its gameplay effect is virtually unnoticable.

Adjust lost soul ally teleport behavior

Instead of teleporting to a random revivable ally when solitary, they
will now prefer to teleport towards allies with high HD (randomized
somewhat). Moreover, they will consider this ally they teleported to
to be their band leader, so they will follow it around instead of possibly
wandering away again.

Hopefully this will cause them to show up in relevant spots more often
and spent less time running into the player while isolated, and disappearing
again.

Give vampire knights a 1-in-4 chance of coming with a pair of phantasmal warriors

New monster flag M_VIGILANT

This flag makes a monster always wake up when the player comes into
LoS and never naturally forget about them (though stairs and teleports
and such still work)

This is intended at the moment for curse skulls, to simulate their
previous behavior of generating awake, but also to keep them from
simply wandering off if you wait long enough while one is outside your
LoS. Possibly it could eventually make sense for other magical sentinels
(eg: golems, once golems are better)

Acquaint Curse Skulls with their inner mushroom

This commit experimentally gives curse skulls a variant on wandering
mushroom AI, making them no longer strictly stationary. Like mushrooms,
they cannot move while the player can see them, however they do not
simply close as close as possible while unseen, since this allows
players to trivially lure them around corners and enter into melee
range the first turn they come into LoS (and they're not very scary in
this case).

Instead, the curse skull will attempt to maintain a maximal distance
from whichever spot the player would need to move to approach them,
while maintaining LoS to this spot. In effect, when you round a corner,
they should be waiting at the end of the corridor instead of right next
to you.

This tracking behavior is player-centric, but since their movement
restrictions (as with mushrooms) are ALSO player-centric, I felt that was
fine. It does make them slightly omniscient when you are near them (and
they are awake and tracking you), but I figure that can be flavorfully
explained by the manner in which they are a 'curse' upon you.

They move at the same speed as flying skulls (15), but have normal action
speed. At the same time I have also greatly reduced their AC, and reduced
their hp somewhat as well. Now that players will be more forced to confront
them, I wanted to make it more reasonable for certain players to actually
kill them (as previously, the only good answer some characters had to them
was to ignore them)

Possibly they are too scary now, but I think it's worth seeing how they
play out this way, and the fact that they cannot approach you while resting
gives you a number of options for positioning them, even if you can't simply
ignore them either.

Don't give tengu reavers a chance for crossbows and longbows

Also correct vault wardens getting orcish equipment by default

Tengu reaver tweaks

Increase their HD, lower hp a little, increase damage of primary melee attack
but lower aux damage slightly, replace sense invis with see invis (sense is
supposed to be reserved for primarily non-visual detection). Remove emergency
blink, and add a second copy of a direct conjuration (which can trigger
battlesphere) to each book that had only one.

Correct a major timing issue with monster battlespheres

Since the change to how monster turns are ordered (being interleaved
based on speed/energy instead of consecutively based on index), monster
battlesphere are no longer nearly guaranteed to act after their caster.
This meant that, from the player's perspective, the battlesphere would
fire one turn AFTER the spell that triggered it. Worse, it would be
aimed at where the player was standing that turn, so it was possible
to dodge by moving.

I have addressed this by making monster battlesphere triggering reset
the battlesphere to a full turn's worth of energy, and then reinsert
it into the monster action queue. (This reinsertion is necessary since
it is quite likely it would not be otherwise examined until the following
turn, no matter how much energy it had.) I do not think that cases where
two copies of the same monster end up in the queue at once will have
any effect, as popped monsters with unsufficient or modified action
energy are simply ignored.

Correct an alarming message when waterstrike is cast on a monster out of LoS

Give Ogres previously forced to start with a short sword a club instead.

This covers OgHus and OgArs.

No slightly sane ogre is going to take a short blade with them as a
backup (or primary) weapon when going on an adventure; they're going to
take a big stick to hit things with, which is what they do best. OgAses
and OgEns are already intelligent enough to do this - they got clubs
instead of daggers (so do TrAses and TrEns; TrHus and TrArs don't get
weapons, so they're not covered by this change).

Slightly more controversially, I'm making this a +1 club instead of a +0
short sword. I don't think anyone will dispute that clubs are terrible
weapons; it's possibly mechanically equivalent to swap out clubs for
daggers and expect them to perform similarly (not for stabbing
purposes), but a club is going to perform noticeably worse than a short
sword. Really, I think it would make more sense for this to be another
weapon type, but I don't think whips or hammers (the maces classed
between clubs and, well, maces) make thematic sense (and hammers are
used essentially nowhere anyway).

Adjustments to layout_grid_shapes.

The individual areas can now have different shapes from each other, and
there are a few more shapes to choose from.

Pan gets the cavey version of the layout half the time.

Finally, the layout's enabled for Zot, where it gets a slightly
distorted version of the shapes.

Update Infusion description

It no longer has an effect with 0 MP.

Adjust Box of Beasts probabilities

Wargs/wolves needed swapping around and some higher-level monsters
have been made available at lower power levels.

Fix formatting and simplify (Spectral Weapon)

Don't cap Makhleb demons

Add spectral weapons to more shared special cases with dancing weapons.

They can no longer be disarmed or self-unwielded.  Nor should they have
a body.

Moved the setting of Spectral Weapon stats to ghost code, like dancing.

By initialising the weapon's stats using a ghost, more properties can be
easily set.  Also vault/wiz-mode placed spectral weapons now get
appropriate stats, as if the spell had been cast at max power and max
weapon skill.

Spectral weapons still shouldn't be placed directly in normal games.