Mailing List Archive

Re: svn commit: r1894021 - /httpd/httpd/trunk/server/util.c
HE CHANGED THE SACRED CODE! ????

????

> Am 08.10.2021 um 11:02 schrieb rpluem@apache.org:
>
> Author: rpluem
> Date: Fri Oct 8 09:02:30 2021
> New Revision: 1894021
>
> URL: http://svn.apache.org/viewvc?rev=1894021&view=rev
> Log:
> * Optimize performance by moving calculation of loop invariant out of the loop
>
> Modified:
> httpd/httpd/trunk/server/util.c
>
> Modified: httpd/httpd/trunk/server/util.c
> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util.c?rev=1894021&r1=1894020&r2=1894021&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/server/util.c (original)
> +++ httpd/httpd/trunk/server/util.c Fri Oct 8 09:02:30 2021
> @@ -505,6 +505,7 @@ AP_DECLARE(int) ap_normalize_path(char *
> int ret = 1;
> apr_size_t l = 1, w = 1, n;
> int decode_unreserved = (flags & AP_NORMALIZE_DECODE_UNRESERVED) != 0;
> + int merge_slashes = (flags & AP_NORMALIZE_MERGE_SLASHES) != 0;
>
> if (!IS_SLASH(path[0])) {
> /* Besides "OPTIONS *", a request-target should start with '/'
> @@ -549,7 +550,7 @@ AP_DECLARE(int) ap_normalize_path(char *
>
> if (w == 0 || IS_SLASH(path[w - 1])) {
> /* Collapse ///// sequences to / */
> - if ((flags & AP_NORMALIZE_MERGE_SLASHES) && IS_SLASH(path[l])) {
> + if (merge_slashes && IS_SLASH(path[l])) {
> do {
> l++;
> } while (IS_SLASH(path[l]));
>
>