[jboss-cvs] JBossAS SVN: r107019 - in projects/jboss-jca/trunk: web/src/main/java/org/jboss/jca/web and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: jesper.pedersen
Date: 2010-07-22 09:22:56 -0400 (Thu, 22 Jul 2010)
New Revision: 107019

Modified:
   projects/jboss-jca/trunk/tools/findbugs/filter.xml
   projects/jboss-jca/trunk/web/src/main/java/org/jboss/jca/web/WARDeployer.java
Log:
Fixed a couple of Findbugs issues

Modified: projects/jboss-jca/trunk/tools/findbugs/filter.xml
===================================================================
--- projects/jboss-jca/trunk/tools/findbugs/filter.xml	2010-07-22 13:00:54 UTC (rev 107018)
+++ projects/jboss-jca/trunk/tools/findbugs/filter.xml	2010-07-22 13:22:56 UTC (rev 107019)
@@ -37,12 +37,24 @@
     <Class name="org.jboss.jca.core.api.WorkManager"/>
     <Bug code="Nm"/>
   </Match>
+  <Match>
+    <Class name="org.jboss.jca.core.connectionmanager.ConnectionManager"/>
+    <Bug code="Nm"/>
+  </Match>
+  <Match>
+    <Class name="org.jboss.jca.core.security.UsersRoles"/>
+    <Bug code="Se,PZLA"/>
+  </Match>
 
   <!--  org.jboss.jca.deployers filter -->
   <Match>
     <Class name="org.jboss.jca.deployers.fungal.RADeployer"/>
     <Bug code="DLS,ST"/>
   </Match>
+  <Match>
+    <Class name="org.jboss.jca.deployers.fungal.RADeployment"/>
+    <Bug code="EI2"/>
+  </Match>
 
   <!--  org.jboss.jca.sjc filter -->
   <Match>

Modified: projects/jboss-jca/trunk/web/src/main/java/org/jboss/jca/web/WARDeployer.java
===================================================================
--- projects/jboss-jca/trunk/web/src/main/java/org/jboss/jca/web/WARDeployer.java	2010-07-22 13:00:54 UTC (rev 107018)
+++ projects/jboss-jca/trunk/web/src/main/java/org/jboss/jca/web/WARDeployer.java	2010-07-22 13:22:56 UTC (rev 107019)
@@ -155,9 +155,9 @@
 
          return new WARDeployment(url, webapp, tmpDeployment, parent);
       }
-      catch (Exception e)
+      catch (Throwable t)
       {
-         throw new DeployException(e.getMessage(), e);
+         throw new DeployException(t.getMessage(), t);
       }
    }
 }