[PATCH REVIEW] fix brace placement for readability

Christian Ambach ambi at samba.org
Wed Oct 17 14:31:35 MDT 2012


On 10/17/2012 09:20 PM, Michael Adam wrote:

> So, I really love this exception from the general rule,

I checked the README.Coding again and it even says that it should be
done this way:
If the beginning statement has to be broken across lines due to length,
the beginning brace should be on a line of its own.

So looks like I missed that detail up to now :)

> but hey, it is "just" a formatting rule, so if the majority would like
> to change it, I'd yield. ;)

No, I understand the motivation and there is no rule change necessary,
we just have to stick to the rule :)

Cheers,
Christian


More information about the samba-technical mailing list