From 598e82e7ad9377138b5a899724083fc5d808a249 Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Mon, 11 Oct 2004 01:03:27 +0000 Subject: r2900: rusty pointed out to me that discard_const() can be done via a macro on systems that have the intptr_t type, and for systems that don't have it, they also almost certainly won't have -Wcast-qual, so we can use a void* cast. (This used to be commit 2132d38f9c5ba59825558d5ba084a514ebc2626b) --- source4/build/m4/rewrite.m4 | 1 + source4/include/includes.h | 24 ++++++++++++++++++++++-- source4/lib/util.c | 22 ---------------------- 3 files changed, 23 insertions(+), 24 deletions(-) (limited to 'source4') diff --git a/source4/build/m4/rewrite.m4 b/source4/build/m4/rewrite.m4 index ef75ba37a2..8d55a5b40a 100644 --- a/source4/build/m4/rewrite.m4 +++ b/source4/build/m4/rewrite.m4 @@ -275,6 +275,7 @@ AC_CHECK_TYPE(ino_t,unsigned) AC_CHECK_TYPE(loff_t,off_t) AC_CHECK_TYPE(offset_t,loff_t) AC_CHECK_TYPE(ssize_t, int) +AC_CHECK_TYPE(intptr_t, void*) ############################################ diff --git a/source4/include/includes.h b/source4/include/includes.h index 06e9b91347..94d15eff3f 100644 --- a/source4/include/includes.h +++ b/source4/include/includes.h @@ -1068,9 +1068,29 @@ time_t timegm(struct tm *tm); #include #endif -#define discard_const_p(type, ptr) ((type *)discard_const(ptr)) - #define TALLOC_ABORT(reason) smb_panic(reason) + +/* + this is a warning hack. The idea is to use this everywhere that we + get the "discarding const" warning from gcc. That doesn't actually + fix the problem of course, but it means that when we do get to + cleaning them up we can do it by searching the code for + discard_const. + + It also means that other error types aren't as swamped by the noise + of hundreds of const warnings, so we are more likely to notice when + we get new errors. + + Please only add more uses of this macro when you find it + _really_ hard to fix const warnings. Our aim is to eventually use + this function in only a very few places. + + Also, please call this via the discard_const_p() macro interface, as that + makes the return type safe. +*/ +#define discard_const(ptr) ((void *)((intptr_t)(ptr))) +#define discard_const_p(type, ptr) ((type *)discard_const(ptr)) + #endif /* _INCLUDES_H */ diff --git a/source4/lib/util.c b/source4/lib/util.c index 94b01f7501..fde5390127 100644 --- a/source4/lib/util.c +++ b/source4/lib/util.c @@ -892,25 +892,3 @@ BOOL all_zero(const char *ptr, uint_t size) } -/* - this is a warning hack. The idea is to use this everywhere that we - get the "discarding const" warning from gcc, effectively moving all - the warnings to this one place. That doesn't actually fix the - problem of course, but it means that when we do get to cleaning them - up we can do it by searching the code for discard_const. - - It also means that other error types aren't as swamped by the noise - of hundreds of const warnings, so we are more likely to notice when - we get new errors. - - Please only add more calls to this function when you find it - _really_ hard to fix const warnings. Our aim is to eventually use - this function in only a very few places. - - Also, please call this via the discard_const_p() macro interface, as that - makes the return type safe. -*/ -void *discard_const(const void *ptr) -{ - return (void *)ptr; -} -- cgit