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

[Bruce Mitchener via lldb-commits]

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;
         }
     }



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;
         }
     }

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits