[Erc-discuss] /list produces incomplete output: analysis and workaround

[Sascha Wilde]

Hi *,

I discovered, that erc /list does not display all channels the server
announces: it turns out that visited and even channels already parted
are omitted.

The reason is, that the function to display the channels is added as
hook to erc-server-322-functions after erc-server-322.  But
erc-server-322 returns a non-NIL value when erc-update-channel-topic
succeeds on a channel -- which in turn leads to the following hook
functions not being called.

I'm not deep enough into the code to call it a solution -- but the
following simple workaround (which simply makes sure that erc-server-322
always returns NIL) seems to work:

--- erc-backend.el           2012-05-14    03:00:11.666556805 +0200
+++ erc-backend.el_patched   2012-05-15    17:48:13.064059897 +0200
@@ -1696,7 +1696,8 @@
     (multiple-value-bind (channel num-users)
         (values-list (cdr (erc-response.command-args parsed)))
       (add-to-list 'erc-channel-list (list channel))
-      (erc-update-channel-topic channel topic))))
+      (erc-update-channel-topic channel topic)))
+  nil)
 
 (defun erc-server-322-message (proc parsed)
   "Display a message for the 322 event."

cheers
sascha
-- 
Sascha Wilde
Well, *my* brain likes to think it's vastly more powerful than any
finite Turing machine but it hasn't proven that to me...
  -- Christopher Koppler in comp.lang.lisp