Hi,
Beirc as it is right now uses a really ugly re-implementation of the
read-message method, because in order to operate correctly, we can't
just use an :around method on read-message like this:
(defmethod read-message ((con my-irc-connection))
(let ((m (call-next-method)))
(dispatch-on-message m)
m))
Why? Because (in the case of a QUIT message) read-message modified the
channel user table already when beirc tries to find out which channels
a user who just quit was on.
So, I think we're specializing a method one level too high. We should
be specializing the irc-message-event method, but it has no connection
argument. Yet.
The attached patch adds a connection argument on which
irc-message-event can be specialized. I would also suggest exporting
irc-message-event and making that method the recommended way to
dispatch on all messages on an irc connection.
I've used this patch with a locally hacked version of beirc for a few
days now and it works very well for me. I hope you all like it,
because I'd really like to see that ugly wart in beirc go away. (-:
--
Andreas Fuchs, (http://|im:asf@|mailto:asf@)boinkor.net, antifuchs