A similar patch for one I've just sent out for clang - removing any default cases in a switch that already has all the enum values specified (only for switches over enums).<br><br>I don't know whether this stylistic/coding convention is consistent between llvm and clang, so thought I'd ask first. Ideally once we get a good warning for this in clang we could turn it on by default.