httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: cvs commit: apache-2.0/src/main http_log.c
Date Fri, 03 Mar 2000 00:54:03 GMT
On Thu, 2 Mar 2000 rbb@apache.org wrote:
> On Thu, 2 Mar 2000, Greg Stein wrote:
> > On 2 Mar 2000 rbb@locus.apache.org wrote:
...
> > >   +#ifdef WIN32
> > >   +#include "apr_win.h"
> > >   +#else
> > >   +#include "apr.h"  /* for ap_signal */
> > >   +#endif
> > 
> > Can this be isolated within apr.h, so that APR users don't have to
> > replicate the above #ifdef every time they include the header?
> 
> I guess we could, but it would be pretty ugly.  I'll look into doing it.

Maybe move apr.h to apr_default.h and have apr.h select between apr_win.h,
apr_default.h, and apr_xxx.h? (where xxx is for other platforms that do
not have configure)

That might keep it clean.

??

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/


Mime
View raw message