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

Proxy.new ---> "Cannot invoke this object" error #4427

Closed
p6rt opened this issue Jul 21, 2015 · 5 comments
Closed

Proxy.new ---> "Cannot invoke this object" error #4427

p6rt opened this issue Jul 21, 2015 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Jul 21, 2015

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

Searchable as RT125663$

@p6rt
Copy link
Author

p6rt commented Jul 21, 2015

From @raiph

What I did

m​: Proxy.new

What I got

rakudo-moar 52b6f2​: OUTPUT«Unhandled exception​: Cannot invoke this object (REPR​: P6opaque, cs = 0)␤ at src/gen/m-BOOTSTRAP.nqp​:1288 (/home/camelia/rakudo-inst-1/share/nqp/lib/Perl6/BOOTSTRAP.moarvm​::10)␤ from src/gen/m-main.nqp​:40 (/home/camelia/rakudo-inst-1/share/perl6/run…»

What I expected

rakudo-moar 52b6f2​: ( no output )


jnthn​: Needs a FETCH/STORE providing; seems we want better error checking for that.

@p6rt
Copy link
Author

p6rt commented Jul 22, 2015

From @moritz

On Tue Jul 21 12​:28​:03 2015, raiph wrote​:

What I did

m​: Proxy.new

What I got

rakudo-moar 52b6f2​: OUTPUT«Unhandled exception​: Cannot invoke this
object (REPR​: P6opaque, cs = 0)␤ at src/gen/m-BOOTSTRAP.nqp​:1288
(/home/camelia/rakudo-inst-
1/share/nqp/lib/Perl6/BOOTSTRAP.moarvm​::10)␤ from src/gen/m-
main.nqp​:40 (/home/camelia/rakudo-inst-1/share/perl6/run…»

What I expected

rakudo-moar 52b6f2​: ( no output )

-----------

What rakudo does now​:

moritz@​hack​:~/p6/rakudo$ ./perl6-m -e 'Proxy.new'
Required named parameter 'FETCH' not passed
  in block <unit> at -e​:1

Fixed with this commit​:

commit 96fed8766ac6f7c8af1a9f25e7a5b54342ffcb2a
Author​: Moritz Lenz <moritz@​faui2k3.org>
Date​: 2015-07-22 16​:17​:02 +0000

  RT #​125663​: Better error message for bare Proxy.new

src/Perl6/Metamodel/BOOTSTRAP.nqp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Now just needs a test.

@p6rt
Copy link
Author

p6rt commented Jul 22, 2015

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

@p6rt
Copy link
Author

p6rt commented Jul 22, 2015

From @moritz

Closing without a test, since the .new method from BOOTSTRAP.pm doesn't throw a typed exception, and so testing would be fragile and/or quite some effort.

@p6rt p6rt closed this as completed Jul 22, 2015
@p6rt
Copy link
Author

p6rt commented Jul 22, 2015

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

@p6rt p6rt added the testneeded label Jan 5, 2020
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