-
Notifications
You must be signed in to change notification settings - Fork 1
LTM alternation captures the wrong stuff when backtracking in Rakudo #3056
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
Comments
From @masak<TimToady> nr: say "abcde" ~~ / (a | b | bc | cde)+»/ |
From @usev6Looks like this was fixed for Moar (because of jnthn++'s work for RT #125285). $ perl6-m -e 'say "abcde" ~~ / (a | b | bc | cde)+»/' I added a test (fudged 'todo' for JVM) to S05-match/capturing-contexts.t with commit Raku/roast@82d271359f. |
1 similar comment
From @usev6Looks like this was fixed for Moar (because of jnthn++'s work for RT #125285). $ perl6-m -e 'say "abcde" ~~ / (a | b | bc | cde)+»/' I added a test (fudged 'todo' for JVM) to S05-match/capturing-contexts.t with commit Raku/roast@82d271359f. |
@usev6 - Status changed from 'new' to 'open' |
From @usev6This works now on the JVM backend as well (test in S05-match/capturing-contexts.t is passing). Fixed with nqp commit Raku/nqp@f22b0b2d0e I'm closing this ticket as 'resolved'. |
1 similar comment
From @usev6This works now on the JVM backend as well (test in S05-match/capturing-contexts.t is passing). Fixed with nqp commit Raku/nqp@f22b0b2d0e I'm closing this ticket as 'resolved'. |
@usev6 - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#116895 (status was 'resolved')
Searchable as RT116895$
The text was updated successfully, but these errors were encountered: