[CF-Devel] Arrow, thrown object and generic attack fixes

[Jan Echternach]

On Tue, Oct 31, 2000 at 02:16:05PM -0800, Peter Mardahl wrote:
>
     
      Speaking of runes, THEY can attack a player in a map while they're in an 
     
     >
     
      inventory, i.e., removed.  Would your patch break this?
     
     
It should be possible that the attacker is in the victim's inventory.
This is a "simple attack" (simple_attack == 1 in attack_ob() and
hit_player()) and it should prevent all things that don't make sense in
this situation, e.g. calculating if the victim can see the attacker, or
triggering runes in the victim (I think I'll test if a door that is not
immune to poison can be poisoned -- the poison shouldn't trigger runes
in that door anymore).

However, I haven't tested yet if this also works for runes.  I'm
wondering if runes in chests that a player has picked up still work
properly, as they're not directly in the players inventory
(rune->env != player, but rune->env->env == player).

>
     
      Should we fix it by inserting the trap in the map with the thing
     
     >
     
      it is attacking?  Shouldn't be difficult to do that.
     
     
Inserting the rune into the victim's inventory (if it was in a
container in the victim's inventory before) should also work.

-- 
Jan