<div dir="ltr">Here are the four you asked me to report:<div><br></div><div><div><a href="https://bugs.swift.org/browse/SR-5127">https://bugs.swift.org/browse/SR-5127</a></div><div>Single-tuple and multiple-argument function types should not be considered equal</div><div> </div><div><a href="https://bugs.swift.org/browse/SR-5128">https://bugs.swift.org/browse/SR-5128</a></div><div>Don't allow swapping a single-tuple function with a multiple-argument function</div><div><br></div><div><a href="https://bugs.swift.org/browse/SR-5129">https://bugs.swift.org/browse/SR-5129</a></div><div>Don't treat func signatures as being the same when they are in fact different</div><div><br></div><div><a href="https://bugs.swift.org/browse/SR-5130">https://bugs.swift.org/browse/SR-5130</a></div><div>Single-tuple and multiple-argument function types should be treated as different types</div></div><div><br></div><div>/Jens</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 6, 2017 at 8:17 PM, Mark Lacey via swift-evolution <span dir="ltr"><<a href="mailto:swift-evolution@swift.org" target="_blank">swift-evolution@swift.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
> On Jun 6, 2017, at 11:09 AM, Vladimir.S <<a href="mailto:svabox@gmail.com">svabox@gmail.com</a>> wrote:<br>
><br>
> On 06.06.2017 19:41, Mark Lacey wrote:<br>
>>> On Jun 6, 2017, at 4:00 AM, Vladimir.S <<a href="mailto:svabox@gmail.com">svabox@gmail.com</a> <mailto:<a href="mailto:svabox@gmail.com">svabox@gmail.com</a>>> wrote:<br>
>>><br>
>>> Mark, could you please also comment this inconsistencies / bugs :<br>
>>> (swift-4.0-DEVELOPMENT-<wbr>SNAPSHOT-2017-06-01-a-osx)<br>
>>><br>
>>> func fooParam(_ x: Int, _ y: Int){}<br>
>>> func fooTuple(_ x: (Int, Int)) {}<br>
>>><br>
>>> print("type of fooParam is", type(of:fooParam))<br>
>>> // result: type of fooParam is (Int, Int) -> ()<br>
>>><br>
>>> print("type of fooTuple is", type(of:fooTuple))<br>
>>> // result: type of fooTuple is (Int, Int) -> ()<br>
>>><br>
>>> print("type of fooTuple as (_:(Int,Int))->Void is", type(of: fooTuple as (_:(Int,Int))->()))<br>
>>> // result: type of fooTuple as (_:(Int,Int))->() is (Int, Int) -> ()<br>
>>><br>
>>><br>
>>> print("type of fooParam == type of fooTuple ?", type(of: fooParam) == type(of: fooTuple))<br>
>>> // result: true<br>
>>><br>
>>> if fooParam is (_: (Int,Int))->() { print("fooParam is (_: (Int,Int))->()") }<br>
>>> // result: fooParam is (_: (Int,Int))->()<br>
>>><br>
>>> if fooTuple is (Int,Int)->() { print("fooTuple is (Int,Int)->()") }<br>
>>> // result: fooTuple is (Int,Int)->()<br>
>>><br>
>>> var closureParam = { (x: Int, y: Int) in }<br>
>>> var closureTuple = { (x: (Int, Int)) in }<br>
>>><br>
>>> print("type of closureParam is", type(of:closureParam))<br>
>>> // result: type of closureParam is (Int, Int) -> ()<br>
>>><br>
>>> print("type of closureTuple is", type(of:closureTuple))<br>
>>> // result: type of closureTuple is (Int, Int) -> ()<br>
>>><br>
>>> if closureParam is (_: (Int,Int))->() { print("closureParam is (_: (Int,Int))->()") }<br>
>>> // result: closureParam is (_: (Int,Int))->()<br>
>>><br>
>>> if closureTuple is (Int,Int)->() { print("closureTuple is (Int,Int)->()") }<br>
>>> // result: closureTuple is (Int,Int)->()<br>
>> Can you open two reports at <a href="http://bugs.swift.org" rel="noreferrer" target="_blank">bugs.swift.org</a> <<a href="http://bugs.swift.org" rel="noreferrer" target="_blank">http://bugs.swift.org</a>>, one for the ‘is’ issue, and one for type(of:)?<br>
>> These (along with the issue with function declarations that Jens mentioned) are all similar issues, but each is in a different part of the compiler.<br>
><br>
> Here they are:<br>
> <a href="https://bugs.swift.org/browse/SR-5114" rel="noreferrer" target="_blank">https://bugs.swift.org/browse/<wbr>SR-5114</a> (typeof)<br>
> <a href="https://bugs.swift.org/browse/SR-5112" rel="noreferrer" target="_blank">https://bugs.swift.org/browse/<wbr>SR-5112</a> (is)<br>
<br>
</div></div>Thanks!<br>
<br>
Mark<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
>> Mark<br>
>>><br>
>>> Thank you.<br>
>>> Vladimir.<br>
>>><br>
>>> On 06.06.2017 11:43, Mark Lacey via swift-evolution wrote:<br>
>>>>> On Jun 6, 2017, at 12:08 AM, Jens Persson via swift-evolution <<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>>> wrote:<br>
>>>>><br>
>>>>> IMHO There seems to be a lot of bugs and inconsistencies left in more than just the reflective type system, for example the following won't compile although the two foo funcs clearly take different types as argument:<br>
>>>>><br>
>>>>> func foo(fun: (Int, Int) -> ()) { print("was given a function of type: (Int, Int) -> ()") }<br>
>>>>> func foo(fun: ((Int, Int)) -> ()) { print("was given a function of type: ((Int, Int)) -> ()") }<br>
>>>> I took a look at this. When determining if we have conflicting declarations, we compute an interface type and this computation is stripping the parens around the tuple in the second example, resulting in these two signatures appearing to be the same, despite the fact that the types of the arguments to the two functions are different.<br>
>>>>> // This will result in error: invalid redeclaration of 'foo(fun:)'<br>
>>>>><br>
>>>>> I would expect this to compile, and I can't understand how this has anything to do with the reflective type system.<br>
>>>>><br>
>>>>><br>
>>>>> Here is another example:<br>
>>>>><br>
>>>>> func add(_ a: Int, _ b: Int) -> Int { return a + b }<br>
>>>>> let a: (Int, Int) -> Int = add<br>
>>>>> let b: ((Int, Int)) -> Int = add // This is OK, unexpectedly<br>
>>>> I didn’t have a chance to look at this yet. I suspect this is related to the swap example that you gave previously.<br>
>>>>> I would not expect it to compile since the add func does not have the type ((Int, Int)) -> Int.<br>
>>>>> I don't think that is a dynamic cast, is it?<br>
>>>> Would you mind opening bugs for all four issues - the two mentioned above and the two from the previous e-mail (with type(of:) and swap examples)? Despite the fact that some of these might have different underlying causes it would be useful to have separate bugs and if they turn out to be the same issue we can dup as appropriate.<br>
>>>> Mark<br>
>>>>><br>
>>>>> /Jens<br>
>>>>><br>
>>>>><br>
>>>>><br>
>>>>><br>
>>>>> On Tue, Jun 6, 2017 at 2:45 AM, John McCall <<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a> <mailto:<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>> <mailto:<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>>> wrote:<br>
>>>>><br>
>>>>>> On Jun 5, 2017, at 12:08 AM, Jens Persson via swift-evolution<br>
>>>>>> <<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>>> wrote:<br>
>>>>>> So the bug in the reflective type system needs to be fixed before SE-0110 can<br>
>>>>>> actually be implemented (so that the statements in its title and text are true<br>
>>>>>> when compared to the actual behavior of the current Swift 4 compiler),<br>
>>>>><br>
>>>>> Gaps in the reflective type system are bugs, but they are not showstopper<br>
>>>>> bugs. We do not even expose any way to query the reflective system today; it<br>
>>>>> basically only affects type equality and dynamic casts that programmers are<br>
>>>>> very unlikely to use. The changes in call type-checking are vastly more<br>
>>>>> important, are implemented (modulo bugs, of course), and by themselves warrant<br>
>>>>> calling SE-0110 implemented.<br>
>>>>><br>
>>>>> John.<br>
>>>>><br>
>>>>>><br>
>>>>>> And yet:<br>
>>>>>><br>
>>>>>> 1. The status of SE-0110 is "Implemented"<br>
>>>>>><br>
>>>>>> 2. These statuses of the following issues are "resolved":<br>
>>>>>> SR-2008: Distinguish between single-tuple and multiple-argument function types<br>
>>>>>> SR-2216: Confusing behavior related to closure types and tuples<br>
>>>>>> SR-296: Fix inconsistencies related to tuples, arg/param lists, type<br>
>>>>>> params, typealiases<br>
>>>>>><br>
>>>>>> Why?<br>
>>>>>><br>
>>>>>> /Jens<br>
>>>>>><br>
>>>>>><br>
>>>>>> On Sun, Jun 4, 2017 at 5:49 PM, Ben Rimmington <<a href="mailto:me@benrimmington.com">me@benrimmington.com</a> <mailto:<a href="mailto:me@benrimmington.com">me@benrimmington.com</a>><br>
>>>>>> <mailto:<a href="mailto:me@benrimmington.com">me@benrimmington.com</a>>> wrote:<br>
>>>>>><br>
>>>>>> I assumed that Swift 3 mode would be the default, so that existing<br>
>>>>>> `#!/usr/bin/swift` scripts continue to work.<br>
>>>>>><br>
>>>>>> -- Ben<br>
>>>>>><br>
>>>>>> > On 3 Jun 2017, at 23:47, Jens Persson <<a href="mailto:jens@bitcycle.com">jens@bitcycle.com</a> <mailto:<a href="mailto:jens@bitcycle.com">jens@bitcycle.com</a>> <mailto:<a href="mailto:jens@bitcycle.com">jens@bitcycle.com</a>>> wrote:<br>
>>>>>> ><br>
>>>>>> > Yes of course, try my demonstration code yourself.<br>
>>>>>> > (In the current dev snapshots, -swift-version 4 is the default and -swift-version 3 is what you need to set if you want 3 compability)<br>
>>>>>> ><br>
>>>>>> >> On Sun, Jun 4, 2017 at 12:37 AM, Ben Rimmington <<a href="mailto:me@benrimmington.com">me@benrimmington.com</a> <mailto:<a href="mailto:me@benrimmington.com">me@benrimmington.com</a>> <mailto:<a href="mailto:me@benrimmington.com">me@benrimmington.com</a>>> wrote:<br>
>>>>>> >><br>
>>>>>> >> Are you using the Swift 4 language mode?<br>
>>>>>> >><br>
>>>>>> >> <<a href="https://swift.org/blog/swift-4-0-release-process/#source-compatibility" rel="noreferrer" target="_blank">https://swift.org/blog/swift-<wbr>4-0-release-process/#source-<wbr>compatibility</a><br>
>>>>>> <<a href="https://swift.org/blog/swift-4-0-release-process/#source-compatibility" rel="noreferrer" target="_blank">https://swift.org/blog/swift-<wbr>4-0-release-process/#source-<wbr>compatibility</a>>><br>
>>>>>> >><br>
>>>>>> >> -- Ben<br>
>>>>>><br>
>>>>>><br>
>>>>>> ______________________________<wbr>_________________<br>
>>>>>> swift-evolution mailing list<br>
>>>>>> <a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>><br>
>>>>>> <a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-<wbr>evolution</a><br>
>>>>>> <<a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-<wbr>evolution</a>><br>
>>>>><br>
>>>>><br>
>>>>> ______________________________<wbr>_________________<br>
>>>>> swift-evolution mailing list<br>
>>>>> <a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>><br>
>>>>> <a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-<wbr>evolution</a><br>
>>>> ______________________________<wbr>_________________<br>
>>>> swift-evolution mailing list<br>
>>>> <a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a> <mailto:<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.<wbr>org</a>><br>
>>>> <a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-<wbr>evolution</a><br>
<br>
______________________________<wbr>_________________<br>
swift-evolution mailing list<br>
<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a><br>
<a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-<wbr>evolution</a><br>
</div></div></blockquote></div><br></div>