You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rp...@apache.org on 2005/11/19 21:59:22 UTC

svn commit: r345686 - in /httpd/httpd/trunk: CHANGES modules/mappers/mod_negotiation.c

Author: rpluem
Date: Sat Nov 19 12:59:16 2005
New Revision: 345686

URL: http://svn.apache.org/viewcvs?rev=345686&view=rev
Log:
* Minor performance tweak by reusing already calculated strlen in get_body.

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

Modified:
    httpd/httpd/trunk/CHANGES
    httpd/httpd/trunk/modules/mappers/mod_negotiation.c

Modified: httpd/httpd/trunk/CHANGES
URL: http://svn.apache.org/viewcvs/httpd/httpd/trunk/CHANGES?rev=345686&r1=345685&r2=345686&view=diff
==============================================================================
--- httpd/httpd/trunk/CHANGES [utf-8] (original)
+++ httpd/httpd/trunk/CHANGES [utf-8] Sat Nov 19 12:59:16 2005
@@ -2,6 +2,10 @@
 Changes with Apache 2.3.0
   [Remove entries to the current 2.0 and 2.2 section below, when backported]
 
+  *) mod_negotiation: Minor performance tweak by reusing already calculated
+     strlen.
+     [Ruediger Pluem, Christophe Jaillet <christophe.jaillet wanadoo.fr>]
+
   *) mod_authz_dbd: SQL authz with Login/Session support [Nick Kew]
 
   *) Fix typo in ProxyStatus syntax error message.

Modified: httpd/httpd/trunk/modules/mappers/mod_negotiation.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/trunk/modules/mappers/mod_negotiation.c?rev=345686&r1=345685&r2=345686&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/mappers/mod_negotiation.c (original)
+++ httpd/httpd/trunk/modules/mappers/mod_negotiation.c Sat Nov 19 12:59:16 2005
@@ -854,7 +854,7 @@
         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') {