irclib: Apply changes to supybot.protocols.irc.maxHistoryLength immediately.

locales/de.po

@@ -955,8 +955,7 @@ msgstr ""
 #: ../src/conf.py:995
 msgid ""
 "Determines how many old messages the bot will\n"
-"    keep around in its history.  Changing this variable will not take effect\n"
-"    until the bot is restarted."
+"    keep around in its history."
 msgstr ""
 
 #: ../src/conf.py:1000

locales/fi.po

@@ -1503,13 +1503,9 @@ msgstr ""
 #: src/conf.py:1019
 msgid ""
 "Determines how many old messages the bot will\n"
-"    keep around in its history.  Changing this variable will not take "
-"effect\n"
-"    until the bot is restarted."
+"    keep around in its history."
 msgstr ""
-"Määrittää kuinka vanhoja viestejä botti pitää historiassaan. Tämän "
-"asetusarvon\n"
-" muuttaminen ei vaikuta ennen uudelleenkäynnistystä."
+"Määrittää kuinka vanhoja viestejä botti pitää historiassaan."
 
 #: src/conf.py:1024
 #, fuzzy

locales/fr.po

@@ -992,9 +992,8 @@ msgstr "Détermine quelle vhost le bot bindera    avant de se connecter au serve
 #: ../src/conf.py:1019
 msgid ""
 "Determines how many old messages the bot will\n"
-"    keep around in its history.  Changing this variable will not take effect\n"
-"    until the bot is restarted."
-msgstr "Détermine combien de vieux messages le bot    gardera dans son historique. Changer cette variable ne prend effet qu'après    avoir redémarré le bot."
+"    keep around in its history."
+msgstr "Détermine combien de vieux messages le bot    gardera dans son historique."
 
 #: ../src/conf.py:1024
 msgid ""

locales/it.po

@@ -1183,11 +1183,9 @@ msgstr ""
 #: src/conf.py:1019
 msgid ""
 "Determines how many old messages the bot will\n"
-"    keep around in its history.  Changing this variable will not take effect\n"
-"    until the bot is restarted."
+"    keep around in its history."
 msgstr ""
-"Determina quanti messaggi vecchi terrà il bot nella cronologia. La modifica\n"
-"    di questa variabile non avrà effetto finché non si riavvia il bot."
+"Determina quanti messaggi vecchi terrà il bot nella cronologia."
 
 #: src/conf.py:1024
 msgid ""

locales/messages.pot

@@ -982,8 +982,7 @@ msgstr ""
 #: src/conf.py:1019
 msgid ""
 "Determines how many old messages the bot will\n"
-"    keep around in its history.  Changing this variable will not take effect\n"
-"    until the bot is restarted."
+"    keep around in its history."
 msgstr ""
 
 #: src/conf.py:1024

src/conf.py

@@ -1038,8 +1038,7 @@ registerGlobalValue(supybot.protocols.irc, 'vhost',
 
 registerGlobalValue(supybot.protocols.irc, 'maxHistoryLength',
     registry.Integer(1000, _("""Determines how many old messages the bot will
-    keep around in its history.  Changing this variable will not take effect
-    until the bot is restarted.""")))
+    keep around in its history.""")))
 
 registerGlobalValue(supybot.protocols.irc, 'throttleTime',
     registry.Float(1.0, _("""A floating point number of seconds to throttle

src/irclib.py

@@ -346,6 +346,9 @@ class IrcState(IrcCommandDispatcher):
                  nicksToHostmasks=None, channels=None):
         if history is None:
             history = RingBuffer(conf.supybot.protocols.irc.maxHistoryLength())
+            def resize_history():
+                self.history.resize(conf.supybot.protocols.irc.maxHistoryLength())
+            conf.supybot.protocols.irc.maxHistoryLength.addCallback(resize_history)
         if supported is None:
             supported = utils.InsensitivePreservingDict()
         if nicksToHostmasks is None:

test/test_irclib.py

@@ -36,10 +36,8 @@ import supybot.conf as conf
 import supybot.irclib as irclib
 import supybot.ircmsgs as ircmsgs
 
-# The test framework used to provide these, but not it doesn't.  We'll add
-# messages to as we find bugs (if indeed we find bugs).
-msgs = []
-rawmsgs = []
+rawmsgs = 'foo bar baz qux quux corge grault garply waldo fred'.split()
+msgs = map(lambda x:ircmsgs.privmsg('#channel', x), rawmsgs)
 
 class IrcMsgQueueTestCase(SupyTestCase):
     mode = ircmsgs.op('#foo', 'jemfinch')
@@ -258,8 +256,7 @@ class IrcStateTestCase(SupyTestCase):
         self.failUnless(st.channels['#foo'].isOp('baz'))
 
     def testHistory(self):
-        if len(msgs) < 10:
-            return
+        assert len(msgs) >= 10
         maxHistoryLength = conf.supybot.protocols.irc.maxHistoryLength
         with maxHistoryLength.context(10):
             state = irclib.IrcState()
@@ -272,6 +269,10 @@ class IrcStateTestCase(SupyTestCase):
             self.assertEqual(len(state.history), maxHistoryLength())
             self.assertEqual(list(state.history),
                              msgs[len(msgs) - maxHistoryLength():])
+            with maxHistoryLength.context(5):
+                self.assertEqual(len(state.history), maxHistoryLength())
+                self.assertEqual(list(state.history),
+                                 msgs[len(msgs) - maxHistoryLength():])
 
     def testWasteland005(self):
         state = irclib.IrcState()