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

咳といいます。

On 2008/12/28, at 15:02, Yugui (Yuki Sonoda) wrote:

> Yuguiです。
>
> On 12/26/08 7:06 PM, Yugui (Yuki Sonoda) wrote:
>> で、全部合わせて添付のパッチのようになりました。どうでしょ 
>> うか。
>> <%# -*- coding: ENCODING-NAME -*-%>
>
> 咳さんから特に反対がなければパッチをコミットして1.9.1 
> に含めたいと思いま
> すが、いかがでしょうか。

返事が遅くてすみません。
ちょっとだけ忙しくてまだちゃんと読めていません。
bin/erbの方は私も同様な修正を考えていました。
Yuguiさんの修正に賛成します。



lib/erb.rbに手が入っていますが、
bin/erbだけmagic comementを考慮するのでしたっけ？
それともERBも考慮するのでしたっけ？

ライブラリ作者としては、ERB.new.runよりも 
ERB.new.resultを、
そしてERB.new.resultよりもメソッド化を、使ってほしい（そう
いう風に誘導したい）と願っているのですが、
メソッド化するときのscript encodingってどうなるんだろう 
とか、
私がよく理解できていないので躊躇してます。

現在のdef_methodはこんな風になってますが
今回の修正後もそれらしく動くでしょうか。

   def def_method(mod, methodname, fname='(ERB)')  # :nodoc:
     mod.module_eval("def #{methodname}\n" + self.src + "\nend\n",  
fname, -1)
   end

大丈夫そうなら、これでOKです。


>
>
> あ、ついでに、bin/erbのオプション処理のパッチも書いて 
> みたので添付しま
> す。今は-s val形式と--long val形式を受け付けますが、
> -sval 形式と--long=val形式も受け付けるようにしてみました。

こっちの修正は了解しました。