Extended Description

1. Clang should warn on a case value that exceeds the range of the type in switch.

int test1(const char foo) {
int success = 0;
switch(foo[0]) {
case 1234: / XXX */
success = 1;
break;
default:
break;
}
return success;
}

2. Clang should warn on switch(bool) since it's almost certainly wrong (developers seem okay with using if-statements instead).

int test2(const char foo) {
int success = 0;
switch (foo && foo[0]) { / XXX */
case 0:
success = 1;
break;
case 1:
break;
}
return success;
}