From: "NARUSE, Yui" <naruse@...>
Date: 2010-02-24T02:07:00+09:00
Subject: [ruby-dev:40469] Re: [ruby-cvs:33954] Ruby:r26739 (trunk): * ext/openssl/ossl_digest.c (GetDigestPtr): Allow to pass the

(2010/02/24 1:57), Akinori MUSHA wrote:
> At Wed, 24 Feb 2010 01:28:24 +0900,
> NARUSE, Yui wrote:
>> 議論なしのAPI変更に加えて、ext/openssl への変更なのであまり感心しないんですが、
> 
> 　メンテナ不在だと思っていたので気軽にいじってしまいました。

メンテナの不在は自由に仕様変更が出来る事を意味しないはずです。
そもそも、仕様変更に寛容だった昔の牧歌的なノリは困ります。
今は RubySpec や各種ドキュメントなど影響範囲が大きいのですから。

また、別の Digest の継承もそうなんですが、ext/openssl への変更はおっかないので
先にMLに差分を投げてレビューを貰って欲しいです。

>> とりあえず、
>> irb(main):002:0> OpenSSL::HMAC.digest(1,"foo","bar")
>> TypeError: wrong argument (Fixnum)! (Expected kind of OpenSSL::Digest)
>> だったエラーが、
>> TypeError: can't convert Fixnum into String
>> というエラーに変わっているのは意図していますか。
> 
> 　そういう変化をもたらすことを予想していたかと言われればイエスです。
> 
> 　もしご存じでしたら、そのテストの意図を教えてください。

ext/openssl でバグが出ると嫌なので差分を見て気付きました。

>> また、API変更をよしとするならば、テストも追加するべきじゃありませんか。
> 
> 　そもそも今のAPIに対するテストがなかったので追加しました。

そのテストはどこですか?

-- 
NARUSE, Yui  <naruse@airemix.jp>