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

subList toIndex should be exclusive, not inclusive. (fromIndex is inclusive.) #1292

Merged
merged 3 commits into from Dec 4, 2013

Conversation

@masover
Copy link

@masover masover commented Dec 3, 2013

Resolves #1274.

@masover
Copy link
Author

@masover masover commented Dec 3, 2013

Well, that's embarrassing. I can't reproduce that build failure on openjdk7, and I don't see how my patch could've caused it. I can try to reproduce on oracle's jdk...

enebo added a commit that referenced this pull request Dec 4, 2013
subList toIndex should be exclusive, not inclusive. (fromIndex is inclusive.)
@enebo enebo merged commit 4157f3b into jruby:master Dec 4, 2013
1 check failed
1 check failed
@enebo
default The Travis CI build could not complete due to an error
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants