Merge r1735906 from trunk:

Avoid a call to 'prep_walk_cache' if possible, just as in 'ap_if_walk' and 'ap_location_walk'
Submitted by: jailletc36
Reviewed by: jailletc36, jim, ylavic

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1800267 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/STATUS b/STATUS
index b34e96ebb167224eed836d2aab77b9cf417fa23e..1cfd5c5c7d7e30e1aaa0f2c5a60270563ef534e5 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -116,12 +116,6 @@ RELEASE SHOWSTOPPERS:
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
-   *) core: Save 160 bytes in the request pool and some cycles if no <If>
-            directive is used.
-      trunk patch: http://svn.apache.org/r1735906
-      2.4.x patch: svn merge -c 1735906 ^/httpd/httpd/trunk .
-      +1: jailletc36, jim, ylavic
-
    *) Easy proposals:
       - mod_authnz_ldap: Log a warning when the LDAP authn provider is configured
                          but an AuthLDAPURL isn't

diff --git a/server/request.c b/server/request.c
index fac5f8c7cd4bbf2e899786c7fa1e7fa30fc28208..7cead3bffb9b8792064647c19b305fd6a62e2373 100644 (file)
--- a/server/request.c
+++ b/server/request.c
@@ -1615,9 +1615,6 @@ AP_DECLARE(int) ap_file_walk(request_rec *r)
         return OK;
     }
 
-    cache = prep_walk_cache(AP_NOTE_FILE_WALK, r);
-    cached = (cache->cached != NULL);
-
     /* No tricks here, there are just no <Files > to parse in this context.
      * We won't destroy the cache, just in case _this_ redirect is later
      * redirected again to a context containing the same or similar <Files >.
@@ -1626,6 +1623,9 @@ AP_DECLARE(int) ap_file_walk(request_rec *r)
         return OK;
     }
 
+    cache = prep_walk_cache(AP_NOTE_FILE_WALK, r);
+    cached = (cache->cached != NULL);
+
     /* Get the basename .. and copy for the cache just
      * in case r->filename is munged by another module
      */