svn commit: r1225442 - /xalan/java/trunk/src/org/apache/xalan/templates/ElemNumber.java

Author: mrglavas
Date: Thu Dec 29 05:36:43 2011
New Revision: 1225442

URL: http://svn.apache.org/viewvc?rev=1225442&view=rev
Log:
Eliminate creation of temporary strings in loops.

Modified:
    xalan/java/trunk/src/org/apache/xalan/templates/ElemNumber.java

Modified: xalan/java/trunk/src/org/apache/xalan/templates/ElemNumber.java
URL: http://svn.apache.org/viewvc/xalan/java/trunk/src/org/apache/xalan/templates/ElemNumber.java?rev=1225442&r1=1225441&r2=1225442&view=diff
==============================================================================
--- xalan/java/trunk/src/org/apache/xalan/templates/ElemNumber.java (original)
+++ xalan/java/trunk/src/org/apache/xalan/templates/ElemNumber.java Thu Dec 29 05:36:43 2011
@@ -1219,7 +1219,7 @@ public class ElemNumber extends ElemTemp
           // then append the formatToken.
           else if (formatTokenizer.isLetterOrDigitAhead())
           {
-            formatTokenString = formatToken;
+            final StringBuffer formatTokenStringBuffer = new StringBuffer(formatToken);
 
             // Append the formatToken string...
             // For instance [2][1][5] with a format value of "1--1. "
@@ -1227,8 +1227,9 @@ public class ElemNumber extends ElemTemp
             while (formatTokenizer.nextIsSep())
             {
               formatToken = formatTokenizer.nextToken();
-              formatTokenString += formatToken;
+              formatTokenStringBuffer.append(formatToken);
             }
+            formatTokenString = formatTokenStringBuffer.toString();
 
             // Record this separator, so it can be used as the 
             // next separator, if the next is the last.
@@ -1943,16 +1944,17 @@ public class ElemNumber extends ElemTemp
       return getZeroString();
     }
 
-    String roman = "";
+    final String roman;
     int place = 0;
 
     if (val <= 3999L)
     {
+      StringBuffer romanBuffer = new StringBuffer();
       do
       {
         while (val >= m_romanConvertTable[place].m_postValue)
         {
-          roman += m_romanConvertTable[place].m_postLetter;
+          romanBuffer.append(m_romanConvertTable[place].m_postLetter);
           val -= m_romanConvertTable[place].m_postValue;
         }
 
@@ -1960,7 +1962,7 @@ public class ElemNumber extends ElemTemp
         {
           if (val >= m_romanConvertTable[place].m_preValue)
           {
-            roman += m_romanConvertTable[place].m_preLetter;
+            romanBuffer.append(m_romanConvertTable[place].m_preLetter);
             val -= m_romanConvertTable[place].m_preValue;
           }
         }
@@ -1968,6 +1970,7 @@ public class ElemNumber extends ElemTemp
         place++;
       }
       while (val > 0);
+      roman = romanBuffer.toString();
     }
     else
     {