On Wed, Feb 22, 2017 at 11:47 AM, Joe Orton <jor...@redhat.com> wrote:
>
> It actually only works like:
>
>    <IfDirective <IfFile>
>
> which is a bit ugly. Quoting the argument is a syntax error. Not sure
> how best to handle this.
>
> (b) for <IfDirective foo> match both "foo" and "<foo".

I'd vote for this, it's very unlikely that some day we want to add a
directive called VirtualHost or If (w/o the leading '<') which may
conflict here, so it shouldn't hurt.

>
> (In core.c the start_if* code is mostly common across all the functions
> and I think can be factored out, so it's possible to make core.c
> simpler/smaller net of even two more container directives.)

+1


Regards,
Yann.

Reply via email to