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

Some operations with Inf.Rat blow up (Inf.Rat.perl) #5952

Closed
p6rt opened this issue Dec 30, 2016 · 4 comments
Closed

Some operations with Inf.Rat blow up (Inf.Rat.perl) #5952

p6rt opened this issue Dec 30, 2016 · 4 comments

Comments

@p6rt
Copy link

p6rt commented Dec 30, 2016

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

Searchable as RT130459$

@p6rt
Copy link
Author

p6rt commented Dec 30, 2016

From @AlexDaniel

Code​:
say 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

What's 20? :)
Anyway, Num-ish Rats were first added in rakudo/rakudo@1f3ca64 (Inf was then represented by 1/0), but then the solution was improved in rakudo/rakudo@498d0a4

As for the solution, it should not blow up like this for sure. But perhaps the whole thing should be changed? I don't know.

Interesting discussion here​: https://irclog.perlgeek.de/perl6-dev/2016-12-30#i_13826116

@p6rt
Copy link
Author

p6rt commented Dec 30, 2017

From @zoffixznet

On Fri, 30 Dec 2016 13​:31​:00 -0800, alex.jakimenko@​gmail.com wrote​:

Code​:
say 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

What's 20? :)
Anyway, Num-ish Rats were first added in
rakudo/rakudo@1f3ca64
(Inf was then represented by 1/0), but then the solution was improved
in
rakudo/rakudo@498d0a4

As for the solution, it should not blow up like this for sure. But
perhaps the whole thing should be changed? I don't know.

Interesting discussion here​: https://irclog.perlgeek.de/perl6-
dev/2016-12-30#i_13826116

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 p6rt closed this as completed Dec 30, 2017
@p6rt
Copy link
Author

p6rt commented Dec 30, 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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant