[Libreoffice-commits] core.git: vcl/headless

[Jan-Marek Glogowski (via logerrit)]

 vcl/headless/svpinst.cxx |    1 -
 1 file changed, 1 deletion(-)

New commits:
commit 4abd2598f7816891af3bd462ffaee57f3423b9f9
Author:     Jan-Marek Glogowski <glogow at fbihome.de>
AuthorDate: Thu Jun 24 18:09:15 2021 +0200
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Thu Jun 24 23:19:45 2021 +0200

    Revert: svp: always release SolarMutex on yield
    
    Keeps the SolarMutexReleaser replacement. Seems a real fix for
    the original busy-lock problem needs further investigation.
    
    This reverts commit b9ea3467383ccdfe4be842e61267256f281487ee.
    
    Change-Id: I4bf6157e7236c30ecbf0b710d8bf9dc39d9ae5dd
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117795
    Tested-by: Eike Rathke <erack at redhat.com>
    Tested-by: Xisco Fauli <xiscofauli at libreoffice.org>
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofauli at libreoffice.org>

diff --git a/vcl/headless/svpinst.cxx b/vcl/headless/svpinst.cxx
index 36859e9b3fb2..638052611360 100644
--- a/vcl/headless/svpinst.cxx
+++ b/vcl/headless/svpinst.cxx
@@ -528,7 +528,6 @@ bool SvpSalInstance::DoYield(bool bWait, bool bHandleAllCurrentEvents)
         {
             if (bSkipPoll)
                 pMutex->m_NonMainWaitingYieldCond.set(); // wake up other threads
-            SolarMutexReleaser aReleaser;
         }
     }
     else // !IsMainThread()