<div dir="ltr">Speaking of coding style: there is no pre-upload check configured in the coreboot git repo, if I am not mistaken, should one be added?<div><br></div><div>As for this particular change - I don't like wasting vertical space, but looks like overwhelming sentiment is supporting this proposal, so I sure can live with it, especially if there is a pre-upload check to enforce it.</div><div><br></div><div>-vb</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, May 16, 2018 at 8:16 AM Patrick Georgi via coreboot <<a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi everybody,<br>
<br>
after just running into an issue on the EC code base, I hereby propose that<br>
going forward, we should always wrap conditional blocks in braces, even<br>
one-liners.<br>
That is:<br>
<br>
if (foo) {<br>
   bar();<br>
}<br>
<br>
instead of<br>
<br>
if (foo)<br>
   bar();<br>
<br>
It doesn't hurt too much but saves us from accidentally adding baz() after<br>
bar(), forgetting to add the - now required - braces. If we get rough<br>
consensus over this, I'd change Coding_Style to match.<br>
<br>
<br>
Thoughts?<br>
Patrick<br>
-- <br>
Google Germany GmbH, ABC-Str. 19, 20354 Hamburg<br>
Registergericht und -nummer: Hamburg, HRB 86891, Sitz der Gesellschaft:<br>
Hamburg<br>
Geschäftsführer: Paul Manicle, Halimah DeLaine Prado<br>
<br>
-- <br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org" target="_blank">coreboot@coreboot.org</a><br>
<a href="https://mail.coreboot.org/mailman/listinfo/coreboot" rel="noreferrer" target="_blank">https://mail.coreboot.org/mailman/listinfo/coreboot</a></blockquote></div>