reference, declaration → definition
definition → references, declarations, derived classes, virtual overrides
reference to multiple definitions → definitions
unreferenced

    1
    2
    3
    4
    5
    6
    7
    8
    9
   10
   11
   12
   13
   14
   15
   16
   17
   18
   19
   20
   21
   22
   23
   24
   25
   26
   27
   28
   29
   30
   31
   32
   33
   34
   35
   36
   37
   38
   39
   40
   41
   42
   43
   44
   45
   46
   47
   48
   49
   50
   51
   52
   53
   54
   55

// RUN: %clang_cc1 -fsyntax-only -verify -Wdangling-else %s

void f(int a, int b, int c, int d, int e) {

  // should warn
  { if (a) if (b) d++; else e++; } // expected-warning {{add explicit braces to avoid dangling else}}
  { if (a) while (b) if (c) d++; else e++; } // expected-warning {{add explicit braces to avoid dangling else}}
  { if (a) switch (b) if (c) d++; else e++; } // expected-warning {{add explicit braces to avoid dangling else}}
  { if (a) for (;;) if (c) d++; else e++; } // expected-warning {{add explicit braces to avoid dangling else}}
  { if (a) if (b) if (d) d++; else e++; else d--; } // expected-warning {{add explicit braces to avoid dangling else}}

  if (a)
    if (b) {
      d++;
    } else e++; // expected-warning {{add explicit braces to avoid dangling else}}

  // shouldn't
  { if (a) if (b) d++; }
  { if (a) if (b) if (c) d++; }
  { if (a) if (b) d++; else e++; else d--; }
  { if (a) if (b) if (d) d++; else e++; else d--; else e--; }
  { if (a) do if (b) d++; else e++; while (c); }

  if (a) {
    if (b) d++;
    else e++;
  }

  if (a) {
    if (b) d++;
  } else e++;
}

// Somewhat more elaborate case that shouldn't warn.
class A {
 public:
  void operator<<(const char* s) {}
};

void HandleDisabledThing() {}
A GetThing() { return A(); }

#define FOO(X) \
   switch (0) default: \
     if (!(X)) \
       HandleDisabledThing(); \
     else \
       GetThing()

void f(bool cond) {
  int x = 0;
  if (cond)
    FOO(x) << "hello"; // no warning
}