Feature #10958: remove "initial_state" argument of Enumerable#{slice_before,chunk}

• Status: Closed
• Assignee: -
• Target version: -

[akr (Akira Tanaka)]

I'd like to remove "initial_state" argument of Enumerable#{slice_before,chunk}.

I think a state management can be implemented using local variable without the argument.
So I'd like to remove it because it is too complex and
not consistent with Enumerable#{slice_when,slice_after} which doesn't have the argument.

Note that I added deprecated warning since Ruby 2.2.
It seems no one complained the warning until now.

[Eregon (Benoit Daloze)]

+1, making these two simpler is definitely an improvement!

[akr (Akira Tanaka)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

Applied in changeset r50174.

---

• enum.c: Enumerable#chunk and Enumerable#slice_before no longer takes
  the initial_state argument. [Feature #10958]

[akr (Akira Tanaka)]

It seems no one against this. I committed.

[naruse (Yui NARUSE)]

• Related to Bug #13202: Enumerable#slice_before does not take both argument and block at the same time, as it ought to. added