<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 15, 2014 at 5:40 PM, Lennart Poettering <span dir="ltr"><<a href="mailto:mztabzr@0pointer.de" target="_blank">mztabzr@0pointer.de</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, 13.12.14 08:32, Matthias Urlichs (<a href="mailto:matthias@urlichs.de">matthias@urlichs.de</a>) wrote:<br>
<br>
> Hi,<br>
><br>
> Zbigniew Jędrzejewski-Szmek:<br>
> > >     wrap a few *_FOREACH macros in curly braces<br>
> > ><br>
> > cppcheck is full of errors anyway. I don't think we should make the code<br>
> > less pretty just to satisfy a checker, and a rarely used one.<br>
> ><br>
> While you may be right about cppcheck, IMHO it's good style to wrap all<br>
> multi-line subordinates in curlies on general principle.<br>
<br>
</span>For the sake of brevity our CODING_STYLE docs suggest to leave out<br>
unnecessary curly braces. Single-line code blocks should not be<br>
enclosed in them, this is not PHP after all...<br></blockquote><div><br></div><div>There's no difference between C and PHP in this case. Perhaps you meant Perl or Go?</div></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Mantas Mikulėnas <<a href="mailto:grawity@gmail.com" target="_blank">grawity@gmail.com</a>></div></div>
</div></div>