sc/source/core/data/table7.cxx | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
New commits:
commit 4726058aa522bf72df4938bf76075ff68f8bacc3
Author: Noel Grandin <noelgran...@gmail.com>
AuthorDate: Thu Aug 1 15:24:52 2024 +0200
Commit: Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Thu Aug 1 16:45:43 2024 +0200
optimise ScTable::dumpColumnRowSizes a little
use an OStringBuffer to accumulate the result
Change-Id: I7d585b48183208200c62fe4f1d9e639cc3b1d466
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171376
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
diff --git a/sc/source/core/data/table7.cxx b/sc/source/core/data/table7.cxx
index f9e7a878c8e5..451c1895c194 100644
--- a/sc/source/core/data/table7.cxx
+++ b/sc/source/core/data/table7.cxx
@@ -527,8 +527,7 @@ OString ScTable::dumpColumnRowSizes(bool bColumns)
// instead just operate on the specialized object.
typedef ScCompressedArray<SCCOL, sal_uInt16> ColWidthsType;
auto dumpColWidths = [this](const ColWidthsType& rWidths) -> OString {
- OString aOutput;
- OString aSegment;
+ OStringBuffer aOutput;
SCCOL nStartCol = 0;
const SCCOL nMaxCol = std::min(rWidths.GetLastPos(),
GetDoc().MaxCol());
size_t nDummy = 0;
@@ -539,12 +538,11 @@ OString ScTable::dumpColumnRowSizes(bool bColumns)
// The last span nEndCol is always MAXCOL+1 for some reason, and
we don't want that.
if (nEndCol > nMaxCol)
nEndCol = nMaxCol;
- aSegment = OString::number(nWidth) + ":" +
OString::number(nEndCol) + " ";
- aOutput += aSegment;
+ aOutput.append(OString::number(nWidth) + ":" +
OString::number(nEndCol) + " ");
nStartCol = nEndCol + 1;
}
- return aOutput;
+ return aOutput.makeStringAndClear();
};
if (bColumns)
