| 
     
      
      
      From: Lawrence M. <we...@gm...> - 2004-05-03 22:32:10
      
     
   | 
Jeremy Maitin-Shepard wrote:
[...]
> It appears this is a basically a non-invasive, toned down
> version of the channel info buffers.  I know replacing channel
> info buffers has been on the todo list for quite a while, and
> if it were expanded to include more of the functionality of
> channel info buffers, it would certainly justify ripping out
> all of the channel info buffer code from erc.el.
> A few comments:
>  - `loop' is really nasty; it makes me cringe just to look at it; are
>    you sure you want to use it? ;)
Well, /I/ quite like LOOP :).  It's certainly more succint than
the equivalent maphash:
(let ((nicks ()))
  (maphash (lambda (k v) (push k nicks))
           erc-channel-users) nicks)
Maybe that's just me :).
Anyway, I found erc-get-channel-nickname-list, which pretty much
does what I want.
[...]
> I also spotted what appear to be several (minor) bugs; I will send some
> fixes later.
I've committed this now (as erc-nicklist.el), feel free to fix
things and/or make improvements.  There's a TODO list in the
comments near the top of the file of things I felt need to be
done.  A few are probably difficult, if not impossible, to
achieve without some major work in making Emacs do clever things
to keep windows together.
I probably won't be able to do much work on stuff in the next
six weeks or so, since I have exams I really ought to be
learning for, but I'll try and answer queries if people have any.
-- 
Lawrence Mitchell <we...@gm...>
 |