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

Rakudo's ObjectRef doesn't proxy .signature properly #760

Closed
p6rt opened this issue Mar 9, 2009 · 5 comments
Closed

Rakudo's ObjectRef doesn't proxy .signature properly #760

p6rt opened this issue Mar 9, 2009 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Mar 9, 2009

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

Searchable as RT63744$

@p6rt
Copy link
Author

p6rt commented Mar 9, 2009

From @masak

<bacek> rakudo​: sub f (Code $c) { say +$c.signature.params }; f(-> $a,
$b? {...})
<p6eval> rakudo 8bbc31​: OUTPUT«Method 'params' not found for invocant
of class '' [...]
* bacek summon masak again...
<masak> raaawr again!
* masak submits
<bacek> rakudo​: my Code $c = -> $a {...}; say $c.signature.perl
<p6eval> rakudo 8bbc31​: OUTPUT«​:(Any $a)␤»
<bacek> rakudo​: my Code $c = -> $a {...}; say $c.signature.perl
<p6eval> rakudo 8bbc31​: OUTPUT«​:(Any $a)␤»
<bacek> rakudo​: my Code $c = -> $a {...}; say $c.signature.perl; sub
f(Code $f) { say $f.signature.perl }; f($c);
<p6eval> rakudo 8bbc31​: OUTPUT«​:(Any $a)␤Failure␤»
<bacek> interesting...
<bacek> Ah!
<bacek> rakudo​: my Code $c = -> $a {...}; say $c.signature.perl; sub
f(Code $f is copy) { say $f.signature.perl }; f($c);
<p6eval> rakudo 8bbc31​: OUTPUT«​:(Any $a)␤Failure␤»
<bacek> rakudo​: my Code $c = -> $a {...}; say $c.signature.perl; sub
f(Code $f is rw) { say $f.signature.perl }; f($c);
<p6eval> rakudo 8bbc31​: OUTPUT«​:(Any $a)␤​:(Any $a)␤»
<bacek> yes. ObjectRef doesn't proxy .signature properly.

@p6rt
Copy link
Author

p6rt commented Mar 9, 2009

From @pmichaud

Now fixed in d16d3c3. We probably want to add a test in the suite, so
I'm leaving the ticket open (assigning to moritz) until that's done.

Thanks!

Pm

@p6rt
Copy link
Author

p6rt commented Mar 9, 2009

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

@p6rt
Copy link
Author

p6rt commented Mar 9, 2009

From @moritz

now tested in t/spec/S06-signature/arity.t (not a perfect location, but
the closest location I could find).

Cheers,
Moritz

@p6rt
Copy link
Author

p6rt commented Mar 9, 2009

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

@p6rt p6rt closed this as completed Mar 9, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant