Kouhei Sutou
kou****@cozmi*****
2008年 11月 21日 (金) 20:44:48 JST
須藤です。 In <1f73f****@mail*****> "[Hiki-dev:01245] Re: Hiki on Rack" on Fri, 21 Nov 2008 00:28:31 +0900, okkez <okkez****@gmail*****> wrote: > > あと、Rack対応以外にもAPIが変わっていたり（escapeHTML -> > > escape_html）、 > > API の変更は Rack 対応は別にしてもするべきだと思います。 > escapeHTML や unescapeHTML の使われ方がバラバラすぎです。 > String に定義するならするでいいのですが、全てそちらを使うように統一すべきです。 > # 少なくともアーカイブに同梱して配布している分に関しては。 > そこら辺の移行をやりやすくするために互換性を考慮しつつパッチを作成してみました。 かずひこさんが判断するんだと思いますが、1つのパッチに複数の 変更（Rack対応とescapeHTML -> escape_html）を含めない方が判 断しやすいと思います。 私だったらescapeHTML -> escape_htmlのパッチを投げて、それが 取り込まれたらRack対応パッチを投げると思います。 # 私もescapeHTMLよりescape_htmlの方が好みです。 > > 日本語のコメントがあったり、 > > コメントには日本語を入れない方が良いのですね。 > 拙いですが、英語っぽくしてみました。 私だったら、国際化対応プログラムのコメントやコミットログやテ ストやspecには日本語は使わないというだけなので、かずひこさん がOKを出すのならOKだと思います。 > > デバッグ用のコードらしいものが残っていたり（raise "OOO"）、 > > 差分を見ていただければ分かると思いますが、デバッグ用のコード > らしきものは私が入れたものではないです。 失礼しました。
