[Lldb-commits] [PATCH] D13084: Fix covered-switch-default warning in FormatManager.

Bruce Mitchener via lldb-commits lldb-commits at lists.llvm.org
Tue Sep 22 19:30:16 PDT 2015


brucem created this revision.
brucem added reviewers: granata.enrico, clayborg.
brucem added a subscriber: lldb-commits.

The default case doesn't need to be here as the switch covers
all possible values. If there's a new "lazy bool" value added
in the future, the compiler would start to warn about the new
case not being covered.

http://reviews.llvm.org/D13084

Files:
  source/DataFormatters/FormatManager.cpp

Index: source/DataFormatters/FormatManager.cpp
===================================================================
--- source/DataFormatters/FormatManager.cpp
+++ source/DataFormatters/FormatManager.cpp
@@ -570,7 +570,6 @@
             case eLazyBoolYes:
                 return true;
             case eLazyBoolCalculate:
-            default:
                 break;
         }
     }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13084.35454.patch
Type: text/x-patch
Size: 387 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150923/689e804f/attachment.bin>


More information about the lldb-commits mailing list