[ruby-dev:37648] Re: M17N of erb(1) Was: Re: Current status of 1.9.1 RC1's issues

咳といいます。

Yuguiさん、ありがとうございます。

On 2008/12/29, at 13:51, Yugui (Yuki Sonoda) wrote:

> On 12/28/08 10:08 PM, Masatoshi SEKI wrote:
>> それともERBも考慮するのでしたっけ？
>
> ERBもmagic commentを考慮します。ERB.newを呼び出 
> した場合はmagic commentと
> 渡された文字列のエンコーディングの衝突が考えられますが、これは 
> evalの仕様
> に倣ってmagic comment優先としました。

了解です。


>> 現在のdef_methodはこんな風になってますが
>> 今回の修正後もそれらしく動くでしょうか。
>
> メソッド化(とモジュール化、クラス化)を考えてい 
> なかったので、その点を修正
> して添付のパッチのようになりました。
>

確認しました。

このままcheckinしていただいてOKです。


気になった点は次の通りです。(どうでもよいことばかりです)

* bin/erb.rbですが、今ならoptparseを使う方が良いので 
しょうか。
   それともこのままの実装で大丈夫ですか?

私がerb.rbを書いた頃はoptparse以前だったか、単に 
知らなかったのか
コマンドオプションの処理を自前で書いてしまいました。
特にこだわりがあるわけではないので、改造しやすい方にしていただいて
かまいません。


* 多値の代入

1.9ではそう書くのがイディオムなのかもしれませんが、多値を 
returnして
それを受け取る場合に「*」で明示的に分解(?)する方 
がよいのですか?

次の変更が気になりました。

+    @src, @enc = *compiler.compile(str)

これは単に次のように書くのはイケてないのでしょうか?

      @src, @enc = compiler.compile(str)