Ceph

Looks basically fine to me (tho i didn't read the whole thing). It leaves a lot of stuff unspecified, though. Notably, they allow all of

if (condition) foo;
 if (condition)
   foo;
 if (condition {
   foo;
 }

I like the linux kernel guidelines, which specify

- no single line if (easy to miss when visually scanning code; sadly i did a lot of this in the older mds code)
- no braces for single line then block

Both specify that the then and else blocks must neither or both have braces.