Re: [spells] Stock Teleport problem

From: Daniel Koepke (dkoepke@california.com)
Date: 01/05/97


On Sun, 5 Jan 1997, Robert wrote:

>   if (victim != NULL)
>     return;

Well, look at it.  This check makes sure victim is NULL before
proceeding (if it's not NULL, it returns).  Then, it goes on,
and uses 'victim' later:

>   act("$n slowly fades out of existance and is gone",
>       FALSE, victim, 0, 0, TO_ROOM);
>   char_from_room(victim);
>   char_to_room(victim, to_room);
>   act("$n slowly fades into existance.", FALSE, victim, 0, 0, TO_ROOM);
>   look_at_room(victim, 0);

I'm not quite certain, but I believe that even in spells flagged
as TAR_SELF_ONLY, a 'victim' is passed (as 'ch').  So you should
change the 'if' check to:

  if (victim != ch)
    return;

Yes, yes, I'm pretty sure that's it...


--
Daniel Koepke
dkoepke@california.com
Forgive me father, for I am sin.


+-----------------------------------------------------------+
| Ensure that you have read the CircleMUD Mailing List FAQ: |
|   http://cspo.queensu.ca/~fletcher/Circle/list_faq.html   |
+-----------------------------------------------------------+



This archive was generated by hypermail 2b30 : 12/18/00 PST