You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2012/02/14 13:00:58 UTC

svn commit: r1243886 - /openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java

Author: rmannibucau
Date: Tue Feb 14 12:00:57 2012
New Revision: 1243886

URL: http://svn.apache.org/viewvc?rev=1243886&view=rev
Log:
fixing exception management

Modified:
    openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java

Modified: openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java?rev=1243886&r1=1243885&r2=1243886&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/ConfigurationFactory.java Tue Feb 14 12:00:57 2012
@@ -88,7 +88,6 @@ import java.io.FileNotFoundException;
 import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
-import java.io.StringReader;
 import java.io.Writer;
 import java.net.URI;
 import java.net.URISyntaxException;
@@ -1318,10 +1317,10 @@ public class ConfigurationFactory implem
 
     private static AppInfo loadDump(final String modulePath, final InputStream input) throws OpenEJBException {
         if (input == null || modulePath == null) {
-            throw new NullPointerException("input can't be null");
+            throw new OpenEJBException("input and modulePath can't be null");
         }
 
-        String read = null;
+        String read;
         try {
             read = IO.slurp(input);
         } catch (IOException e) {