httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: cvs commit: httpd-2.0/server config.c
Date Sun, 07 Jan 2001 23:38:46 GMT
On Sun, Jan 07, 2001 at 11:16:17PM +0000, Ben Laurie wrote:
> Greg Stein wrote:
> > 
> > On Sun, Jan 07, 2001 at 11:06:01PM +0000, Ben Laurie wrote:
> > > Ben Laurie wrote:
> > > > > >           /* MIME type arguments */
> > > > > >                while (p > handler && p[-1] == ' ')
> > > > > >               --p;            /* strip trailing spaces */
> > > > > >   -       handler_len = p - handler;
> > > > > >   +       *p='\0';
> > > > >
> > > > > How does this work? p is "const char *".
> > > >
> > > > Good question! My compiler compiles it! What gives?
> > >
> > > Sigh. It's -Werror disappearing. I keep forgetting it doesn't go in by
> > > default. Please can we fix that?
> > 
> > Feel free. It's in httpd-2.0/configure.in and in apr/configure.in
> 
> I'd like to know why it isn't in the _other_ ones, too.

aprutil: just haven't got around to it.

pcre? not really "ours", so maybe people haven't bothered to extensively
change it. otherwise, I'd say another case of not having a Round Tuit.

> Also, why configure is so brain-dead you can't do:
> 
> CFLAGS="-Wall -Werror" ./configure

Because it is brain-dead. Somebody got smart and thought "EXTRA_CFLAGS" is
the right way to do it. Not the standard, and needs to change.

Cheers,
-g

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

Mime
View raw message