summaryrefslogtreecommitdiff
path: root/chart2/qa
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2018-08-28 15:09:33 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2018-08-29 09:05:18 +0200
commit4c91b89d8ce9c34179f31854dc88bd0a9fa84cba (patch)
tree1fe9cc9db455779d33c24320fedc1e25888b3e5c /chart2/qa
parenta1f31211920bfae1a21ea375fa5280c9c6595e15 (diff)
new loplugin:oustringbuffer
look for places where we are appending the temporary result of adding strings together, to an OUStringBuffer, where we could rather call append repeatedly and avoid the temporary creation Change-Id: I481435124291ac7fb54b91a78344a9fe5b379a82 Reviewed-on: https://gerrit.libreoffice.org/59708 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'chart2/qa')
-rw-r--r--chart2/qa/extras/chart2dump/chart2dump.cxx4
1 files changed, 2 insertions, 2 deletions
diff --git a/chart2/qa/extras/chart2dump/chart2dump.cxx b/chart2/qa/extras/chart2dump/chart2dump.cxx
index ceb818c70e0f..e06ce2f76415 100644
--- a/chart2/qa/extras/chart2dump/chart2dump.cxx
+++ b/chart2/qa/extras/chart2dump/chart2dump.cxx
@@ -230,7 +230,7 @@ protected:
OUStringBuffer aBufer;
for (const OUString& seqItem : rSeq)
{
- aBufer.append(seqItem + ";");
+ aBufer.append(seqItem).append(";");
}
return aBufer.makeStringAndClear();
}
@@ -240,7 +240,7 @@ protected:
OUStringBuffer aBufer;
for (const double& vectorItem : rVector)
{
- aBufer.append(OUString::number(vectorItem) + ";");
+ aBufer.append(OUString::number(vectorItem)).append(";");
}
return aBufer.makeStringAndClear();
}