ditch SYNCHRONIZED_START/END

Change-Id: I1c0c015f371346efc5712ead7abe9e7f2ec0f9d0

framework/inc/general.h

@@ -48,25 +48,6 @@
  */
 namespace css = ::com::sun::star;
 
-namespace framework {
-
-//-----------------------------------------------------------------------------
-/** Those macro is used to make it more clear where a synchronized block will start.
-    Because normal documentation code wont be recognized by some developers to be real
-    I need something where they are thinking about.
-
-    At least this macro will do nothing ... it's empty.
-    But it should make the code more clear .-))
-*/
-#define SYNCHRONIZED_START
-
-//-----------------------------------------------------------------------------
-/** Same then SYNCHRONIZED_START ... but instead it mark the end of such code block.
-*/
-#define SYNCHRONIZED_END
-
-} // namespace framework
-
 #endif // #ifndef __FRAMEWORK_GENERAL_H_
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */

framework/source/dispatch/dispatchprovider.cxx

@@ -230,8 +230,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt
     // I.II) "_default"
     //  This is a combination of search an empty task for recycling - or create a new one.
     //-----------------------------------------------------------------------------------------------------
-    else
-    if (sTargetFrameName==SPECIALTARGET_DEFAULT)
+    else if (sTargetFrameName==SPECIALTARGET_DEFAULT)
     {
         if (implts_isLoadableContent(aURL))
             xDispatcher = implts_getOrCreateDispatchHelper( E_DEFAULTDISPATCHER, xDesktop );

framework/source/services/desktop.cxx

@@ -271,8 +271,7 @@ sal_Bool SAL_CALL Desktop::terminate()
 {
     TransactionGuard aTransaction( m_aTransactionManager, E_HARDEXCEPTIONS );
 
-    SYNCHRONIZED_START
-        ReadGuard aReadLock( m_aLock );
+    ReadGuard aReadLock( m_aLock ); // start synchronize
 
     css::uno::Reference< css::frame::XTerminateListener > xPipeTerminator    = m_xPipeTerminator;
     css::uno::Reference< css::frame::XTerminateListener > xQuickLauncher     = m_xQuickLauncher;
@@ -282,8 +281,7 @@ sal_Bool SAL_CALL Desktop::terminate()
     css::lang::EventObject                                aEvent             ( static_cast< ::cppu::OWeakObject* >(this) );
     ::sal_Bool                                            bAskQuickStart     = !m_bSuspendQuickstartVeto                  ;
 
-        aReadLock.unlock();
-    SYNCHRONIZED_END
+    aReadLock.unlock(); // end synchronize
 
     //-------------------------------------------------------------------------------------------------------------
     // Ask normal terminate listener. They could stop terminate without closing any open document.
@@ -1179,8 +1177,8 @@ void SAL_CALL Desktop::dispose()
         fprintf( stderr, "This used to be an assertion failure: Desktop disposed before terminating it,\n"
                  "but nothing bad seems to happen anyway?\n" );
 #endif
-    SYNCHRONIZED_START
-        WriteGuard aWriteLock( m_aLock );
+
+    WriteGuard aWriteLock( m_aLock ); // start synchronize
 
     // Look for multiple calls of this method!
     // If somewhere call dispose() twice - he will be stopped here realy!!!
@@ -1201,8 +1199,7 @@ void SAL_CALL Desktop::dispose()
     // and reject all new incoming requests!
     m_aTransactionManager.setWorkingMode( E_BEFORECLOSE );
 
-        aWriteLock.unlock();
-    SYNCHRONIZED_END
+    aWriteLock.unlock(); // end synchronize
 
     // Following lines of code can be called outside a synchronized block ...
     // Because our transaction manager will block all new requests to this object.
@@ -1898,11 +1895,9 @@ void Desktop::impl_sendNotifyTerminationEvent()
 //=============================================================================
 ::sal_Bool Desktop::impl_closeFrames(::sal_Bool bAllowUI)
 {
-    SYNCHRONIZED_START
-        ReadGuard aReadLock( m_aLock );
+    ReadGuard aReadLock( m_aLock ); // start synchronize
     css::uno::Sequence< css::uno::Reference< css::frame::XFrame > > lFrames = m_aChildTaskContainer.getAllElements();
-        aReadLock.unlock();
-    SYNCHRONIZED_END
+    aReadLock.unlock(); // end synchronize
 
     ::sal_Int32 c                = lFrames.getLength();
     ::sal_Int32 i                = 0;