From ec4babad021218b75dfe8534985d7db525edde69 Mon Sep 17 00:00:00 2001
From: Noel Grandin <noelgrandin@gmail.com>
Date: Sun, 29 Jan 2023 19:09:31 +0200
Subject: no need to lock SolarMutex over the whole method here

Change-Id: Ifcac67c0f4e149fe7e1d923d7efede9552b034a8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146308
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
---
 basic/source/classes/sbxmod.cxx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'basic')

diff --git a/basic/source/classes/sbxmod.cxx b/basic/source/classes/sbxmod.cxx
index 00bfac58dd71..f38e729185db 100644
--- a/basic/source/classes/sbxmod.cxx
+++ b/basic/source/classes/sbxmod.cxx
@@ -2375,9 +2375,9 @@ public:
     virtual void SAL_CALL documentEventOccured( const document::DocumentEvent& rEvent ) override
     {
         // early disposing on document event "OnUnload", to be sure Basic still exists when calling VBA "UserForm_Terminate"
-        SolarMutexGuard g;
         if( rEvent.EventName == GlobalEventConfig::GetEventName( GlobalEventId::CLOSEDOC ) )
         {
+            SolarMutexGuard g;
             removeListener();
             mbDisposed = true;
             if ( mpUserForm )
-- 
cgit