<div dir="ltr">I personally prefer always including the braces, but, of course, I'll abide by the local style rules.  That said, is the local style rule implemented in clang-format?  I've never noticed it removing nor adding syntactically unnecessary braces.<div><br></div><div>A few times, I've had a bit of a panic attack on code of this form:</div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><font face="monospace">if (condition)</font></div><div><font face="monospace">  // there's a comment here</font></div><div><font face="monospace">  Statement = Something;</font></div><div><font face="monospace">else</font></div><div><font face="monospace">  ComputeThisOrThat();</font></div></blockquote><font face="monospace"><br></font><div><font face="arial, sans-serif">It's correct, but it looks like it's not, especially when it's nested within a clause of another "outer" if statement.</font></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 15, 2020 at 2:35 PM David Greene via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.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">Matt Arsenault via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> writes:<br>
<br>
> I think braces should be added in all contexts, and the more contexts<br>
> the better. It eliminates any inconsistency or attempt to contextually<br>
> interpret rules. It also reduces merge conflicts, since something<br>
> eventually something will probably be added inside any control flow<br>
> statement. I’ve suffered through many painful merges trying to find<br>
> where the braces went wrong, usually due to switch statements. The<br>
> sometimes-braces-sometimes-not combined with the lack of indentation<br>
> for switch cases leads to way more time figuring out braces than<br>
> should be necessary.<br>
<br>
+1.  In addition, the lack of braces can lead to subtle bugs if one adds<br>
a line to an existing single-line conditional statement.  Best to be<br>
safe and always use braces.<br>
<br>
                -David<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>