compilerplugins/clang/badstatics.cxx | 1 svx/source/inc/StylesPreviewWindow.hxx | 2 - svx/source/tbxctrls/StylesPreviewWindow.cxx | 46 ++++++++++++++++++++++++---- 3 files changed, 42 insertions(+), 7 deletions(-)
New commits: commit c20bb38fea30127835c62c9546ca02edf2182737 Author: Szymon Kłos <[email protected]> AuthorDate: Fri Jun 2 18:31:55 2023 +0200 Commit: Szymon Kłos <[email protected]> CommitDate: Sat Jun 3 11:17:20 2023 +0200 Styles preview cache cleanup on exit Change-Id: I28929d4137008ebcca1733837d0b2112b6859a89 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152563 Reviewed-by: Caolán McNamara <[email protected]> Tested-by: Jenkins CollaboraOffice <[email protected]> Reviewed-by: Szymon Kłos <[email protected]> diff --git a/compilerplugins/clang/badstatics.cxx b/compilerplugins/clang/badstatics.cxx index 7c604bc5ee66..98375f212dd2 100644 --- a/compilerplugins/clang/badstatics.cxx +++ b/compilerplugins/clang/badstatics.cxx @@ -202,6 +202,7 @@ public: || (loplugin::DeclCheck(pVarDecl).Var("aPreviewCache") .Class("StylesPreviewWindow_Base").GlobalNamespace()) // TODO: temp disable //svtools/source/control/ctrlbox.cxx, empty at exit + || name == "gStylePreviewCache" // svx/source/tbxctrls/StylesPreviewWindow.cxx || name == "aLogger" // FormulaLogger& FormulaLogger::get() in sc/source/core/tool/formulalogger.cxx || name == "s_aUncommittedRegistrations" // sw/source/uibase/dbui/dbmgr.cxx || (loplugin::DeclCheck(pVarDecl).Var("aAllListeners") diff --git a/svx/source/inc/StylesPreviewWindow.hxx b/svx/source/inc/StylesPreviewWindow.hxx index 33aaf513f28d..78ebcf94b13a 100644 --- a/svx/source/inc/StylesPreviewWindow.hxx +++ b/svx/source/inc/StylesPreviewWindow.hxx @@ -127,8 +127,6 @@ private: void UpdateStylesList(); void UpdateSelection(); bool Command(const CommandEvent& rEvent); - - static std::map<OUString, VclPtr<VirtualDevice>> aPreviewCache; }; class StylesPreviewWindow_Impl final : public InterimItemWindow, public StylesPreviewWindow_Base diff --git a/svx/source/tbxctrls/StylesPreviewWindow.cxx b/svx/source/tbxctrls/StylesPreviewWindow.cxx index d5e6791cb412..cd65ff839b1d 100644 --- a/svx/source/tbxctrls/StylesPreviewWindow.cxx +++ b/svx/source/tbxctrls/StylesPreviewWindow.cxx @@ -60,7 +60,36 @@ #include <vcl/commandevent.hxx> -std::map<OUString, VclPtr<VirtualDevice>> StylesPreviewWindow_Base::aPreviewCache; +namespace +{ +class StylePreviewCache +{ + static std::map<OUString, VclPtr<VirtualDevice>> gStylePreviewCache; + static int gStylePreviewCacheClients; + +public: + static std::map<OUString, VclPtr<VirtualDevice>>& Get() { return gStylePreviewCache; } + + static void ClearCache() + { + for (auto& aPreview : gStylePreviewCache) + aPreview.second.disposeAndClear(); + + gStylePreviewCache.clear(); + } + + static void RegisterClient() { gStylePreviewCacheClients++; } + static void UnregisterClient() + { + gStylePreviewCacheClients--; + if (!gStylePreviewCacheClients) + ClearCache(); + } +}; + +std::map<OUString, VclPtr<VirtualDevice>> StylePreviewCache::gStylePreviewCache; +int StylePreviewCache::gStylePreviewCacheClients; +} StyleStatusListener::StyleStatusListener( StylesPreviewWindow_Base* pPreviewControl, @@ -103,8 +132,10 @@ StylePoolChangeListener::~StylePoolChangeListener() EndListening(*m_pStyleSheetPool); } -void StylePoolChangeListener::Notify(SfxBroadcaster& /*rBC*/, const SfxHint& /*rHint*/) +void StylePoolChangeListener::Notify(SfxBroadcaster& /*rBC*/, const SfxHint& rHint) { + if (rHint.GetId() == SfxHintId::StyleSheetModified) + StylePreviewCache::ClearCache(); m_pPreviewControl->RequestStylesListUpdate(); } @@ -378,6 +409,8 @@ StylesPreviewWindow_Base::StylesPreviewWindow_Base( , m_aUpdateTask(*this) , m_aDefaultStyles(std::move(aDefaultStyles)) { + StylePreviewCache::RegisterClient(); + m_xStylesView->connect_selection_changed(LINK(this, StylesPreviewWindow_Base, Selected)); m_xStylesView->connect_item_activated(LINK(this, StylesPreviewWindow_Base, DoubleClick)); m_xStylesView->connect_command(LINK(this, StylesPreviewWindow_Base, DoCommand)); @@ -424,6 +457,8 @@ StylesPreviewWindow_Base::~StylesPreviewWindow_Base() m_aUpdateTask.Stop(); + StylePreviewCache::UnregisterClient(); + try { m_xStatusListener->dispose(); @@ -465,8 +500,9 @@ void StylesListUpdateTask::Invoke() VclPtr<VirtualDevice> StylesPreviewWindow_Base::GetCachedPreview(const std::pair<OUString, OUString>& rStyle) { - if (aPreviewCache.find(rStyle.second) != aPreviewCache.end()) - return aPreviewCache[rStyle.second]; + auto aFound = StylePreviewCache::Get().find(rStyle.second); + if (aFound != StylePreviewCache::Get().end()) + return StylePreviewCache::Get()[rStyle.second]; else { VclPtr<VirtualDevice> pImg = VclPtr<VirtualDevice>::Create(); @@ -475,7 +511,7 @@ StylesPreviewWindow_Base::GetCachedPreview(const std::pair<OUString, OUString>& StyleItemController aStyleController(rStyle); aStyleController.Paint(*pImg); - aPreviewCache[rStyle.second] = pImg; + StylePreviewCache::Get()[rStyle.second] = pImg; return pImg; }
