On Mon, Jul 25, 2011 at 6:55 PM, Avi Kivity <a...@redhat.com> wrote: > It's already allowed by the example; there are about 1800 instances in the > tree; and disallowing it would lead to > > if (a) { > ... > } else { > if (b) { > ... > } else { > if (c) { > ... > } else { > if (d) { > ... > } else { > ... > } > } > } > } > > instead of > > if (a) { > ... > } else if (b) { > ... > } else if (c) { > ... > } else if (d) { > ... > } else { > ... > } > > which is more readable. > > Signed-off-by: Avi Kivity <a...@redhat.com>
Acked-by: Blue Swirl <blauwir...@gmail.com> > --- > CODING_STYLE | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/CODING_STYLE b/CODING_STYLE > index 5ecfa22..6e61c49 100644 > --- a/CODING_STYLE > +++ b/CODING_STYLE > @@ -68,6 +68,10 @@ keyword. Example: > printf("a was something else entirely.\n"); > } > > +Note that 'else if' is considered a single statement; otherwise a long if/ > +else if/else if/.../else sequence would need an indent for every else > +statement. > + > An exception is the opening brace for a function; for reasons of tradition > and clarity it comes on a line by itself: > > -- > 1.7.5.3 > >