[coreboot-gerrit] Change in coreboot[master]: Documentation/contributing/coding_style: remove bug-causing rule

[ron minnich (Code Review)]

Attention is currently required from: Stefan Reinauer, Paul Menzel, David Hendricks, Julius Werner, Felix Held.
ron minnich has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51825 )

Change subject: Documentation/contributing/coding_style: remove bug-causing rule
......................................................................

Patch Set 1: Code-Review+2

(1 comment)

Patchset:

PS1: 
I did a quick google search and it does seem that people now accept the value of "always use {}" for C. Using {} on single lines is a recommended practice:
https://wiki.sei.cmu.edu/confluence/display/c/EXP19-C.+Use+braces+for+the+bo...

Key point: " indentation gives strong (but misleading) guidance to the structure."

absolutely. White space has no meaning in C, and I worry about depending on something with no meaning to provide meaning.

From a convenience point of view, I use 3 languages for firmware now, and 2 of them (Rust, Go) don't allow the option of NOT using {}. I am finding it jarring going back and forth now to C. I realize this is a little thing, but it it's nicer when such similar languages are a little bit more the same.