> This patch fixes the compilation problem with m_kick.c. It also prevents 
> users from setting +x if they have not authed with a service (ie have 
> FLAGS_ACCOUNT set). This is to stop people setting +x and thinking they 
> are now host hidden (ie they dont check with /whois), when they actually 
> won't be until they auth. Umode +x now means 'this host is hidden' rather 
> than 'this host may be hidden'.

Because the file is MIME-encoded, I can't easily quote it.  I have two
comments--first is, please don't bundle different changes like this.  We
may wish to accept one part and reject another--bundling them makes more
work for us.  Second is that there's a better fix for the m_kick--don't
add = and (), just move the ! to where it's supposed to be...and add
whitespace around the relation.

I'd also like to take the opportunity to remind people that I'd prefer
if you could attach the patches to reports in the trackers--I've been
known to lose patches in my mailbox.
-- 
Kevin L. Mitchell <[EMAIL PROTECTED]>

Reply via email to