httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rodent of Unusual Size <Ken.C...@Golux.Com>
Subject Re: [PATCH] Add '+' and '-' to IndexOptions keywords
Date Fri, 25 Sep 1998 21:36:32 GMT
Ben Hyde wrote:
> Piffle.  Your implying legalese when you use the word.

:-)  Which word, precedent or legalese?

> >> Can we disallow the confusing case, i.e. mixing incremental
> >> and absolute in the same statement?
> >
> >We could, and the user would get an error message if they were
> >combined
> that's a good thing, right, he wrote junk and we told him so

No, he did't write junk -- he wrote something that may have
result different than he intended.  Not the same thing.
Someone may *want* to add a keyword to the end of an existing
IndexOptions statement.  I don't buy it.  :-)  It's also
different from the behaviour of the Options directive with
the same syntactic form.

>    IndexOptions FancyIndexing
>    IndexOptions SuppressLastModified
> isn't
>    IndexOptions FancyIndexing SuppressLastModified

It is with the patch.

> That's why this operation has to use RAW_ARGS now.

No, it uses RAW_ARGS because it never got converted to
ITERATE.  (Probably based on the Options code, which is
also RAW_ARGS.)  ITERATE is perfectly reasonable for this
directive, in both its original and modified forms.  I'll
probably fix that later.  I'm on a clean-up-mod_autoindex
kick at the moment.

#ken    P-)}

Ken Coar                    <http://Web.Golux.Com/coar/>
Apache Group member         <>
"Apache Server for Dummies" <http://Web.Golux.Com/coar/ASFD/>

View raw message