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 [EMAIL PROTECTED] 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); > > >
