-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
新正規表現エンジンで後方参照を使っていると正しくマッチしない場合がある #479
Comments
パターンから考えるにバックトラックからNFAに変わったせいだよね。 とりあえず問題を避けるだけなら |
正規表現エンジン周りについて知識が無いのですが,
確かにそれはおっしゃる通りですね…
とりあえずはそれで対処しようと思います. |
|
おお,ありがとうございます.とりあえずのほうはそれでいきます. ちなみに,この件はバグ報告したほうが良いでしょうか? |
|
した方が良いと思います。
7.4.029 で re=0 と re=2 は同じになってしまったのではないかと思います。 |
うーむ,NFA エンジンは未実装の機能とかもあるらしいので, |
承知しました. |
oh! そうなのですか…。
なるほど。そういう事であれば @rhysd さんの比較が正しいですね。 |
https://groups.google.com/d/msg/vim_dev/HcZVZT7fQno/ID_SndJv6CcJ
|
いえ,ありがとうございます. |
@k-takata おお,ありがとうございます.確かに |
こんな感じでどうでしょう. |
ここは違うと思います。 |
レビューありがとうございます.
な感じでしょうか. |
いいと思います。 |
バグレポート出しました. |
@rhysd 👍 乙です。 vim_devにレポートを出した場合はその反応をみてからクローズする運用になっています。 vim_devに投稿が出てきたらリンクをここにコメントしてもらえると助かります。 (なので再オープンさせてもらいますね:smile:) |
おお,よくルール分かっていなくて申し訳ないです. |
todo入りしました。
|
todo.txtの上から3番目に入ったので、重要度は高いと認識されたようです。(すぐ直るとは限りませんが。) |
おお,チェックありがとうございます. |
7.4.100で直ったようです。 |
こちらでも確認しました。 |
todo.txt からは削除されていないようなのですが,これは問題ないのでしょうか? |
todo.txtの更新はruntimeファイルの更新時に行われますので、タイムラグがあります。 |
なるほど,分かりました.ありがとうございます. |
問題点
は
set re=1
のとき\1
がa
にマッチしますが,set re=0
またはset re=2
のときはマッチしないようです.は問題なくマッチするので,グループ化部分と後方参照部分の間に何か文字が挟まるとマッチしなくなる気がしています(推測)
再現環境
The text was updated successfully, but these errors were encountered: