[systemd-devel] [systemd-commits] 2 commits - src/core src/journal-remote src/network
mztabzr at 0pointer.de
Mon Dec 15 07:40:54 PST 2014
On Sat, 13.12.14 08:32, Matthias Urlichs (matthias at urlichs.de) wrote:
> 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 code
> > 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...
Lennart Poettering, Red Hat
More information about the systemd-devel