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 thinks 0/0 is in -∞^..^∞ range, previously it was an error ((-∞^..^∞).in-range: 0/0) #5947

Closed
p6rt opened this issue Dec 30, 2016 · 4 comments
Labels
regression Issue did not exist previously

Comments

@p6rt
Copy link

p6rt commented Dec 30, 2016

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

Searchable as RT130452$

@p6rt
Copy link
Author

p6rt commented Dec 30, 2016

From @AlexDaniel

Code​:
(-â��^..^â��).in-range​: 0/0

Result (2015.12,2016.03)​:
X​::OutOfRange exception produced no message
  in block <unit> at /tmp/WOkgMZTuHS line 1

Result (2016.04,HEAD)​:
True

Bisectable points to rakudo/rakudo@a424f07

Basically, 0/0 is NaN, so I was expecting False as a result. Throwing an error is also OK (though the previous one was LTA).

@p6rt
Copy link
Author

p6rt commented Dec 30, 2016

From @zoffixznet

On Thu, 29 Dec 2016 19​:18​:12 -0800, alex.jakimenko@​gmail.com wrote​:

Code​:
(-â��^..^â��).in-range​: 0/0

Result (2015.12,2016.03)​:
X​::OutOfRange exception produced no message
in block <unit> at /tmp/WOkgMZTuHS line 1

Result (2016.04,HEAD)​:
True

Bisectable points to
rakudo/rakudo@a424f07

Basically, 0/0 is NaN, so I was expecting False as a result. Throwing
an error is also OK (though the previous one was LTA).

Thank you for the report. This is now fixed.

Fixes​:
rakudo/rakudo@a567eb4
rakudo/rakudo@b2332c2

Tests​:
Raku/roast@b5049e033a
rakudo/rakudo@b2332c2

@p6rt
Copy link
Author

p6rt commented Dec 30, 2016

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

@p6rt p6rt closed this as completed Dec 30, 2016
@p6rt
Copy link
Author

p6rt commented Dec 30, 2016

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

@p6rt p6rt added the regression Issue did not exist previously label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Issue did not exist previously
Projects
None yet
Development

No branches or pull requests

1 participant