[ruby-dev:11598] Re: marshal.c パッチ

なかだです。

At Sun, 19 Nov 2000 04:37:48 +0900,
Kazuhiro NISHIYAMA <[e-mail]> wrote:
> それからMARSHAL_MAJORとMARSHAL_MINORのチェックのところを変更して、
> 新しいのか古いのかわかりやすくしました。

  ちと直しすぎでは。(can read) といっておいて読まないってのは…。


--- marshal.c~	Sun Nov 19 11:15:43 2000
+++ marshal.c	Sun Nov 19 11:16:52 2000
@@ -1031,19 +1031,17 @@ marshal_load(argc, argv)
 	volatile VALUE hash;	/* protect from GC */
 
 	minor = r_byte(&arg);
-	if (minor == MARSHAL_MINOR) {
-	    arg.symbol = st_init_numtable();
-	    arg.data   = hash = rb_hash_new();
-	    if (NIL_P(proc)) arg.proc = 0;
-	    else             arg.proc = proc;
-	    v = rb_ensure(load, (VALUE)&arg, load_ensure, (VALUE)&arg);
-	}
-	else if (minor < MARSHAL_MINOR) {
+	if (minor < MARSHAL_MINOR) {
 	    rb_warn("old marshal file format (can read)");
 	}
-	else {
+	else if (minor > MARSHAL_MINOR) {
 	    rb_warn("new marshal file format (can read)");
 	}
+	arg.symbol = st_init_numtable();
+	arg.data   = hash = rb_hash_new();
+	if (NIL_P(proc)) arg.proc = 0;
+	else             arg.proc = proc;
+	v = rb_ensure(load, (VALUE)&arg, load_ensure, (VALUE)&arg);
     }
     else if (major < MARSHAL_MAJOR) {
 	rb_raise(rb_eTypeError, "old marshal file format (can't read)");


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦