While we are at it, could we add a inout/mutating version of forEach (unless something like that exists and I missed it)? I have been trying to figure out how to run through an array and call a mutating method on each element without copying everything, and I am at a loss…
Thanks, Jon > On Jan 22, 2017, at 11:27 PM, Russ Bishop via swift-evolution > <swift-evolution@swift.org> wrote: > >> >> On Jan 22, 2017, at 10:56 PM, Chris Eidhof <ch...@eidhof.nl >> <mailto:ch...@eidhof.nl>> wrote: >> >> Not as a direct reply to Russ, but just to reiterate: to me, there are two >> clear benefits of using the `inout` version of reduce: >> >> 1. The performance (currently discussed at length) >> 2. Readability (because we can use mutating methods on `inout` arguments). >> >> Even if the compiler were to optimize the unnecessary copy of `return arr + >> [el]` away, there are still a lot of other mutable methods that you might >> want to use within the reduce closure. So I think the proposal is still very >> valid even if the compiler optimizations would magically appear tomorrow. >> >> To push this proposal forward a little bit, I'd like to come up with a good >> name. It seems like we shouldn't overload `reduce`, but choose a different >> name, so that we don't stress the typechecker. Any other suggestions? >> >> On Mon, Jan 23, 2017 at 7:11 AM, Russ Bishop <xen...@gmail.com >> <mailto:xen...@gmail.com>> wrote: >> -- >> Chris Eidhof > > > Sorry for the derail! > > reduce(mutating:_:) { } is still my favorite; You can take mutating to mean > we will copy the value now but mutate it later. > > > Some alternatives: > > reduce(forMutating:_:) { } > > reduce(forInout:_:) { } > > reduce(initial:_:) { } > > reduce(copying:mutate:) { } > > // just kidding... > reduce(copyForLaterMutating:_:) { } > > > > It should definitely be some form of reduce. > > Russ > _______________________________________________ > swift-evolution mailing list > swift-evolution@swift.org <mailto:swift-evolution@swift.org> > https://lists.swift.org/mailman/listinfo/swift-evolution > <https://lists.swift.org/mailman/listinfo/swift-evolution>
_______________________________________________ swift-evolution mailing list swift-evolution@swift.org https://lists.swift.org/mailman/listinfo/swift-evolution
