[ruby-dev:32335] Re: x=0;defined?(foo(x)) dumps core

遠藤です。

さっきのパッチは不完全で def foo;end;x=0;defined?(foo(x)) で
落ちるようになってました。すみません。

もとのコードがどういう意図で書かれていたのかよくわからないのですが、
これが正解でしょうか。

$ ./ruby -e 'p defined?(foo)'
nil
$ ./ruby -e 'p defined?(foo(x))'
nil
$ ./ruby -e 'x=0;p defined?(foo(x))'
nil
$ ./ruby -e 'def foo;end;p defined?(foo)'
"method"
$ ./ruby -e 'def foo;end;p defined?(foo(x))'
nil
$ ./ruby -e 'def foo;end;x=0;p defined?(foo(x))'
"method"

# 最後の jump とか要らない気もします。


Index: compile.c
===================================================================
--- compile.c	(revision 13997)
+++ compile.c	(working copy)
@@ -2295,13 +2295,11 @@
 	if (node->nd_args) {
 	    lfalse = NEW_LABEL(nd_line(node));
 	    defined_expr(iseq, ret, node->nd_args, lfinish, Qfalse);
+	    ADD_INSNL(ret, nd_line(node), branchunless, lfalse);
 	}
 	if (!self) {
 	    LABEL *lcont = NEW_LABEL(nd_line(node));

-	    if (lfalse) {
-		ADD_INSNL(ret, nd_line(node), branchunless, lfalse);
-	    }
 	    defined_expr(iseq, ret, node->nd_recv, lfinish, Qfalse);
 	    ADD_INSNL(ret, nd_line(node), branchif, lcont);
 	    if (lfalse) {
@@ -2319,9 +2317,8 @@
 	    ADD_INSN(ret, nd_line(node), putself);
 	    ADD_INSN3(ret, nd_line(node), defined, INT2FIX(DEFINED_FUNC),
 		      ID2SYM(node->nd_mid), needstr);
+	    ADD_INSNL(ret, nd_line(node), jump, lfinish);
 	    if (lfalse) {
-		ADD_INSNL(ret, nd_line(node), branchif, lfinish);
-
 		ADD_LABEL(ret, lfalse);
 		ADD_INSN(ret, nd_line(node), putnil);
 		ADD_INSNL(ret, nd_line(node), jump, lfinish);

--
Yusuke ENDOH <[e-mail]>