ack! bad :)

> sprintf(buf, "[Announcement by $n] Player %s and Player %s have done
> something", victim1->name, victim2->name);
> 
> act_new(buf, ch, NULL, NULL, TO_ALL, POS_DEAD);
> 
> Other than that, I'm not sure what you're trying to do...

You should never dereference '->name' directly unless it's an immortal.
If you do, your players lose the benefits of invisibility and blindness.
If you have to refer to a player, and act won't cut it, use PERS.

----------------------
sprintf(buf, "[Announcement] Player %s and Player %s have done
something", PERS(victim, ch), PERS(victim2, ch));
 
act_new(buf, ch, NULL, NULL, TO_ALL, POS_DEAD);
------------------------ 

I know it was just sample code, but implies bad practice :)

Jason, who has a 60 line PERS() function.

Reply via email to