As I mentioned above, I agree that better diagnostics for near-misses are necessary, but they are possible without new syntax. There is no win in avoiding unintentional behavior because, without a default implementation, these issues are caught at compile time already.<br><div class="gmail_quote"><div dir="ltr">On Tue, Sep 20, 2016 at 10:14 Vladimir.S via swift-evolution &lt;<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
 &gt; extension P {<br>
 &gt; implement func foo() -&gt; [String : String] { return [:] }<br>
 &gt; }<br>
<br>
Yes, it seems like we need `implement` (or `override` as another<br>
suggestion) in protocol extension also just for the same reasons - be clear<br>
about our intention regarding implementing the requirement, to show that<br>
this func *depends* on the previous definition of P protocol and to avoid<br>
possible mistakes related to protocol conformance.<br>
<br>
On 20.09.2016 17:38, Charles Srstka wrote:<br>
&gt;&gt; On Sep 20, 2016, at 8:17 AM, Vladimir.S via swift-evolution<br>
&gt;&gt; &lt;<a href="mailto:swift-evolution@swift.org" target="_blank">swift-evolution@swift.org</a> &lt;mailto:<a href="mailto:swift-evolution@swift.org" target="_blank">swift-evolution@swift.org</a>&gt;&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; On 20.09.2016 3:03, Xiaodi Wu via swift-evolution wrote:<br>
&gt;&gt;&gt; I definitely think Vladimir&#39;s suggestion is a great starting point, IMO.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; However, I think it could be improved in one key respect where previous<br>
&gt;&gt;&gt; proposals using `override` are superior. Namely, the proposed `implement`<br>
&gt;&gt;&gt; keyword adds no additional safety when a type implements a protocol<br>
&gt;&gt;&gt; requirement that doesn&#39;t have a default implementation. This is because, if<br>
&gt;&gt;<br>
&gt;&gt; Yes, *at the moment of writing* the type&#39;s code there could be no default<br>
&gt;&gt; implementation for protocol requirement. But, *at the moment of<br>
&gt;&gt; compilation* such default implementation could appear.<br>
&gt;&gt;<br>
&gt;&gt; Let&#39;s discuss such scenario in case we&#39;ll take your suggestion:<br>
&gt;&gt;<br>
&gt;&gt; You got SomeClass.swift file, 3rd party file you don&#39;t want to change or<br>
&gt;&gt; changes are not allowed. Content:<br>
&gt;&gt;<br>
&gt;&gt; public protocol SomeProtocol {<br>
&gt;&gt; func foo()<br>
&gt;&gt; }<br>
&gt;&gt;<br>
&gt;&gt; public class SomeClass : SomeProtocol {<br>
&gt;&gt; func foo() {...} // no default implementation *at the moment of writing*,<br>
&gt;&gt; no need in `overload`<br>
&gt;&gt; }<br>
&gt;&gt;<br>
&gt;&gt; Now, you adds SomeClass.swift file to your project and in some *other*<br>
&gt;&gt; file you write:<br>
&gt;&gt;<br>
&gt;&gt; extension SomeProtocol {<br>
&gt;&gt; func foo() {...}<br>
&gt;&gt; }<br>
&gt;&gt;<br>
&gt;&gt; As you see, you don&#39;t control the SomeClass.swift but you suggest in this<br>
&gt;&gt; case SomeClass.foo() should be defined with `override`.<br>
&gt;&gt;<br>
&gt;&gt; With &#39;implement&#39; SomeClass.foo() will be marked initially and will save<br>
&gt;&gt; us if protocol&#39;s requirement PLUS default implementation changed.<br>
&gt;<br>
&gt; Requiring the ‘implement’ keyword can help us even if no default<br>
&gt; implementation is involved. Consider:<br>
&gt;<br>
&gt; protocol P {<br>
&gt; func foo() -&gt; [String : Any]<br>
&gt; }<br>
&gt;<br>
&gt; struct S : P {<br>
&gt; func foo() -&gt; [String : String] { return [:] }<br>
&gt; }<br>
&gt;<br>
&gt; We will get an error here that S does not conform to P. However, this is<br>
&gt; not the correct error, since S in fact *tries* to conform to P, but it has<br>
&gt; a mistake in a method signature. This misleads us as to the true nature of<br>
&gt; the problem, and if S has enough members in it that we fail to spot the<br>
&gt; existing foo(), we might solve the problem by reimplementing foo(), and<br>
&gt; leaving the original foo() as dangling dead code. Having an ‘implement’<br>
&gt; keyword on the existing foo() function would change the compiler error to<br>
&gt; let us know that we have an existing foo() that is incorrectly declared.<br>
&gt;<br>
&gt; In addition, ‘implement’ can help us when the declaration in question *is*<br>
&gt; the default implementation:<br>
&gt;<br>
&gt; protocol P {<br>
&gt; func foo() -&gt; [String : Any]<br>
&gt; }<br>
&gt;<br>
&gt; extension P {<br>
&gt; implement func foo() -&gt; [String : String] { return [:] }<br>
&gt; }<br>
&gt;<br>
&gt; Here we will get an error with the proposed ‘implement’ keyword, because<br>
&gt; foo() does not have a signature matching anything in the protocol, whereas<br>
&gt; without ‘implement’ we would happily and silently generate a useless<br>
&gt; dangling function that would never be used, and then pass the buck to the<br>
&gt; concrete type that implements P:<br>
&gt;<br>
&gt; protocol P {<br>
&gt; func foo() -&gt; [String : Any]<br>
&gt; }<br>
&gt;<br>
&gt; extension P {<br>
&gt; func foo() -&gt; [String : String] { return [:] } // The error is here:<br>
&gt; }<br>
&gt;<br>
&gt; struct S : P {} // But it gets reported here.<br>
&gt;<br>
&gt; Charles<br>
&gt;<br>
_______________________________________________<br>
swift-evolution mailing list<br>
<a href="mailto:swift-evolution@swift.org" target="_blank">swift-evolution@swift.org</a><br>
<a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank">https://lists.swift.org/mailman/listinfo/swift-evolution</a><br>
</blockquote></div>