> 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, it looks OK,
though I'd suggest making it possible to turn it off for local operators.
-- 
Kevin L. Mitchell <[EMAIL PROTECTED]>

Reply via email to