<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">I don’t agree with that. To me, everything between <span style="color: rgb(181, 27, 71); font-family: Menlo; font-size: 11px;" class="">return </span>and <span style="color: rgb(181, 27, 71); font-family: Menlo; font-size: 11px;" class="">if</span> (including the <span style="color: rgb(181, 27, 71); font-family: Menlo; font-size: 11px;" class="">return</span> itself) only being executed if the condition is true seems like the logical way of interpreting that syntax.</div><div class=""><br class=""></div><div class="">I do, however, see how people skimming through code might be confused, but a <span style="color: rgb(181, 27, 71); font-family: Menlo; font-size: 11px;" class="">return</span> with code after it should make it obvious that there’s more to it than always returning at that point.</div><br class=""><div><blockquote type="cite" class=""><div class="">On 05 Aug 2016, at 04:58, rintaro ishizaki <<a href="mailto:fs.output@gmail.com" class="">fs.output@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Limiting for `<font face="monospace, monospace" class="">return</font>` introduces another problem.<div class=""><br class=""></div><div class=""><font face="monospace, monospace" class="">mutating func increment() -> Val {</font></div><div class=""><font face="monospace, monospace" class=""> self = self.incremented()</font></div><div class=""><font face="monospace, monospace" class=""> return self</font></div><div class=""><font face="monospace, monospace" class="">}</font></div><div class=""><br class=""></div><div class=""><font face="monospace, monospace" class="">return val.increment() if val == sentinel</font></div><div class=""><font face="monospace, monospace" class="">print(val)</font></div><div class=""><br class=""></div><div class="">I think, it's not obvious `<font face="monospace, monospace" class="">val`</font> after `<font face="monospace, monospace" class="">return ... if</font>` is incremented or not.</div><div class=""><br class=""></div></div><div class="gmail_extra"><br class=""><div class="gmail_quote">2016-08-05 9:11 GMT+09:00 Julian Dunskus via swift-evolution <span dir="ltr" class=""><<a href="mailto:swift-evolution@swift.org" target="_blank" class="">swift-evolution@swift.org</a>></span>:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 05 Aug 2016, at 01:24, Brent Royal-Gordon <<a href="mailto:brent@architechies.com" class="">brent@architechies.com</a>> wrote:<br class="">
><br class="">
>> On Aug 4, 2016, at 6:38 AM, Julian Dunskus via swift-evolution <<a href="mailto:swift-evolution@swift.org" class="">swift-evolution@swift.org</a>> wrote:<br class="">
>><br class="">
>> return nil unless let thing = things[index]<br class="">
><br class="">
> On the `unless` part, at least, see the "Rename guard to unless" section in Commonly Rejected Changes: <<a href="https://github.com/apple/swift-evolution/blob/master/commonly_proposed.md" rel="noreferrer" target="_blank" class="">https://github.com/apple/<wbr class="">swift-evolution/blob/master/<wbr class="">commonly_proposed.md</a>><br class="">
><br class="">
<br class="">
</span>I read that, see paragraph three of the email you replied to (my second in this thread)<br class="">
<div class="HOEnZb"><div class="h5">______________________________<wbr class="">_________________<br class="">
swift-evolution mailing list<br class="">
<a href="mailto:swift-evolution@swift.org" class="">swift-evolution@swift.org</a><br class="">
<a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank" class="">https://lists.swift.org/<wbr class="">mailman/listinfo/swift-<wbr class="">evolution</a><br class="">
</div></div></blockquote></div><br class=""></div>
</div></blockquote></div><br class=""></body></html>