[jBoss-Dev] CVS update: jboss/src/main/org/jboss/ejbContainerFactory.java

[jBoss CVS Development]

  User: juha    
  Date: 00/08/26 13:10:42

  Modified:    src/main/org/jboss/ejb ContainerFactory.java
  Log:
  Fixes an NPE bug on resource-refs on jboss.xml when a match
  was not found in ejb-jar.xml.
  
  Revision  Changes    Path
  1.37      +10 -2     jboss/src/main/org/jboss/ejb/ContainerFactory.java
  
  Index: ContainerFactory.java
  ===================================================================
  RCS file: /products/cvs/ejboss/jboss/src/main/org/jboss/ejb/ContainerFactory.java,v
  retrieving revision 1.36
  retrieving revision 1.37
  diff -u -r1.36 -r1.37
  --- ContainerFactory.java     2000/08/22 19:27:16     1.36
  +++ ContainerFactory.java     2000/08/26 20:10:41     1.37
  @@ -76,7 +76,7 @@
   *   @author <a href="mailto:[EMAIL PROTECTED]">Juha Lindfors</a>
   *   @author <a href="mailto:[EMAIL PROTECTED]">Sebastien Alborini</a>
   *
  -*   @version $Revision: 1.36 $
  +*   @version $Revision: 1.37 $
   */
   public class ContainerFactory
        extends org.jboss.util.ServiceMBeanSupport
  @@ -603,8 +603,16 @@
   
                        // Register deployment. Use the original name in the hashtable
                        deployments.put(origUrl, app);
  -             } catch (Throwable e)
  +             }
  +        catch (Throwable e)
                {
  +            if (e instanceof NullPointerException) {
  +                // Avoids useless 'null' messages on a server trace.
  +                // Let's be honest and spam them with a stack trace.
  +                // NPE should be considered an internal server error anyways.
  +                //Logger.exception(e);
  +            }
  +            
                        Logger.debug(e.getMessage());
   
                        app.stop();