On Monday, 26 January 2015 at 20:58:36 UTC, Dicebot wrote:
Don't forget that there is already http://dlang.org/phobos/std_algorithm.html#.group which matches groupBy in this context (only groups consequitive elements). It would be totally awkward to have those named differently (which was why I have suggested to change the name from chunkBy to groupBy in the very beginning)

This seems valid to me. I'd suggest to keep the name and implementation as it is and add the "common" groupBy function as an example.

https://github.com/D-Programming-Language/phobos/pull/2915

Reply via email to