From 0e97a7dbd0b83c0fbbdb34f52a17d1d73a632f84 Mon Sep 17 00:00:00 2001 From: matz Date: Fri, 14 Mar 2008 07:45:26 +0000 Subject: [PATCH] * lib/cgi.rb (CGI::Cookie::initialize): performance patch from Makoto Kuwata in [ruby-dev:34048]. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_8@15778 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ lib/cgi.rb | 17 +++++++++++------ 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index cb57a79747..ab092ca682 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Fri Mar 14 16:35:11 2008 Yukihiro Matsumoto + + * lib/cgi.rb (CGI::Cookie::initialize): performance patch from + Makoto Kuwata in [ruby-dev:34048]. + Fri Mar 14 15:49:05 2008 Nobuyoshi Nakada * configure.in (RUBY_LIB_PREFIX): use libdir. diff --git a/lib/cgi.rb b/lib/cgi.rb index ccc48d8836..e616534221 100644 --- a/lib/cgi.rb +++ b/lib/cgi.rb @@ -792,11 +792,16 @@ class CGI # # These keywords correspond to attributes of the cookie object. def initialize(name = "", *value) - options = if name.kind_of?(String) - { "name" => name, "value" => value } - else - name - end + if name.kind_of?(String) + @name = name + @value = value + %r|^(.*/)|.match(ENV["SCRIPT_NAME"]) + @path = ($1 or "") + @secure = false + return super(@value) + end + + options = name unless options.has_key?("name") raise ArgumentError, "`name' required" end @@ -880,7 +885,7 @@ class CGI if cookies.has_key?(name) values = cookies[name].value + values end - cookies[name] = Cookie::new({ "name" => name, "value" => values }) + cookies[name] = Cookie::new(name, *values) end cookies