[ruby-dev:12137] Re: eval.c: NODE_FOR

  豊福です。

まつもとさん
> バグです。修正も豊福さんの示された通りです。ありがとうござい
> ます。

  私が言うまでもないででしょうが、実際の修正のときは
PUSH_ITER, POP_ITER も if else の外にくくり出していた
だけるとありがたい。

> |  それからここの
> |    ruby_sourcefile = file;
> |    ruby_sourceline = line;
> |はいらなそうな気もするのですが必要になるのはどういう
> |ケースですか。
> 
> eachの呼び出しでエラーになった際に行番号がずれることがあるの
> です。特に in の後に式が複数行になったときに。

  うっ、そうですか。

  for i in xxx {...} は xxx.each {|i| ...} にほぼ対応
するので NODE_ITER の

    PUSH_ITER(ITER_PRE);
    result = rb_eval(self, node->nd_iter);
    POP_ITER();

を each向けに展開すると

    PUSH_ITER(ITER_PRE);
    BEGIN_CALLARGS;
    recv = rb_eval(self, node->nd_recv);
    SETUP_ARGS(node->nd_args); # each は引数がないので実質この行はいらない
    END_CALLARGS;
    result = rb_call(CLASS_OF(recv),recv,each,0,0,0);
    POP_ITER();

と ruby_sourceline が出てこなかったのでいらないと
思ったのですが読みが浅かったか。


> このパッチを取り込もうと思うのですが、ChangeLogエントリはど
> のようにするのがお好みですか？
> 一応、姓名とアドレスをローマ字表記で、ということなんですが、
> 豊福さんって「つよし」だっけ「たけし」だっけ？

  たまーに間違われるのですが Javaなどの翻訳をされて
いる豊福剛さんは私とは別の人です。珍しい名前だから
姓だけでいいかと思ってましたが名も書かないとまずい
かな。

  豊福親信 (TOYOFUKU Chikanobu) です。

---
                        豊福
                        [e-mail]