Skip to content
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 doing callwith inside a multi method #1361

Closed
p6rt opened this issue Oct 13, 2009 · 5 comments
Closed

Null PMC access when doing callwith inside a multi method #1361

p6rt opened this issue Oct 13, 2009 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Oct 13, 2009

Migrated from rt.perl.org#69756 (status was 'resolved')

Searchable as RT69756$

@p6rt
Copy link
Author

p6rt commented Oct 13, 2009

From @masak

<masak> rakudo​: multi sub f(0) {}; multi sub f($n) { say $n;
callwith($n - 1) }; f(3)
<p6eval> rakudo 54cfe4​: OUTPUT«3␤Null PMC access in clone()␤ [...]
* masak submits rakudobug

jnthn++ informs me that the above code is likely not what I want.
Nevertheless, I expected it do produce something a bit less internal
than a Null PMC access error. Seems jnthn did, too.

@p6rt
Copy link
Author

p6rt commented Jul 28, 2010

@coke - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Oct 11, 2011

From @coke

On Tue Oct 13 16​:05​:16 2009, masak wrote​:

<masak> rakudo​: multi sub f(0) {}; multi sub f($n) { say $n;
callwith($n - 1) }; f(3)
<p6eval> rakudo 54cfe4​: OUTPUT«3␤Null PMC access in clone()␤ [...]
* masak submits rakudobug

jnthn++ informs me that the above code is likely not what I want.
Nevertheless, I expected it do produce something a bit less internal
than a Null PMC access error. Seems jnthn did, too.

NPA gone​:

21​:19 < [Coke]> rakudo​: multi sub f(0) {}; multi sub f($n) { say $n;
  callwith($n - 1) }; f(3)
21​:19 <+p6eval> rakudo 38907e​: OUTPUT«3␤»

Closable with tests.
--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Oct 11, 2011

From @moritz

Now tested in t/spec/S12-methods/defer-call.t.

@p6rt
Copy link
Author

p6rt commented Oct 11, 2011

@moritz - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant