[systemd-devel] [systemd-commits] 2 commits - src/core src/journal-remote src/network
grawity at gmail.com
Mon Dec 15 07:47:15 PST 2014
On Mon, Dec 15, 2014 at 5:40 PM, Lennart Poettering <mztabzr at 0pointer.de>
> On Sat, 13.12.14 08:32, Matthias Urlichs (matthias at urlichs.de) wrote:
> > Hi,
> > Zbigniew Jędrzejewski-Szmek:
> > > > wrap a few *_FOREACH macros in curly braces
> > > >
> > > cppcheck is full of errors anyway. I don't think we should make the
> > > less pretty just to satisfy a checker, and a rarely used one.
> > >
> > While you may be right about cppcheck, IMHO it's good style to wrap all
> > multi-line subordinates in curlies on general principle.
> For the sake of brevity our CODING_STYLE docs suggest to leave out
> unnecessary curly braces. Single-line code blocks should not be
> enclosed in them, this is not PHP after all...
There's no difference between C and PHP in this case. Perhaps you meant
Perl or Go?
Mantas Mikulėnas <grawity at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the systemd-devel