[ruby-dev:18372] Re: [ruby-cvs] ruby/lib/uri: * eval.c (ruby_run): should set toplevel visibility again here.

At Wed, 25 Sep 2002 21:58:47 +0900,
Kazuhiro NISHIYAMA wrote:
> > この変更の意図は名前(REGEXPと付いているのに文字列)ですか。
> > それとも/#{mailbox_pattern}/にアクセスしたいということでしょうか。
> > 
> > check_toの中で使っているため、
> > この変更だと1.6.7ではちょっとまずいようです。

こんな感じでしょうか。

Index: mailto.rb
===================================================================
RCS file: /src/ruby/lib/uri/mailto.rb,v
retrieving revision 1.1.2.2
diff -u -u -u -r1.1.2.2 mailto.rb
--- mailto.rb	2002/09/25 07:07:43	1.1.2.2
+++ mailto.rb	2002/09/25 13:39:50
@@ -46,22 +46,24 @@
     # hname      =  *urlc
     # hvalue     =  *urlc
     # header     =  hname "=" hvalue
-    header_pattern = "(?:[^?=&]*=[^?=&]*)"
-    HEADER_REGEXP = /#{header_pattern}/
+    HEADER_REGEXP_S = "(?:[^?=&]*=[^?=&]*)"
+    HEADER_REGEXP = Regexp.new(HEADER_REGEXP_S)
     # headers    =  "?" header *( "&" header )
     # to         =  #mailbox
     # mailtoURL  =  "mailto:" [ to ] [ headers ]
-    mailbox_pattern = "(?:[^(),%?=&]|#{PATTERN::ESCAPED})"
-    MAILBOX_REGEXP = /#{mailbox_pattern}/
-    MAILTO_REGEXP = Regexp.new("
+    MAILBOX_REGEXP_S = "(?:[^(),%?=&]|#{PATTERN::ESCAPED})"
+    MAILBOX_REGEXP = Regexp.new(MAILBOX_REGEXP_S)
+    MAILTO_REGEXP_S = "
       \\A
-      (#{mailbox_pattern}*?)                         (?# 1: to)
+      (#{MAILBOX_REGEXP_S}*?)                         (?# 1: to)
       (?:
         \\?
-        (#{header_pattern}(?:\\&#{header_pattern})*) (?# 2: headers)
+        (#{HEADER_REGEXP_S}(?:\\&#{HEADER_REGEXP_S})*) (?# 2: headers)
       )?
       \\z
-    ", Regexp::EXTENDED, 'N').freeze
+    "
+    MAILTO_REGEXP = Regexp.new(MAILTO_REGEXP_S,
+			       Regexp::EXTENDED, 'N').freeze
 
 =begin
 
@@ -155,7 +157,7 @@
       return true unless v
       return true if v.size == 0
 
-      if OPAQUE !~ v || /\A#{MAILBOX_REGEXP}*\z/o !~ v
+      if OPAQUE !~ v || /\A#{MAILBOX_REGEXP_S}*\z/o !~ v
 	raise InvalidComponentError,
 	  "bad component(expected opaque component): #{v}"
       end
@@ -191,7 +193,7 @@
       return true if v.size == 0
 
       if OPAQUE !~ v || 
-	  /\A(#{HEADER_REGEXP}(?:\&#{HEADER_REGEXP})*)\z/o !~ v
+	  /\A(#{HEADER_REGEXP_S}(?:\&#{HEADER_REGEXP_S})*)\z/o !~ v
 	raise InvalidComponentError,
 	  "bad component(expected opaque component): #{v}"
       end


> ところでoはつけないんでしょうか?

　そもそも一度しか通らないのでは。

-- 
                     /
                    /__  __            Akinori.org / MUSHA.org
                   / )  )  ) )  /     FreeBSD.org / Ruby-lang.org
Akinori MUSHA aka / (_ /  ( (__(  @ iDaemons.org / and.or.jp

"When I leave I don't know what I'm hoping to find
              When I leave I don't know what I'm leaving behind.."