Hi,

When reviewing code, should I complain about excessive curly brackets in
code like this?

    if (foo) {
        bar = 0;
    }

Pulseaudio source code follows quite consistently the unwritten rule
that you shouldn't use curly braces unless necessary, so I was surprised
to notice that the unwritten rule was really unwritten (ie. not
mentioned in http://pulseaudio.org/wiki/CodingStyle). So, should I add
the rule to that page, or should I stop caring about this?

(As a personal preference, I actually would like to have "excessive"
curly braces, but even more than that I would like consistency.)

-- 
Tanu

_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss

Reply via email to