Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: jruby/jruby
base: 8aac6ace6266
Choose a base ref
...
head repository: jruby/jruby
compare: 6cba9d34f11b
Choose a head ref
  • 2 commits
  • 2 files changed
  • 2 contributors

Commits on Oct 13, 2016

  1. Don't dup every element in Enumerator#drop 

    In 06f0441 (JRUBY-6892) Enumerable#drop was changed to call #dup on every element that ended up in the result, but that might not work. Not all objects correctly implement #dup, or are even allocatable.

#take was not changed, and looking at the differences between #take and #drop the only thing that stood out was that the signature of the #each call was different. Changing from Arity.NO_ARGUMENTS to Arity.ONE_REQUIRED make the issue that 06f0441 tried to fix go away.
    @iconara
    iconara committed Oct 13, 2016
    Configuration menu
    Copy the full SHA
    833660a View commit details
    Browse the repository at this point in the history

Commits on Oct 18, 2016

  1. Merge pull request #4226 from iconara/fix-4218-1.7 

    Don't dup every element in Enumerator#drop (1.7)
    @headius
    headius committed Oct 18, 2016
    Configuration menu
    Copy the full SHA
    6cba9d3 View commit details
    Browse the repository at this point in the history