Bug #4546: ruby-indent-beg-re の値が壊れている

• Status: Closed
• Assignee: mrkn (Kenta Murata)
• Target version: 1.9.3
• ruby -v: trunk

[mrkn (Kenta Murata)]

vim-ruby の indent/ruby.vim を修正しようと思い ruby-mode.el を読んでいたところ
ruby-indent-beg-re の値が壊れているような気がしたので報告します。

r19205 で regexp-opt を使って正規表現を最適化するよう改善されていますが、
そのときに \| が抜けてしまっています。

r19204 の 場合:
"\(\s *\(class\|module\|def\)\)\|if\|unless\|case\|while\|until\|for\|begin"

r19205 の場合:
"\(\s *\(class\|def\|module\)\)\(?:begin\|case\|for\|if\|un\(?:less\|til\)\|while\)"

このように class, def, module を囲む括弧の直後にあった \| が落ちています。

それでも、なぜか適切にインデントできていて不思議だったので少しだけ調査してみました。
r19205 以降では、ruby-beginning-of-indent が classif や defbegin のような不適切な語をインデントの開始として
判断してしまいますが、このような語が存在しない場合はバッファの先頭まで戻っていました。
ですから、以下のパッチを当ててこの間違いを修正すると、長いファイルのインデント計算が
若干高速化されるかもしれません。

diff --git a/misc/ruby-mode.el b/misc/ruby-mode.el
index c799d8c..9023b77 100644
--- a/misc/ruby-mode.el
+++ b/misc/ruby-mode.el
@@ -72,7 +72,7 @@
"Regexp to match")

(defconst ruby-indent-beg-re

• (concat "\(\s *" (regexp-opt '("class" "module" "def") t) "\)"

• (concat "\(\s *" (regexp-opt '("class" "module" "def") t) "\)\|"
  (regexp-opt '("if" "unless" "case" "while" "until" "for" "begin")))
  "Regexp to match where the indentation gets deeper.")

[mrkn (Kenta Murata)]

=begin
新しい Redmine に慣れてなくて、patch が崩れてしまいました。
以下に再掲します。

diff --git a/misc/ruby-mode.el b/misc/ruby-mode.el
index c799d8c..9023b77 100644
--- a/misc/ruby-mode.el
+++ b/misc/ruby-mode.el
@@ -72,7 +72,7 @@
"Regexp to match")

(defconst ruby-indent-beg-re

• (concat "\(\s *" (regexp-opt '("class" "module" "def") t) "\)"

• (concat "\(\s *" (regexp-opt '("class" "module" "def") t) "\)\|"
  (regexp-opt '("if" "unless" "case" "while" "until" "for" "begin")))
  "Regexp to match where the indentation gets deeper.")

=end

[naruse (Yui NARUSE)]

• Status changed from Open to Assigned
• Assignee set to nobu (Nobuyoshi Nakada)

[nahi (Hiroshi Nakamura)]

• Target version changed from 2.0.0 to 1.9.3

[nobu (Nobuyoshi Nakada)]

• Assignee changed from nobu (Nobuyoshi Nakada) to mrkn (Kenta Murata)

いれといてください。

[mrkn (Kenta Murata)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r32338.
Kenta, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• misc/ruby-mode.el (ruby-indent-beg-re): Fix broken regular
  expression. Fixes #4546