You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by dw...@apache.org on 2010/06/15 04:15:13 UTC
svn commit: r954695 -
/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java
Author: dwoods
Date: Tue Jun 15 02:15:13 2010
New Revision: 954695
URL: http://svn.apache.org/viewvc?rev=954695&view=rev
Log:
OPENJPA-1700 FindBugs - This method might drop an exception. In general, exceptions should be handled or reported in some way, or they should be thrown out of the method.
Modified:
openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java
Modified: openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java?rev=954695&r1=954694&r2=954695&view=diff
==============================================================================
--- openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java (original)
+++ openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configurations.java Tue Jun 15 02:15:13 2010
@@ -626,8 +626,13 @@ public class Configurations {
throw new NestableRuntimeException(
_loc.get("naming-err", name).getMessage(), ne);
} finally {
- if (ctx != null)
- try { ctx.close(); } catch (Exception e) {}
+ if (ctx != null) {
+ try {
+ ctx.close();
+ } catch (NamingException ne) {
+ // ignore
+ }
+ }
}
}