[llvm-commits] Removing redundant default cases

David Blaikie dblaikie at gmail.com
Fri Sep 23 19:22:04 PDT 2011


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).

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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110923/fcb379fc/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm_excess_default_fixes.diff
Type: application/octet-stream
Size: 34056 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110923/fcb379fc/attachment.obj>


More information about the llvm-commits mailing list