Simplify the code a bit.

Change-Id: I57581fb0f3439ddc4a4848466f6925d962b04aea

framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx

@@ -1483,14 +1483,9 @@ Reference< ui::XAcceleratorConfiguration > SAL_CALL ModuleUIConfigurationManager
     if ( m_bDisposed )
         throw DisposedException();
 
-    Reference< XComponentContext > xContext   = m_xContext;
-    OUString                   aModule = m_aModuleIdentifier;
-
     if ( !m_xModuleAcceleratorManager.is() )
-    {
-        Reference< ui::XAcceleratorConfiguration >  xManager = ui::ModuleAcceleratorConfiguration::createWithModuleIdentifier(xContext, aModule);
-        m_xModuleAcceleratorManager = xManager;
-    }
+        m_xModuleAcceleratorManager = ui::ModuleAcceleratorConfiguration::
+            createWithModuleIdentifier(m_xContext, m_aModuleIdentifier);
 
     return m_xModuleAcceleratorManager;
 }

framework/source/uiconfiguration/uiconfigurationmanager.cxx

@@ -1178,24 +1178,11 @@ Reference< XAcceleratorConfiguration > SAL_CALL UIConfigurationManager::getShort
     // SAFE ->
     ResetableGuard aGuard( m_aLock );
 
-    if (m_xAccConfig.is())
-        return m_xAccConfig;
+    if (!m_xAccConfig.is())
+        m_xAccConfig = DocumentAcceleratorConfiguration::
+            createWithDocumentRoot(m_xContext, m_xDocConfigStorage);
 
-    Reference< XComponentContext >    xContext = m_xContext;
-    Reference< XStorage >             xDocumentRoot = m_xDocConfigStorage;
-
-    aGuard.unlock();
-    // <- SAFE
-
-    Reference< XAcceleratorConfiguration >      xAccConfig = DocumentAcceleratorConfiguration::createWithDocumentRoot(xContext, xDocumentRoot);
-
-    // SAFE ->
-    aGuard.lock();
-    m_xAccConfig = xAccConfig;
-    aGuard.unlock();
-    // <- SAFE
-
-    return xAccConfig;
+    return m_xAccConfig;
 }
 
 Reference< XInterface > SAL_CALL UIConfigurationManager::getEventsManager() throw (::com::sun::star::uno::RuntimeException)