<div dir="ltr"><div dir="ltr">I use this idiom too in the layout code. I guess I just prefer a more compact code.</div><div dir="ltr">(I don't feel too strongly about it though)</div><div dir="ltr"><br></div><div dir="ltr">Alan.<br></div><div dir="ltr"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Feb 7, 2019 at 7:31 PM Alex Christensen <<a href="mailto:achristensen@apple.com">achristensen@apple.com</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">If you search for “return completionHandler” in WebKit you will find over a hundred instances.  Most if not all of them return void.  It means call the completion handler and return.  I probably wrote most of them and obviously think it’s a fabulous idiom.<br>
<br>
> On Feb 7, 2019, at 2:32 PM, Geoffrey Garen <<a href="mailto:ggaren@apple.com" target="_blank">ggaren@apple.com</a>> wrote:<br>
> <br>
> FWIW, I’ve always felt conflicted about this case.<br>
> <br>
> I very much prefer early return to if/else chains.<br>
> <br>
> However, “return f()” when f returns void is a bit mind bending.<br>
> <br>
> So, I can’t use my preferred style in functions that return void. Boo. <br>
> <br>
> Geoff<br>
> <br>
>> On Feb 7, 2019, at 12:47 PM, Daniel Bates <<a href="mailto:dbates@webkit.org" target="_blank">dbates@webkit.org</a>> wrote:<br>
>> <br>
>> Hi all,<br>
>> <br>
>> Something bothers me about code like:<br>
>> <br>
>> void f();<br>
>> void g()<br>
>> {<br>
>>    if (...)<br>
>>        return f();<br>
>>    return f();<br>
>> }<br>
>> <br>
>> I prefer:<br>
>> <br>
>> void g()<br>
>> {<br>
>>    if (...) {<br>
>>        f();<br>
>>        return<br>
>>    }<br>
>>    f();<br>
>> }<br>
>> <br>
>> Does it bother you? For the former? For the latter? Update our style guide?<br>
>> <br>
>> Opinions, please.<br>
>> <br>
>> Dan<br>
>> _______________________________________________<br>
>> webkit-dev mailing list<br>
>> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
>> <a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
> <br>
> _______________________________________________<br>
> webkit-dev mailing list<br>
> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
> <a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
</blockquote></div>