From df9b58de719d0c6e8e696f2015de8ae903173fb3 Mon Sep 17 00:00:00 2001
From: Godin <mandrikov@gmail.com>
Date: Tue, 23 Nov 2010 20:58:08 +0000
Subject: SONAR-1979: Add missing FindBugs rule - Dead store due to switch
 statement fall through to throw

---
 .../sonar/plugins/findbugs/profile-sonar-way-findbugs.xml    |  3 +++
 .../src/main/resources/org/sonar/plugins/findbugs/rules.xml  | 12 +++++++++++-
 2 files changed, 14 insertions(+), 1 deletion(-)

(limited to 'plugins/sonar-findbugs-plugin')

diff --git a/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/profile-sonar-way-findbugs.xml b/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/profile-sonar-way-findbugs.xml
index d3b7e6c9fd6..4de545b9191 100644
--- a/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/profile-sonar-way-findbugs.xml
+++ b/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/profile-sonar-way-findbugs.xml
@@ -37,6 +37,9 @@
   <Match>
     <Bug pattern="SF_DEAD_STORE_DUE_TO_SWITCH_FALLTHROUGH"/>
   </Match>
+  <Match>
+    <Bug pattern="SF_DEAD_STORE_DUE_TO_SWITCH_FALLTHROUGH_TO_THROW"/>
+  </Match>
   <Match>
     <Bug pattern="EQ_COMPARETO_USE_OBJECT_EQUALS"/>
   </Match>
diff --git a/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/rules.xml b/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/rules.xml
index f26445e2cc8..4f5be6190c2 100644
--- a/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/rules.xml
+++ b/plugins/sonar-findbugs-plugin/src/main/resources/org/sonar/plugins/findbugs/rules.xml
@@ -4299,7 +4299,17 @@ public @NonNegative Integer example(@Negative Integer value) {
       to a switch fall through. It is likely that you forgot to put a 
       break or return at the end of the previous case.
       </p>]]></description>
-  </rule>   
+  </rule>
+  <rule key="SF_DEAD_STORE_DUE_TO_SWITCH_FALLTHROUGH_TO_THROW" priority="CRITICAL">
+    <name>
+      <![CDATA[Dead store due to switch statement fall through to throw]]></name>
+    <configKey><![CDATA[SF_DEAD_STORE_DUE_TO_SWITCH_FALLTHROUGH_TO_THROW]]></configKey>
+    <category name="Reliability"/>
+    <description><![CDATA[<p>
+      A value stored in the previous switch case is ignored here due to a switch fall through to a place where an exception is thrown. 
+      It is likely that you forgot to put a break or return at the end of the previous case.
+      </p>]]></description>
+  </rule>
   <rule key="NM_CLASS_NAMING_CONVENTION" priority="MAJOR">
     <name>
       <![CDATA[Class names should start with an upper case letter]]></name>
-- 
cgit v1.2.3