apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cmpil...@collab.net
Subject Re: cvs commit: apr/file_io/win32 filestat.c
Date Mon, 29 Jan 2001 20:15:13 GMT
Hey, all.  This fix also fixed a build problem I was having in Win98,
but I'm worried that the fix might be skirting a bigger issue.  Code
exists in apr_private.h that "turns off" the use of Windows' GDI
library (by #define'ing NOGDI) among other things.  The recent
inclusion of <aclapi.h>, on Win98 platforms, eventually results in
<wtypes.h> being included, part of which depends on the GDI headers
being available.  I do NOT expect this to happen on WinNT, but as I
don't have an NT box here, I can't verify that.

At any rate, the REASON the fix below helped is because the GDI stuff
wasn't opted out until the following conclusion of apr_private.h.  I
general get concerned when the order of header inclusion means the
difference between a successful and a failed build.

Can someone provide some insight as to why APR explicitly attempts to
bar inclusion of the GDI, USER, and 3 other classes of header files in
apr_private.h?

Thanks.

--Mike

stoddard@apache.org writes:

> stoddard    01/01/29 09:10:22
> 
>   Modified:    file_io/win32 filestat.c
>   Log:
>   This fixes a funky compile problem for me.
>   
>   Revision  Changes    Path
>   1.44      +1 -1      apr/file_io/win32/filestat.c
>   
>   Index: filestat.c
>   ===================================================================
>   RCS file: /home/cvs/apr/file_io/win32/filestat.c,v
>   retrieving revision 1.43
>   retrieving revision 1.44
>   diff -u -r1.43 -r1.44
>   --- filestat.c	2001/01/29 06:21:40	1.43
>   +++ filestat.c	2001/01/29 17:10:19	1.44
>   @@ -52,6 +52,7 @@
>     * <http://www.apache.org/>.
>     */
>    
>   +#include <aclapi.h>
>    #include "apr_private.h"
>    #include "win32/fileio.h"
>    #include "apr_file_io.h"
>   @@ -62,7 +63,6 @@
>    #include <sys/stat.h>
>    #include "atime.h"
>    #include "misc.h"
>   -#include <aclapi.h>
>    
>    static apr_status_t free_localheap(void *heap) {
>        LocalFree(heap);
>   
>   
>   

Mime
View raw message