Ingo Oeser <[EMAIL PROTECTED]> wrote: > > What about using a brace more: > >>diff --git a/include/linux/netfilter.h b/include/linux/netfilter.h >>--- a/include/linux/netfilter.h >>+++ b/include/linux/netfilter.h >>@@ -29,7 +29,7 @@ >> #define NF_VERDICT_QMASK 0xffff0000 >> #define NF_VERDICT_QBITS 16 >> >>-#define NF_QUEUE_NR(x) ((x << NF_VERDICT_QBITS) & NF_VERDICT_QMASK || >>NF_QUEUE) >>+#define NF_QUEUE_NR(x) ((x << NF_VERDICT_QBITS) & NF_VERDICT_QMASK | >>NF_QUEUE) > > #define NF_QUEUE_NR(x) ((x << NF_VERDICT_QBITS) & (NF_VERDICT_QMASK | > NF_QUEUE))
Actually, these two versions are different. & is higher than | so Harald's version is actually #define NF_QUEUE_NR(x) (((x << NF_VERDICT_QBITS) & NF_VERDICT_QMASK) | NF_QUEUE) Which is it? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
