We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Migrated from rt.perl.org#120511 (status was 'resolved')
Searchable as RT120511$
The text was updated successfully, but these errors were encountered:
17:50 < timotimo> nqp: say("\r\na" ~~ /<?[\w]>"\r\na"/); 17:50 <+camelia> nqp-moarvm: OUTPUT«Substring length (-6) cannot be negativeStr» 17:50 <+camelia> ..nqp-jvm, nqp-parrot: OUTPUT«» 17:50 < timotimo> nqp: say("\r\na" ~~ /"\r\na"/); 17:50 <+camelia> nqp-moarvm, nqp-jvm, nqp-parrot: OUTPUT«
the implementation of CCLASS_NEWLINE doesn't have a condition for the addition to pos if it matched a \r\n.
I'll fix this bug on a train ride today.
Sorry, something went wrong.
actually, the regex just has to have \n, but it b0rks with \w as well, because that mixes it in.
18:03 < timotimo> nqp: say("\r\na" ~~ /<?[\n]>"\r\na"/); 18:03 <+camelia> nqp-moarvm: OUTPUT«Substring length (-6) cannot be negativeStr» 18:03 <+camelia> ..nqp-jvm, nqp-parrot: OUTPUT«» 18:03 < timotimo> nqp: say("\r\na" ~~ /"\r\na"/); 18:03 <+camelia> nqp-moarvm, nqp-jvm, nqp-parrot: OUTPUT«
solved the bug, but it needs to get a test.
ideally, the test would land in qregex/somewhere, but that's giving me trouble with \r\n and different levels of escaping ...
Added test to roast instead of nqp: Raku/roast@c82bbd55ff
@FROGGS - Status changed from 'new' to 'resolved'
No branches or pull requests
Migrated from rt.perl.org#120511 (status was 'resolved')
Searchable as RT120511$
The text was updated successfully, but these errors were encountered: