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.

Sign up for GitHub

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

Jump to bottom

Range#eql? returns incorrect result for float vs int #3956

Closed
djberg96 opened this issue Jun 7, 2016 · 1 comment
Closed

Range#eql? returns incorrect result for float vs int #3956

djberg96 opened this issue Jun 7, 2016 · 1 comment
Labels
beginner core
Milestone
JRuby 9.1.3.0

Comments

@djberg96
Copy link
Contributor

djberg96 commented Jun 7, 2016 

>ruby -v
jruby 9.1.1.0 (2.3.0) 2016-05-19 fe84e89 Java HotSpot(TM) 64-Bit Server VM 25.45-b02 on 1.8.0_45-b14 +jit [darwin-x86_64]

The docs for Range#eql? in MRI say a Range object is eql? if start1 && start2 are eql? and end1 && end2 are eql? However, it seems JRuby is treating a float as an int:

Range.new(1.0, 25).eql?(Range.new(1, 25)) # true, but should be false

Since 1.eql?(1.0) is false, the above should be false as well.

Courtesy of berger_spec.
@kares
Copy link
Member

kares commented Jun 14, 2016

fixed with updates from #3960

@kares kares closed this as completed Jun 14, 2016
@kares kares added this to the JRuby 9.1.3.0 milestone Jun 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
beginner core
Projects
None yet
Milestone
JRuby 9.1.3.0
Development

No branches or pull requests
2 participants
@kares @djberg96