commit b0556da3a0966c9c68c6a57909524058d0bb5e07
author Luboš Luňák <l.lunak@collabora.com> Thu May 28 16:18:58 2020 +0200
committer Noel Grandin <noel.grandin@collabora.co.uk> Wed Jun 03 11:37:43 2020 +0200
tree df65661c4e8fee6fdff1632b95dd033fb17d6f2f
parent 35ec66f8766bd3d8f17c9e0ea68d1e260a9fbc32

purge shared string pool if ScDocument is closed (tdf#125428)

Especially with the testcases given in the bugreport the undo history
would keep many unused shared strings.

Change-Id: I57fcad73532fdcbad0a1298fbd06cb35c058bf96
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95045
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 861fbd998f2b526c2aea073c9471613bf728fa75)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95323
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

sc/source/core/data/documen2.cxx

diff --git a/sc/source/core/data/documen2.cxx b/sc/source/core/data/documen2.cxx
index cc45c83..3cbe637 100644
--- a/sc/source/core/data/documen2.cxx
+++ b/sc/source/core/data/documen2.cxx
@@ -390,6 +390,9 @@
    SAL_WARN_IF( pAutoNameCache, "sc.core", "AutoNameCache still set in dtor" );

    mpFormulaGroupCxt.reset();
    // Purge unused items if the string pool will be still used (e.g. by undo history).
    if(mpCellStringPool.use_count() > 1)
        mpCellStringPool->purge();
    mpCellStringPool.reset();
}