[cvslog] (2008-06-30 16:41:57 UTC) Module eggdrop1.6: Change committed!

cvslog cvs at tsss.org
Mon Jun 30 10:41:57 CST 2008


CVSROOT    : /usr/local/cvsroot
Module     : eggdrop1.6
Commit time: 2008-06-30 16:41:57 UTC
Commited by: Tothwolf <tothwolf at techmonkeys.org>

Modified files:
     doc/UPDATES1.6 src/mod/irc.mod/mode.c

Log message:

set CHAN_PEND when calling refresh_who_chan() to avoid possibly sending extra WHO requests to the server if a WHO was already sent.
Found by: Tothwolf - Patch by: thommey

---------------------- diff included ----------------------
Index: eggdrop1.6/doc/UPDATES1.6
diff -u eggdrop1.6/doc/UPDATES1.6:1.656 eggdrop1.6/doc/UPDATES1.6:1.657
--- eggdrop1.6/doc/UPDATES1.6:1.656	Sun Jun 29 11:39:41 2008
+++ eggdrop1.6/doc/UPDATES1.6	Mon Jun 30 11:41:47 2008
@@ -1,4 +1,4 @@
-$Id: UPDATES1.6,v 1.656 2008-06-29 16:39:41 guppy Exp $
+$Id: UPDATES1.6,v 1.657 2008-06-30 16:41:47 tothwolf Exp $
 
 Eggdrop Changes (since v1.6.0)
     _____________________________________________________________________
@@ -12,6 +12,10 @@
 
 
   1.6.20:
+    - set CHAN_PEND when calling refresh_who_chan() to avoid possibly sending
+      extra WHO requests to the server if a WHO was already sent.
+    * Found by: Tothwolf - Patch by: thommey
+
     - always send a "WHO chname" instead of sending a "WHO nick" which doesn't
       have the same behaviour across all irc daemons.
     - improved use_354 support as we weren't using it everywhere we could
Index: eggdrop1.6/src/mod/irc.mod/mode.c
diff -u eggdrop1.6/src/mod/irc.mod/mode.c:1.84 eggdrop1.6/src/mod/irc.mod/mode.c:1.85
--- eggdrop1.6/src/mod/irc.mod/mode.c:1.84	Sun Jun 29 11:39:42 2008
+++ eggdrop1.6/src/mod/irc.mod/mode.c	Mon Jun 30 11:41:47 2008
@@ -4,7 +4,7 @@
  *   channel mode changes and the bot's reaction to them
  *   setting and getting the current wanted channel modes
  *
- * $Id: mode.c,v 1.84 2008-06-29 16:39:42 guppy Exp $
+ * $Id: mode.c,v 1.85 2008-06-30 16:41:47 tothwolf Exp $
  */
 /*
  * Copyright (C) 1997 Robey Pointer
@@ -388,6 +388,7 @@
     if (channel_pending(chan))
       return;
     putlog(LOG_MISC, chan->dname, CHAN_BADCHANMODE, chan->dname, who);
+    chan->status |= CHAN_PEND;
     refresh_who_chan(chan->name);
     return;
   }
@@ -479,6 +480,7 @@
     if (channel_pending(chan))
       return;
     putlog(LOG_MISC, chan->dname, CHAN_BADCHANMODE, chan->dname, who);
+    chan->status |= CHAN_PEND;
     refresh_who_chan(chan->name);
     return;
   }
@@ -566,6 +568,7 @@
     if (channel_pending(chan))
       return;
     putlog(LOG_MISC, chan->dname, CHAN_BADCHANMODE, chan->dname, who);
+    chan->status |= CHAN_PEND;
     refresh_who_chan(chan->name);
     return;
   }
@@ -614,10 +617,12 @@
    * check to see if they have +v or +h
    */
   if (!(m->flags & (CHANVOICE | STOPWHO))) {
+    chan->status |= CHAN_PEND;
     refresh_who_chan(chan->name);
     m->flags |= STOPWHO;
   }
   if (!(m->flags & (CHANHALFOP | STOPWHO))) {
+    chan->status |= CHAN_PEND;
     refresh_who_chan(chan->name);
     m->flags |= STOPWHO;
   }
@@ -653,6 +658,7 @@
     if (channel_pending(chan))
       return;
     putlog(LOG_MISC, chan->dname, CHAN_BADCHANMODE, chan->dname, who);
+    chan->status |= CHAN_PEND;
     refresh_who_chan(chan->name);
     return;
   }
@@ -698,6 +704,7 @@
     putlog(LOG_MODES, chan->dname, "TS resync (%s): %s deopped by %s",
            chan->dname, who, from);
   if (!(m->flags & (CHANVOICE | STOPWHO))) {
+    chan->status |= CHAN_PEND;
     refresh_who_chan(chan->name);
     m->flags |= STOPWHO;
   }
@@ -1167,6 +1174,7 @@
             if (channel_pending(chan))
               break;
             putlog(LOG_MISC, chan->dname, CHAN_BADCHANMODE, chan->dname, op);
+            chan->status |= CHAN_PEND;
             refresh_who_chan(chan->name);
           } else {
             simple_sprintf(s, "%s!%s", m->nick, m->userhost);
----------------------- End of diff -----------------------



More information about the Changes mailing list