diff options
| author | Rusty Russell <rusty@rustcorp.com.au> | 2002-05-21 22:23:11 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@penguin.transmeta.com> | 2002-05-21 22:23:11 -0700 |
| commit | 190cd5e56e8416f60e158ac1ad884d5e13cdf5f1 (patch) | |
| tree | 3fcb18b7d9060f6cad382bc26f3f067d1ba4f4b6 /include/linux | |
| parent | aa2364b8080994fa14dc50662052045f03024513 (diff) | |
[PATCH] min/max elimination in netfilter.h
Rusty Russell <rusty@rustcorp.com.au>: Trivial patch to remove minmax macros:
Hi Harald,
The MIN & MAX etc. macros aren't used anywhere, and kernel.h
has the new min & max anyway, which should be used.
Cheers,
Rusty.
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/netfilter.h | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/include/linux/netfilter.h b/include/linux/netfilter.h index 46c2b72e33f3..5899abd26a44 100644 --- a/include/linux/netfilter.h +++ b/include/linux/netfilter.h @@ -158,26 +158,5 @@ extern void nf_invalidate_cache(int pf); #define NF_HOOK(pf, hook, skb, indev, outdev, okfn) (okfn)(skb) #endif /*CONFIG_NETFILTER*/ -/* From arch/i386/kernel/smp.c: - * - * Why isn't this somewhere standard ?? - * - * Maybe because this procedure is horribly buggy, and does - * not deserve to live. Think about signedness issues for five - * seconds to see why. - Linus - */ - -/* Two signed, return a signed. */ -#define SMAX(a,b) ((ssize_t)(a)>(ssize_t)(b) ? (ssize_t)(a) : (ssize_t)(b)) -#define SMIN(a,b) ((ssize_t)(a)<(ssize_t)(b) ? (ssize_t)(a) : (ssize_t)(b)) - -/* Two unsigned, return an unsigned. */ -#define UMAX(a,b) ((size_t)(a)>(size_t)(b) ? (size_t)(a) : (size_t)(b)) -#define UMIN(a,b) ((size_t)(a)<(size_t)(b) ? (size_t)(a) : (size_t)(b)) - -/* Two unsigned, return a signed. */ -#define SUMAX(a,b) ((size_t)(a)>(size_t)(b) ? (ssize_t)(a) : (ssize_t)(b)) -#define SUMIN(a,b) ((size_t)(a)<(size_t)(b) ? (ssize_t)(a) : (ssize_t)(b)) #endif /*__KERNEL__*/ - #endif /*__LINUX_NETFILTER_H*/ |
