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

Inf.Rat.perl produces a really bad error message (Inf.Rat.perl) #5822

Closed
p6rt opened this issue Nov 24, 2016 · 5 comments
Closed

Inf.Rat.perl produces a really bad error message (Inf.Rat.perl) #5822

p6rt opened this issue Nov 24, 2016 · 5 comments
Labels
LTA Less Than Awesome; typically an error message that could be better

Comments

@p6rt
Copy link

p6rt commented Nov 24, 2016

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

Searchable as RT130171$

@p6rt
Copy link
Author

p6rt commented Nov 24, 2016

From @AlexDaniel

*Code​:*
Inf.Rat.perl

*Result​:*
P6opaque​: get_boxed_ref could not unbox for the representation '20' of type
Num
  in block <unit> at -e line 1

There is no Num in the code, there is no 20, and the rest is just internal
stuff that makes no sense to the user.

In some sense, this is a subset of
https://rt-archive.perl.org/perl6/Ticket/Display.html?id=128857

See this IRC log for some background​:
https://irclog.perlgeek.de/perl6/2016-11-24#i_13624557

@p6rt
Copy link
Author

p6rt commented Nov 24, 2016

From @nwc10

On Thu, 24 Nov 2016 11​:28​:06 -0800, alex.jakimenko@​gmail.com wrote​:

*Code​:*
Inf.Rat.perl

*Result​:*
P6opaque​: get_boxed_ref could not unbox for the representation '20' of type
Num
in block <unit> at -e line 1

There is no Num in the code, there is no 20, and the rest is just internal
stuff that makes no sense to the user.

In some sense, this is a subset of
https://rt-archive.perl.org/perl6/Ticket/Display.html?id=128857

See this IRC log for some background​:
https://irclog.perlgeek.de/perl6/2016-11-24#i_13624557

More discussion starts here​:

http://irclog.perlgeek.de/perl6/2016-11-24#i_13624763

19​:35 < nwc10> this is also a bug?
19​:35 < nwc10> m​: say Inf.Rat*0
19​:35 <+camelia> rakudo-moar 61a18c​: OUTPUT«Type check failed in binding to nu; expected Int but got Num (NaN)␤ in block <unit> at <tmp> line 1␤␤»
19​:36 < viki> nwc10​: that's the same cause
19​:36 < nwc10> OK. I'll add that to the ticket unless someone else has already

19​:37 < viki> nwc10​: there're a couple of related tickets​: https://rt.perl.org/Ticket/Display.html?id=127341#ticket-history and https://rt-archive.perl.org/perl6/Ticket/Display.html?id=128857#ticket-history
19​:38 < viki> And this is the commit that explains what problem we solved by the current situation​: rakudo/rakudo@498d0a4
19​:38 < viki> So any new solutions should solve that problem too.

@p6rt
Copy link
Author

p6rt commented Dec 30, 2017

From @zoffixznet

On Thu, 24 Nov 2016 11​:42​:36 -0800, nicholas wrote​:

On Thu, 24 Nov 2016 11​:28​:06 -0800, alex.jakimenko@​gmail.com wrote​:

*Code​:*
Inf.Rat.perl

*Result​:*
P6opaque​: get_boxed_ref could not unbox for the representation '20'
of type
Num
in block <unit> at -e line 1

There is no Num in the code, there is no 20, and the rest is just
internal
stuff that makes no sense to the user.

In some sense, this is a subset of
https://rt-archive.perl.org/perl6/Ticket/Display.html?id=128857

See this IRC log for some background​:
https://irclog.perlgeek.de/perl6/2016-11-24#i_13624557

More discussion starts here​:

http://irclog.perlgeek.de/perl6/2016-11-24#i_13624763

19​:35 < nwc10> this is also a bug?
19​:35 < nwc10> m​: say Inf.Rat*0
19​:35 <+camelia> rakudo-moar 61a18c​: OUTPUT«Type check failed in
binding to nu; expected Int but got Num (NaN)␤ in block <unit> at
<tmp> line 1␤␤»
19​:36 < viki> nwc10​: that's the same cause
19​:36 < nwc10> OK. I'll add that to the ticket unless someone else has
already

19​:37 < viki> nwc10​: there're a couple of related tickets​:
https://rt-archive.perl.org/perl6/Ticket/Display.html?id=127341#ticket-history and
https://rt-archive.perl.org/perl6/Ticket/Display.html?id=128857#ticket-history
19​:38 < viki> And this is the commit that explains what problem we
solved by the current situation​:
rakudo/rakudo@498d0a4
19​:38 < viki> So any new solutions should solve that problem too.

Thank you for the report. This is now fixed.

Fix​: rakudo/rakudo@042cb7413f87277
Test​: Raku/roast@c3086244fd0c6feea

@p6rt
Copy link
Author

p6rt commented Dec 30, 2017

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

@p6rt
Copy link
Author

p6rt commented Dec 30, 2017

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

@p6rt p6rt closed this as completed Dec 30, 2017
@p6rt p6rt added the LTA Less Than Awesome; typically an error message that could be better label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LTA Less Than Awesome; typically an error message that could be better
Projects
None yet
Development

No branches or pull requests

1 participant