diff options
Diffstat (limited to 'extensions')
-rw-r--r-- | extensions/source/ole/makefile.mk | 7 | ||||
-rw-r--r-- | extensions/source/ole/oleobjw.cxx | 21 | ||||
-rw-r--r-- | extensions/source/ole/unoconversionutilities.hxx | 332 | ||||
-rw-r--r-- | extensions/source/ole/unoobjw.cxx | 48 | ||||
-rw-r--r-- | extensions/source/ole/unoobjw.hxx | 10 |
5 files changed, 212 insertions, 206 deletions
diff --git a/extensions/source/ole/makefile.mk b/extensions/source/ole/makefile.mk index c83d92daf..73852fa76 100644 --- a/extensions/source/ole/makefile.mk +++ b/extensions/source/ole/makefile.mk @@ -2,9 +2,9 @@ # # $RCSfile: makefile.mk,v $ # -# $Revision: 1.6 $ +# $Revision: 1.7 $ # -# last change: $Author: jl $ $Date: 2002-06-05 13:21:38 $ +# last change: $Author: jl $ $Date: 2002-09-13 06:23:07 $ # # The Contents of this file are made available subject to the terms of # either of the following licenses @@ -152,7 +152,8 @@ SLOFILES= \ $(SLO)$/olethread.obj \ $(SLO)$/oledll.obj \ $(SLO)$/jscriptclasses.obj \ - $(SLO)$/ole2uno.obj + $(SLO)$/ole2uno.obj + diff --git a/extensions/source/ole/oleobjw.cxx b/extensions/source/ole/oleobjw.cxx index b62426f98..01e8dac97 100644 --- a/extensions/source/ole/oleobjw.cxx +++ b/extensions/source/ole/oleobjw.cxx @@ -2,9 +2,9 @@ * * $RCSfile: oleobjw.cxx,v $ * - * $Revision: 1.9 $ + * $Revision: 1.10 $ * - * last change: $Author: jl $ $Date: 2002-06-05 13:21:38 $ + * last change: $Author: jl $ $Date: 2002-09-13 06:23:08 $ * * The Contents of this file are made available subject to the terms of * either of the following licenses @@ -158,7 +158,7 @@ hash_map<sal_uInt32,sal_uInt32> AdapterToWrapperMap; // adapted interface which is then used to locate the entry in AdapterToWrapperMap. hash_map<sal_uInt32,sal_uInt32> WrapperToAdapterMap; -//hash_map<sal_uInt32, WeakReference<XInterface> > ComPtrToWrapperMap; +hash_map<sal_uInt32, WeakReference<XInterface> > ComPtrToWrapperMap; /***************************************************************************** class implementation IUnknownWrapper_Impl @@ -176,12 +176,15 @@ IUnknownWrapper_Impl::IUnknownWrapper_Impl( Reference<XMultiServiceFactory>& xFa IUnknownWrapper_Impl::~IUnknownWrapper_Impl() { MutexGuard guard(getBridgeMutex()); - acquire(); // make sure we don't delete us twice - Reference<XInterface> xInt( static_cast<XWeak*>(this), UNO_QUERY); + XInterface * xIntRoot = (OWeakObject *)this; +#ifdef _DEBUG + acquire(); // make sure we don't delete us twice because of Reference + OSL_ASSERT( Reference<XInterface>( static_cast<XWeak*>(this), UNO_QUERY).get() == xIntRoot ); +#endif // remove entries in global maps typedef hash_map<sal_uInt32, sal_uInt32>::iterator _IT; - _IT it= WrapperToAdapterMap.find( (sal_uInt32) xInt.get()); + _IT it= WrapperToAdapterMap.find( (sal_uInt32) xIntRoot); if( it != WrapperToAdapterMap.end()) { sal_uInt32 adapter= it->second; @@ -190,9 +193,9 @@ IUnknownWrapper_Impl::~IUnknownWrapper_Impl() WrapperToAdapterMap.erase( it); } -// IT_Com it_c= ComPtrToWrapperMap.find( (sal_uInt32) m_pUnknown); -// if(it_c != ComPtrToWrapperMap.end()) -// ComPtrToWrapperMap.erase(it_c); + IT_Com it_c= ComPtrToWrapperMap.find( (sal_uInt32) m_pUnknown); + if(it_c != ComPtrToWrapperMap.end()) + ComPtrToWrapperMap.erase(it_c); o2u_attachCurrentThread(); diff --git a/extensions/source/ole/unoconversionutilities.hxx b/extensions/source/ole/unoconversionutilities.hxx index 53701a550..3c742f7c2 100644 --- a/extensions/source/ole/unoconversionutilities.hxx +++ b/extensions/source/ole/unoconversionutilities.hxx @@ -2,9 +2,9 @@ * * $RCSfile: unoconversionutilities.hxx,v $ * - * $Revision: 1.11 $ + * $Revision: 1.12 $ * - * last change: $Author: jl $ $Date: 2002-06-05 13:21:38 $ + * last change: $Author: jl $ $Date: 2002-09-13 06:23:08 $ * * The Contents of this file are made available subject to the terms of * either of the following licenses @@ -100,9 +100,9 @@ typedef hash_map<sal_uInt32, sal_uInt32>::iterator CIT_Wrap; // IUnknownWrapperImpl. It is the responsibility of the wrapper to remove the entry when // it is being destroyed. // Used to ensure that an Automation object is always mapped to the same UNO objects. -// extern hash_map<sal_uInt32, WeakReference<XInterface> > ComPtrToWrapperMap; -// typedef hash_map<sal_uInt32, WeakReference<XInterface> >::iterator IT_Com; -// typedef hash_map<sal_uInt32, WeakReference<XInterface> >::const_iterator CIT_Com; +extern hash_map<sal_uInt32, WeakReference<XInterface> > ComPtrToWrapperMap; +typedef hash_map<sal_uInt32, WeakReference<XInterface> >::iterator IT_Com; +typedef hash_map<sal_uInt32, WeakReference<XInterface> >::const_iterator CIT_Com; // Maps XInterface pointers to a weak reference of its wrapper class (i.e. // InterfaceOleWrapper_Impl). It is the responsibility of the wrapper to remove the entry when @@ -112,7 +112,11 @@ typedef hash_map<sal_uInt32, sal_uInt32>::iterator CIT_Wrap; // must be returned. extern hash_map<sal_uInt32, WeakReference<XInterface> > UnoObjToWrapperMap; typedef hash_map<sal_uInt32, WeakReference<XInterface> >::iterator IT_Uno; -typedef hash_map<sal_uInt32, WeakReference<XInterface> >::const_iterator CIT_Uno; +typedef hash_map<sal_uInt32, WeakReference<XInterface> >::const_iterator CIT_Uno; + + + + // createUnoObjectWrapper gets a wrapper instance by calling createUnoWrapperInstance // and initializes it via XInitialization. The wrapper object is required to implement // XBridgeSupplier so that it can convert itself to IDispatch. @@ -161,7 +165,7 @@ public: static sal_Bool isJScriptArray(const VARIANT* pvar); - + Sequence<Type> getImplementedInterfaces(IUnknown* pUnk); protected: // helper function for Sequence conversion @@ -979,6 +983,12 @@ SAFEARRAY* UnoConversionUtilities<T>::createUnoSequenceWrapper(const Any& rSeq) return pArray; } +/* The argument rObj can contain +- UNO struct +- UNO interface +- UNO interface created by this bridge (adapter factory) +- UNO interface created by this bridge ( COM Wrapper) +*/ template<class T> IDispatch* UnoConversionUtilities<T>::createUnoObjectWrapper(const Any& rObj) { @@ -988,58 +998,63 @@ IDispatch* UnoConversionUtilities<T>::createUnoObjectWrapper(const Any& rObj) Reference<XInvocation> xInv; Reference<XInterface> xInt; rObj >>= xInt; + //make sure we have the main XInterface which is used with a map + xInt=Reference<XInterface>(xInt, UNO_QUERY); //If there is already a wrapper for the UNO object then use it + + Reference<XInterface> xIntWrapper; + // Does a UNO wrapper exist already ? if(xInt.is()) { IT_Uno it_uno= UnoObjToWrapperMap.find( (sal_uInt32) xInt.get()); if(it_uno != UnoObjToWrapperMap.end()) { - Reference<XInterface> wrapper( it_uno->second); + xIntWrapper= it_uno->second; + OSL_ENSURE( xIntWrapper.is(),"Automation bridge: Mapping failure"); } } - if(xInt.is()) + // Is the object a COM wrapper ( either XInvocation, or Adapter object) + // or does it suppy an IDispatch by its own ? + if(xInt.is() && ! xIntWrapper.is()) { - // can the object provide a Wrapper on its own - if( ! convertSelfToIDispatch( xInt, &pDispatch) ) - { - // figure out if the UNO object is a Wrapper through global maps - typedef hash_map<sal_uInt32,sal_uInt32>::iterator _IT; - _IT it= AdapterToWrapperMap.find( (sal_uInt32) xInt.get()); - if( it != AdapterToWrapperMap.end() ) - { - Reference<XInterface> xIntWrapper( (XInterface*)it->second); - convertSelfToIDispatch( xIntWrapper, &pDispatch); // should always work on the objects in the map - } - else - xInv= Reference<XInvocation>( xInt, UNO_QUERY); - } - } - - if (! pDispatch && !xInv.is()) + Reference<XInterface> xIntComWrapper= xInt; + typedef hash_map<sal_uInt32,sal_uInt32>::iterator _IT; + // Adapter? then get the COM wrapper to which the adapter delegates its calls + _IT it= AdapterToWrapperMap.find( (sal_uInt32) xInt.get()); + if( it != AdapterToWrapperMap.end() ) + xIntComWrapper= reinterpret_cast<XInterface*>(it->second); + + // the object can be a COM wrapper, or any other UNO object that supports + // a bridge (XBridgeSupplier) and provide an IDispatch on its own. + convertSelfToIDispatch(xIntComWrapper, &pDispatch); + } + // If we have no UNO wrapper nor the IDispatch yet then we have to create + // a wrapper. For that we need an XInvocation from the UNO object. + if( !xIntWrapper.is() && ! pDispatch) { - // create a XInvocation object for the uno interface or struct - Reference<XSingleServiceFactory> xInvFactory= getInvocationFactory(rObj); - if( xInvFactory.is()) + // get an XInvocation or create one using the invocation service + Reference<XInvocation> xInv(xInt, UNO_QUERY); + if( ! xInv.is()) { - Sequence<Any> params(1); - params.getArray()[0] = rObj; - Reference<XInterface> xInt = xInvFactory->createInstanceWithArguments(params); - xInv= Reference<XInvocation>(xInt, UNO_QUERY); + Reference<XSingleServiceFactory> xInvFactory= getInvocationFactory(rObj); + if( xInvFactory.is()) + { + Sequence<Any> params(1); + params.getArray()[0] = rObj; + Reference<XInterface> xInt = xInvFactory->createInstanceWithArguments(params); + xInv= Reference<XInvocation>(xInt, UNO_QUERY); + } } - } - OSL_ENSURE(xInv.is() || pDispatch, "no invocation interface"); - if( !pDispatch) - { - Reference< XInterface > xIntWrapper= createUnoWrapperInstance(); - if( xIntWrapper.is()) - { - Reference< XInitialization > xInitWrapper( xIntWrapper, UNO_QUERY); + if( xInv.is()) + { + Reference< XInterface > xNewWrapper= createUnoWrapperInstance(); + Reference< XInitialization > xInitWrapper( xNewWrapper, UNO_QUERY); if( xInitWrapper.is() ) { VARTYPE vartype= getVarType( rObj); - + if( xInt.is()) { Any params[3]; @@ -1056,23 +1071,21 @@ IDispatch* UnoConversionUtilities<T>::createUnoObjectWrapper(const Any& rObj) xInitWrapper->initialize( Sequence<Any>(params, 2)); } - // Get the IDispatch - // A wrapper is expected to implement XBridgeSupplier in order to - // convert itself to an IDispatch: - IDispatch* pDisp; - if( convertSelfToIDispatch( xIntWrapper, &pDisp)) - pDispatch= pDisp; - + xIntWrapper= xNewWrapper; // put the newly created object into a map. If the same object will // be mapped again and there is already a wrapper then the old wrapper // will be used. if(xInt.is()) // only interfaces - UnoObjToWrapperMap[(sal_uInt32) xInt.get()]= WeakReference<XInterface>(xIntWrapper); + UnoObjToWrapperMap[(sal_uInt32) xInt.get()]= xIntWrapper; } } } - - return pDispatch; + + // get IDispatch from the UNO wrapper + if( !pDispatch) + convertSelfToIDispatch(xIntWrapper, &pDispatch); + + return pDispatch; } template<class T> @@ -1375,10 +1388,10 @@ Any UnoConversionUtilities<T>::createOleObjectWrapper(IUnknown* pUnknown, const Any ret; Type desiredType= aType == VOID_TYPE ? getCppuType((Reference<XInvocation>*) 0) : aType; - Reference<XInterface> xInt; if (pUnknown == NULL) { + Reference<XInterface> xInt; if( aType.getTypeClass() == TypeClass_INTERFACE) ret.setValue( &xInt, aType); else if( aType.getTypeClass() == TypeClass_STRUCT) @@ -1390,7 +1403,7 @@ Any UnoConversionUtilities<T>::createOleObjectWrapper(IUnknown* pUnknown, const // and we extract the original UNO object. CComQIPtr<IUnoObjectWrapper> spUno( pUnknown); if( spUno) - { + { // it is a wrapper Reference<XInterface> xInt; if( SUCCEEDED( spUno->getOriginalUnoObject( &xInt))) { @@ -1406,117 +1419,25 @@ Any UnoConversionUtilities<T>::createOleObjectWrapper(IUnknown* pUnknown, const else { // "pUnknown" is a real COM object. + // If the object implements UNO interfaces then get the types. + Sequence<Type> seqTypes= getImplementedInterfaces(pUnknown); // Before we create a new wrapper object we check if there is an existing wrapper - // CIT_Com cit_currWrapper= ComPtrToWrapperMap.find( reinterpret_cast<sal_uInt32>(pUnknown)); -// sal_Bool bSuccess= sal_False; -// if(cit_currWrapper != ComPtrToWrapperMap.end()) -// { -// WeakReference<XInterface> xweak= cit_currWrapper->second; -// Reference<XInterface> xint= xweak; -// // get the Adapter from the WrapperToAdapterMap. This is dangerous since the stored pointer -// // might be invalid already. We should store the adapter in a map with a weak reference. The -// // invocation adapter factory does not support adapter which can be kept weekly. -// CIT_Wrap cit_w= WrapperToAdapterMap.find((sal_uInt32) xint.get()); -// if( cit_w != WrapperToAdapterMap.end()) -// { -// Reference<XInterface> xIntAdapt(reinterpret_cast<XInterface*>( cit_w->second)); -// if(xIntAdapt.is()) -// { -// ret= xIntAdapt->queryInterface(desiredType); -// bSuccess= sal_True; -// } -// } -// else -// { -// ret= xint->queryInterface(desiredType); -// bSuccess= sal_True; -// } -// } -// if(bSuccess == sal_False) -// { + Reference<XInterface> xIntWrapper; + CIT_Com cit_currWrapper= ComPtrToWrapperMap.find( reinterpret_cast<sal_uInt32>(pUnknown)); + if(cit_currWrapper != ComPtrToWrapperMap.end()) + { + WeakReference<XInterface> xweak= cit_currWrapper->second; + xIntWrapper= xweak; + //When the wrapper is destructed it must remove the entry in the map + //therefore we must always get a hard reference + OSL_ENSURE(xIntWrapper.is(),"OLE Automation bridge"); + } + else + { //There is no existing wrapper, therefore we create one for the real COM object Reference<XInterface> xInt= createComWrapperInstance(); if( xInt.is()) { - Reference<XInvocation> xInv( xInt, UNO_QUERY); - - Sequence<Type> seqTypes; - //The wrapper implements already XInvocation and XInterface. Therefore if the - //param aType is one of those we can use the wrapper object directly otherwise - //we have to create an Adapter interface. - //If aType is void then we assume the type to be XInterface - - if( desiredType == getCppuType((Reference<XInvocation>*) 0)) - { - ret <<= xInt; -// ComPtrToWrapperMap[reinterpret_cast<sal_uInt32>(pUnknown)]= xInt; - } - else - { - Reference< XInterface> xIntAdapterFac; - xIntAdapterFac= m_smgr->createInstance( INTERFACE_ADAPTER_FACTORY); - // We create an adapter object that does not only implement the required type but also - // all types that the COM object pretends to implement. An COM object must therefore - // support the property "_implementedInterfaces". - CComDispatchDriver disp( pUnknown); - if( disp) - { - CComVariant var; - HRESULT hr= S_OK; - // There are two different property names possible. - if( FAILED( hr= disp.GetPropertyByName( SUPPORTED_INTERFACES_PROP, &var))) - { - hr= disp.GetPropertyByName( SUPPORTED_INTERFACES_PROP2, &var); - } - if (SUCCEEDED( hr)) - { - // we exspect an array( SafeArray or IDispatch) of Strings. - Any anyNames; - if( variantToAny2( &var, anyNames, getCppuType( (Sequence<Any>*) 0))) - { - Sequence<Any> seqAny; - if( anyNames >>= seqAny) - { - seqTypes.realloc( seqAny.getLength()); - for( sal_Int32 i=0; i < seqAny.getLength(); i++) - { - OUString typeName; - seqAny[i] >>= typeName; - seqTypes[i]= Type( TypeClass_INTERFACE, typeName); - } - } - } - } - } - - Reference<XInterface> xIntAdapted; - sal_Int32 seqTypesLen = seqTypes.getLength(); - - if( seqTypesLen > 0) - { - Reference< XInvocationAdapterFactory2> xAdapterFac( xIntAdapterFac, UNO_QUERY); - if( xAdapterFac.is()) - xIntAdapted= xAdapterFac->createAdapter( xInv, seqTypes); - } - else - { - Reference<XInvocationAdapterFactory> xAdapterFac( xIntAdapterFac, UNO_QUERY); - if( xAdapterFac.is()) - xIntAdapted= xAdapterFac->createAdapter( xInv, desiredType); - } - if( xIntAdapted.is()) - { - ret= xIntAdapted->queryInterface( desiredType); - // Put the pointer to the wrapper object and the interface pointer of the adapted interface - // in a global map. Thus we can determine in a call to createUnoObjectWrapper whether the UNO - // object is a wrapped COM object. In that case we extract the original COM object rather than - // creating a wrapper around the UNO object. - typedef hash_map<sal_uInt32,sal_uInt32>::value_type VALUE; - AdapterToWrapperMap.insert( VALUE( (sal_uInt32) xIntAdapted.get(), (sal_uInt32) xInt.get())); - WrapperToAdapterMap.insert( VALUE( (sal_uInt32) xInt.get(), (sal_uInt32) xIntAdapted.get())); -// ComPtrToWrapperMap[reinterpret_cast<sal_uInt32>(pUnknown)]= xInt; - } - } // initialize the COM wrapper ( IUnknown + Type s) Reference<XInitialization> xInit( xInt, UNO_QUERY); if( xInit.is()) @@ -1533,9 +1454,61 @@ Any UnoConversionUtilities<T>::createOleObjectWrapper(IUnknown* pUnknown, const } xInit->initialize( Sequence<Any>( params, 2)); + xIntWrapper= xInt; } } -// } + } + + if( xIntWrapper.is()) + { + // we have a wrapper object + //The wrapper implements already XInvocation and XInterface. Therefore if the + //param aType is one of those we can use the wrapper object directly otherwise + //we have to create an Adapter interface. + //If aType is void then we assume the type to be XInterface + if( desiredType == getCppuType((Reference<XInvocation>*) 0)) + { + ret <<= xIntWrapper; + // remember the wrapper object + ComPtrToWrapperMap[reinterpret_cast<sal_uInt32>(pUnknown)]= xIntWrapper; + } + else + { + Reference< XInterface> xIntAdapterFac; + xIntAdapterFac= m_smgr->createInstance( INTERFACE_ADAPTER_FACTORY); + // We create an adapter object that does not only implement the required type but also + // all types that the COM object pretends to implement. An COM object must therefore + // support the property "_implementedInterfaces". + Reference<XInterface> xIntAdapted; + sal_Int32 seqTypesLen = seqTypes.getLength(); + Reference<XInvocation> xInv( xIntWrapper, UNO_QUERY); + if( seqTypesLen > 0) + { + Reference< XInvocationAdapterFactory2> xAdapterFac( xIntAdapterFac, UNO_QUERY); + if( xAdapterFac.is()) + xIntAdapted= xAdapterFac->createAdapter( xInv, seqTypes); + } + else + { + Reference<XInvocationAdapterFactory> xAdapterFac( xIntAdapterFac, UNO_QUERY); + if( xAdapterFac.is()) + xIntAdapted= xAdapterFac->createAdapter( xInv, desiredType); + } + if( xIntAdapted.is()) + { + ret= xIntAdapted->queryInterface( desiredType); + // Put the pointer to the wrapper object and the interface pointer of the adapted interface + // in a global map. Thus we can determine in a call to createUnoObjectWrapper whether the UNO + // object is a wrapped COM object. In that case we extract the original COM object rather than + // creating a wrapper around the UNO object. + typedef hash_map<sal_uInt32,sal_uInt32>::value_type VALUE; + AdapterToWrapperMap.insert( VALUE( (sal_uInt32) xIntAdapted.get(), (sal_uInt32) xIntWrapper.get())); + WrapperToAdapterMap.insert( VALUE( (sal_uInt32) xIntWrapper.get(), (sal_uInt32) xIntAdapted.get())); + ComPtrToWrapperMap[reinterpret_cast<sal_uInt32>(pUnknown)]= xIntWrapper; + } + } + + } } } @@ -1938,6 +1911,43 @@ VARTYPE UnoConversionUtilities<T>::mapTypeClassToVartype( TypeClass type) return ret; } +template<class T> +Sequence<Type> UnoConversionUtilities<T>::getImplementedInterfaces(IUnknown* pUnk) +{ + Sequence<Type> seqTypes; + CComDispatchDriver disp( pUnk); + if( disp) + { + CComVariant var; + HRESULT hr= S_OK; + // There are two different property names possible. + if( FAILED( hr= disp.GetPropertyByName( SUPPORTED_INTERFACES_PROP, &var))) + { + hr= disp.GetPropertyByName( SUPPORTED_INTERFACES_PROP2, &var); + } + if (SUCCEEDED( hr)) + { + // we exspect an array( SafeArray or IDispatch) of Strings. + Any anyNames; + if( variantToAny2( &var, anyNames, getCppuType( (Sequence<Any>*) 0))) + { + Sequence<Any> seqAny; + if( anyNames >>= seqAny) + { + seqTypes.realloc( seqAny.getLength()); + for( sal_Int32 i=0; i < seqAny.getLength(); i++) + { + OUString typeName; + seqAny[i] >>= typeName; + seqTypes[i]= Type( TypeClass_INTERFACE, typeName); + } + } + } + } + } + return seqTypes; +} + // This function tries to the change the type of a value (contained in the Any) // to the smallest possible that can hold the value. This is actually done only diff --git a/extensions/source/ole/unoobjw.cxx b/extensions/source/ole/unoobjw.cxx index d3c436e07..622b0cfec 100644 --- a/extensions/source/ole/unoobjw.cxx +++ b/extensions/source/ole/unoobjw.cxx @@ -2,9 +2,9 @@ * * $RCSfile: unoobjw.cxx,v $ * - * $Revision: 1.13 $ + * $Revision: 1.14 $ * - * last change: $Author: jl $ $Date: 2002-08-06 10:44:34 $ + * last change: $Author: jl $ $Date: 2002-09-13 06:23:08 $ * * The Contents of this file are made available subject to the terms of * either of the following licenses @@ -162,7 +162,11 @@ InterfaceOleWrapper_Impl::InterfaceOleWrapper_Impl( Reference<XMultiServiceFacto InterfaceOleWrapper_Impl::~InterfaceOleWrapper_Impl() { - UnoObjToWrapperMap.erase( (unsigned long) m_xOrigin.get()); + MutexGuard guard(getBridgeMutex()); + // remove entries in global map + IT_Uno it= UnoObjToWrapperMap.find( (sal_uInt32) m_xOrigin.get()); + if(it != UnoObjToWrapperMap.end()) + UnoObjToWrapperMap.erase(it); } STDMETHODIMP InterfaceOleWrapper_Impl::QueryInterface(REFIID riid, LPVOID FAR * ppv) @@ -194,25 +198,17 @@ STDMETHODIMP InterfaceOleWrapper_Impl::QueryInterface(REFIID riid, LPVOID FAR * STDMETHODIMP_(ULONG) InterfaceOleWrapper_Impl::AddRef() { - return m_refCount.acquire(); + acquire(); + // does not need to guard because one should not rely on the return value of + // AddRef anyway + return m_refCount; } STDMETHODIMP_(ULONG) InterfaceOleWrapper_Impl::Release() { - ULONG n; - - { - MutexGuard guard(getBridgeMutex()); - - n = m_refCount.release(); - } - - if (n == 0) - { - delete this; - } - - return n; + ULONG n= m_refCount; + release(); + return n - 1; } // IUnoObjectWrapper -------------------------------------------------------- @@ -615,14 +611,14 @@ Any SAL_CALL InterfaceOleWrapper_Impl::createBridge(const Any& modelDepObject, } // XInterface ------------------------------------------------------ -void SAL_CALL InterfaceOleWrapper_Impl::acquire( ) throw() -{ - AddRef(); -} -void SAL_CALL InterfaceOleWrapper_Impl::release( ) throw() -{ - Release(); -} +// void SAL_CALL InterfaceOleWrapper_Impl::acquire( ) throw() +// { +// AddRef(); +// } +// void SAL_CALL InterfaceOleWrapper_Impl::release( ) throw() +// { +// Release(); +// } // XInitialization -------------------------------------------------- diff --git a/extensions/source/ole/unoobjw.hxx b/extensions/source/ole/unoobjw.hxx index f1f66e9a4..f02004302 100644 --- a/extensions/source/ole/unoobjw.hxx +++ b/extensions/source/ole/unoobjw.hxx @@ -2,9 +2,9 @@ * * $RCSfile: unoobjw.hxx,v $ * - * $Revision: 1.6 $ + * $Revision: 1.7 $ * - * last change: $Author: jl $ $Date: 2002-06-05 13:21:38 $ + * last change: $Author: jl $ $Date: 2002-09-13 06:23:08 $ * * The Contents of this file are made available subject to the terms of * either of the following licenses @@ -213,10 +213,6 @@ public: virtual HRESULT STDMETHODCALLTYPE GetNameSpaceParent( /* [out] */ IUnknown __RPC_FAR *__RPC_FAR *ppunk); - //XInterface ---------------------------------------------------------- - void SAL_CALL InterfaceOleWrapper_Impl::acquire( ) throw(); - void SAL_CALL InterfaceOleWrapper_Impl::release( ) throw(); - // XBridgeSupplier2 --------------------------------------------------- virtual Any SAL_CALL createBridge(const Any& modelDepObject, const Sequence<sal_Int8>& ProcessId, @@ -256,7 +252,7 @@ protected: sal_Bool getInvocationInfoForCall(DISPID id, InvocationInfo& info); - vos::ORefCount m_refCount; +// vos::ORefCount m_refCount; Reference<XInvocation> m_xInvocation; Reference<XExactName> m_xExactName; Reference<XInterface> m_xOrigin; |