> On 21 Jun 2016, at 01:52, Brent Royal-Gordon via swift-evolution
> <[email protected]> wrote:
>
> * I'm very uncomfortable with the amount of weight `accumulatingResultBy`
> adds to `reduce`. `combinedBy` seems perfectly cromulent to me. I'm even more
> concerned by your suggestion in the pull request body of
> `accumulating(startingFrom:combiningBy:)`. `reduce` is a subtle and slightly
> confusing operation; adding more words to its call sites will not solve that
> problem. If you want to invent a new name from whole cloth, I would probably
> use something like `combining(with initialResult: T, by nextResult: (T,
> Element) -> T)`. (For that matter, while we're working in this area,
> `sequence(first:next:)` could use a similar coat of paint.)
Didn’t have time to give it a thorough read, but `accumulatingResultBy` did
stand out negatively for me as being a bit too verbose.
_______________________________________________
swift-evolution mailing list
[email protected]
https://lists.swift.org/mailman/listinfo/swift-evolution
