framework/source/fwe/xml/statusbardocumenthandler.cxx |   12 ------------
 1 file changed, 12 deletions(-)

New commits:
commit a67b622ce51b4a8d0d36a62e4749f4ed0f17cfd2
Author:     Noel Grandin <[email protected]>
AuthorDate: Tue Jan 11 10:14:04 2022 +0200
Commit:     Noel Grandin <[email protected]>
CommitDate: Tue Jan 11 13:21:01 2022 +0100

    OReadStatusBarDocumentHandler doesnt need SolarMutex
    
    it is only ever used from a single thread. This was there
    since the beginning.
    
    Change-Id: Ib001db94c7a5b2520d9a6435e21c27dca60d1dd0
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128271
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <[email protected]>

diff --git a/framework/source/fwe/xml/statusbardocumenthandler.cxx 
b/framework/source/fwe/xml/statusbardocumenthandler.cxx
index e04299d764d1..eb1542287d2b 100644
--- a/framework/source/fwe/xml/statusbardocumenthandler.cxx
+++ b/framework/source/fwe/xml/statusbardocumenthandler.cxx
@@ -186,8 +186,6 @@ void SAL_CALL OReadStatusBarDocumentHandler::startDocument()
 
 void SAL_CALL OReadStatusBarDocumentHandler::endDocument()
 {
-    SolarMutexGuard g;
-
     if ( m_bStatusBarStartFound )
     {
         OUString aErrorMessage = getErrorLineString() + "No matching start or 
end element 'statusbar' found!";
@@ -198,8 +196,6 @@ void SAL_CALL OReadStatusBarDocumentHandler::endDocument()
 void SAL_CALL OReadStatusBarDocumentHandler::startElement(
     const OUString& aName, const Reference< XAttributeList > &xAttribs )
 {
-    SolarMutexGuard g;
-
     StatusBarHashMap::const_iterator pStatusBarEntry = m_aStatusBarMap.find( 
aName );
     if ( pStatusBarEntry == m_aStatusBarMap.end() )
         return;
@@ -396,8 +392,6 @@ void SAL_CALL OReadStatusBarDocumentHandler::startElement(
 
 void SAL_CALL OReadStatusBarDocumentHandler::endElement(const OUString& aName)
 {
-    SolarMutexGuard g;
-
     StatusBarHashMap::const_iterator pStatusBarEntry = m_aStatusBarMap.find( 
aName );
     if ( pStatusBarEntry == m_aStatusBarMap.end() )
         return;
@@ -448,15 +442,11 @@ void SAL_CALL 
OReadStatusBarDocumentHandler::processingInstruction(
 void SAL_CALL OReadStatusBarDocumentHandler::setDocumentLocator(
     const Reference< XLocator > &xLocator)
 {
-    SolarMutexGuard g;
-
     m_xLocator = xLocator;
 }
 
 OUString OReadStatusBarDocumentHandler::getErrorLineString()
 {
-    SolarMutexGuard g;
-
     if ( m_xLocator.is() )
         return "Line: " + OUString::number( m_xLocator->getLineNumber() ) + " 
- ";
     else
@@ -483,8 +473,6 @@ 
OWriteStatusBarDocumentHandler::~OWriteStatusBarDocumentHandler()
 
 void OWriteStatusBarDocumentHandler::WriteStatusBarDocument()
 {
-    SolarMutexGuard g;
-
     m_xWriteDocumentHandler->startDocument();
 
     // write DOCTYPE line!

Reply via email to