Return-Path: 
 <dev-return-29579-apmail-httpd-dev-archive=httpd.apache.org@httpd.apache.org>
Delivered-To: apmail-httpd-dev-archive@httpd.apache.org
Received: (qmail 37073 invoked by uid 500); 12 May 2002 06:05:54 -0000
Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
list-help: <mailto:dev-help@httpd.apache.org>
list-unsubscribe: <mailto:dev-unsubscribe@httpd.apache.org>
list-post: <mailto:dev@httpd.apache.org>
Delivered-To: mailing list dev@httpd.apache.org
Received: (qmail 37001 invoked from network); 12 May 2002 06:05:53 -0000
Errors-To: <wrowe@rowe-clan.net>
Message-Id: <5.1.0.14.2.20020512005252.0260d4c0@pop3.rowe-clan.net>
X-Sender: wrowe%rowe-clan.net@pop3.rowe-clan.net
X-Mailer: QUALCOMM Windows Eudora Version 5.1
Date: Sun, 12 May 2002 00:54:26 -0500
To: dev@httpd.apache.org
From: "William A. Rowe, Jr." <wrowe@rowe-clan.net>
Subject: Re: cvs commit: httpd-2.0/server request.c
In-Reply-To: <20020512034547.86927.qmail@icarus.apache.org>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; format=flowed
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N

My gut instinct?  You break something here.

ap_location_walk, IIRC, sets up the default_conf.  Which means the
default_conf may not be brought in - given the case you've optimized
for here.  I might be mistaken, and don't have the energy to research
this early morning, but I thought I better point it out sooner.

At 10:45 PM 5/11/2002, brianp@apache.org wrote:
>brianp      02/05/11 20:45:47
>
>   Modified:    server   request.c
>   Log:
>   Optimization: skip cache setup in location_walk() if the vhost
>   config contains no <Location> blocks
>
>   Revision  Changes    Path
>   1.113     +2 -2      httpd-2.0/server/request.c
>
>   Index: request.c
>   ===================================================================
>   RCS file: /home/cvs/httpd-2.0/server/request.c,v
>   retrieving revision 1.112
>   retrieving revision 1.113
>   diff -u -r1.112 -r1.113
>   --- request.c 24 Apr 2002 04:20:10 -0000      1.112
>   +++ request.c 12 May 2002 03:45:47 -0000      1.113
>   @@ -1192,8 +1192,6 @@
>        walk_cache_t *cache;
>        const char *entry_uri;
>
>   -    cache = prep_walk_cache(AP_NOTE_LOCATION_WALK, r);
>   -
>        /* No tricks here, there are no <Locations > to parse in this vhost.
>         * We won't destroy the cache, just in case _this_ redirect is later
>         * redirected again to a vhost with <Location > blocks to optimize.
>   @@ -1201,6 +1199,8 @@
>        if (!num_sec) {
>            return OK;
>        }
>   +
>   +    cache = prep_walk_cache(AP_NOTE_LOCATION_WALK, r);
>
>        /* Location and LocationMatch differ on their behaviour w.r.t. 
> multiple
>         * slashes.  Location matches multiple slashes with a single slash,
>
>
>