Date: Sun, 12 Jul 2020 13:01:59 +1000 From: Luke Mewburn <lu...@netbsd.org> Message-ID: <20200712030159.gh12...@mewburn.net>
| | IMHO, permitting braces to be consistently used: | | - Adds to clarity of intent. | | - Aids code review. | | - Avoids gotofail: https://en.wikipedia.org/wiki/Unreachable_code#goto_fail_bug Permitting the braces is probably no big deal, but does none of that. Actually using the extra braces might, but unless you change "permitting" to "requiring", that's unlikely to happen a lot. I simply cannot see myself changing if (p == NULL) return NULL; into if (p == NULL) { return NULL; } Aside from anything else, the closing brace occupies an extra line (and often two, as those are often followed by blank lines) which means two less lines of context I get to see in my window (however big the window is - enlarging it still means 2 less lines of context than would be possible) - and that's for each time this is done. But as long as they're just permitted, and not required, then I don't have a big problem with it - but note that if I'm working on code written like that, I'm likely to delete non-required meaningless braces (just as cleaning up trailing whitespace, fixing tab vs space indentation, and wrapping long lines, etc). kre