You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by gs...@locus.apache.org on 2000/12/07 11:54:18 UTC
cvs commit: httpd-2.0/build buildexports.awk
gstein 00/12/07 02:54:17
Modified: build buildexports.awk
Log:
*) simplify the AWK script a bit
*) do not put the (const void *) cast into the code (it isn't needed)
Revision Changes Path
1.3 +3 -8 httpd-2.0/build/buildexports.awk
Index: buildexports.awk
===================================================================
RCS file: /home/cvs/httpd-2.0/build/buildexports.awk,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -u -r1.2 -r1.3
--- buildexports.awk 2000/12/05 18:29:06 1.2
+++ buildexports.awk 2000/12/07 10:54:15 1.3
@@ -1,8 +1,3 @@
-{
- if ($1 ~ /^APR_/)
- print "#if", $1;
- if ($1 ~ /^apr?_/)
- print "const void *ap_hack_" $1 " = (const void *)" $1 ";";
- if ($1 ~ /^\/APR_/)
- print "#endif /*", substr($1,2), "*/";
-}
+/^APR_/ { print "#if", $1 }
+/^\t*apr?_/ { print "const void *ap_hack_" $1 " = " $1 ";" }
+/^\/APR_/ { print "#endif /*", substr($1,2), "*/" }
Re: cvs commit: httpd-2.0/build buildexports.awk
Posted by Jeff Trawick <tr...@bellsouth.net>.
gstein@locus.apache.org writes:
> gstein 00/12/07 02:54:17
>
> Modified: build buildexports.awk
> Log:
...
> *) do not put the (const void *) cast into the code (it isn't
> needed)
I'm afraid that it is. Without it, we get a warning for every line on
some platforms. Here is a sample on Tru64:
> cc: Warning: exports.c, line 41: In the initializer for ap_hack_apr_dso_load, the referenced type of the pointer value "apr_dso_load" is "function (pointer to pointer to struct apr_dso_handle_t, pointer to const char, pointer to struct apr_pool_t) returning int", which is not compatible with "const void". (ptrmismatch)
> const void *ap_hack_apr_dso_load = apr_dso_load;
> -----------------------------------^
I'll fix shortly.
--
Jeff Trawick | trawickj@bellsouth.net | PGP public key at web site:
http://www.geocities.com/SiliconValley/Park/9289/
Born in Roswell... married an alien...