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 &quot;bar&quot; &lt;baz&gt;"
   def CGI::escapeHTML(string)
-    string.gsub(/&/n, '&amp;').gsub(/\"/n, '&quot;').gsub(/>/n, '&gt;').gsub(/</n, '&lt;')
+    string.gsub(/&/, '&amp;').gsub(/\"/, '&quot;').gsub(/>/, '&gt;').gsub(/</, '&lt;')
   end
 
 
@@ -367,22 +368,23 @@
   #   CGI::unescapeHTML("Usage: foo &quot;bar&quot; &lt;baz&gt;")
   #      # => "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