[ruby-dev:13240] Re: SizedQueue#pop causes deadlock

In message <[e-mail]>,
 `"Akinori MUSHA" <[e-mail]>' wrote:
> At Fri, 18 May 2001 00:10:07 +0900,
> U.Nakamura <[e-mail]> wrote:
> > sha1_new() 内の rb_obj_call_init() は不要ではないかと思うの
> > ですが、どうでしょう?
> > これがあると、new に引数を与えると ArgmentError になります。
> 
> 　その通りでした。下で引数がある場合には sha1_update(obj, arg)
> しているので、 initialize() を呼ぶ必要はありませんね。

このへんをどう書くかいつも迷うのですが、拡張ライブラリのクラス
を継承した場合などに initialize() を呼んで欲しくても、new() の
中で rb_scan_args() や Check_Type() していると、引数の数や評価
の仕方を変更できなくて困ることがあります。

そこで、デフォルトの initialize() を C で実装して、引数の処理
についてはそちらで行うようにすれば、サブクラスの initialize() 
で引数を調整して super() を呼べば済むので便利だと思うのですが、
いかがでしょうか？

そうなっていないものもあるので、お作法としてどうあるべきかとい
う所から始めたほうがいいかもしれませんけれど。

--
ごとうゆうぞう

Index: sha1-ruby.c
===================================================================
RCS file: /home/ncvs/ruby-lang/ruby/ext/sha1/sha1-ruby.c,v
retrieving revision 1.2
diff -u -r1.2 sha1-ruby.c
--- sha1-ruby.c	2001/05/17 16:13:34	1.2
+++ sha1-ruby.c	2001/05/18 03:17:56
@@ -64,22 +64,32 @@
 }
 
 static VALUE
+sha1_init(argc, argv, self)
+    int argc;
+    VALUE* argv;
+    VALUE self;
+{
+    VALUE arg;
+
+    rb_scan_args(argc, argv, "01", &arg);
+    if (!NIL_P(arg)){
+	sha1_update(self, arg);
+    }
+    return self;
+}
+
+static VALUE
 sha1_new(argc, argv, class)
     int argc;
     VALUE* argv;
     VALUE class;
 {
-    VALUE arg, obj;
+    VALUE obj;
     SHA1_CTX *sha1;
 
-    rb_scan_args(argc, argv, "01", &arg);
-    if (!NIL_P(arg)) Check_Type(arg, T_STRING);
-
     obj = Data_Make_Struct(class, SHA1_CTX, 0, free, sha1);
     SHA1Init(sha1);
-    if (!NIL_P(arg)) {
-	sha1_update(obj, arg);
-    }
+    rb_obj_call_init(obj, argc, argv);
 
     return obj;
 }
@@ -93,6 +103,7 @@
     rb_define_singleton_method(cSHA1, "sha1", sha1_new, -1);
 
     rb_define_method(cSHA1, "update", sha1_update, 1);
+    rb_define_method(cSHA1, "initialize", sha1_init, -1);
     rb_define_method(cSHA1, "<<", sha1_update, 1);
     rb_define_method(cSHA1, "digest", sha1_digest, 0);
     rb_define_method(cSHA1, "hexdigest", sha1_hexdigest, 0);