sc/source/core/data/column3.cxx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
New commits: commit 13be1527c0e5a5c13d73e73af8c1bc7df0a2f603 Author: Luboš Luňák <l.lu...@collabora.com> AuthorDate: Tue Feb 23 13:54:10 2021 +0100 Commit: Luboš Luňák <l.lu...@collabora.com> CommitDate: Tue Feb 23 17:47:53 2021 +0100 delete from mdds in reverse order (tdf#139820) This is a more elegant solution from Noel, compared to my previous commit. Deleting in reverse order means the underlying std::vector used by mdds does not have possible to-be-deleted items to move around that forward order would have. Change-Id: I31f0774e302308bcb70595cb899c977ee5966c7e Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111395 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lu...@collabora.com> diff --git a/sc/source/core/data/column3.cxx b/sc/source/core/data/column3.cxx index 14d1ebdc09c4..edf99f02bef5 100644 --- a/sc/source/core/data/column3.cxx +++ b/sc/source/core/data/column3.cxx @@ -978,7 +978,8 @@ void ScColumn::DeleteCells( aFunc.getSpans().getSpans(aSpans); // Delete the cells for real. - std::for_each(aSpans.begin(), aSpans.end(), EmptyCells(rBlockPos, *this)); + // tdf#139820: Deleting in reverse order is more efficient. + std::for_each(aSpans.rbegin(), aSpans.rend(), EmptyCells(rBlockPos, *this)); CellStorageModified(); aFunc.getSpans().swap(rDeleted); _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits