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

Fix Fixnum#pred overflow #3422

Closed
wants to merge 1 commit into from
Closed

Conversation

@grddev
Copy link
Contributor

@grddev grddev commented Oct 24, 2015

Without this (-2**63+1).pred returns a positive number. Given that the implementation hasn't changed (substantially) since the initial version in 2008, it seems all JRuby versions should be affected.

@kares
Copy link
Member

@kares kares commented Nov 10, 2015

thanks, put this one onto jruby-1_7 it shall get to master with the branch merge

@kares kares closed this Nov 10, 2015
@kares kares added this to the JRuby 1.7.23 milestone Nov 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants