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
Null PMC access when calling 'callsame' directly from a sub in Rakudo #1310
Comments
From @masak<masak> rakudo: sub foo($n) { say $n; if $n { callsame($n-1) } }; foo(1) Hm, maybe I should have used 'callwith' in that piece of code? Anyway, the bug manifests itself for <call next> X~ <same with>. |
From @kylehaThis is an automatically generated mail to inform you that tests are now available in t/spec/S06-advanced_subroutine_features/callsame.t commit d19774bc271975862491aee6427f9e8d59532716 [t/spec] Added test for RT69314 - Null PMC access when calling 'callsame' directly from a sub in Rakudo. Inline Patchdiff --git a/t/spec/S06-advanced_subroutine_features/callsame.t b/t/spec/S06-advanced_subroutine_features/callsame.t
index d9062ca..65b6108 100644
--- a/t/spec/S06-advanced_subroutine_features/callsame.t
+++ b/t/spec/S06-advanced_subroutine_features/callsame.t
@@ -16,6 +16,20 @@ plan *;
is @called, <Int Num>, 'multi with "callsame" worked';
}
+# RT 69314
+
+{
+ sub rt69314($n) {
+ if $n {
+ callsame($n-1);
+ }
+ };
+
+ #?rakudo todo 'Calling callsame directly from a sub'
+ lives_ok {rt69314(1)}, 'Calling callsame directly from a sub works';
+
+}
+
done_testing;
# vim: ft=perl6 |
The RT System itself - Status changed from 'new' to 'open' |
From @jnthnOn Wed Sep 23 03:16:28 2009, masak wrote:
Now:
Which is right - callsame takes no arguments. And if you just do
Which I think is also right. Tagging testneeded. /jnthn |
From @tadzikTest fixed in |
@jnthn - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#69314 (status was 'resolved')
Searchable as RT69314$
The text was updated successfully, but these errors were encountered: