diff options
author | Caolán McNamara <caolanm@redhat.com> | 2015-12-30 14:03:56 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2015-12-30 14:05:23 +0000 |
commit | 18b9a6d3e1401ee7660af7c7cea2b847360cb07c (patch) | |
tree | c07dfbb99030a0dab049f0fb2f85486e7282e535 /qadevOOo | |
parent | 1d3abcb04397b521a91446ecfd3e46c9c49e0e39 (diff) |
coverity#1326889 SBSC: String concatenation in loop using + operator
Change-Id: Ie572d3f8aab49de491b54c206de8dda01fc1fcaa
Diffstat (limited to 'qadevOOo')
-rw-r--r-- | qadevOOo/tests/java/ifc/xml/sax/_XDocumentHandler.java | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/qadevOOo/tests/java/ifc/xml/sax/_XDocumentHandler.java b/qadevOOo/tests/java/ifc/xml/sax/_XDocumentHandler.java index 4e8b9216618e..d6e5a51d8ca6 100644 --- a/qadevOOo/tests/java/ifc/xml/sax/_XDocumentHandler.java +++ b/qadevOOo/tests/java/ifc/xml/sax/_XDocumentHandler.java @@ -241,20 +241,20 @@ public class _XDocumentHandler extends MultiMethodTest { log.println("StartElement Processing XML data ...") ; for(int i = 0; i < xmlData.length; i++) { String[] elem = xmlData[i] ; - String xmlTag = "" ; if ("start".equals(elem[0])) { - xmlTag += "<" ; + StringBuilder xmlTag = new StringBuilder(); + xmlTag.append("<"); String tagName = elem[1] ; - xmlTag += tagName ; + xmlTag.append(tagName); XMLTools.AttributeList attr = new XMLTools.AttributeList() ; for (int j = 2; j < elem.length; j+=3) { attr.add(elem[j], elem[j+1], elem[j+2]); - xmlTag += " " + elem[j] + "(" + elem[j+1] + - ")=\"" + elem[j+2] + "\"" ; + xmlTag.append(" ").append(elem[j]).append("(").append(elem[j+1]).append( + ")=\"").append(elem[j+2]).append("\""); } - xmlTag += ">" ; + xmlTag.append(">"); - log.println(xmlTag) ; + log.println(xmlTag.toString()) ; oObj.startElement(tagName, attr) ; } else if ("end".equals(elem[0])) { |