clang warned that a trailing else was impossible; it was right because
a preceding 'if' was of the form:

 if (a == b || c)

instead of:

 if (a == b || a == c)

since 'c' is a non-zero constant, the 'if' was always true, making
the next 'else' impossible.