[Catalyst-commits] r6240 - in trunk/Catalyst-Runtime: . lib/Catalyst

[andyg at dev.catalyst.perl.org]

Author: andyg
Date: 2007-03-29 20:44:12 +0100 (Thu, 29 Mar 2007)
New Revision: 6240

Modified:
   trunk/Catalyst-Runtime/Changes
   trunk/Catalyst-Runtime/lib/Catalyst/Engine.pm
Log:
Add a note on the performance of APR::Request

Modified: trunk/Catalyst-Runtime/Changes
===================================================================
--- trunk/Catalyst-Runtime/Changes	2007-03-29 19:35:16 UTC (rev 6239)
+++ trunk/Catalyst-Runtime/Changes	2007-03-29 19:44:12 UTC (rev 6240)
@@ -5,7 +5,8 @@
           * $c->uri_for (approx. 8x faster)
           * $c->engine->prepare_path (approx. 27x faster)
           * $c->engine->prepare_query_parameters (approx. 5x faster)
-        - If libapreq2 is installed, URIs are decoded using a faster C function.
+        - If libapreq2 is installed, URIs are decoded using a C function that is
+          approx. 12x faster than URI::Escape.
         - Updated HTTP::Body dependency to 0.9 which fixes the following issues:
           * Handle when IE sometimes sends an extra CRLF after the POST body.
           * Empty fields in multipart/form-data POSTs are no longer ignored.

Modified: trunk/Catalyst-Runtime/lib/Catalyst/Engine.pm
===================================================================
--- trunk/Catalyst-Runtime/lib/Catalyst/Engine.pm	2007-03-29 19:35:16 UTC (rev 6239)
+++ trunk/Catalyst-Runtime/lib/Catalyst/Engine.pm	2007-03-29 19:44:12 UTC (rev 6240)
@@ -650,6 +650,7 @@
     my $self = shift;
     
     if ( HAS_APR ) {
+        # This function is ~12x faster than URI::Escape
         return APR::Request::decode(@_);
     }