What is your evaluation of the proposal?<br>Strong +1. The syntax is clear and obvious, without looking at documentation, one could almost guess it, and that's always great.<br><br> • Is the problem being addressed significant enough to warrant a change to Swift?<br>Yup! Using objective-c APIs is still the bread and butter in Apple's platforms, where a lot of Swift code is written. Stringly-typed selectors are incredibly fragile. The target-selector pattern will still feel a bit foreign in Swift, but this will at least make it look like we're passing a partially-applied function. <br><br> • Does this proposal fit well with the feel and direction of Swift?<br>Yes, particularly with the proposed syntax to name any function.<br><br> • How much effort did you put into your review? A glance, a quick reading, or an in-depth study?<br>I followed the discussion on the topic prior to the proposal these past weeks, and read the proposal from begging to end. <br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 18, 2016 at 10:21 AM Félix Cloutier <<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div># What is your evaluation of the proposal?</div><div><br></div></div><div style="word-wrap:break-word"><div>I support it.</div><div><br></div># Is the problem being addressed significant enough to warrant a change to Swift?<div><br></div><div>One great advantage of being able to name a selector from a method is that tools can tell that some method from some class is being used as a callback. It can't tell that just from a selector string. This will be useful for refactoring tools.</div><div><br></div><div>I think that selectors are going to die a very slow death, so we might as well have decent support for them.</div><div><br><div># Does this proposal fit well with the feel and direction of Swift?</div><div><br></div><div>I guess?</div><div><br></div><div># If you have you used other languages or libraries with a similar feature, how do you feel that this proposal compares to those?</div><div><br></div><div>Selectors are pretty unique to Objective-C.</div><div><br></div><div># How much effort did you put into your review? A glance, a quick reading, or an in-depth study?</div><div><br></div><div>I read the proposal and was already convinced. I followed a little bit of discussion and everything looked good.</div></div></div><div style="word-wrap:break-word"><div><div><br><div>
<br><span style="color:rgb(0,0,0);font-family:'Lucida Grande';font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;display:inline!important;float:none">Félix</span>
</div></div></div></div><div style="word-wrap:break-word"><div><div>
<br><div><blockquote type="cite"><div>Le 18 janv. 2016 à 11:02:27, Joe Groff via swift-evolution <<a href="mailto:swift-evolution@swift.org" target="_blank">swift-evolution@swift.org</a>> a écrit :</div><br><div><div style="word-wrap:break-word"><p style="word-wrap:break-word;margin:1.3125em 0px;line-height:1.3125em"><font color="#111111" face="Helvetica Neue, Helvetica, Arial, Verdana, sans-serif"><span style="font-size:14px">Hello Swift community,<br><br>The review of “Referencing the Objective-C selector of a method” begins now and runs through January 23, 2016. The proposal is available here:<br><br><span style="white-space:pre-wrap"> </span><a href="https://github.com/apple/swift-evolution/blob/master/proposals/0022-objc-selectors.md" target="_blank">https://github.com/apple/swift-evolution/blob/master/proposals/0022-objc-selectors.md</a><br><br>Reviews are an important part of the Swift evolution process. All reviews should be sent to the swift-evolution mailing list at<br><br><span style="white-space:pre-wrap"> </span><a href="https://lists.swift.org/mailman/listinfo/swift-evolution" target="_blank">https://lists.swift.org/mailman/listinfo/swift-evolution</a><br><br>or, if you would like to keep your feedback private, directly to the review manager.<br><br><b>What goes into a review?<br></b><br>The goal of the review process is to improve the proposal under review through constructive criticism and, eventually, determine the direction of Swift. When writing your review, here are some questions you might want to answer in your review:<br><br></span></font></p><div><font color="#111111" face="Helvetica Neue, Helvetica, Arial, Verdana, sans-serif"><span style="font-size:14px"><span style="white-space:pre-wrap"> </span>• What is your evaluation of the proposal?<br></span></font></div><div><font color="#111111" face="Helvetica Neue, Helvetica, Arial, Verdana, sans-serif"><span style="font-size:14px"><span style="white-space:pre-wrap"> </span>• Is the problem being addressed significant enough to warrant a change to Swift?<br></span></font></div><div><font color="#111111" face="Helvetica Neue, Helvetica, Arial, Verdana, sans-serif"><span style="font-size:14px"><span style="white-space:pre-wrap"> </span>• Does this proposal fit well with the feel and direction of Swift?<br></span></font></div><div><font color="#111111" face="Helvetica Neue, Helvetica, Arial, Verdana, sans-serif"><span style="font-size:14px"><span style="white-space:pre-wrap"> </span>• If you have you used other languages or libraries with a similar feature, how do you feel that this proposal compares to those?<br></span></font></div><div><font color="#111111" face="Helvetica Neue, Helvetica, Arial, Verdana, sans-serif"><span style="font-size:14px"><span style="white-space:pre-wrap"> </span>• How much effort did you put into your review? A glance, a quick reading, or an in-depth study?<br></span></font></div><p style="word-wrap:break-word;margin:1.3125em 0px;line-height:1.3125em"><font color="#111111" face="Helvetica Neue, Helvetica, Arial, Verdana, sans-serif"><span style="font-size:14px">More information about the Swift evolution process is available at<br><br><span style="white-space:pre-wrap"> </span><a href="https://github.com/apple/swift-evolution/blob/master/process.md" target="_blank">https://github.com/apple/swift-evolution/blob/master/process.md</a><br><br>Thank you,<br><br>-Joe<br><br>Review Manager</span></font></p></div>_______________________________________________<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" target="_blank">https://lists.swift.org/mailman/listinfo/swift-evolution</a><br></div></blockquote></div><br></div></div></div>_______________________________________________<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 dir="ltr">-- <br></div>Javier Soto