You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/05/10 22:21:04 UTC

svn commit: r1101631 - /commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java

Author: simonetripodi
Date: Tue May 10 20:21:03 2011
New Revision: 1101631

URL: http://svn.apache.org/viewvc?rev=1101631&view=rev
Log:
bodyText can be a StringBuilder that's faster (and Digester instances are not thread-safe by default, so...)

Modified:
    commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java

Modified: commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java
URL: http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java?rev=1101631&r1=1101630&r2=1101631&view=diff
==============================================================================
--- commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java (original)
+++ commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/Digester.java Tue May 10 20:21:03 2011
@@ -144,13 +144,13 @@ public class Digester extends DefaultHan
     /**
      * The body text of the current element.
      */
-    private StringBuffer bodyText = new StringBuffer();
+    private StringBuilder bodyText = new StringBuilder();
 
 
     /**
      * The stack of body text string buffers for surrounding elements.
      */
-    protected Stack<StringBuffer> bodyTexts = new Stack<StringBuffer>();
+    protected Stack<StringBuilder> bodyTexts = new Stack<StringBuilder>();
 
 
     /**
@@ -1388,7 +1388,7 @@ public class Digester extends DefaultHan
         if (debug) {
             log.debug("  Pushing body text '" + bodyText.toString() + "'");
         }
-        bodyText = new StringBuffer();
+        bodyText = new StringBuilder();
 
         // the actual element name is either in localName or qName, depending 
         // on whether the parser is namespace aware