rleland     2004/04/23 22:13:28

  Modified:    src/share/org/apache/struts/action ActionServlet.java
  Log:
  Document some of the unhandled exceptions.
  
  Revision  Changes    Path
  1.176     +16 -4     
jakarta-struts/src/share/org/apache/struts/action/ActionServlet.java
  
  Index: ActionServlet.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/action/ActionServlet.java,v
  retrieving revision 1.175
  retrieving revision 1.176
  diff -u -r1.175 -r1.176
  --- ActionServlet.java        8 Apr 2004 22:07:56 -0000       1.175
  +++ ActionServlet.java        24 Apr 2004 05:13:28 -0000      1.176
  @@ -300,6 +300,11 @@
   
               // :FIXME: Why is this dependent on the container's version of 
commons-logging?
               // Shouldn't this depend on the version packaged with Struts?
  +            /*
  +              Reason: LogFactory.release(classLoader); was added as
  +              an attempt to investigate the OutOfMemory error reported on Bugzilla 
#14042.
  +              It was committed for version 1.136 by craigmcc
  +            */
           }
   
       }
  @@ -823,6 +828,13 @@
                     // FIXME Whenever we fail silently, we must document a valid 
reason
                     // for doing so.  Why should we fail silently if a property can't 
be set on
                     // the plugin?
  +                    /**
  +                     * Between version 1.138-1.140 cedric made these changes.
  +                     * The exceptions are caught to deal with containers applying 
strict security.
  +                     * This was in response to bug #15736
  +                     *
  +                     * Recommend that we make the currentPlugInConfigObject part of 
the PlugIn Interface if we can, Rob
  +                     */
                   }
                   plugIns[i].init(this, config);
   
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to