config/3983: __GNUC__ and __GNUC_MINOR__ cpp symbols tested incorrectly

[John Tobey <jt...@banta-im.com>]

>Number:         3983
>Category:       config
>Synopsis:       __GNUC__ and __GNUC_MINOR__ cpp symbols tested incorrectly
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    apache
>State:          open
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Tue Mar  2 13:20:01 PST 1999
>Last-Modified:
>Originator:     jtobey@banta-im.com
>Organization:
apache
>Release:        1.3.4
>Environment:
GCC
>Description:
In src/include/ap_config.h, an attempt to test for GNU CC version 2.7 or greater
will fail when GCC 3.0 is released
>How-To-Repeat:
You can't trigger this problem until GCC reaches version 3.0.
>Fix:
--- apache_1.3.4/src/include/ap_config.h~       Fri Jan  1 14:04:38 1999
+++ apache_1.3.4/src/include/ap_config.h        Tue Mar  2 16:01:19 1999
@@ -95,7 +95,8 @@
  * means.  In particular it's missing inline and the __attribute__
  * stuff.  So we hack around it.  PR#1613. -djg
  */
-#if !defined(__GNUC__) || __GNUC__ < 2 || __GNUC_MINOR__ < 7 || defined(NEXT)
+#if !defined(__GNUC__) || __GNUC__ < 2 \
+  || (__GNUC__ == 2 && __GNUC_MINOR__ < 7) || defined(NEXT)
 #define ap_inline
 #define __attribute__(__x)
 #define ENUM_BITFIELD(e,n,w)  signed int n : w
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, ]
[you need to include <ap...@Apache.Org> in the Cc line ]
[and leave the subject line UNCHANGED.  This is not done]
[automatically because of the potential for mail loops. ]
[If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request ]
[from a developer.                                      ]
[Reply only with text; DO NOT SEND ATTACHMENTS!         ]