ICU-8268 concatenation of strings using + in a loop is inefficient. Use StringBuilder.

X-SVN-Rev: 30720

diff --git a/icu4j/main/classes/core/src/com/ibm/icu/text/DateTimePatternGenerator.java b/icu4j/main/classes/core/src/com/ibm/icu/text/DateTimePatternGenerator.java
index 3cf27bade0b9d3f45020b46841b255493c6b20e7..11719a92cfed76eec1f9fbb2b82488c0102d833c 100644 (file)
--- a/icu4j/main/classes/core/src/com/ibm/icu/text/DateTimePatternGenerator.java
+++ b/icu4j/main/classes/core/src/com/ibm/icu/text/DateTimePatternGenerator.java
@@ -1756,13 +1756,16 @@ public class DateTimePatternGenerator implements Freezable<DateTimePatternGenera
     }
 
     private static String showMask(int mask) {
-        String result = "";
+        StringBuilder result = new StringBuilder();
         for (int i = 0; i < TYPE_LIMIT; ++i) {
-            if ((mask & (1<<i)) == 0) continue;
-            if (result.length() != 0) result += " | ";
-            result += FIELD_NAME[i] + " ";
+            if ((mask & (1<<i)) == 0)
+                continue;
+            if (result.length() != 0)
+                result.append(" | ");
+            result.append(FIELD_NAME[i]);
+            result.append(" ");
         }
-        return result;
+        return result.toString();
     }
 
     static private String[] CLDR_FIELD_APPEND = {

diff --git a/icu4j/main/classes/core/src/com/ibm/icu/text/RBBIDataWrapper.java b/icu4j/main/classes/core/src/com/ibm/icu/text/RBBIDataWrapper.java
index 357fb74236e42cba42e148cf8cbd6ef5deee72ea..046061aee573ca46efea353bfc9186b6c5344d8e 100644 (file)
--- a/icu4j/main/classes/core/src/com/ibm/icu/text/RBBIDataWrapper.java
+++ b/icu4j/main/classes/core/src/com/ibm/icu/text/RBBIDataWrapper.java
@@ -384,11 +384,11 @@ final class RBBIDataWrapper {
         } else {
             int n;
             int state;
-            String header = " Row  Acc Look  Tag";
+            StringBuilder header = new StringBuilder(" Row  Acc Look  Tag");
             for (n=0; n<fHeader.fCatCount; n++) {
-                header += intToString(n, 5);     
+                header.append(intToString(n, 5));
             }
-            System.out.println(header);
+            System.out.println(header.toString());
             for (n=0; n<header.length(); n++) {
                 System.out.print("-");
             }