https://redmine.ruby-lang.org/
https://redmine.ruby-lang.org/favicon.ico?1711330511
2011-12-29T02:44:03Z
Ruby Issue Tracking System
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=22993
2011-12-29T02:44:03Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>= 現状の非互換について</p>
<p>Rubyへの取り込み方:<br>
cd Onigmo<br>
cp reg{comp,enc,error,exec,parse,syntax}.c reg{enc,int,parse}.h ~/ruby<br>
cp oniguruma.h ~/ruby/include/ruby/</p>
<p>Onigmo: edf9df1cefc28472085a857116153529253ffab5 (tmp/ruby-2.0.x)<br>
Ruby: r34150</p>
<blockquote>
<ol>
<li>Failure:<br>
test_parse_utf8(TestDateParse)<br>
[/home/naruse/ruby/test/date/test_date_parse.rb:651]:<br>
<"日本"> expected but was<br>
.</li>
</ol>
</blockquote>
<p>原因: 解析に正規表現を用いており、<br>
[:alpha:] が ASCII にしかマッチしない問題を踏んでいる。</p>
<blockquote>
<ol start="2">
<li>Failure:<br>
test_iso_8859_1(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</li>
</ol>
<blockquote>
<ol start="3">
<li>Failure:<br>
test_iso_8859_10(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</li>
</ol>
</blockquote>
<ol start="4">
<li>
<p>Failure:<br>
test_iso_8859_13(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
<li>
<p>Failure:<br>
test_iso_8859_14(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
<li>
<p>Failure:<br>
test_iso_8859_15(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
<li>
<p>Failure:<br>
test_iso_8859_16(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
<li>
<p>Failure:<br>
test_iso_8859_2(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
<li>
<p>Failure:<br>
test_iso_8859_3(TestISO8859) [(eval):5]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
<li>
<p>Failure:<br>
test_iso_8859_4(TestISO8859) [(eval):7]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
<li>
<p>Failure:<br>
test_iso_8859_9(TestISO8859) [(eval):5]:<br>
Expected /^SS$/i to match "\xDF".</p>
</li>
</ol>
</blockquote>
<p>未調査<br>
大文字小文字テーブル対応が壊れた?</p>
<blockquote>
<ol start="12">
<li>
<p>Failure:<br>
test_regexp_named_class(TestM17N) [/home/naruse/ruby/test/ruby/test_m17n.rb:543]:<br>
Expected /[[:space:]]/ to match " ".</p>
</li>
<li>
<p>Failure:<br>
test_posix_bracket(TestRegexp) [/home/naruse/ruby/test/ruby/test_regexp.rb:716]:<br>
Expected /\A[[:digit:]]+\z/ to match "0123456789".</p>
</li>
</ol>
</blockquote>
<p>POSIX 文字クラスが ASCII の範囲なため。</p>
<blockquote>
<ol start="14">
<li>Failure:<br>
test_unicode(TestRegexp) [/home/naruse/ruby/test/ruby/test_regexp.rb:792]:<br>
Expected /^\u0149$/i to match "ʼn".</li>
</ol>
</blockquote>
<p>2)-11) と同じ問題?</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=22994
2011-12-29T04:23:18Z
kosaki (Motohiro KOSAKI)
kosaki.motohiro@gmail.com
<ul></ul><blockquote>
<p>Ruby 1.9 では正規表現エンジンや M17N の基盤として Oniguruma を用いています。<br>
これを 2.0 では Oniguruma の改造版である、k-takata さんの Onigmo に置き換えようという話です。<br>
<a href="https://github.com/k-takata/Onigmo/tree/tmp/ruby-2.0.x" class="external">https://github.com/k-takata/Onigmo/tree/tmp/ruby-2.0.x</a></p>
</blockquote>
<p>現状 onigurumaのupstreamは活動が止まっているという認識なので乗り換え自体は賛成です。<br>
ただ将来的には libonigmo.so を動的にリンクするだけで済むような形態に移行できたらなあと<br>
思っており(実際にディストリからそういう要望をつい最近受けたばかりであるし)、k-takataさんと<br>
今後どのような形態で協力していくのか一回お話ししにいった方がいいんじゃないかと思ったり思わなかったり。k-tanakaさんにRubyのコミッタになって貰うか、なるせさんがonigumoのコミッタになるとかがベストだと思いますが、そういう敷居をあげる話をうっちゃっておくとしても、なんらかのコミュニケーションパスが確立されていて欲しいなあ。とかとか</p>
<a name="非互換変更は-Ruby-30まで入れられないから気にしてもしょうが無いかなあ"></a>
<h1 >非互換変更は Ruby 3.0まで入れられないから気にしてもしょうが無いかなあ・・・<a href="#非互換変更は-Ruby-30まで入れられないから気にしてもしょうが無いかなあ" class="wiki-anchor">¶</a></h1>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=22998
2011-12-30T15:37:18Z
k_takata (Ken Takata)
<ul></ul><p>POSIXブラケットは、Perlでは/a指定時にはASCII範囲に限定されるので、Onigmoではそれに合わせてあります。<br>
個人的にはあまり変更したくないのですが、変更すべきでしょうか。<br>
変更するとしたら、挙動オプションを1つ追加して、Ruby文法の場合はそれを有効にすることになると思いますが。</p>
<p>残りの問題は未調査です。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23003
2011-12-30T19:41:36Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>Motohiro KOSAKI wrote:</p>
<blockquote>
<p>ただ将来的には libonigmo.so を動的にリンクするだけで済むような形態に移行できたらなあと<br>
思っており(実際にディストリからそういう要望をつい最近受けたばかりであるし)、</p>
</blockquote>
<p>Ruby は Ruby 文法挙動オプション(コンパイルオプション)を指定した鬼車を用いているので、<br>
普通のバイナリではたとえソースが一致していてもダメです。<br>
この状況は鬼雲でも変わりませんから、そのような形態に移行するのは不可能なんじゃないかと思います。</p>
<blockquote>
<p>k-takataさんと<br>
今後どのような形態で協力していくのか一回お話ししにいった方がいいんじゃないかと思ったり思わなかったり。</p>
</blockquote>
<p>そうですねぇ、現時点で鬼雲で追加された機能を Ruby 側でどうするかはちょっと悩んでいるところではあります。</p>
<blockquote>
<p>k-tanakaさんにRubyのコミッタになって貰うか、なるせさんがonigumoのコミッタになるとかがベストだと思いますが、<br>
そういう敷居をあげる話をうっちゃっておくとしても、なんらかのコミュニケーションパスが確立されていて欲しいなあ。<br>
とかとか</p>
</blockquote>
<p>k-tanaka さんがコミッタになって頂けるとわたしが楽ですね。</p>
<p>Ken Takata wrote:</p>
<blockquote>
<p>POSIXブラケットは、Perlでは/a指定時にはASCII範囲に限定されるので、Onigmoではそれに合わせてあります。<br>
個人的にはあまり変更したくないのですが、変更すべきでしょうか。<br>
変更するとしたら、挙動オプションを1つ追加して、Ruby文法の場合はそれを有効にすることになると思いますが。</p>
</blockquote>
<p>既定路線では 2.0 では非互換を導入しないという予定ですので、変更されない場合は 2.0 ではマージできない、<br>
と言うことになると思います。<br>
とはいえ、この辺はまつもとさんのさじ加減一つですが。</p>
<p>ところで、Ruby って /a のカスタマイズではなく、/u のカスタマイズではないでしょうか。<br>
残りの大文字小文字の問題はこれが起因なんじゃないかと思います。<br>
ので、仮に変更するとしたら、POSIXブラケットの変更ではなく、\d\w\sの変更でしょう。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23007
2011-12-30T23:03:18Z
k_takata (Ken Takata)
<ul></ul><p>Yui NARUSE wrote:</p>
<blockquote>
<p>この状況は鬼雲でも変わりませんから、そのような形態に移行するのは不可能なんじゃないかと思います。</p>
</blockquote>
<p>同意見です。エンコーディング周りの変更もそう簡単にはマージできそうにないですし。</p>
<blockquote>
<p>k-tanaka さんがコミッタになって頂けるとわたしが楽ですね。</p>
</blockquote>
<p>Rubyのビルドすらしたことのない人がコミッタになるのは互いの不幸の元ではないかと。</p>
<blockquote>
<p>既定路線では 2.0 では非互換を導入しないという予定ですので、変更されない場合は 2.0 ではマージできない、<br>
と言うことになると思います。</p>
</blockquote>
<p>なるほど、そうなってしまいますか。</p>
<blockquote>
<p>ところで、Ruby って /a のカスタマイズではなく、/u のカスタマイズではないでしょうか。<br>
残りの大文字小文字の問題はこれが起因なんじゃないかと思います。<br>
ので、仮に変更するとしたら、POSIXブラケットの変更ではなく、\d\w\sの変更でしょう。</p>
</blockquote>
<p>すみません。よく状況が把握できていません。<br>
そうだとすると、regparse.c の OnigSyntaxRuby 構造体の中の ONIG_OPTION_ASCII_RANGE を<br>
0 に書き換えると、大文字小文字の部分はパスしますか?</p>
<p>もし /u のカスタマイズだと、\d\w\sが常にASCIIの範囲内になってしまって、<br>
/u/a の有用性がずいぶん落ちてしまいますね。(非互換を導入しないならそれも致し方なしですか。)</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23009
2011-12-30T23:39:55Z
k_takata (Ken Takata)
<ul></ul><p>すみません。大文字小文字の問題は、Onigmo 5.11.4でcase-insensitiveなBM法を<br>
導入したときに混入したバグと判明しました。(どう修正すべきかまだ分かっていませんが。)<br>
これが修正できれば、/a で POSIX 文字クラスを ASCII に限定しないオプションと合わせることで、<br>
非互換点は解決できそうです。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23024
2011-12-31T23:48:22Z
k_takata (Ken Takata)
<ul><li><strong>File</strong> <a href="/attachments/2363">0001-add-ONIG_SYN_POSIX_BRACKET_ALWAYS_ALL_RANGE-option.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2363/0001-add-ONIG_SYN_POSIX_BRACKET_ALWAYS_ALL_RANGE-option.patch">0001-add-ONIG_SYN_POSIX_BRACKET_ALWAYS_ALL_RANGE-option.patch</a> added</li><li><strong>File</strong> <a href="/attachments/2364">0002-bug-ss-i-doesn-t-match-x-DF.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2364/0002-bug-ss-i-doesn-t-match-x-DF.patch">0002-bug-ss-i-doesn-t-match-x-DF.patch</a> added</li></ul><p>POSIX 文字クラスを ASCII に限定しないオプションの追加と、大文字小文字問題の修正パッチを作成しました。<br>
特に問題なければ、Onigmo本体に反映したいと思います。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23132
2012-01-06T21:31:54Z
k_takata (Ken Takata)
<ul><li><strong>File</strong> <a href="/attachments/2372">0001-add-ONIG_SYN_POSIX_BRACKET_ALWAYS_ALL_RANGE-option.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2372/0001-add-ONIG_SYN_POSIX_BRACKET_ALWAYS_ALL_RANGE-option.patch">0001-add-ONIG_SYN_POSIX_BRACKET_ALWAYS_ALL_RANGE-option.patch</a> added</li><li><strong>File</strong> <a href="/attachments/2373">0002-bug-ss-i-doesn-t-match-x-DF.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2373/0002-bug-ss-i-doesn-t-match-x-DF.patch">0002-bug-ss-i-doesn-t-match-x-DF.patch</a> added</li></ul><p>前回のパッチはOnigmoのmasterブランチで作成したものでしたが、<br>
tmp/ruby-2.0.xブランチにはそのまま適用できませんでした。<br>
tmp/ruby-2.0.xブランチに適用できるようにパッチを作成し直しました。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23146
2012-01-07T22:40:09Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>ありがとうございます、大文字小文字問題は確かに直りました。</p>
<p>残っているのが一点、以下がマッチしなくなっています。<br>
Bug <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: \b and \B don't match properly (Closed)" href="https://redmine.ruby-lang.org/issues/5208">#5208</a> と関連しているような気もしますが。</p>
<p>/\u3042\b[ a]/=~"\u3042a" #=> nil</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23150
2012-01-08T00:11:45Z
k_takata (Ken Takata)
<ul></ul><p>おや、おかしいですね。<br>
手元では、/(?a)あ\b[ a]/ は "あa" にマッチしています。nilを返すべきところを返さなくなったということでしょうか。<br>
(Ruby 1.9.3では、nilを返すようですが。)</p>
<p>私としては、Bug #5208にも書いたように\b,\Bは\w,\Wと連動しているべきだと考えていますので、マッチしていないとすれば想定外ですので、もう少し詳しく教えていただけますでしょうか。</p>
<p><a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: \b and \B don't match properly (Closed)" href="https://redmine.ruby-lang.org/issues/5208">#5208</a> note-3より</p>
<blockquote>
<p>考えたんですが 1.9 では現状のままとします。<br>
/u /a が将来取り込まれたらその時に改めて考えましょう。</p>
</blockquote>
<p>ちょうど今がそのときでしょうか。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23151
2012-01-08T00:39:47Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>Ken Takata wrote:</p>
<blockquote>
<p>おや、おかしいですね。<br>
手元では、/(?a)あ\b[ a]/ は "あa" にマッチしています。nilを返すべきところを返さなくなったということでしょうか。<br>
(Ruby 1.9.3では、nilを返すようですが。)</p>
</blockquote>
<p>あああ、その通りです。書いててごっちゃになってました。</p>
<blockquote>
<p>私としては、Bug #5208にも書いたように\b,\Bは\w,\Wと連動しているべきだと考えていますので、マッチしていないとすれば想定外ですので、もう少し詳しく教えていただけますでしょうか。</p>
<p><a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: \b and \B don't match properly (Closed)" href="https://redmine.ruby-lang.org/issues/5208">#5208</a> note-3より</p>
<blockquote>
<p>考えたんですが 1.9 では現状のままとします。<br>
/u /a が将来取り込まれたらその時に改めて考えましょう。</p>
</blockquote>
<p>ちょうど今がそのときでしょうか。</p>
</blockquote>
<p>/u /a は Perl の挙動にあってる方がいいと思うんですが、無指定だとなぁ、うーん。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23178
2012-01-09T05:28:39Z
k_takata (Ken Takata)
<ul><li><strong>File</strong> <a href="/attachments/2376">0003-support-for-Ruby-1.9.3-compatible-b-B-and-POSIX-brac.patch</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/2376/0003-support-for-Ruby-1.9.3-compatible-b-B-and-POSIX-brac.patch">0003-support-for-Ruby-1.9.3-compatible-b-B-and-POSIX-brac.patch</a> added</li></ul><p>/a /u はPerlに合わせて、無指定あるいは /d だとRuby 1.9.3と同じ仕様とすると添付のパッチのような感じでしょうか。<br>
この仕様がよいのか正直決めかねているところですが。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23231
2012-01-10T22:45:04Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>0003-support-for-Ruby-1.9.3-compatible-b-B-and-POSIX-brac.patch で既存のテストが全て通ることを確認しました。</p>
<p>しかし、確かに perlre を見直すと、/d で 1.9 仕様というのもちょっと引っかかりますねぇ、うーん。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23232
2012-01-10T22:47:33Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>そういえば、/u って Ruby だと UTF-8 文字コード指定で既に存在するんですよね。<br>
これもどうするか考えなければというか、いっそ /a /u 入れないのがいいんだろうか…。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23327
2012-01-15T14:38:08Z
k_takata (Ken Takata)
<ul></ul><p>私としては、せっかく (?au) を入れたので使えないのはもったいないなと思います。Perlの他にはPythonでも /u 相当が使えることですし需要はあるでしょう。<br>
/u がぶつかる問題は、大文字で /A/U にするとか?</p>
<p>/d は確かに悩ましいですが、/d で 1.9 仕様とするか、1.9 仕様は破棄して /a をデフォルトにする(/d は無し)か、どちらかしかないのではないかと思っています。(/a で 1.9 仕様とするのは \b の挙動を考えると選択肢からは除外したい。)</p>
<p>なお、#note-1 のマージ方法だと、enc ディレクトリ以下をコピーしていないため以下の制限があります。</p>
<ol>
<li>Unicode系エンコーディングで、\X, \R が仕様通りに動作しない。(OnigEncodingType にflagsメンバを追加して、それを参照しているため。)(鬼雲新機能)</li>
<li>Unicode系エンコーディングで、\p{In_XXX} によるUnicodeブロック名が使えない。(鬼雲新機能)</li>
<li>Unicode系エンコーディングで、大文字小文字変換テーブルがUnicode 6.0に対応していない。(鬼雲新機能)</li>
<li>Unicode系エンコーディングで、\p{C} が正しくない。(\p{Cn} の範囲が含まれていない。)(Ruby 1.9のバグ)</li>
<li>Unicode系エンコーディングで、\p{LC} (\p{Cased_Letter}) が使えない。(鬼雲新機能)</li>
<li>Unicode系エンコーディングで、本来は内部実装用の \p{NEWLINE} が使えるようになってしまっている。(鬼車5.9.2のバグ(?))</li>
<li>Unicode系エンコーディングで、なぜか 0x09~0x0d が \p{Print} に含まれている。(鬼車5.9.2のバグ(?))</li>
<li>sjis/euc_jpで、全角アルファベット、キリル文字、ギリシャ文字の大文字小文字同一視検索ができない。(鬼雲新機能)</li>
<li>sjis/euc_jpで、\p{Han}, \p{Latin}, \p{Greek}, \p{Cyrillic} が使えない。(鬼雲新機能)</li>
</ol>
<p>8,9についてはJIS X 0208を基準にしているので、JIS X 0212や0213はどうするのかという課題があります。<br>
tool/enc-unicode.rb (と tool/CaseFolding.py) もマージが必要です。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23375
2012-01-20T17:02:26Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>Ken Takata wrote:</p>
<blockquote>
<p>私としては、せっかく (?au) を入れたので使えないのはもったいないなと思います。Perlの他にはPythonでも /u 相当が使えることですし需要はあるでしょう。<br>
/u がぶつかる問題は、大文字で /A/U にするとか?</p>
</blockquote>
<p>/A /U は今度は Perl が /A /U を導入した時に困るので、2.0 で /u を deprecated にして、どこかで変更ですかねぇ。</p>
<blockquote>
<p>/d は確かに悩ましいですが、/d で 1.9 仕様とするか、1.9 仕様は破棄して /a をデフォルトにする(/d は無し)か、どちらかしかないのではないかと思っています。(/a で 1.9 仕様とするのは \b の挙動を考えると選択肢からは除外したい。)</p>
</blockquote>
<p>そうですね、/d で 1.9 ですかね。</p>
<blockquote>
<p>なお、#note-1 のマージ方法だと、enc ディレクトリ以下をコピーしていないため以下の制限があります。<br>
tool/enc-unicode.rb (と tool/CaseFolding.py) もマージが必要です。</p>
</blockquote>
<p><a href="https://github.com/nurse/ruby/tree/onigmo" class="external">https://github.com/nurse/ruby/tree/onigmo</a> マージしました。<br>
以下のような変更をしています。r34236 は未マージです。</p>
<ul>
<li>enc/cp932.c 内の sjis.c への参照を shift_jis.c へと変更</li>
<li>enc/cp932.c から enc/windows_31j.c に rename</li>
<li>Windows-31J のエンコーディング定義を enc/shift_jis.c から enc/windows_31j.c に移動</li>
<li>enc/koi8.c を削除 (KOI8ってエンコーディングは存在しないはず。これが何者なのかわからない)</li>
<li>enc/cp1251.c を削除 (Ruby では enc/windows_1251.c)</li>
</ul>
<blockquote>
<p>8,9についてはJIS X 0208を基準にしているので、JIS X 0212や0213はどうするのかという課題があります。</p>
</blockquote>
<p>まず、JIS X 0213 については、現在の Ruby は黙殺しています。<br>
仮にサポートするとしても、Shift_JISX0213/EUC-JISX0213 などの 0213 系エンコーディングでの話でしょう。<br>
また、SJIS 系エンコーディングでは JIS X 0212 を含むエンコーディングが存在しないのでこれも関係しません。<br>
EUC-JP と eucJP-ms は JIS X 0212 を含むので、対応するのはありかもしれません。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23386
2012-01-21T11:52:40Z
k_takata (Ken Takata)
<ul></ul><p>Onigmo 5.13.0を公開しました。ONIG_SYNTAX_RUBYにて、/d で1.9仕様としています。<br>
tmp/ruby-2.0.xブランチも更新し、r34236もマージしています。(masterブランチにはr34236をマージすべきか判断が付かなかったので保留しています。)<br>
enc/shift_jis.cは少し手を入れています。enc/windows_31j.cを使っても、\p{Han} の範囲が変更されていなかったようですので。</p>
<blockquote>
<p>まず、JIS X 0213 については、現在の Ruby は黙殺しています。<br>
そうであれば、Onigmoも当分そのままにしておこうと思います。</p>
</blockquote>
<blockquote>
<p>EUC-JP と eucJP-ms は JIS X 0212 を含むので、対応するのはありかもしれません。<br>
こちらは頭の片隅にでも留めておきます。</p>
</blockquote>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23399
2012-01-22T23:13:43Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>Ken Takata wrote:</p>
<blockquote>
<p>Onigmo 5.13.0を公開しました。ONIG_SYNTAX_RUBYにて、/d で1.9仕様としています。<br>
tmp/ruby-2.0.xブランチも更新し、r34236もマージしています。(masterブランチにはr34236をマージすべきか判断が付かなかったので保留しています。)</p>
</blockquote>
<p>r34236 をそのまま取り込むべきかは迷う所ですが、不正な文字の場合の戻り値が不統一という<br>
視点自体は妥当なものなんじゃないかと思っています。</p>
<p>というわけで、テストの失敗もでなくなったので、特に異論が無ければ2月にはマージしますかね。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23772
2012-02-13T23:33:53Z
mame (Yusuke Endoh)
mame@ruby-lang.org
<ul></ul><p>ほとんどの議論は横道 or 枝葉末節っぽい感じがしたので読んでないのですが、<br>
結論としては onigmo を取り込む方向なんでしょうか。<br>
あと、k-takata さんにはコミッタになって頂ける予定でしょうか。</p>
<a name="もしそうだったら早速ですが-1200-1201-あたりをご検討願いたいと"></a>
<h1 >もしそうだったら、早速ですが <a class="issue tracker-2 status-6 priority-4 priority-default closed" title="Feature: Possibility for using named and normal groups together in regular expressions (Rejected)" href="https://redmine.ruby-lang.org/issues/1200">#1200</a> 、#1201 あたりをご検討願いたいと<a href="#もしそうだったら早速ですが-1200-1201-あたりをご検討願いたいと" class="wiki-anchor">¶</a></h1>
<a name="考えております"></a>
<h1 >考えております。<a href="#考えております" class="wiki-anchor">¶</a></h1>
<p>--<br>
Yusuke Endoh <a href="mailto:mame@tsg.ne.jp" class="email">mame@tsg.ne.jp</a></p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23806
2012-02-14T08:15:02Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul></ul><p>Yusuke Endoh wrote:</p>
<blockquote>
<p>ほとんどの議論は横道 or 枝葉末節っぽい感じがしたので読んでないのですが、<br>
結論としては onigmo を取り込む方向なんでしょうか。</p>
</blockquote>
<p>わたしが 1.9.3 やってて遅れているだけですね。<br>
終わったら取り込みます。</p>
<blockquote>
<p>あと、k-takata さんにはコミッタになって頂ける予定でしょうか。</p>
</blockquote>
<p>なって頂けない感じなので当面はわたしがマージしようかと思っています。</p>
<blockquote>
<a name="もしそうだったら早速ですが-1200-1201-あたりをご検討願いたいと"></a>
<h1 >もしそうだったら、早速ですが <a class="issue tracker-2 status-6 priority-4 priority-default closed" title="Feature: Possibility for using named and normal groups together in regular expressions (Rejected)" href="https://redmine.ruby-lang.org/issues/1200">#1200</a> 、#1201 あたりをご検討願いたいと<a href="#もしそうだったら早速ですが-1200-1201-あたりをご検討願いたいと" class="wiki-anchor">¶</a></h1>
<a name="考えております"></a>
<h1 >考えております。<a href="#考えております" class="wiki-anchor">¶</a></h1>
</blockquote>
<p>なって頂けなくてもこの辺りは見て頂いているようなので検討していただけることを期待していますが、<br>
<a class="issue tracker-2 status-6 priority-4 priority-default closed" title="Feature: Possibility for using named and normal groups together in regular expressions (Rejected)" href="https://redmine.ruby-lang.org/issues/1200">#1200</a> は本家でなく Ruby 側のデザインで許可していない部分なので却下ですね、しました。<br>
<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Add relative group reference in back reference with nest level in regular expressions (e.g. \k<-5... (Closed)" href="https://redmine.ruby-lang.org/issues/1201">#1201</a> も named capture 使えよって感がありますがどうかな。</p>
Ruby master - Feature #5820: Merge Onigmo to Ruby 2.0
https://redmine.ruby-lang.org/issues/5820?journal_id=23944
2012-02-17T16:42:28Z
naruse (Yui NARUSE)
naruse@airemix.jp
<ul><li><strong>Status</strong> changed from <i>Assigned</i> to <i>Closed</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>This issue was solved with changeset r34663.<br>
Yui, thank you for reporting this issue.<br>
Your contribution to Ruby is greatly appreciated.<br>
May Ruby be with you.</p>
<hr>
<ul>
<li>Merge Onigmo-5.13.1. <a href="/issues/5820">[ruby-dev:45057]</a> [Feature <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Feature: Merge Onigmo to Ruby 2.0 (Closed)" href="https://redmine.ruby-lang.org/issues/5820">#5820</a>]<br>
<a href="https://github.com/k-takata/Onigmo" class="external">https://github.com/k-takata/Onigmo</a><br>
cp reg{comp,enc,error,exec,parse,syntax}.c reg{enc,int,parse}.h<br>
cp oniguruma.h<br>
cp tool/enc-unicode.rb<br>
cp -r enc/</li>
</ul>