basic/source/basmgr/basicmanagerrepository.cxx |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit c319ccfc37ee6d05e08227799045ff26ea985432
Author: Stephan Bergmann <[email protected]>
Date:   Fri Aug 22 09:10:40 2014 +0200

    Revert "Simplify ImplRepository::impl_getLocationForModel return type"
    
    This reverts commit 72305ed49d4c9ba968d7bf6d8d5ae564bb92147d, which was 
bogus,
    cf. the comment in ImplRepository::getDocumentBasicManager about recursive
    calls.

diff --git a/basic/source/basmgr/basicmanagerrepository.cxx 
b/basic/source/basmgr/basicmanagerrepository.cxx
index a292ccc..bcf0855 100644
--- a/basic/source/basmgr/basicmanagerrepository.cxx
+++ b/basic/source/basmgr/basicmanagerrepository.cxx
@@ -107,7 +107,7 @@ namespace basic
             @precond
                 our mutex is locked
         */
-        BasicManager*
+        BasicManager*&
                 impl_getLocationForModel( const Reference< XModel >& 
_rxDocumentModel );
 
         /** creates a new BasicManager instance for the given model
@@ -232,7 +232,7 @@ namespace basic
             thus a recursive call of this function will find and return it
             without creating another instance.
          */
-        BasicManager* pBasicManager = impl_getLocationForModel( 
_rxDocumentModel );
+        BasicManager*& pBasicManager = impl_getLocationForModel( 
_rxDocumentModel );
         if ( pBasicManager == NULL )
             impl_createManagerForModel( pBasicManager, _rxDocumentModel );
 
@@ -363,12 +363,12 @@ namespace basic
     }
 
 
-    BasicManager* ImplRepository::impl_getLocationForModel( const Reference< 
XModel >& _rxDocumentModel )
+    BasicManager*& ImplRepository::impl_getLocationForModel( const Reference< 
XModel >& _rxDocumentModel )
     {
         Reference< XInterface > xNormalized( _rxDocumentModel, UNO_QUERY );
         DBG_ASSERT( _rxDocumentModel.is(), 
"ImplRepository::impl_getLocationForModel: invalid model!" );
 
-        BasicManager* location = m_aStore[ xNormalized ];
+        BasicManager*& location = m_aStore[ xNormalized ];
         return location;
     }
 
_______________________________________________
Libreoffice-commits mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to