From: Stefan Metzmacher <metze@samba.org>
Date: Sun, 15 Apr 2007 21:13:48 +0000 (+0000)
Subject: r22227: merge from samba4:
X-Git-Tag: samba-misc-tags/initial-v3-0-unstable~712
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=8f4a39127f7b560a7a609e74423ccb88f53e545e;p=thirdparty%2Fsamba.git

r22227: merge from samba4:

move discard_const macros to librelace

metze
---

diff --git a/source/lib/replace/replace.h b/source/lib/replace/replace.h
index 7d6dcec7f12..959d44b33e7 100644
--- a/source/lib/replace/replace.h
+++ b/source/lib/replace/replace.h
@@ -79,6 +79,29 @@
 #include <stddef.h>
 #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/source/lib/replace/strptime.c b/source/lib/replace/strptime.c
index 8fb919472e1..d415b7826e5 100644
--- a/source/lib/replace/strptime.c
+++ b/source/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);
 }