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
chars that look different in .fc break :i regex matches #6132
Comments
From @zoffixznetThe mere presence of this ligature, breaks the :i regex match for 'T': IOninja m: say "st T" ~~ m:i/T/ Note that the issue goes away without `:i` or if the space is removed. Bisectable points to a version bump that brought this commit: MoarVM/MoarVM@ab426c1 |
From @zoffixznetLooks like might be related to char count: https://irclog.perlgeek.de/perl6-dev/2017-03-08#i_14225428 10:54 jnthn m: say "st".fc.chars 10:59 m: say "ststT" ~~ m:i/T/ |
From @zoffixznetAnother datapoint. Use .match method with such a regex hangs: 13:14 m: "ststT".match(m:i/T/) |
From @zoffixznetOoops. Never mind. That hang is due to RT#130940 As for this ticket, I've added fudged tests to cover it now: Raku/roast@2fa32e11eb Please note, this ticket is now marked as a release blocker, with the release scheduled to occur on March 18, 2017. |
From @samcvOn Wed, 08 Mar 2017 05:36:41 -0800, cpan@zoffix.com wrote:
I have fixed this as of this commit MoarVM/MoarVM@5845784 There is now a nqp op that used to not be callable as nqp::eqatic, even though it was called another way during the regex. There are now some tests in the nqp repo that test this case insensitive op which is the one that was breaking the regex. |
The RT System itself - Status changed from 'new' to 'open' |
@samcv - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#130953 (status was 'resolved')
Searchable as RT130953$
The text was updated successfully, but these errors were encountered: