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
Calling some methods on Match.orig causes NPA #1377
Comments
From @moritz15:37 <@moritz_> rakudo: 'a' ~~ /a/; say ($/.orig).rindex('a', 1) Likely cause: 15:35 <@moritz_> rakudo: 'a' ~~ /a/; say $/.orig.PARROT |
From @kylehaThis is an automatically generated mail to inform you that tests are now available in t/spec/S05-match/capturing-contexts.t commit ad5306eaedfd60179a505fce0b2203f0c07712c1 [t/spec] Test for RT 70003: $/.orig produces CodeString Inline Patchdiff --git a/t/spec/S05-match/capturing-contexts.t b/t/spec/S05-match/capturing-contexts.t
index 93f0e37..8f97ae9 100644
--- a/t/spec/S05-match/capturing-contexts.t
+++ b/t/spec/S05-match/capturing-contexts.t
@@ -102,6 +102,13 @@ is_run( q{'aa' ~~ /(.)$1/},
is $/, 'RT', 'Matched as intended in void context';
}
+# RT #70003
+{
+ 'a' ~~ /a/;
+ #?rakudo skip 'RT 70003'
+ is ($/.orig).rindex('a'), 0, 'rindex() works on $/.orig';
+}
+
done_testing;
# vim: ft=perl6 |
The RT System itself - Status changed from 'new' to 'open' |
From @moritzNeeds better tests. Right now the example fail()s, but this fails too:
|
From @moritzTests are better now, and passing. |
@moritz - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#70003 (status was 'resolved')
Searchable as RT70003$
The text was updated successfully, but these errors were encountered: