Remove some (hopefully) useless behavior code

This block of code didn't actually seem to do what it claims it does
(since it doesn't set new foe properly and ends up getting reset before
the monster can even take a single action with the new values), and
it seems likely it would be bad if it actually did (allied monsters
randomly wandering off from the player for no reason?).

Behavioral code is convoluted enough as it is, without more bits of
it being maintained when they don't even do anything. (I hope.)

1 files changed, 1 insertions, 10 deletions

diff --git a/crawl-ref/source/mon-behv.cc b/crawl-ref/source/mon-behv.cc
index fda0306657..8910ef0c1f 100644
--- a/crawl-ref/source/mon-behv.cc
+++ b/crawl-ref/source/mon-behv.cc
@@ -848,16 +848,7 @@ void handle_behaviour(monster* mon)
                 // (or wasn't even attempted) and we need to set our target
                 // the traditional way.
 
-                // Sometimes, your friends will wander a bit.
-                if (isFriendly && !mons_is_avatar(mon->type)
-                    && one_chance_in(8))
-                {
-                    set_random_target(mon);
-                    mon->foe = MHITNOT;
-                    new_beh  = BEH_WANDER;
-                }
-                else
-                    mon->target = PLAYER_POS;
+                mon->target = PLAYER_POS;
             }
             else
             {