On 07/25/2011 10:55 AM, Avi Kivity 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>

Applied.  Thanks.

Regards,

Anthony Liguori

---
  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:



Reply via email to