<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Apr 26, 2016 at 4:28 PM, Tony Allevato <span dir="ltr"><<a href="mailto:allevato@google.com" target="_blank">allevato@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><span class=""><div dir="ltr">On Sun, Apr 24, 2016 at 2:57 AM Nicola Salmoria via swift-evolution <<a href="mailto:swift-evolution@swift.org" target="_blank">swift-evolution@swift.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> > func isEqual(to other: Self) ->Bool<br>
> > func isLess(than other: Self) ->Bool<br>
> > func isLessThanOrEqual(to other: Self) ->Bool<br>
><br>
> I'm still not sure why these are methods instead of operators.<br>
<br>
I think this is an *excellent* choice, and I hope it is the first step to completely removing operators from protocols.<br>
<br>
IMHO throwing operators into protocols is inconsistent and confusing. Having regular methods and a single generic version of the operator that calls down on the type’s methods is clearer and guarantees that generic code can avoid ambiguities by calling the methods directly, instead of having to rely only on heavily overloaded global operators.<br></blockquote><div><br></div></span><div>I personally disagree on this point. To me, a protocol describes a set of requirements for a type to fulfill, which includes things other than methods. Just as a protocol can define initializers, properties, and associated types that a type must define in order to conform, it makes sense that a protocol would also define which operators a conforming type must support.</div></div></div></blockquote><div><br></div><div>Well, I'm not sure about that. A protocol describes what a type can do, so it's debatable whether a global function is within this scope.<br><br></div><div>Operators are magically special: you can declare them inside a protocol and require them to be available for conformance, even if they don't belong to the type. You can't do the same thing for normal global functions, yet conceptually global functions and operators are the same thing.<br></div><br><div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>Introducing a mapping between names and operators poses a few problems:</div><div><br></div><div>– IMO, they are overly verbose and add noise to the definition. This makes the language look less clean (I'm getting visions of NSDecimalNumber).</div><div>– They expose two ways to accomplish the same thing (writing `x.isEqual(to: y)` and `x == y`).</div><div>– Do certain operators automatically get mapped to method names with appropriate signatures across all types, or does a conforming type still have to provide that mapping by implementing the operators separately? If it's the latter, that's extra work for the author of the type writing the protocol. If it's the former, does it make sense to automatically push these operators for all types? Should any type that has an `add` method automatically get `+` as a synonym as well? That may not be desirable.</div></div></div></blockquote><div><br></div><div>The difference at the protocol declaration is between:<br><br>protocol Equatable {<br> func ==(lhs: Self, rhs: Self) -> Bool<br>}<br></div><div><br>and:<br><br>protocol Equatable {<br> func isEqual(to other: Self) -> Bool<br>}<br><br>func ==<T: Equatable>(lhs: T, rhs: T) -> Bool {<br> return lhs.isEqual(to: rhs)<br>}<br></div><div><br></div><div>so the latter is a bit more verbose, but arguably clearer in intent, and not different from how you would define any generic global function using a protocol, or from how you can define protocol extensions with default implementations that take advantage of the protocol's core methods.<br></div><div><br></div><div>The difference for the conformance is between:<br></div><div><br></div><div>extension Foo : Equatable { }<br></div><div><br>func ==(lhs: Foo, rhs: Foo) -> Bool {<br> return <comparison><br>}<br><br></div><div>and:<br><br></div><div>extension Bar : Equatable {<br> func isEqual(to: Bar) -> Bool {<br> return <comparison><br> }<br>}<br></div><div><br></div><div>the former way to define the conformance can be confusing to newbies. The latter is straightforward and consistent with the usual way to adopt a protocol.<br><br></div><div>The == operator looks exactly the same at its use points, but the way how it's implemented is different.<br>In the former case, it's many overloads of a global function, which can stress the compiler's type inference and doesn't offer an obvious way to disambiguate in case of ambiguities.<br></div><div>In the latter case, there is only one generic definition of ==, which automatically applies to all types that conform to the protocol.<br></div><div> <br></div><div>Nicola<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>I'm very supportive of the floating-point protocol proposal in general, but I feel the arithmetic and comparison operations should be exposed by operators alone and not by methods, where there is a suitable operator that has the intended meaning.</div><div><span style="line-height:1.5"><br></span></div><div><span style="line-height:1.5"> </span><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
—<br>
Nicola<br>
<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></div>
</blockquote></div><br></div></div>