<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 24, 2015 at 5:20 PM, Richard <span dir="ltr"><<a href="mailto:legalize@xmission.com" target="_blank" class="cremed">legalize@xmission.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">In <a href="http://reviews.llvm.org/D8532#145854" target="_blank" class="cremed">http://reviews.llvm.org/D8532#145854</a>, @djasper wrote:<br>
<br>
> I don't feel strongly about this, and I can see some of your reasoning.<br>
>  However, an "if (a) return true; else return false;" is very suspect to me<br>
>  and I think "return a;" is more readable, independent of whether it is at<br>
>  the end of a chain or not.<br>
<br>
<br>
</span>This is my view on readability as well and one of the reasons I wrote this check for `clang-tidy`.<br>
<br>
These patches arose from me trying out the new check on a "real codebase" instead of my single lint based test file.  A couple of issues arose from applying the new check on the code that is helping me to improve my check beyond just handling the basics, so this is all great feedback for me, even if the patches don't get accepted.<br>
<br>
The LLVM coding rule of "no `else` after a `return`, `continue`, etc." probably should be turned into a clang-tidy check, at least for detection if not correction.<br></blockquote><div><br></div><div>I have committed an initial version of such a check in r226025.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">
<br>
<a href="http://reviews.llvm.org/D8532" target="_blank" class="cremed">http://reviews.llvm.org/D8532</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank" class="cremed">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>