[ruby-dev:31358] Re: invalid string for Date.parse

なかだです。

At Wed, 8 Aug 2007 15:42:33 +0900,
Yukihiro Matsumoto wrote in [ruby-dev:31357]:
> [ruby-core:11860]によれば、1.8.6(とtrunk)では
> 
>   % ruby -r date -e 'p Date.parse("junk").to_s'
>   "2007-06-01"
> 
> となります。どうも、junではじまっているので6月とみなしている
> ようです。以前はエラーになっていたので、これはあまり望ましい
> 仕様変更ではないと思います。

とりあえず全部\bとoオプションを付けてみました。


Index: lib/date/format.rb
===================================================================
--- lib/date/format.rb	(revision 12899)
+++ lib/date/format.rb	(working copy)
@@ -404,12 +404,12 @@ class Date
 	case s
 	when 'A', 'a'
-	  return unless str.sub!(/\A(#{Format::DAYS.keys.join('|')})/io, '') ||
-			str.sub!(/\A(#{Format::ABBR_DAYS.keys.join('|')})/io, '')
+	  return unless str.sub!(/\A(#{Format::DAYS.keys.join('|')})\b/io, '') ||
+			str.sub!(/\A(#{Format::ABBR_DAYS.keys.join('|')})\b/io, '')
 	  val = Format::DAYS[$1.downcase] || Format::ABBR_DAYS[$1.downcase]
 	  return unless val
 	  e.wday = val
 	when 'B', 'b', 'h'
-	  return unless str.sub!(/\A(#{Format::MONTHS.keys.join('|')})/io, '') ||
-			str.sub!(/\A(#{Format::ABBR_MONTHS.keys.join('|')})/io, '')
+	  return unless str.sub!(/\A(#{Format::MONTHS.keys.join('|')})\b/io, '') ||
+			str.sub!(/\A(#{Format::ABBR_MONTHS.keys.join('|')})\b/io, '')
 	  val = Format::MONTHS[$1.downcase] || Format::ABBR_MONTHS[$1.downcase]
 	  return unless val
@@ -685,5 +685,5 @@ class Date
 
   def self._parse_day(str, e) # :nodoc:
-    if str.sub!(/\b(#{Format::ABBR_DAYS.keys.join('|')})[^-\d\s]*/in, ' ')
+    if str.sub!(/\b(#{Format::ABBR_DAYS.keys.join('|')})\b[^-\d\s]*/ion, ' ')
       e.wday = Format::ABBR_DAYS[$1.downcase]
       true
@@ -774,5 +774,5 @@ class Date
 		/'?(\d+)[^-\d\s]*
 		 \s*
-		 (#{Format::ABBR_MONTHS.keys.join('|')})[^-\d\s']*
+		 (#{Format::ABBR_MONTHS.keys.join('|')})\b[^-\d\s']*
 		 (?:
 		   \s*
@@ -781,5 +781,5 @@ class Date
 		   ('?-?\d+(?:(?:st|nd|rd|th)\b)?)
 		 )?
-		/inx,
+		/ionx,
 		' ') # '
       s3e(e, $4, Format::ABBR_MONTHS[$2.downcase], $1,
@@ -791,5 +791,5 @@ class Date
   def self._parse_us(str, e) # :nodoc:
     if str.sub!(
-		/\b(#{Format::ABBR_MONTHS.keys.join('|')})[^-\d\s']*
+		/\b(#{Format::ABBR_MONTHS.keys.join('|')})\b[^-\d\s']*
 		 \s*
 		 ('?\d+)[^-\d\s']*
@@ -800,5 +800,5 @@ class Date
 		   ('?-?\d+)
 		 )?
-		/inx,
+		/ionx,
 		' ') # '
       s3e(e, $4, Format::ABBR_MONTHS[$1.downcase], $2,
@@ -854,10 +854,10 @@ class Date
 
   def self._parse_vms(str, e) # :nodoc:
-    if str.sub!(/('?-?\d+)-(#{Format::ABBR_MONTHS.keys.join('|')})[^-]*
-		-('?-?\d+)/inx, ' ')
+    if str.sub!(/('?-?\d+)-(#{Format::ABBR_MONTHS.keys.join('|')})\b[^-]*
+		-('?-?\d+)/ionx, ' ')
       s3e(e, $3, Format::ABBR_MONTHS[$2.downcase], $1)
       true
-    elsif str.sub!(/\b(#{Format::ABBR_MONTHS.keys.join('|')})[^-]*
-		-('?-?\d+)(?:-('?-?\d+))?/inx, ' ')
+    elsif str.sub!(/\b(#{Format::ABBR_MONTHS.keys.join('|')})\b[^-]*
+		-('?-?\d+)(?:-('?-?\d+))?/ionx, ' ')
       s3e(e, $3, Format::ABBR_MONTHS[$1.downcase], $2)
       true
@@ -887,5 +887,5 @@ class Date
 
   def self._parse_mon(str, e) # :nodoc:
-    if str.sub!(/\b(#{Format::ABBR_MONTHS.keys.join('|')})\S*/in, ' ')
+    if str.sub!(/\b(#{Format::ABBR_MONTHS.keys.join('|')})\b\S*/ion, ' ')
       e.mon = Format::ABBR_MONTHS[$1.downcase]
       true


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦