[swift-evolution] Proposal: Remove implicit tuple splat behavior from function applications
Chris Lattner
clattner at apple.com
Wed Jan 27 13:10:30 CST 2016
> On Jan 27, 2016, at 11:03 AM, swift at lng.la wrote:
>
> I'm +1 if this will eventually be replaced with a better-designed version, but I do use it and would be sad to see it go. I've found it very helpful for testing:
Hi Jarod, where are you using this feature?
>
> protocol APIType {
> func getChatMessages(channel channel: ChatChannel, limit: Int, success: (chatMessages: [ChatMessage]) -> Void, failure: (error: NSError) -> Void)
> }
>
> class FakeAPI: APIType {
> var getChatMessagesEndpoint = FakeAPIEndpoint(
> calls: [(channel: ChatChannel, limit: Int)](),
> successStubs: [[ChatMessage]](),
> failureStubs: [NSError]()
> )
>
> func getChatMessages(channel channel: ChatChannel, limit: Int, success: (chatMessages: [ChatMessage]) -> Void, failure: (error: NSError) -> Void) {
> getChatMessagesEndpoint.handleCall(success, failure, callParams: (channel: channel, limit: limit))
If you are referring to this line, then there is no change. The only affected case is in call sites that take a single parameter without a label. Passing tuples as values will not be change.
-Chris
> }
> }
>
> struct FakeAPIEndpoint<CallParams, SuccessParams, FailureParams> {
> private(set) var calls: [CallParams]
> private(set) var successStubs: [SuccessParams]
> private(set) var failureStubs: [FailureParams]
>
> mutating func stubSuccess(params: SuccessParams) {
> successStubs.append(params)
> }
>
> mutating func stubFailure(params: FailureParams) {
> failureStubs.append(params)
> }
>
> private mutating func handleCall(success: (SuccessParams) -> Void, _ failure: (FailureParams) -> Void, call: CallParams) {
> calls.append(call)
>
> if successStubs.count > 0 {
> success(successStubs.removeFirst())
> }
> else if failureStubs.count > 0 {
> failure(failureStubs.removeFirst())
> }
> }
> }
>
> It's really nice to have that handleCall method that can call the success or failure callback automatically rather than reimplementing that functionality in every fake API call.
>
> Jarod
>
>> On Jan 27, 2016, at 10:16, Chris Lattner via swift-evolution <swift-evolution at swift.org <mailto:swift-evolution at swift.org>> wrote:
>>
>>> On Jan 27, 2016, at 2:18 AM, Nisse Bergman <nisse at potmo.com <mailto:nisse at potmo.com>> wrote:
>>>
>>> -1 I use this in both my mocking API and in JSON deserialising/serializing.
>>> I think this is a great thing to have.
>>
>> Ok, I’m definitely interested in knowing more. Please include a code sample, showing both the declaration being splatted into and the call sites. Otherwise, I can’t tell how much value this feature is adding, and what the pain would be if it were removed. Thanks!
>>
>> -Chris
>>
>>>
>>>> On 27 Jan 2016, at 11:07, David Waite via swift-evolution <swift-evolution at swift.org <mailto:swift-evolution at swift.org>> wrote:
>>>>
>>>> I used this just last week! But only as an illustration
>>>>
>>>> +1
>>>>
>>>> One comment though, a splat operator need not be purely syntactic sugar - it could be the way arrays are applied to variadic functions.
>>>>
>>>> -DW
>>>> _______________________________________________
>>>> swift-evolution mailing list
>>>> swift-evolution at swift.org <mailto:swift-evolution at swift.org>
>>>> https://lists.swift.org/mailman/listinfo/swift-evolution
>>>
>>
>> _______________________________________________
>> swift-evolution mailing list
>> swift-evolution at swift.org <mailto:swift-evolution at swift.org>
>> https://lists.swift.org/mailman/listinfo/swift-evolution
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-evolution/attachments/20160127/bce37196/attachment.html>
More information about the swift-evolution
mailing list