> -----Original Message----- > From: Kev [mailto:[EMAIL PROTECTED]] > Sent: Tuesday, March 19, 2002 5:16 PM > To: LordLuke > Cc: [EMAIL PROTECTED] > Subject: Re: [Patches] LIST_CHAN > > > > diff -b -u -d -r1.25.2.2 channel.h > > --- include/channel.h 2002/01/28 21:06:16 1.25.2.2 > > +++ include/channel.h 2002/03/19 01:53:12 > > @@ -100,7 +100,7 @@ > > /* channel not shown but names are */ > > #define HiddenChannel(x) ((x) && ((x)->mode.mode & = > > MODE_PRIVATE)) > > /* channel visible */ > > -#define ShowChannel(v,c) (PubChannel(c) || = > > find_channel_member((v),(c))) > > +#define ShowChannel(v,c) (PubChannel(c) || = > > find_channel_member((v),(c)) || (IsAnOper(v)) && HasPriv(v, = > > PRIV_LIST_CHAN)) > > (ugh, quoted printable) > > You have an ambiguity here; I think you meant to have the > second ')' after > the IsAnOper() test to after the HasPriv() test. Otherwise,
right > it looks OK, > though I'd suggest making it possible to turn it off for > local operators. > -- Will add it tonight LL
