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

At Fri, 18 May 2001 21:05:45 +0900,
[e-mail] wrote:
> At Fri, 18 May 2001 12:58:27 +0900,
> GOTOU Yuuzou <[e-mail]> wrote:
> > そこで、デフォルトの initialize() を C で実装して、引数の処理
> > についてはそちらで行うようにすれば、サブクラスの initialize() 
> > で引数を調整して super() を呼べば済むので便利だと思うのですが、
> > いかがでしょうか？
> > 
> > そうなっていないものもあるので、お作法としてどうあるべきかとい
> > う所から始めたほうがいいかもしれませんけれど。
> 
>   そうなっていないのは古い仕様なのでは。少なくとも 1.6 以降はそ
> うすべきだと思います。

　標準添付のもの、特に昔からあるものはお手本にならないといけない
ですね。md5 の方も直しましょうか。

Index: ext/md5/md5init.c
===================================================================
RCS file: /src/ruby/ext/md5/md5init.c,v
retrieving revision 1.10
diff -u -r1.10 md5init.c
--- ext/md5/md5init.c	2001/05/16 21:16:02	1.10
+++ ext/md5/md5init.c	2001/05/18 14:38:51
@@ -76,20 +76,32 @@
 }
 
 static VALUE
+md5i_init(argc, argv, self)
+    int argc;
+    VALUE* argv;
+    VALUE self;
+{
+    VALUE str;
+
+    rb_scan_args(argc, argv, "01", &str);
+
+    if (!NIL_P(str)) md5i_update(self, str);
+
+    return self;
+}
+
+static VALUE
 md5i_new(argc, argv, class)
     int argc;
     VALUE* argv;
     VALUE class;
 {
-    VALUE obj, str;
+    VALUE obj;
     md5_state_t *md5;
 
     obj = Data_Make_Struct(class, md5_state_t, 0, free, md5);
     md5_init(md5);
-    rb_scan_args(argc, argv, "01", &str);
-    if (argc == 1) {
-	md5i_update(obj, str);
-    }
+    rb_obj_call_init(obj, argc, argv);
 
     return obj;
 }
@@ -102,6 +114,7 @@
     rb_define_singleton_method(cMD5, "new", md5i_new, -1);
     rb_define_singleton_method(cMD5, "md5", md5i_new, -1);
 
+    rb_define_method(cMD5, "initialize", md5i_init, -1);
     rb_define_method(cMD5, "update", md5i_update, 1);
     rb_define_method(cMD5, "<<", md5i_update, 1);
     rb_define_method(cMD5, "digest", md5i_digest, 0);


　あと、 curses の Window, socket の各クラス, tcltklib の TclTkIp,
tk の TkKernel にも initialize がないようです。

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

"Freeze this moment a little bit longer, make each impression
  a little bit stronger..  Experience slips away -- Time stand still"