aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
diff options
context:
space:
mode:
authorDavid Heinemeier Hansson <david@loudthinking.com>2005-09-15 05:30:56 +0000
committerDavid Heinemeier Hansson <david@loudthinking.com>2005-09-15 05:30:56 +0000
commitb37d7a70f9a6d9a114fb21b525fbf1eb9bfcca38 (patch)
tree6544c5f9ea60ac3a22c34f926d95494ddea5659d /actionpack
parentc1505e3e2b1a4f0d6d9c29731cac1dc096c13d2e (diff)
downloadrails-b37d7a70f9a6d9a114fb21b525fbf1eb9bfcca38.tar.gz
rails-b37d7a70f9a6d9a114fb21b525fbf1eb9bfcca38.tar.bz2
rails-b37d7a70f9a6d9a114fb21b525fbf1eb9bfcca38.zip
Speed up cookie use by decreasing string copying #2194 [skae]
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@2245 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'actionpack')
-rw-r--r--actionpack/CHANGELOG2
-rw-r--r--actionpack/lib/action_controller/cgi_ext/cookie_performance_fix.rb14
2 files changed, 9 insertions, 7 deletions
diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG
index a6ea1e9649..21d8145ff1 100644
--- a/actionpack/CHANGELOG
+++ b/actionpack/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*
+* Speed up cookie use by decreasing string copying #2194 [skae]
+
* Fixed access to "Host" header with requests made by crappy old HTTP/1.0 clients #2124 [Marcel Molina]
* Added easy assignment of fragment cache store through use of symbols for included stores (old way still works too)
diff --git a/actionpack/lib/action_controller/cgi_ext/cookie_performance_fix.rb b/actionpack/lib/action_controller/cgi_ext/cookie_performance_fix.rb
index 1c30f82b19..d73cb0355f 100644
--- a/actionpack/lib/action_controller/cgi_ext/cookie_performance_fix.rb
+++ b/actionpack/lib/action_controller/cgi_ext/cookie_performance_fix.rb
@@ -71,28 +71,28 @@ class CGI #:nodoc:
# Convert the Cookie to its string representation.
def to_s
buf = ""
- buf += @name + '='
+ buf << @name << '='
if @value.kind_of?(String)
- buf += CGI::escape(@value)
+ buf << CGI::escape(@value)
else
- buf += @value.collect{|v| CGI::escape(v) }.join("&")
+ buf << @value.collect{|v| CGI::escape(v) }.join("&")
end
if @domain
- buf += '; domain=' + @domain
+ buf << '; domain=' << @domain
end
if @path
- buf += '; path=' + @path
+ buf << '; path=' << @path
end
if @expires
- buf += '; expires=' + CGI::rfc1123_date(@expires)
+ buf << '; expires=' << CGI::rfc1123_date(@expires)
end
if @secure == true
- buf += '; secure'
+ buf << '; secure'
end
buf