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
Regex matches in void context don't set $/ in Rakudo #1032
Comments
From @masak<pmichaud> rakudo: my $x = "test"; given $x { say |
From @kylehaThis is an automatically generated mail to inform you that tests are now available in t/spec/S05-match/capturing-contexts.t commit 3ace0ce336a3780127e8d1ef8f5e4c62dc446c96 [t/spec] Test for RT 66252: match in void context does not set $/ Inline Patchdiff --git a/t/spec/S05-match/capturing-contexts.t b/t/spec/S05-match/capturing-contexts.t
index 609480f..93f0e37 100644
--- a/t/spec/S05-match/capturing-contexts.t
+++ b/t/spec/S05-match/capturing-contexts.t
@@ -93,6 +93,15 @@ is_run( q{'aa' ~~ /(.)$1/},
},
'match with non-existent capture emits a warning' );
+# RT #66252
+{
+ $_ = 'RT 66252';
+ /(R.)/;
+ #?rakudo 2 todo 'RT 66252'
+ isa_ok $/, 'Match', 'Match object in $/ after match in void context';
+ is $/, 'RT', 'Matched as intended in void context';
+}
+
done_testing;
# vim: ft=perl6 |
The RT System itself - Status changed from 'new' to 'open' |
From @moritzcorrected tests now pass. |
@moritz - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#66252 (status was 'resolved')
Searchable as RT66252$
The text was updated successfully, but these errors were encountered: