From bb36705c8d360a2ba865a3d8118c52afa1e46f4e Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Sun, 15 Apr 2007 21:13:13 +0000 Subject: r22226: move discard_const macros to librelace metze (This used to be commit c2cfee6d25718fac35bd4ed982c7424f1c3ed0b7) --- source4/lib/replace/replace.h | 23 +++++++++++++++++++++++ source4/lib/replace/strptime.c | 2 +- 2 files changed, 24 insertions(+), 1 deletion(-) (limited to 'source4/lib/replace') diff --git a/source4/lib/replace/replace.h b/source4/lib/replace/replace.h index 7d6dcec7f1..959d44b33e 100644 --- a/source4/lib/replace/replace.h +++ b/source4/lib/replace/replace.h @@ -79,6 +79,29 @@ #include #endif +/** + 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))) + +/** Type-safe version of discard_const */ +#define discard_const_p(type, ptr) ((type *)discard_const(ptr)) + #ifndef HAVE_STRERROR extern char *sys_errlist[]; #define strerror(i) sys_errlist[i] diff --git a/source4/lib/replace/strptime.c b/source4/lib/replace/strptime.c index 8fb919472e..d415b7826e 100644 --- a/source4/lib/replace/strptime.c +++ b/source4/lib/replace/strptime.c @@ -974,7 +974,7 @@ strptime_internal (rp, fmt, tm, decided, era_cnt) if (want_xday && !have_yday) day_of_the_year (tm); - return (char *) rp; + return discard_const_p(char, rp); } -- cgit