ruby-changes:3122
From: ko1@a...
Date: 24 Dec 2007 23:59:44 +0900
Subject: [ruby-changes:3122] akr - Ruby:r14614 (trunk): * lib/cgi.rb (CGI::escape): m17nized.
akr 2007-12-24 23:59:34 +0900 (Mon, 24 Dec 2007) New Revision: 14614 Modified files: trunk/ChangeLog trunk/lib/cgi.rb Log: * lib/cgi.rb (CGI::escape): m17nized. (CGI::unescape): ditto. (CGI::escapeHTML): ditto. (CGI::unescapeHTML): ditto. http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/lib/cgi.rb?r1=14614&r2=14613 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/ChangeLog?r1=14614&r2=14613 Index: ChangeLog =================================================================== --- ChangeLog (revision 14613) +++ ChangeLog (revision 14614) @@ -1,3 +1,10 @@ +Mon Dec 24 23:55:29 2007 Tanaka Akira <akr@f...> + + * lib/cgi.rb (CGI::escape): m17nized. + (CGI::unescape): ditto. + (CGI::escapeHTML): ditto. + (CGI::unescapeHTML): ditto. + Mon Dec 24 23:32:24 2007 NAKAMURA Usaku <usa@r...> * transcode_data_japanese.c (select_iso_2022_mode): '\e' is not valid. Index: lib/cgi.rb =================================================================== --- lib/cgi.rb (revision 14613) +++ lib/cgi.rb (revision 14614) @@ -339,8 +339,8 @@ # url_encoded_string = CGI::escape("'Stop!' said Fred") # # => "%27Stop%21%27+said+Fred" def CGI::escape(string) - string.gsub(/([^ a-zA-Z0-9_.-]+)/n) do - '%' + $1.unpack('H2' * $1.size).join('%').upcase + string.gsub(/([^ a-zA-Z0-9_.-]+)/) do + '%' + $1.unpack('H2' * $1.bytesize).join('%').upcase end.tr(' ', '+') end @@ -349,8 +349,9 @@ # string = CGI::unescape("%27Stop%21%27+said+Fred") # # => "'Stop!' said Fred" def CGI::unescape(string) - string.tr('+', ' ').gsub(/((?:%[0-9a-fA-F]{2})+)/n) do - [$1.delete('%')].pack('H*') + enc = string.encoding + string.tr('+', ' ').gsub(/((?:%[0-9a-fA-F]{2})+)/) do + [$1.delete('%')].pack('H*').force_encoding(enc) end end @@ -359,7 +360,7 @@ # CGI::escapeHTML('Usage: foo "bar" <baz>') # # => "Usage: foo "bar" <baz>" def CGI::escapeHTML(string) - string.gsub(/&/n, '&').gsub(/\"/n, '"').gsub(/>/n, '>').gsub(/</n, '<') + string.gsub(/&/, '&').gsub(/\"/, '"').gsub(/>/, '>').gsub(/</, '<') end @@ -367,22 +368,23 @@ # CGI::unescapeHTML("Usage: foo "bar" <baz>") # # => "Usage: foo \"bar\" <baz>" def CGI::unescapeHTML(string) - string.gsub(/&(amp|quot|gt|lt|\#[0-9]+|\#x[0-9A-Fa-f]+);/n) do + enc = string.encoding + string.gsub(/&(amp|quot|gt|lt|\#[0-9]+|\#x[0-9A-Fa-f]+);/) do match = $1.dup case match when 'amp' then '&' when 'quot' then '"' when 'gt' then '>' when 'lt' then '<' - when /\A#0*(\d+)\z/n then + when /\A#0*(\d+)\z/ then if Integer($1) < 256 - Integer($1).chr + Integer($1).chr.force_encoding(enc) else "&##{$1};" end - when /\A#x([0-9a-f]+)\z/ni then + when /\A#x([0-9a-f]+)\z/i then if $1.hex < 256 - $1.hex.chr + $1.hex.chr.force_encoding(enc) else "&#x#{$1};" end -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml