diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-26 17:16:16 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-27 11:35:24 +0200 |
| commit | b83cc2b0108a4ec2087221a6150a66578788e5b4 (patch) | |
| tree | b3684057b66602ddf01546dcb8306fddbad69a38 /basic/source/runtime/methods1.cxx | |
| parent | 86be39afd5b142f7cbdbe0107b394c5924c414cc (diff) | |
loplugin:stringloop in basic, framework, sax, svtools
Change-Id: I2bad74a8f103e9dc68c8e0d0e6315697068d2f6d
Reviewed-on: https://gerrit.libreoffice.org/58135
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'basic/source/runtime/methods1.cxx')
| -rw-r--r-- | basic/source/runtime/methods1.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/basic/source/runtime/methods1.cxx b/basic/source/runtime/methods1.cxx index 8c8c3dcbcd7d..4b59ea081c28 100644 --- a/basic/source/runtime/methods1.cxx +++ b/basic/source/runtime/methods1.cxx @@ -1550,20 +1550,20 @@ void SbRtl_Join(StarBASIC *, SbxArray & rPar, bool) { aDelim = " "; } - OUString aRetStr; + OUStringBuffer aRetStr; short nLower, nUpper; pArr->GetDim( 1, nLower, nUpper ); short aIdx[1]; for (aIdx[0] = nLower; aIdx[0] <= nUpper; ++aIdx[0]) { OUString aStr = pArr->Get(aIdx)->GetOUString(); - aRetStr += aStr; + aRetStr.append(aStr); if (aIdx[0] != nUpper) { - aRetStr += aDelim; + aRetStr.append(aDelim); } } - rPar.Get(0)->PutString( aRetStr ); + rPar.Get(0)->PutString( aRetStr.makeStringAndClear() ); } else { |