[ruby-dev:21151] Re: Marshal upgrade

新井です。

In message "[ruby-dev:21088] Re: Marshal upgrade"
  on 03 Aug 2003 09:24:33 +0900,
  [e-mail] (Yukihiro Matsumoto) wrote:
> まつもと ゆきひろです
> 
> In message "[ruby-dev:21087] Re: Marshal upgrade"
>     on 03/08/03, Koji Arai <[e-mail]> writes:
> 
> |> これによって、今まで undocumented だった _dump_data/_load_data は
> |> 必要なくなるのでしょうか？
> 
> 必要なくなります。undocumentのままで良いと思います。

大した話じゃないのですが、ソース(marshal.c)上

		if (!rb_respond_to(obj, s_dump_data)) {
		    rb_raise(rb_eTypeError,
			     "class %s needs to have instance method `_dump_data'",
			     rb_obj_classname(obj));
		}

っと _dump_data の記述が残っているのはよろしくないですかね？
s_dump_data の存在チェックをしたあとに「marshal_dump を定義
しろ」というのも変ですが

Wed Aug  6 00:48:37 2003  Koji Arai  <[e-mail]>

	* marshal.c (w_object): should recommend marshal_dump rather than
	_dump_data.

Index: marshal.c
===================================================================
RCS file: /usr/local/cvsup/ruby/ruby/marshal.c,v
retrieving revision 1.95
diff -u -r1.95 marshal.c
--- marshal.c	4 Aug 2003 05:28:50 -0000	1.95
+++ marshal.c	5 Aug 2003 15:48:35 -0000
@@ -641,7 +641,7 @@
 		w_class(TYPE_DATA, obj, arg);
 		if (!rb_respond_to(obj, s_dump_data)) {
 		    rb_raise(rb_eTypeError,
-			     "class %s needs to have instance method `_dump_data'",
+			     "class %s needs to have instance method `marshal_dump'",
 			     rb_obj_classname(obj));
 		}
 		v = rb_funcall(obj, s_dump_data, 0);

--
新井康司 (Koji Arai)