Feature #1190
closed
- Target version changed from v0.31 to v0.32
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.
- Translation missing: en.field_position set to 709
- Translation missing: en.field_story_points set to 5
- Translation missing: en.field_position deleted (
720)
- Translation missing: en.field_position set to 720
- Assignee set to Sage Weil
- Status changed from New to Resolved
Also available in: Atom
PDF