string.h needs to be included before monkeying around with strchr et al,

since some system header files do their own monkeying around with those
functions in ways that don't get along with us unless we include their
definitions first

examples: gcc 2.8.1 on Solaris
          gcc 2.95.2 on AIX

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@88113 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/include/httpd.h b/include/httpd.h
index 329115208d582f52f1747e62fc28ee4199b15531..c315d14669b9ed498c89ec18f8b0a27395463b91 100644 (file)
--- a/include/httpd.h
+++ b/include/httpd.h
@@ -1523,7 +1523,12 @@ AP_DECLARE(extern const char *) ap_psignature(const char *prefix, request_rec *r
   /* The C library has functions that allow const to be silently dropped ...
      these macros detect the drop in maintainer mode, but use the native
      methods for normal builds
+
+     Note that on some platforms (e.g., AIX with gcc, Solaris with gcc), string.h needs 
+     to be included before the macros are defined or compilation will fail.
   */
+#include <string.h>
+
 #ifdef AP_DEBUG
 
 #undef strchr