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

Character properties ignore the ignore case flag #41

Closed
k-takata opened this Issue Aug 7, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@k-takata
Owner

k-takata commented Aug 7, 2014

/(?i)\p{lower}\p{upper}/ =~ "Ab"       → unmatch
/(?i)[\p{lower}][\p{upper}/] =~ "Ab"   → match

If they are in character class, the flag is not ignored. It is inconsistent.
Oniguruma 5.9.5 also behaves like this.

Perl 5.16 doesn't ignore the flag.

@k-takata k-takata added the bug label Aug 7, 2014

k-takata added a commit that referenced this issue Aug 7, 2014

Comment out some tests
They are different issue. See #41.

k-takata added a commit that referenced this issue Aug 8, 2014

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Aug 8, 2014

Owner

Fixed with 020d218.

Owner

k-takata commented Aug 8, 2014

Fixed with 020d218.

@k-takata k-takata closed this Aug 8, 2014

k-takata added a commit to k-takata/bregonig that referenced this issue Sep 13, 2014

Ver.3.06
* Onigmo (Oniguruma-mod) 5.15.0 for bregonig.dll を使用。
  https://github.com/k-takata/Onigmo/tree/Onigmo-5.15.0_for_bregonig
  - Unicode 7.0 に対応
  - Oniguruma 5.9.5 をマージ
  - 大量のグループを使うと落ちる問題を修正
    k-takata/Onigmo#24
  - /\x{1ffc}/i =~ "\x1ff3" がマッチしない問題を修正
  - UTF-16/32 で /[a-c#]+\W/ =~ "def#" がマッチしない問題を修正
  - /(?i)\u0149\u0149/ =~ "\u0149\u0149" がマッチしない問題を修正
    k-takata/Onigmo#40
  - 文字クラスの中で /w を使い、/i オプションを指定したときの問題を修正
    k-takata/Onigmo#4
  - 文字プロパティが /i オプションを無視する問題を修正
    k-takata/Onigmo#41
  - "ab" =~ /(?!^a).*b/ がマッチしない問題を修正
    k-takata/Onigmo#44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment