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 calling .perl on callframe.my in Rakudo #2151

Closed
p6rt opened this issue Sep 9, 2010 · 7 comments
Closed

Null PMC access when calling .perl on callframe.my in Rakudo #2151

p6rt opened this issue Sep 9, 2010 · 7 comments

Comments

@p6rt
Copy link

p6rt commented Sep 9, 2010

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

Searchable as RT77754$

@p6rt
Copy link
Author

p6rt commented Sep 9, 2010

From @masak

<moritz_> rakudo​: say callframe().callframe(1).my.perl
<p6eval> rakudo 859f2d​: OUTPUT«Null PMC access in isa_pmc()␤ in
'&infix​:<=>' at line 1␤ in 'CallFrame​::my' at line 7047​:CORE.setting
[...]
<moritz_> that isa_pmc NPA is new, I think
<moritz_> ie it worked when I implemented it, back in the days :-)
* masak submits rakudobug
<pmichaud> why are we getting a ParrotInterpret...... oh, I see the problem.
<pmichaud> The problem isn't with CallFrame.perl, it's with
CallFrame's $!interp.perl
<pmichaud> the default .perl is attempting to call .perl on all of the
attributes
<pmichaud> so, we just need a .perl for CallFrame that does something
more reasonable, like fail. :-)

@p6rt
Copy link
Author

p6rt commented Oct 8, 2011

From @coke

On Thu Sep 09 09​:02​:22 2010, masak wrote​:

<moritz_> rakudo​: say callframe().callframe(1).my.perl
<p6eval> rakudo 859f2d​: OUTPUT«Null PMC access in isa_pmc()␤ in
'&infix​:<=>' at line 1␤ in 'CallFrame​::my' at line 7047​:CORE.setting
[...]
<moritz_> that isa_pmc NPA is new, I think
<moritz_> ie it worked when I implemented it, back in the days :-)
* masak submits rakudobug
<pmichaud> why are we getting a ParrotInterpret...... oh, I see the
problem.
<pmichaud> The problem isn't with CallFrame.perl, it's with
CallFrame's $!interp.perl
<pmichaud> the default .perl is attempting to call .perl on all of the
attributes
<pmichaud> so, we just need a .perl for CallFrame that does something
more reasonable, like fail. :-)

Output is now​:
16​:53 < [Coke]> rakudo​: say callframe().callframe(1).my.perl
16​:53 <+p6eval> rakudo 88c5a5​: OUTPUT«not yet implemented␤ in method
callframe
  at src/gen/CORE.setting​:6343␤ in block <anon> at
  /tmp/KUGjkHLZrt​:1␤ in <anon> at /tmp/KUGjkHLZrt​:1␤»

--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Oct 8, 2011

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented May 25, 2015

From @usev6

This fails now with X​::NYI and an adequate error message​:

$ perl6 -e 'say callframe().callframe(1).my.perl'
Callframe.callframe not yet implemented. Sorry.
  in block <unit> at -e​:1
  in any <unit-outer> at -e​:1

I wonder whether this ticket could be closed now or how a proper test would look like.

1 similar comment
@p6rt
Copy link
Author

p6rt commented May 25, 2015

From @usev6

This fails now with X​::NYI and an adequate error message​:

$ perl6 -e 'say callframe().callframe(1).my.perl'
Callframe.callframe not yet implemented. Sorry.
  in block <unit> at -e​:1
  in any <unit-outer> at -e​:1

I wonder whether this ticket could be closed now or how a proper test would look like.

@p6rt
Copy link
Author

p6rt commented Oct 13, 2017

From @zoffixznet

Tests​: rakudo/rakudo@80cee36223

@p6rt
Copy link
Author

p6rt commented Oct 13, 2017

@zoffixznet - 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