From 7dcda40b1ac8642e7d379bede367b4919b69e08d Mon Sep 17 00:00:00 2001 From: jomo Date: Sat, 21 Jun 2014 12:47:25 +0200 Subject: only using recipient list when iterating --- mentio.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'mentio.py') diff --git a/mentio.py b/mentio.py index 1d29d57..bda1e6d 100644 --- a/mentio.py +++ b/mentio.py @@ -13,9 +13,9 @@ def onChat(event): if not event.isCancelled(): sender = event.getPlayer() words = event.getMessage().split(" ") - recipients = list(event.getRecipients()) + recipients = event.getRecipients() - for recipient in recipients[:]: + for recipient in recipients.tolist(): rec_words = words[:] # copy for i in range(len(rec_words)): word = rec_words[i] @@ -25,9 +25,10 @@ def onChat(event): # player was mentioned if rec_words != words: - try: # list might not be mutable + try: recipients.remove(recipient) # don't send original message except: + # list might not be mutable, ignoring. Receiver will get the message twice pass message = " ".join([sender.getDisplayName(), arrow] + rec_words) msg(recipient, message, usecolor = False) -- cgit v1.2.3