coverity#1327008 SBSC: String concatenation in loop using + operator

Change-Id: I2e205bd87760740e1b57647a9a3bdd928be09dfd

1 files changed, 8 insertions, 5 deletions

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java
index 8875c21c902f..29e7740c9bba 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java
@@ -416,14 +416,17 @@ public class TextStyle extends Style implements Cloneable {
      * Dump this {@code Style} as a Comma Separated Value (CSV) line.
      */
     public void dumpCSV() {
-        String attributes = "";
+        StringBuilder attributes = new StringBuilder();
         for (int bitVal = 0x01; bitVal <= 0x20; bitVal = bitVal << 1) {
             if ((bitVal & mask) != 0) {
-                attributes += toCSV(((bitVal & values) != 0) ? "yes" : "no");
-            } else attributes += toCSV(null);  // unspecified
+                attributes.append(toCSV(((bitVal & values) != 0) ? "yes" : "no"));
+            }
+            else {
+                attributes.append(toCSV(null));  // unspecified
+            }
         }
         System.out.println(toCSV(name) + toCSV(family) + toCSV(parent)
-        + toCSV(fontName) + toCSV("" + sizeInPoints) + attributes + toLastCSV(null));
+        + toCSV(fontName) + toCSV("" + sizeInPoints) + attributes.toString() + toLastCSV(null));
     }
 
     /**
@@ -569,4 +572,4 @@ public class TextStyle extends Style implements Cloneable {
         }
         return false;
     }
-}
\ No newline at end of file
+}