Merge r345686 from trunk:

* Minor performance tweak by reusing already calculated strlen in get_body.

Submitted by: Christophe Jaillet <christophe.jaillet wanadoo.fr>
Reviewed by: Ruediger Pluem

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

diff --git a/CHANGES b/CHANGES
index dcb32e833fc354f95bcc2eb88946df76e341b6ba..b4e14a8eff6156a487c8be326ea0004ef195ef8e 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,10 @@
                                                         -*- coding: utf-8 -*-
 Changes with Apache 2.1.10
 
+  *) mod_negotiation: Minor performance tweak by reusing already calculated
+     strlen.
+     [Ruediger Pluem, Christophe Jaillet <christophe.jaillet wanadoo.fr>]
+
   *) Remove support for 'On' and 'Off' for AuthBasicProvider and
      AuthDigestProvider.  [Joshua Slive, Justin Erenkrantz]
 

diff --git a/modules/mappers/mod_negotiation.c b/modules/mappers/mod_negotiation.c
index 86a50d797e7892311519c0dd0da75b967d63794e..37c665eb946a34bf721282ac0e0061c5ed31e1f4 100644 (file)
--- a/modules/mappers/mod_negotiation.c
+++ b/modules/mappers/mod_negotiation.c
@@ -854,7 +854,7 @@ static apr_off_t get_body(char *buffer, apr_size_t *len, const char *tag,
         return -1;
     }
     bodylen = endbody - buffer;
-    endbody += strlen(tag);
+    endbody += taglen;
     /* Skip all the trailing cruft after the end tag to the next line */
     while (*endbody) {
         if (*endbody == '\n') {