[ruby-dev:23035] Re: [ruby-talk:93584] date/format.rb fix

なひです。

主にふなばさんに相談というかお願いです。
date/format.rbの_parseに関連して、こんなのがruby-talkに流れてました。

> From: "Brad Hilton" <[e-mail]>
> To: "ruby-talk ML" <[e-mail]>
> Sent: Wednesday, February 25, 2004 5:19 AM

> The _parse() function of date/format.rb does not handle fractions of
> seconds.  Eg, the string
> 
> 2004-02-12 12:23:45.581297-07
> 
> causes the parser to miss the timezone offset.  Postgresql returns
> timestamp columns like this, thus breaking ruby-dbi.
> 
> A simple change to the regex can fix this:
> 
> --- date/format.rb.orig      2004-02-24 12:07:42.000000000 -0800
> +++ date/format.rb   2004-02-24 12:03:12.000000000 -0800
> @@ -236,7 +236,7 @@ class Date
> 
>       # time
>       if str.sub!(
> -               /(\d+):(\d+)(?::(\d+))?
> +               /(\d+):(\d+)(?::(\d+)(?:\.\d+)?)?
>                   (?:
>                     \s*
>                     ([ap])(?:m\b|\.m\.)
> 
> Would someone please consider applying this fix?

0$ ruby -rdate -e 'p DateTime.parse("2004-01-01T00:00:00+09").to_s'
"2004-01-01T00:00:00+0900"
0$ ruby -rdate -e 'p DateTime.parse("2004-01-01T00:00:00.0+09").to_s'
"2004-01-01T00:00:00Z"

と、秒数に小数点が付いていると、以後の部分の解析に失敗する（無視される）
というものです。

彼の修正だと小数点以下が単に無視されて嬉しくないんですが、
Floatで扱っていいものか、どうせ最後はRationalだし、では誰が
Rationalの面倒をみるべきか?と考えると、迂闊に手を出せないところに
潜ってしまいそうです。また、「小数点以下は扱わない」と、例外を
上げるという対応もあるのかもしれません。

というわけでふなばさんどうでしょう。

ちなみにxsd/datatypes.rbのXSD::XSDDateTimeでは、XML Schemaにあわせて
独自に解析をやっており、ここの修正の影響は受けません。依存してるのは、
標準添付ライブラリ中では、date.rbを除くと、optparse.rbくらいかな。