[ruby-dev:35988] Re: block parameter for Delagator

遠藤です。

2008/08/26 16:33 Keiju ISHITSUKA <[e-mail]>:
> --
> require "delegate"
>
> d1 = SimpleDelegator.new([1,2])
> d2 = SimpleDelegator.new([3,4])
>
>
> [d1, d2].each{|a, b| p a, b}
> --
>
> Ruby1.8では, 配列の要素が, 各a, b に代入されます.
> Ruby1.9 では, a だけに d1, d2代入され, b は nil になります.
>
> これって, 1.9 の仕様です? 言語の仕様? or Delegator の仕様?


バグかどうかはわかりませんが直してみました。
が、以下のコードが 3 倍遅くなりました。どんなもんでしょう。

$ time ./ruby.org -e '([1] * 5000000).each {|a, b| a }'

real    0m1.180s
user    0m0.740s
sys     0m0.430s

$ time ./ruby.new -e '([1] * 5000000).each {|a, b| a }'

real    0m3.420s
user    0m2.930s
sys     0m0.480s


一応パッチです。

Index: vm_insnhelper.c
===================================================================
--- vm_insnhelper.c	(revision 18871)
+++ vm_insnhelper.c	(working copy)
@@ -704,6 +704,7 @@
 	int i;
 	int argc = orig_argc;
 	const int m = iseq->argc;
+	VALUE ary;

 	th->mark_stack_len = argc;

@@ -714,8 +715,7 @@
 	 */
 	if (!(iseq->arg_simple & 0x02) &&
 	    (m + iseq->arg_post_len) > 0 &&
-	    argc == 1 && TYPE(argv[0]) == T_ARRAY) {
-	    VALUE ary = argv[0];
+	    argc == 1 && !NIL_P(ary = rb_check_array_type(argv[0]))) {
 	    th->mark_stack_len = argc = RARRAY_LEN(ary);

 	    CHECK_STACK_OVERFLOW(th->cfp, argc);


> また, 1.8 の様に振る舞うようにDelegator似のクラス側で対応することって出
> 来ますでしょうか?

たぶん、できないような気がします。

-- 
Yusuke ENDOH <[e-mail]>