You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@turbine.apache.org by il...@apache.org on 2001/03/16 02:00:07 UTC
cvs commit: jakarta-turbine/src/java/org/apache/turbine/services/rundata DefaultTurbineRunData.java
ilkka 01/03/15 17:00:06
Modified: src/java/org/apache/turbine/services/rundata
DefaultTurbineRunData.java
Log:
The instantiation of FormMessages has been delegated to the RunData
implementation to happen on demand. This reduces memory
allocation in cases where FormMessages are not in use.
Revision Changes Path
1.2 +5 -1 jakarta-turbine/src/java/org/apache/turbine/services/rundata/DefaultTurbineRunData.java
Index: DefaultTurbineRunData.java
===================================================================
RCS file: /home/cvs/jakarta-turbine/src/java/org/apache/turbine/services/rundata/DefaultTurbineRunData.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- DefaultTurbineRunData.java 2001/03/16 00:06:37 1.1
+++ DefaultTurbineRunData.java 2001/03/16 01:00:04 1.2
@@ -106,7 +106,7 @@
* @author <a href="mailto:jon@latchkey.com">Jon S. Stevens</a>
* @author <a href="mailto:bhoeneis@ee.ethz.ch">Bernie Hoeneisen</a>
* @author <a href="mailto:dlr@finemaltcoding.com">Daniel Rall</a>
- * @version $Id: DefaultTurbineRunData.java,v 1.1 2001/03/16 00:06:37 ilkka Exp $
+ * @version $Id: DefaultTurbineRunData.java,v 1.2 2001/03/16 01:00:04 ilkka Exp $
*/
public class DefaultTurbineRunData
extends RecyclableSupport
@@ -770,6 +770,10 @@
*/
public FormMessages getMessages()
{
+ if (this.messages == null)
+ {
+ this.messages = new FormMessages();
+ }
return this.messages;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
DefaultTurbineRunData.java
Posted by Jon Stevens <jo...@latchkey.com>.
on 3/15/01 5:00 PM, "ilkka@apache.org" <il...@apache.org> wrote:
> + if (this.messages == null)
> + {
> + this.messages = new FormMessages();
> + }
This should be 4 space indentations.
thanks,
-jon
--
If you come from a Perl or PHP background, JSP is a way to take
your pain to new levels. --Anonymous
<http://jakarta.apache.org/velocity/ymtd/ymtd.html>
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org