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
Rakudo allows reversed ranges in character classes in regexes, shouldn't #1458
Comments
From @masak<masak> mberends: I made great strides with GGE yesterday on the bus. |
From @kylehaThis is an automatically generated mail to inform you that tests are now available in t/spec/S05-metasyntax/charset.t commit 8cf9ce6e75dec38eaf2e2c5c5474bc1336d468d2 [t/spec] Test for RT 71702: lethal reverse range in charset Inline Patchdiff --git a/t/spec/S05-metasyntax/charset.t b/t/spec/S05-metasyntax/charset.t
index 1aeba3f..329348c 100644
--- a/t/spec/S05-metasyntax/charset.t
+++ b/t/spec/S05-metasyntax/charset.t
@@ -12,7 +12,7 @@ be valid perl6.
=end pod
-plan 25;
+plan *;
if !eval('("a" ~~ /a/)') {
skip_rest "skipped tests - rules support appears to be missing";
@@ -60,6 +60,12 @@ ok( "foo" ~~ /<[f] #`[comment] + [o]>/, 'comment embedded in charset works' );
#?rakudo skip 'large \\x char spec in regex (RT #67122) (noauto)'
ok "\x[10001]" ~~ /<[\x10000..\xEFFFF]>/, 'large \\x char spec';
+#?rakudo todo 'RT 71702: lethal reverse range in charset'
+eval_dies_ok( "'RT 71702' ~~ /<[d..b]>? RT/",
+ 'reverse range in charset is lethal (RT 71702)' );
+
}
+done_testing;
+
# vim: ft=perl6 |
The RT System itself - Status changed from 'new' to 'open' |
From @bbkr[13:26] <bbkr> std: say "foo" ~~ /<[d..b]>? foo/ I'm not sure if reversed ranges are allowed in current spec, so I leave |
From @jnthnOn Tue Dec 29 03:14:18 2009, masak wrote:
Dies now:
Tagging testneeded. /jnthn |
From @diakopterOn Fri Apr 20 06:02:21 2012, jnthn@jnthn.net wrote:
19:27 <+dalek> roast: 6a03ec5 | diakopter++ | S05-mass/rx.t: |
@diakopter - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#71702 (status was 'resolved')
Searchable as RT71702$
The text was updated successfully, but these errors were encountered: