You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by rl...@apache.org on 2004/04/24 07:13:28 UTC
cvs commit: jakarta-struts/src/share/org/apache/struts/action ActionServlet.java
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: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org