[ruby-dev:45382] Re: Marshal.dumpにおけるインスタンス変数の取り扱いについて

2012年3月18日0:26 Yukihiro Matsumoto <[e-mail]>:

> |ところで, すごーく細かい話なのですが,
> |
> |rb_marshal_define_compat() で
> |
> |    :
> |    compat = ALLOC(marshal_compat_t);
> |    compat->newclass = Qnil;
> |    compat->oldclass = Qnil;
> |    compat->newclass = newclass;
> |    compat->oldclass = oldclass;
> |    :
> |
> |と一度Qnilを代入しているのですが, これはなぜでしょうか?
>
> なんでなんでしょうねえ。akrさんのr13413からそうみたいなんで、
> 彼に聞いてみないと。

r13413 では rb_gc_register_address が間に入っていたので、
そこで GC が起きることへの対策ですね。

% svn diff -c 13413 --diff-cmd diff -x '-u -p'
...
+    compat = ALLOC(marshal_compat_t);
+    compat->newclass = Qnil;
+    compat->oldclass = Qnil;
+    rb_gc_register_address(&compat->newclass);
+    rb_gc_register_address(&compat->oldclass);
+    compat->newclass = newclass;
+    compat->oldclass = oldclass;
+    compat->dumper = dumper;
+    compat->loader = loader;
...

rb_gc_register_address 内では ALLOC するので GC が起きる可能性があり、
GC が起きたときに compat->newclass や compat->oldclass にゴミが入っていると
よろしくなかったんじゃないでしょうか。
-- 
[田中 哲][たなか あきら][Tanaka Akira]