diff --git a/extensions/source/ole/oleobjw.cxx b/extensions/source/ole/oleobjw.cxx index b07c9bf067ac..8238088e641f 100644 --- a/extensions/source/ole/oleobjw.cxx +++ b/extensions/source/ole/oleobjw.cxx @@ -77,16 +77,16 @@ std::unordered_map WrapperToAdapterMap; std::unordered_map > ComPtrToWrapperMap; -IUnknownWrapper_Impl::IUnknownWrapper_Impl( Reference const & xFactory, - sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass): - UnoConversionUtilities( xFactory, unoWrapperClass, comWrapperClass), +IUnknownWrapper::IUnknownWrapper( Reference const & xFactory, + sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass): + UnoConversionUtilities( xFactory, unoWrapperClass, comWrapperClass), m_pxIdlClass( nullptr), m_eJScript( JScriptUndefined), m_bComTlbIndexInit(false), m_bHasDfltMethod(false), m_bHasDfltProperty(false) { } -IUnknownWrapper_Impl::~IUnknownWrapper_Impl() +IUnknownWrapper::~IUnknownWrapper() { o2u_attachCurrentThread(); MutexGuard guard(getBridgeMutex()); @@ -112,7 +112,7 @@ IUnknownWrapper_Impl::~IUnknownWrapper_Impl() ComPtrToWrapperMap.erase(it_c); } -Any IUnknownWrapper_Impl::queryInterface(const Type& t) +Any IUnknownWrapper::queryInterface(const Type& t) { if (t == cppu::UnoType::get() && !m_bHasDfltMethod ) return Any(); @@ -130,14 +130,14 @@ Any IUnknownWrapper_Impl::queryInterface(const Type& t) XInitialization, XAutomationObject, XDefaultProperty, XDefaultMethod, XDirectInvocation, XAutomationInvocation >::queryInterface(t); } -Reference SAL_CALL IUnknownWrapper_Impl::getIntrospection() +Reference SAL_CALL IUnknownWrapper::getIntrospection() { Reference ret; return ret; } -Any SAL_CALL IUnknownWrapper_Impl::invokeGetProperty( const OUString& aPropertyName, const Sequence< Any >& aParams, Sequence< sal_Int16 >& aOutParamIndex, Sequence< Any >& aOutParam ) +Any SAL_CALL IUnknownWrapper::invokeGetProperty( const OUString& aPropertyName, const Sequence< Any >& aParams, Sequence< sal_Int16 >& aOutParamIndex, Sequence< Any >& aOutParam ) { Any aResult; try @@ -159,13 +159,13 @@ Any SAL_CALL IUnknownWrapper_Impl::invokeGetProperty( const OUString& aPropertyN catch ( const Exception& e ) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::invokeGetProperty ! Message : \n" + + "IUnknownWrapper::invokeGetProperty ! Message : \n" + e.Message); } return aResult; } -Any SAL_CALL IUnknownWrapper_Impl::invokePutProperty( const OUString& aPropertyName, const Sequence< Any >& aParams, Sequence< sal_Int16 >& aOutParamIndex, Sequence< Any >& aOutParam ) +Any SAL_CALL IUnknownWrapper::invokePutProperty( const OUString& aPropertyName, const Sequence< Any >& aParams, Sequence< sal_Int16 >& aOutParamIndex, Sequence< Any >& aOutParam ) { Any aResult; try @@ -187,14 +187,14 @@ Any SAL_CALL IUnknownWrapper_Impl::invokePutProperty( const OUString& aPropertyN catch ( const Exception& e ) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::invokePutProperty ! Message : \n" + + "IUnknownWrapper::invokePutProperty ! Message : \n" + e.Message); } return aResult; } -Any SAL_CALL IUnknownWrapper_Impl::invoke( const OUString& aFunctionName, +Any SAL_CALL IUnknownWrapper::invoke( const OUString& aFunctionName, const Sequence< Any >& aParams, Sequence< sal_Int16 >& aOutParamIndex, Sequence< Any >& aOutParam ) { @@ -242,19 +242,19 @@ Any SAL_CALL IUnknownWrapper_Impl::invoke( const OUString& aFunctionName, catch (const Exception & e) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::invoke ! Message : \n" + + "IUnknownWrapper::invoke ! Message : \n" + e.Message); } catch(...) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::Invoke !"); + "IUnknownWrapper::Invoke !"); } return ret; } -void SAL_CALL IUnknownWrapper_Impl::setValue( const OUString& aPropertyName, +void SAL_CALL IUnknownWrapper::setValue( const OUString& aPropertyName, const Any& aValue ) { if ( ! m_spDispatch ) @@ -409,7 +409,7 @@ void SAL_CALL IUnknownWrapper_Impl::setValue( const OUString& aPropertyName, catch (const Exception & e) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::setValue ! Message : \n" + + "IUnknownWrapper::setValue ! Message : \n" + e.Message); } @@ -417,11 +417,11 @@ void SAL_CALL IUnknownWrapper_Impl::setValue( const OUString& aPropertyName, { throw RuntimeException( "[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::setValue !"); + "IUnknownWrapper::setValue !"); } } -Any SAL_CALL IUnknownWrapper_Impl::getValue( const OUString& aPropertyName ) +Any SAL_CALL IUnknownWrapper::getValue( const OUString& aPropertyName ) { if ( ! m_spDispatch ) { @@ -551,19 +551,19 @@ Any SAL_CALL IUnknownWrapper_Impl::getValue( const OUString& aPropertyName ) catch (const Exception & e) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::getValue ! Message : \n" + + "IUnknownWrapper::getValue ! Message : \n" + e.Message); } catch (...) { throw RuntimeException( "[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::getValue !"); + "IUnknownWrapper::getValue !"); } return ret; } -sal_Bool SAL_CALL IUnknownWrapper_Impl::hasMethod( const OUString& aName ) +sal_Bool SAL_CALL IUnknownWrapper::hasMethod( const OUString& aName ) { if ( ! m_spDispatch ) { @@ -600,18 +600,18 @@ sal_Bool SAL_CALL IUnknownWrapper_Impl::hasMethod( const OUString& aName ) catch (const Exception & e) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::hasMethod ! Message : \n" + + "IUnknownWrapper::hasMethod ! Message : \n" + e.Message); } catch (...) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::hasMethod !"); + "IUnknownWrapper::hasMethod !"); } return ret; } -sal_Bool SAL_CALL IUnknownWrapper_Impl::hasProperty( const OUString& aName ) +sal_Bool SAL_CALL IUnknownWrapper::hasProperty( const OUString& aName ) { if ( ! m_spDispatch ) { @@ -649,19 +649,19 @@ sal_Bool SAL_CALL IUnknownWrapper_Impl::hasProperty( const OUString& aName ) catch (const Exception & e) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::hasProperty ! Message : \n" + + "IUnknownWrapper::hasProperty ! Message : \n" + e.Message); } catch (...) { throw RuntimeException("[automation bridge] unexpected exception in " - "IUnknownWrapper_Impl::hasProperty !"); + "IUnknownWrapper::hasProperty !"); } return ret; } -Any SAL_CALL IUnknownWrapper_Impl::createBridge( const Any& modelDepObject, +Any SAL_CALL IUnknownWrapper::createBridge( const Any& modelDepObject, const Sequence< sal_Int8 >& /*aProcessId*/, sal_Int16 sourceModelType, sal_Int16 destModelType ) { @@ -707,10 +707,10 @@ Any SAL_CALL IUnknownWrapper_Impl::createBridge( const Any& modelDepObject, @exception InvocationTargetException @RuntimeException */ -Any IUnknownWrapper_Impl::invokeWithDispIdUnoTlb(const OUString& sFunctionName, - const Sequence< Any >& Params, - Sequence< sal_Int16 >& OutParamIndex, - Sequence< Any >& OutParam) +Any IUnknownWrapper::invokeWithDispIdUnoTlb(const OUString& sFunctionName, + const Sequence< Any >& Params, + Sequence< sal_Int16 >& OutParamIndex, + Sequence< Any >& OutParam) { Any ret; HRESULT hr= S_OK; @@ -928,7 +928,7 @@ Any IUnknownWrapper_Impl::invokeWithDispIdUnoTlb(const OUString& sFunctionName, if( !SUCCEEDED( CComObject::CreateInstance( &pParamObject))) { throw BridgeRuntimeError( - "[automation bridge]IUnknownWrapper_Impl::" + "[automation bridge]IUnknownWrapper::" "invokeWithDispIdUnoTlb\n" "Could not create out parameter at index: " + OUString::number(static_cast(i))); @@ -948,7 +948,7 @@ Any IUnknownWrapper_Impl::invokeWithDispIdUnoTlb(const OUString& sFunctionName, CComDispatchDriver dispDriver( pDisp); if(FAILED( dispDriver.PutPropertyByName( L"0", &varParam))) throw BridgeRuntimeError( - "[automation bridge]IUnknownWrapper_Impl::" + "[automation bridge]IUnknownWrapper::" "invokeWithDispIdUnoTlb\n" "Could not set property \"0\" for the in/out " "param!"); @@ -1071,14 +1071,14 @@ Any IUnknownWrapper_Impl::invokeWithDispIdUnoTlb(const OUString& sFunctionName, catch (IllegalArgumentException & e) { e.Message = - "[automation bridge]IUnknownWrapper_Impl::invokeWithDispIdUnoTlb\n" + "[automation bridge]IUnknownWrapper::invokeWithDispIdUnoTlb\n" "Could not convert return value! \n Message: \n" + e.Message; throw; } catch (CannotConvertException & e) { e.Message = - "[automation bridge]IUnknownWrapper_Impl::invokeWithDispIdUnoTlb\n" + "[automation bridge]IUnknownWrapper::invokeWithDispIdUnoTlb\n" "Could not convert return value! \n Message: \n" + e.Message; throw; } @@ -1141,7 +1141,7 @@ Any IUnknownWrapper_Impl::invokeWithDispIdUnoTlb(const OUString& sFunctionName, // XInitialization -void SAL_CALL IUnknownWrapper_Impl::initialize( const Sequence< Any >& aArguments ) +void SAL_CALL IUnknownWrapper::initialize( const Sequence< Any >& aArguments ) { // 1.parameter is IUnknown // 2.parameter is a boolean which indicates if the COM pointer was a IUnknown or IDispatch @@ -1187,7 +1187,7 @@ void SAL_CALL IUnknownWrapper_Impl::initialize( const Sequence< Any >& aArgument { getFuncDesc( usName, &aDescGet ); if ( !aDescGet ) - throw BridgeRuntimeError( "[automation bridge]IUnknownWrapper_Impl::initialize() Failed to get Function or Property desc. for " + usName ); + throw BridgeRuntimeError( "[automation bridge]IUnknownWrapper::initialize() Failed to get Function or Property desc. for " + usName ); } // now for some funny heuristics to make basic understand what to do // a single aDescGet ( that doesn't take any params ) would be @@ -1212,14 +1212,14 @@ void SAL_CALL IUnknownWrapper_Impl::initialize( const Sequence< Any >& aArgument catch( const Exception& e ) { throw RuntimeException( - "[automation bridge] unexpected exception in IUnknownWrapper_Impl::initialize() error message: \n" + e.Message ); + "[automation bridge] unexpected exception in IUnknownWrapper::initialize() error message: \n" + e.Message ); } } } // XDirectInvocation -uno::Any SAL_CALL IUnknownWrapper_Impl::directInvoke( const OUString& aName, const uno::Sequence< uno::Any >& aParams ) +uno::Any SAL_CALL IUnknownWrapper::directInvoke( const OUString& aName, const uno::Sequence< uno::Any >& aParams ) { Any aResult; @@ -1464,7 +1464,7 @@ uno::Any SAL_CALL IUnknownWrapper_Impl::directInvoke( const OUString& aName, con return aResult; } -sal_Bool SAL_CALL IUnknownWrapper_Impl::hasMember( const OUString& aName ) +sal_Bool SAL_CALL IUnknownWrapper::hasMember( const OUString& aName ) { if ( ! m_spDispatch ) { @@ -1479,11 +1479,11 @@ sal_Bool SAL_CALL IUnknownWrapper_Impl::hasMember( const OUString& aName ) // UnoConversionUtilities -------------------------------------------------------------------------------- -Reference< XInterface > IUnknownWrapper_Impl::createUnoWrapperInstance() +Reference< XInterface > IUnknownWrapper::createUnoWrapperInstance() { if( m_nUnoWrapperClass == INTERFACE_OLE_WRAPPER_IMPL) { - Reference xWeak= static_cast( new InterfaceOleWrapper_Impl( + Reference xWeak= static_cast( new InterfaceOleWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } @@ -1496,15 +1496,15 @@ Reference< XInterface > IUnknownWrapper_Impl::createUnoWrapperInstance() else return Reference(); } -Reference IUnknownWrapper_Impl::createComWrapperInstance() +Reference IUnknownWrapper::createComWrapperInstance() { - Reference xWeak= static_cast( new IUnknownWrapper_Impl( + Reference xWeak= static_cast( new IUnknownWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } -void IUnknownWrapper_Impl::getMethodInfo(const OUString& sName, TypeDescription& methodInfo) +void IUnknownWrapper::getMethodInfo(const OUString& sName, TypeDescription& methodInfo) { TypeDescription desc= getInterfaceMemberDescOfCurrentCall(sName); if( desc.is()) @@ -1515,7 +1515,7 @@ void IUnknownWrapper_Impl::getMethodInfo(const OUString& sName, TypeDescription& } } -void IUnknownWrapper_Impl::getAttributeInfo(const OUString& sName, TypeDescription& attributeInfo) +void IUnknownWrapper::getAttributeInfo(const OUString& sName, TypeDescription& attributeInfo) { TypeDescription desc= getInterfaceMemberDescOfCurrentCall(sName); if( desc.is()) @@ -1527,7 +1527,7 @@ void IUnknownWrapper_Impl::getAttributeInfo(const OUString& sName, TypeDescripti } } } -TypeDescription IUnknownWrapper_Impl::getInterfaceMemberDescOfCurrentCall(const OUString& sName) +TypeDescription IUnknownWrapper::getInterfaceMemberDescOfCurrentCall(const OUString& sName) { TypeDescription ret; @@ -1568,7 +1568,7 @@ TypeDescription IUnknownWrapper_Impl::getInterfaceMemberDescOfCurrentCall(const return ret; } -bool IUnknownWrapper_Impl::isJScriptObject() +bool IUnknownWrapper::isJScriptObject() { if( m_eJScript == JScriptUndefined) { @@ -1606,10 +1606,10 @@ bool IUnknownWrapper_Impl::isJScriptObject() @RuntimeException @BridgeRuntimeError */ -Any IUnknownWrapper_Impl::invokeWithDispIdComTlb(const OUString& sFuncName, - const Sequence< Any >& Params, - Sequence< sal_Int16 >& OutParamIndex, - Sequence< Any >& OutParam) +Any IUnknownWrapper::invokeWithDispIdComTlb(const OUString& sFuncName, + const Sequence< Any >& Params, + Sequence< sal_Int16 >& OutParamIndex, + Sequence< Any >& OutParam) { // Get type info for the call. It can be a method call or property put or // property get operation. @@ -1618,11 +1618,11 @@ Any IUnknownWrapper_Impl::invokeWithDispIdComTlb(const OUString& sFuncName, return invokeWithDispIdComTlb( aFuncDesc, sFuncName, Params, OutParamIndex, OutParam ); } -Any IUnknownWrapper_Impl::invokeWithDispIdComTlb(FuncDesc& aFuncDesc, - const OUString& sFuncName, - const Sequence< Any >& Params, - Sequence< sal_Int16 >& OutParamIndex, - Sequence< Any >& OutParam) +Any IUnknownWrapper::invokeWithDispIdComTlb(FuncDesc& aFuncDesc, + const OUString& sFuncName, + const Sequence< Any >& Params, + Sequence< sal_Int16 >& OutParamIndex, + Sequence< Any >& OutParam) { Any ret; HRESULT result; @@ -2090,9 +2090,9 @@ Any IUnknownWrapper_Impl::invokeWithDispIdComTlb(FuncDesc& aFuncDesc, return ret; } -void IUnknownWrapper_Impl::getFuncDescForInvoke(const OUString & sFuncName, - const Sequence & seqArgs, - FUNCDESC** pFuncDesc) +void IUnknownWrapper::getFuncDescForInvoke(const OUString & sFuncName, + const Sequence & seqArgs, + FUNCDESC** pFuncDesc) { int nUnoArgs = seqArgs.getLength(); const Any * arArgs = seqArgs.getConstArray(); @@ -2143,14 +2143,14 @@ void IUnknownWrapper_Impl::getFuncDescForInvoke(const OUString & sFuncName, *pFuncDesc = aFuncDesc.Detach(); } } -bool IUnknownWrapper_Impl::getDispid(const OUString& sFuncName, DISPID * id) +bool IUnknownWrapper::getDispid(const OUString& sFuncName, DISPID * id) { OSL_ASSERT(m_spDispatch); LPOLESTR lpsz = const_cast (o3tl::toW(sFuncName.getStr())); HRESULT hr = m_spDispatch->GetIDsOfNames(IID_NULL, &lpsz, 1, LOCALE_USER_DEFAULT, id); return hr == S_OK; } -void IUnknownWrapper_Impl::getFuncDesc(const OUString & sFuncName, FUNCDESC ** pFuncDesc) +void IUnknownWrapper::getFuncDesc(const OUString & sFuncName, FUNCDESC ** pFuncDesc) { OSL_ASSERT( * pFuncDesc == nullptr); @@ -2217,8 +2217,8 @@ void IUnknownWrapper_Impl::getFuncDesc(const OUString & sFuncName, FUNCDESC ** p //else no entry found for sFuncName, pFuncDesc will not be filled in } -void IUnknownWrapper_Impl::getPropDesc(const OUString & sFuncName, FUNCDESC ** pFuncDescGet, - FUNCDESC** pFuncDescPut, VARDESC** pVarDesc) +void IUnknownWrapper::getPropDesc(const OUString & sFuncName, FUNCDESC ** pFuncDescGet, + FUNCDESC** pFuncDescPut, VARDESC** pVarDesc) { OSL_ASSERT( * pFuncDescGet == nullptr && * pFuncDescPut == nullptr); buildComTlbIndex(); @@ -2287,7 +2287,7 @@ void IUnknownWrapper_Impl::getPropDesc(const OUString & sFuncName, FUNCDESC ** p //else no entry for sFuncName, pFuncDesc will not be filled in } -VARTYPE IUnknownWrapper_Impl::getUserDefinedElementType( ITypeInfo* pTypeInfo, const DWORD nHrefType ) +VARTYPE IUnknownWrapper::getUserDefinedElementType( ITypeInfo* pTypeInfo, const DWORD nHrefType ) { VARTYPE _type( VT_NULL ); if ( pTypeInfo ) @@ -2331,7 +2331,7 @@ VARTYPE IUnknownWrapper_Impl::getUserDefinedElementType( ITypeInfo* pTypeInfo, c return _type; } -VARTYPE IUnknownWrapper_Impl::getElementTypeDesc(const TYPEDESC *desc) +VARTYPE IUnknownWrapper::getElementTypeDesc(const TYPEDESC *desc) { VARTYPE _type( VT_NULL ); @@ -2357,7 +2357,7 @@ VARTYPE IUnknownWrapper_Impl::getElementTypeDesc(const TYPEDESC *desc) return _type; } -void IUnknownWrapper_Impl::buildComTlbIndex() +void IUnknownWrapper::buildComTlbIndex() { if ( ! m_bComTlbIndexInit) { @@ -2384,12 +2384,12 @@ void IUnknownWrapper_Impl::buildComTlbIndex() } else { - sError = "[automation bridge] IUnknownWrapper_Impl::buildComTlbIndex, " + sError = "[automation bridge] IUnknownWrapper::buildComTlbIndex, " "ITypeInfo::GetNames failed."; } } else - sError = "[automation bridge] IUnknownWrapper_Impl::buildComTlbIndex, " + sError = "[automation bridge] IUnknownWrapper::buildComTlbIndex, " "ITypeInfo::GetFuncDesc failed."; } @@ -2412,18 +2412,18 @@ void IUnknownWrapper_Impl::buildComTlbIndex() } else { - sError = "[automation bridge] IUnknownWrapper_Impl::buildComTlbIndex, " + sError = "[automation bridge] IUnknownWrapper::buildComTlbIndex, " "ITypeInfo::GetNames failed."; } } else - sError = "[automation bridge] IUnknownWrapper_Impl::buildComTlbIndex, " + sError = "[automation bridge] IUnknownWrapper::buildComTlbIndex, " "ITypeInfo::GetVarDesc failed."; } } else - sError = "[automation bridge] IUnknownWrapper_Impl::buildComTlbIndex, " + sError = "[automation bridge] IUnknownWrapper::buildComTlbIndex, " "ITypeInfo::GetTypeAttr failed."; if (sError.getLength()) @@ -2437,7 +2437,7 @@ void IUnknownWrapper_Impl::buildComTlbIndex() } } -ITypeInfo* IUnknownWrapper_Impl::getTypeInfo() +ITypeInfo* IUnknownWrapper::getTypeInfo() { if( !m_spDispatch) { diff --git a/extensions/source/ole/oleobjw.hxx b/extensions/source/ole/oleobjw.hxx index 6c673f178c82..793e00a12a18 100644 --- a/extensions/source/ole/oleobjw.hxx +++ b/extensions/source/ole/oleobjw.hxx @@ -53,16 +53,16 @@ typedef std::unordered_multimap TLBFuncIndexMap; // This class wraps an IDispatch and maps XInvocation calls to IDispatch calls on the wrapped object. // If m_TypeDescription is set then this class represents an UNO interface implemented in a COM component. // The interface is not a real interface in terms of an abstract class but is realized through IDispatch. -class IUnknownWrapper_Impl : public WeakImplHelper< XBridgeSupplier2, XInitialization, XAutomationObject, XDefaultProperty, XDefaultMethod, XDirectInvocation, XAutomationInvocation >, +class IUnknownWrapper : public WeakImplHelper< XBridgeSupplier2, XInitialization, XAutomationObject, XDefaultProperty, XDefaultMethod, XDirectInvocation, XAutomationInvocation >, - public UnoConversionUtilities + public UnoConversionUtilities { public: - IUnknownWrapper_Impl(Reference const &xFactory, - sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass); + IUnknownWrapper(Reference const &xFactory, + sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass); - ~IUnknownWrapper_Impl() override; + ~IUnknownWrapper() override; //XInterface Any SAL_CALL queryInterface(const Type& t) override; diff --git a/extensions/source/ole/servprov.cxx b/extensions/source/ole/servprov.cxx index 15e2ae246d87..60959ae323c4 100644 --- a/extensions/source/ole/servprov.cxx +++ b/extensions/source/ole/servprov.cxx @@ -43,8 +43,8 @@ using namespace com::sun::star::bridge::ModelDependent; // {82154420-0FBF-11d4-8313-005004526AB4} DEFINE_GUID(OID_ServiceManager, 0x82154420, 0xfbf, 0x11d4, 0x83, 0x13, 0x0, 0x50, 0x4, 0x52, 0x6a, 0xb4); -ProviderOleWrapper_Impl::ProviderOleWrapper_Impl(const Reference& smgr, - const Reference& xSFact, GUID const * pGuid) +ProviderOleWrapper::ProviderOleWrapper(const Reference& smgr, + const Reference& xSFact, GUID const * pGuid) : m_xSingleServiceFactory(xSFact), m_guid(*pGuid), m_smgr( smgr) @@ -59,11 +59,11 @@ ProviderOleWrapper_Impl::ProviderOleWrapper_Impl(const Reference& smgr, - const Reference& xInst, - GUID const * pGuid ) +OneInstanceOleWrapper::OneInstanceOleWrapper( const Reference& smgr, + const Reference& xInst, + GUID const * pGuid ) : m_refCount(0) , m_xInst(xInst) , m_guid(*pGuid) @@ -188,11 +188,11 @@ OneInstanceOleWrapper_Impl::OneInstanceOleWrapper_Impl( const Reference &smgr): - UnoConversionUtilities( smgr) +OleConverter::OleConverter( const Reference &smgr): + UnoConversionUtilities( smgr) { } // The XMultiServiceFactory is later set by XInitialization -OleConverter_Impl2::OleConverter_Impl2( const Reference& smgr, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass ): - UnoConversionUtilities( smgr, unoWrapperClass, comWrapperClass ) +OleConverter::OleConverter( const Reference& smgr, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass ): + UnoConversionUtilities( smgr, unoWrapperClass, comWrapperClass ) { } -OleConverter_Impl2::~OleConverter_Impl2() +OleConverter::~OleConverter() { } // XBridgeSupplier -------------------------------------------------------------- -Any SAL_CALL OleConverter_Impl2::createBridge(const Any& modelDepObject, - const Sequence< sal_Int8 >& ProcessId, - sal_Int16 sourceModelType, - sal_Int16 destModelType) +Any SAL_CALL OleConverter::createBridge(const Any& modelDepObject, + const Sequence< sal_Int8 >& ProcessId, + sal_Int16 sourceModelType, + sal_Int16 destModelType) { Any ret; sal_uInt8 arId[16]; @@ -393,19 +393,19 @@ Any SAL_CALL OleConverter_Impl2::createBridge(const Any& modelDepObject, return ret; } -OUString OleConverter_Impl2::getImplementationName() +OUString OleConverter::getImplementationName() { return m_nUnoWrapperClass == INTERFACE_OLE_WRAPPER_IMPL ? OUString("com.sun.star.comp.ole.OleConverter2") : OUString("com.sun.star.comp.ole.OleConverterVar1"); } -sal_Bool OleConverter_Impl2::supportsService(OUString const & ServiceName) +sal_Bool OleConverter::supportsService(OUString const & ServiceName) { return cppu::supportsService(this, ServiceName); } -css::uno::Sequence OleConverter_Impl2::getSupportedServiceNames() +css::uno::Sequence OleConverter::getSupportedServiceNames() { if (m_nUnoWrapperClass == INTERFACE_OLE_WRAPPER_IMPL) { @@ -419,7 +419,7 @@ css::uno::Sequence OleConverter_Impl2::getSupportedServiceNames() // XInitialize ------------------------------------------------------------------------------ // the first argument is an XMultiServiceFactory if at all -void SAL_CALL OleConverter_Impl2::initialize( const Sequence< Any >& aArguments ) +void SAL_CALL OleConverter::initialize( const Sequence< Any >& aArguments ) { if( aArguments.getLength() == 1 && aArguments[0].getValueTypeClass() == TypeClass_INTERFACE) { @@ -431,11 +431,11 @@ void SAL_CALL OleConverter_Impl2::initialize( const Sequence< Any >& aArguments } // UnoConversionUtilities ------------------------------------------------------------------- -Reference< XInterface > OleConverter_Impl2::createUnoWrapperInstance() +Reference< XInterface > OleConverter::createUnoWrapperInstance() { if( m_nUnoWrapperClass == INTERFACE_OLE_WRAPPER_IMPL) { - Reference xWeak= static_cast( new InterfaceOleWrapper_Impl( + Reference xWeak= static_cast( new InterfaceOleWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } @@ -449,15 +449,15 @@ Reference< XInterface > OleConverter_Impl2::createUnoWrapperInstance() return Reference(); } -Reference< XInterface > OleConverter_Impl2::createComWrapperInstance() +Reference< XInterface > OleConverter::createComWrapperInstance() { - Reference xWeak= static_cast( new IUnknownWrapper_Impl( + Reference xWeak= static_cast( new IUnknownWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } -OleClient_Impl::OleClient_Impl( const Reference& smgr): - UnoConversionUtilities( smgr) +OleClient::OleClient( const Reference& smgr): + UnoConversionUtilities( smgr) { Reference xInt;// = m_smgr->createInstance(L"com.sun.star.bridge.OleBridgeSupplier2"); @@ -468,35 +468,35 @@ OleClient_Impl::OleClient_Impl( const Reference& smgr): } } -OleClient_Impl::~OleClient_Impl() +OleClient::~OleClient() { } -Sequence< OUString > SAL_CALL OleClient_Impl::getAvailableServiceNames() +Sequence< OUString > SAL_CALL OleClient::getAvailableServiceNames() { Sequence< OUString > ret; return ret; } -OUString OleClient_Impl::getImplementationName() +OUString OleClient::getImplementationName() { return OUString("com.sun.star.comp.ole.OleClient"); } -sal_Bool OleClient_Impl::supportsService(OUString const & ServiceName) +sal_Bool OleClient::supportsService(OUString const & ServiceName) { return cppu::supportsService(this, ServiceName); } -css::uno::Sequence OleClient_Impl::getSupportedServiceNames() +css::uno::Sequence OleClient::getSupportedServiceNames() { return css::uno::Sequence{ "com.sun.star.bridge.OleObjectFactory", "com.sun.star.bridge.oleautomation.Factory"}; } -Reference SAL_CALL OleClient_Impl::createInstance(const OUString& ServiceSpecifier) +Reference SAL_CALL OleClient::createInstance(const OUString& ServiceSpecifier) { Reference ret; HRESULT result; @@ -543,17 +543,17 @@ Reference SAL_CALL OleClient_Impl::createInstance(const OUString& Se return ret; } -Reference SAL_CALL OleClient_Impl::createInstanceWithArguments(const OUString& ServiceSpecifier, const Sequence< Any >& /*Arguments*/) +Reference SAL_CALL OleClient::createInstanceWithArguments(const OUString& ServiceSpecifier, const Sequence< Any >& /*Arguments*/) { return createInstance( ServiceSpecifier); } // UnoConversionUtilities ----------------------------------------------------------------------------- -Reference< XInterface > OleClient_Impl::createUnoWrapperInstance() +Reference< XInterface > OleClient::createUnoWrapperInstance() { if( m_nUnoWrapperClass == INTERFACE_OLE_WRAPPER_IMPL) { - Reference xWeak= static_cast( new InterfaceOleWrapper_Impl( + Reference xWeak= static_cast( new InterfaceOleWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } @@ -567,14 +567,14 @@ Reference< XInterface > OleClient_Impl::createUnoWrapperInstance() return Reference< XInterface>(); } // UnoConversionUtilities ----------------------------------------------------------------------------- -Reference< XInterface > OleClient_Impl::createComWrapperInstance( ) +Reference< XInterface > OleClient::createComWrapperInstance( ) { - Reference xWeak= static_cast( new IUnknownWrapper_Impl( + Reference xWeak= static_cast( new IUnknownWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } -OleServer_Impl::OleServer_Impl( const Reference& smgr): +OleServer::OleServer( const Reference& smgr): m_smgr( smgr) { Reference xInt = m_smgr->createInstance("com.sun.star.bridge.oleautomation.BridgeSupplier"); @@ -588,7 +588,7 @@ OleServer_Impl::OleServer_Impl( const Reference& smgr): (void) provideInstance( m_smgr, &OID_ServiceManager ); } -OleServer_Impl::~OleServer_Impl() +OleServer::~OleServer() { for (auto const& elem : m_wrapperList) { @@ -598,26 +598,26 @@ OleServer_Impl::~OleServer_Impl() m_wrapperList.clear(); } -OUString OleServer_Impl::getImplementationName() +OUString OleServer::getImplementationName() { return OUString("com.sun.star.comp.ole.OleServer"); } -sal_Bool OleServer_Impl::supportsService(OUString const & ServiceName) +sal_Bool OleServer::supportsService(OUString const & ServiceName) { return cppu::supportsService(this, ServiceName); } -css::uno::Sequence OleServer_Impl::getSupportedServiceNames() +css::uno::Sequence OleServer::getSupportedServiceNames() { return css::uno::Sequence{ "com.sun.star.bridge.OleApplicationRegistration", "com.sun.star.bridge.oleautomation.ApplicationRegistration"}; } -bool OleServer_Impl::provideService(const Reference& xSFact, GUID const * guid) +bool OleServer::provideService(const Reference& xSFact, GUID const * guid) { - IClassFactoryWrapper* pFac = new ProviderOleWrapper_Impl( m_smgr, xSFact, guid); + IClassFactoryWrapper* pFac = new ProviderOleWrapper( m_smgr, xSFact, guid); pFac->AddRef(); @@ -626,10 +626,10 @@ bool OleServer_Impl::provideService(const Reference& xSFa return pFac->registerClass(); } -bool OleServer_Impl::provideInstance(const Reference& xInst, GUID const * guid) +bool OleServer::provideInstance(const Reference& xInst, GUID const * guid) { IClassFactoryWrapper* pFac = - new OneInstanceOleWrapper_Impl( m_smgr, xInst, guid ); + new OneInstanceOleWrapper( m_smgr, xInst, guid ); pFac->AddRef(); m_wrapperList.push_back(pFac); diff --git a/extensions/source/ole/servprov.hxx b/extensions/source/ole/servprov.hxx index d8725f35cc49..d7caa9785df0 100644 --- a/extensions/source/ole/servprov.hxx +++ b/extensions/source/ole/servprov.hxx @@ -44,7 +44,7 @@ Reference OleServer_CreateInstance( const Reference& smgr, - const Reference& xSFactory, GUID const * pGuid); - virtual ~ProviderOleWrapper_Impl(); + ProviderOleWrapper( const Reference& smgr, + const Reference& xSFactory, GUID const * pGuid); + virtual ~ProviderOleWrapper(); bool registerClass() override; bool deregisterClass() override; @@ -106,7 +106,7 @@ protected: /***************************************************************************** - OneInstanceOleWrapper_Impl + OneInstanceOleWrapper Provides an single UNO object as OLE object. Handle the OLE registration by overriding the abstract methods from @@ -117,12 +117,12 @@ protected: *****************************************************************************/ -class OneInstanceOleWrapper_Impl : public IClassFactoryWrapper +class OneInstanceOleWrapper : public IClassFactoryWrapper { public: - OneInstanceOleWrapper_Impl( const Reference& smgr, const Reference& xInst, GUID const * pGuid ); - virtual ~OneInstanceOleWrapper_Impl(); + OneInstanceOleWrapper( const Reference& smgr, const Reference& xInst, GUID const * pGuid ); + virtual ~OneInstanceOleWrapper(); bool registerClass() override; bool deregisterClass() override; @@ -149,22 +149,21 @@ protected: // Implementation of the UNO service com.sun.star.bridge.OleBridgeSupplier2. // This class realizes the service com.sun.star.bridge.OleBridgeSupplier2 and -// com.sun.star.bridge.OleBridgeSupplierVar1. The class implements XBridgeSupplier2 -// instead of XBridgeSuppplier as done by class OleConverter_Impl. The XBridgeSupplier2 +// com.sun.star.bridge.OleBridgeSupplierVar1. The class implements XBridgeSupplier2 which // interface does not need a Maschine Id in its createBridge function anymore, // If an UNO interface is to be converted then the member m_nUnoWrapperClass determines -// what wrapper class is to be used. There are currently InterfaceOleWrapper_Impl and +// what wrapper class is to be used. There are currently InterfaceOleWrapper and // UnoObjectWrapperRemoteOpt. The first is used for the OleBridgeSupplier2 and the // latter for OleBridgeSupplierVar1. // The m_nComWrapperClass specifies the class which is used as wrapper for COM interfaces. -// Currently there is only one class available ( IUnknownWrapper_Impl). -class OleConverter_Impl2 : public WeakImplHelper, - public UnoConversionUtilities +// Currently there is only one class available (IUnknownWrapper). +class OleConverter : public WeakImplHelper, + public UnoConversionUtilities { public: - explicit OleConverter_Impl2( const Reference& smgr); - OleConverter_Impl2( const Reference& smgr, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass ); - virtual ~OleConverter_Impl2() override; + explicit OleConverter( const Reference& smgr); + OleConverter( const Reference& smgr, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass ); + virtual ~OleConverter() override; // XBridgeSupplier2 --------------------------------------------------- @@ -191,12 +190,12 @@ protected: // Implementation of the UNO service com.sun.star.bridge.OleObjectFactory. -class OleClient_Impl : public WeakImplHelper, - public UnoConversionUtilities +class OleClient : public WeakImplHelper, + public UnoConversionUtilities { public: - explicit OleClient_Impl( const Reference& smgr); - ~OleClient_Impl() override; + explicit OleClient( const Reference& smgr); + ~OleClient() override; // XMultiServiceFactory Reference SAL_CALL createInstance(const OUString& ServiceSpecifier) override; @@ -219,7 +218,7 @@ protected: /***************************************************************************** - OleServer_Impl + OleServer Implementation of the UNO service com.sun.star.bridge.OleApplicationRegistration. Register the calling application as OLE automation server for @@ -228,11 +227,11 @@ protected: *****************************************************************************/ -class OleServer_Impl : public cppu::WeakImplHelper +class OleServer : public cppu::WeakImplHelper { public: - explicit OleServer_Impl( const Reference &smgr); - ~OleServer_Impl() override; + explicit OleServer( const Reference &smgr); + ~OleServer() override; OUString SAL_CALL getImplementationName() override; diff --git a/extensions/source/ole/servreg.cxx b/extensions/source/ole/servreg.cxx index aee019b79c1e..cdecefae695c 100644 --- a/extensions/source/ole/servreg.cxx +++ b/extensions/source/ole/servreg.cxx @@ -27,25 +27,25 @@ using namespace cppu; Reference ConverterProvider_CreateInstance2( const Reference & xSMgr) { - Reference xService = *new OleConverter_Impl2( xSMgr); + Reference xService = *new OleConverter( xSMgr); return xService; } Reference ConverterProvider_CreateInstanceVar1( const Reference & xSMgr) { - Reference xService = *new OleConverter_Impl2( xSMgr, UNO_OBJECT_WRAPPER_REMOTE_OPT, IUNKNOWN_WRAPPER_IMPL); + Reference xService = *new OleConverter( xSMgr, UNO_OBJECT_WRAPPER_REMOTE_OPT, IUNKNOWN_WRAPPER_IMPL); return xService; } Reference OleClient_CreateInstance( const Reference & xSMgr) { - Reference xService = *new OleClient_Impl( xSMgr); + Reference xService = *new OleClient( xSMgr); return xService; } Reference OleServer_CreateInstance( const Reference & xSMgr) { - Reference xService = *new OleServer_Impl(xSMgr); + Reference xService = *new OleServer(xSMgr); return xService; } diff --git a/extensions/source/ole/unoconversionutilities.hxx b/extensions/source/ole/unoconversionutilities.hxx index f52a3c3430f7..ddc64ce666ad 100644 --- a/extensions/source/ole/unoconversionutilities.hxx +++ b/extensions/source/ole/unoconversionutilities.hxx @@ -70,7 +70,7 @@ extern std::unordered_map WrapperToAdapterMap; extern std::unordered_map > ComPtrToWrapperMap; // 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 +// InterfaceOleWrapper). It is the responsibility of the wrapper to remove the entry when // it is being destroyed. It is used to ensure the identity of objects. That is, an UNO interface // is mapped to IDispatch which is kept alive in the COM environment. If the same // UNO interface is mapped again to COM then the IDispach of the first mapped instance @@ -1624,7 +1624,7 @@ void UnoConversionUtilities::variantToAny( const VARIANT* pVariant, Any& rAny // UNO interfaces in which case it has to support the SUPPORTED_INTERFACES_PROP (see // #define) property. That property contains all names of interfaces. // "pUnknown" is wrapped by a COM wrapper object that implements XInvocation, e.g. -// IUnknownWrapper_Impl. Additionally an object of type "aType" is created by help +// IUnknownWrapper. Additionally an object of type "aType" is created by help // of the INTERFACE_ADAPTER_FACTORY (see #define) service. The implementation of // "aType" calls on the COM wrapper's XInvocation::invoke. If the COM object supports // more than one UNO interfaces, as can be determined by the property diff --git a/extensions/source/ole/unoobjw.cxx b/extensions/source/ole/unoobjw.cxx index dab94a65385e..ad2748f4252a 100644 --- a/extensions/source/ole/unoobjw.cxx +++ b/extensions/source/ole/unoobjw.cxx @@ -87,14 +87,14 @@ static void writeExcepinfo(EXCEPINFO * pInfo, const OUString& message) } } -InterfaceOleWrapper_Impl::InterfaceOleWrapper_Impl( Reference const & xFactory, - sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass): - UnoConversionUtilities( xFactory, unoWrapperClass, comWrapperClass), +InterfaceOleWrapper::InterfaceOleWrapper( Reference const & xFactory, + sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass): + UnoConversionUtilities( xFactory, unoWrapperClass, comWrapperClass), m_defaultValueType( 0) { } -InterfaceOleWrapper_Impl::~InterfaceOleWrapper_Impl() +InterfaceOleWrapper::~InterfaceOleWrapper() { MutexGuard guard(getBridgeMutex()); // remove entries in global map @@ -103,7 +103,7 @@ InterfaceOleWrapper_Impl::~InterfaceOleWrapper_Impl() UnoObjToWrapperMap.erase(it); } -STDMETHODIMP InterfaceOleWrapper_Impl::QueryInterface(REFIID riid, LPVOID FAR * ppv) +STDMETHODIMP InterfaceOleWrapper::QueryInterface(REFIID riid, LPVOID FAR * ppv) { HRESULT ret= S_OK; @@ -130,7 +130,7 @@ STDMETHODIMP InterfaceOleWrapper_Impl::QueryInterface(REFIID riid, LPVOID FAR * return ret; } -STDMETHODIMP_(ULONG) InterfaceOleWrapper_Impl::AddRef() +STDMETHODIMP_(ULONG) InterfaceOleWrapper::AddRef() { acquire(); // does not need to guard because one should not rely on the return value of @@ -138,7 +138,7 @@ STDMETHODIMP_(ULONG) InterfaceOleWrapper_Impl::AddRef() return m_refCount; } -STDMETHODIMP_(ULONG) InterfaceOleWrapper_Impl::Release() +STDMETHODIMP_(ULONG) InterfaceOleWrapper::Release() { ULONG n= m_refCount; release(); @@ -146,17 +146,17 @@ STDMETHODIMP_(ULONG) InterfaceOleWrapper_Impl::Release() } // IUnoObjectWrapper -------------------------------------------------------- -STDMETHODIMP InterfaceOleWrapper_Impl::getWrapperXInterface( Reference* pXInt) +STDMETHODIMP InterfaceOleWrapper::getWrapperXInterface( Reference* pXInt) { pXInt->set( static_cast( this), UNO_QUERY); return pXInt->is() ? S_OK : E_FAIL; } -STDMETHODIMP InterfaceOleWrapper_Impl::getOriginalUnoObject( Reference* pXInt) +STDMETHODIMP InterfaceOleWrapper::getOriginalUnoObject( Reference* pXInt) { *pXInt= m_xOrigin; return m_xOrigin.is() ? S_OK : E_FAIL; } -STDMETHODIMP InterfaceOleWrapper_Impl::getOriginalUnoStruct( Any * pStruct) +STDMETHODIMP InterfaceOleWrapper::getOriginalUnoStruct( Any * pStruct) { HRESULT ret= E_FAIL; if( !m_xOrigin.is()) @@ -175,21 +175,21 @@ STDMETHODIMP InterfaceOleWrapper_Impl::getOriginalUnoStruct( Any * pStruct) return ret; } -STDMETHODIMP InterfaceOleWrapper_Impl::GetTypeInfoCount( unsigned int * /*pctinfo*/ ) +STDMETHODIMP InterfaceOleWrapper::GetTypeInfoCount( unsigned int * /*pctinfo*/ ) { return E_NOTIMPL ; } -STDMETHODIMP InterfaceOleWrapper_Impl::GetTypeInfo(unsigned int /*itinfo*/, LCID /*lcid*/, ITypeInfo ** /*pptinfo*/) +STDMETHODIMP InterfaceOleWrapper::GetTypeInfo(unsigned int /*itinfo*/, LCID /*lcid*/, ITypeInfo ** /*pptinfo*/) { return E_NOTIMPL; } -STDMETHODIMP InterfaceOleWrapper_Impl::GetIDsOfNames(REFIID /*riid*/, - OLECHAR ** rgszNames, - unsigned int cNames, - LCID /*lcid*/, - DISPID * rgdispid ) +STDMETHODIMP InterfaceOleWrapper::GetIDsOfNames(REFIID /*riid*/, + OLECHAR ** rgszNames, + unsigned int cNames, + LCID /*lcid*/, + DISPID * rgdispid ) { HRESULT ret = DISP_E_UNKNOWNNAME; try @@ -312,7 +312,7 @@ STDMETHODIMP InterfaceOleWrapper_Impl::GetIDsOfNames(REFIID /*riid*/, // A JScriptValue (ValueObject) object is a COM object in that it implements IDispatch and the // IJScriptValue object interface. Such objects are provided by all UNO wrapper // objects used within a JScript script. To obtain an instance one has to call -// "_GetValueObject() or Bridge_GetValueObject()" on an UNO wrapper object (class InterfaceOleWrapper_Impl). +// "_GetValueObject() or Bridge_GetValueObject()" on an UNO wrapper object (class InterfaceOleWrapper). // A value object is appropriately initialized within the script and passed as // parameter to an UNO object method or property. The convertDispparamsArgs function // can easily find out that a param is such an object by querying for the @@ -324,7 +324,7 @@ STDMETHODIMP InterfaceOleWrapper_Impl::GetIDsOfNames(REFIID /*riid*/, // Normal JScript object parameter can be mixed with JScriptValue object. If an // VARIANT contains an VT_DISPATCH that is no JScriptValue than the type information // is used to find out about the required type. -void InterfaceOleWrapper_Impl::convertDispparamsArgs(DISPID id, +void InterfaceOleWrapper::convertDispparamsArgs(DISPID id, unsigned short /*wFlags*/, DISPPARAMS* pdispparams, Sequence& rSeq) { HRESULT hr= S_OK; @@ -341,7 +341,7 @@ void InterfaceOleWrapper_Impl::convertDispparamsArgs(DISPID id, InvocationInfo info; if( ! getInvocationInfoForCall( id, info)) throw BridgeRuntimeError( - "[automation bridge]InterfaceOleWrapper_Impl::convertDispparamsArgs \n" + "[automation bridge]InterfaceOleWrapper::convertDispparamsArgs \n" "Could not obtain type information for current call."); for (int i = 0; i < countArgs; i++) @@ -406,7 +406,7 @@ void InterfaceOleWrapper_Impl::convertDispparamsArgs(DISPID id, }// end for / iterating over all parameters } -bool InterfaceOleWrapper_Impl::getInvocationInfoForCall( DISPID id, InvocationInfo& info) +bool InterfaceOleWrapper::getInvocationInfoForCall( DISPID id, InvocationInfo& info) { bool bTypesAvailable= false; @@ -463,11 +463,11 @@ bool InterfaceOleWrapper_Impl::getInvocationInfoForCall( DISPID id, InvocationI } // XBridgeSupplier2 --------------------------------------------------- -// only bridges itself ( this instance of InterfaceOleWrapper_Impl)from UNO to IDispatch -// If sourceModelType is UNO than any UNO interface implemented by InterfaceOleWrapper_Impl +// only bridges itself ( this instance of InterfaceOleWrapper)from UNO to IDispatch +// If sourceModelType is UNO than any UNO interface implemented by InterfaceOleWrapper // can bridged to IDispatch ( if destModelType == OLE). The IDispatch is // implemented by this class. -Any SAL_CALL InterfaceOleWrapper_Impl::createBridge(const Any& modelDepObject, +Any SAL_CALL InterfaceOleWrapper::createBridge(const Any& modelDepObject, const Sequence& /*ProcessId*/, sal_Int16 sourceModelType, sal_Int16 destModelType) @@ -499,7 +499,7 @@ Any SAL_CALL InterfaceOleWrapper_Impl::createBridge(const Any& modelDepObject, } // XInitialization -------------------------------------------------- -void SAL_CALL InterfaceOleWrapper_Impl::initialize( const Sequence< Any >& aArguments ) +void SAL_CALL InterfaceOleWrapper::initialize( const Sequence< Any >& aArguments ) { switch( aArguments.getLength() ) { @@ -517,16 +517,16 @@ void SAL_CALL InterfaceOleWrapper_Impl::initialize( const Sequence< Any >& aArgu m_xExactName.set( m_xInvocation, UNO_QUERY); } -Reference< XInterface > InterfaceOleWrapper_Impl::createUnoWrapperInstance() +Reference< XInterface > InterfaceOleWrapper::createUnoWrapperInstance() { - Reference xWeak= static_cast( new InterfaceOleWrapper_Impl( + Reference xWeak= static_cast( new InterfaceOleWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } -Reference InterfaceOleWrapper_Impl::createComWrapperInstance() +Reference InterfaceOleWrapper::createComWrapperInstance() { - Reference xWeak= static_cast( new IUnknownWrapper_Impl( + Reference xWeak= static_cast( new IUnknownWrapper( m_smgr, m_nUnoWrapperClass, m_nComWrapperClass)); return Reference( xWeak, UNO_QUERY); } @@ -776,14 +776,14 @@ static bool writeBackOutParameter(VARIANTARG* pDest, VARIANT* pSource) return ret; } -STDMETHODIMP InterfaceOleWrapper_Impl::Invoke(DISPID dispidMember, - REFIID /*riid*/, - LCID /*lcid*/, - unsigned short wFlags, - DISPPARAMS * pdispparams, - VARIANT * pvarResult, - EXCEPINFO * pexcepinfo, - unsigned int * puArgErr ) +STDMETHODIMP InterfaceOleWrapper::Invoke(DISPID dispidMember, + REFIID /*riid*/, + LCID /*lcid*/, + unsigned short wFlags, + DISPPARAMS * pdispparams, + VARIANT * pvarResult, + EXCEPINFO * pexcepinfo, + unsigned int * puArgErr ) { comphelper::ProfileZone aZone("COM Bridge"); HRESULT ret = S_OK; @@ -861,14 +861,14 @@ STDMETHODIMP InterfaceOleWrapper_Impl::Invoke(DISPID dispidMember, } catch(const Exception& e) { - OUString message= "InterfaceOleWrapper_Impl::Invoke : \n" + + OUString message= "InterfaceOleWrapper::Invoke : \n" + e.Message; writeExcepinfo(pexcepinfo, message); ret = DISP_E_EXCEPTION; } catch(...) { - OUString message= "InterfaceOleWrapper_Impl::Invoke : \n" + OUString message= "InterfaceOleWrapper::Invoke : \n" "Unexpected exception"; writeExcepinfo(pexcepinfo, message); ret = DISP_E_EXCEPTION; @@ -877,7 +877,7 @@ STDMETHODIMP InterfaceOleWrapper_Impl::Invoke(DISPID dispidMember, return ret; } -HRESULT InterfaceOleWrapper_Impl::doInvoke( DISPPARAMS * pdispparams, VARIANT * pvarResult, +HRESULT InterfaceOleWrapper::doInvoke( DISPPARAMS * pdispparams, VARIANT * pvarResult, EXCEPINFO * pexcepinfo, unsigned int * puArgErr, OUString& name, Sequence& params) { @@ -954,14 +954,14 @@ HRESULT InterfaceOleWrapper_Impl::doInvoke( DISPPARAMS * pdispparams, VARIANT * } catch(const Exception & e) { - OUString message= "InterfaceOleWrapper_Impl::doInvoke : \n" + + OUString message= "InterfaceOleWrapper::doInvoke : \n" + e.Message; writeExcepinfo(pexcepinfo, message); ret = DISP_E_EXCEPTION; } catch( ... ) { - OUString message= "InterfaceOleWrapper_Impl::doInvoke : \n" + OUString message= "InterfaceOleWrapper::doInvoke : \n" "Unexpected exception"; writeExcepinfo(pexcepinfo, message); ret = DISP_E_EXCEPTION; @@ -969,8 +969,8 @@ HRESULT InterfaceOleWrapper_Impl::doInvoke( DISPPARAMS * pdispparams, VARIANT * return ret; } -HRESULT InterfaceOleWrapper_Impl::doGetProperty( DISPPARAMS * /*pdispparams*/, VARIANT * pvarResult, - EXCEPINFO * pexcepinfo, OUString& name) +HRESULT InterfaceOleWrapper::doGetProperty( DISPPARAMS * /*pdispparams*/, VARIANT * pvarResult, + EXCEPINFO * pexcepinfo, OUString& name) { HRESULT ret= S_OK; @@ -993,13 +993,13 @@ HRESULT InterfaceOleWrapper_Impl::doGetProperty( DISPPARAMS * /*pdispparams*/, V } catch(const Exception& e) { - OUString message= "InterfaceOleWrapper_Impl::doGetProperty : \n" + + OUString message= "InterfaceOleWrapper::doGetProperty : \n" + e.Message; writeExcepinfo(pexcepinfo, message); } catch( ... ) { - OUString message= "InterfaceOleWrapper_Impl::doInvoke : \n" + OUString message= "InterfaceOleWrapper::doInvoke : \n" "Unexpected exception"; writeExcepinfo(pexcepinfo, message); ret = DISP_E_EXCEPTION; @@ -1007,7 +1007,7 @@ HRESULT InterfaceOleWrapper_Impl::doGetProperty( DISPPARAMS * /*pdispparams*/, V return ret; } -HRESULT InterfaceOleWrapper_Impl::doSetProperty( DISPPARAMS * /*pdispparams*/, VARIANT * /*pvarResult*/, +HRESULT InterfaceOleWrapper::doSetProperty( DISPPARAMS * /*pdispparams*/, VARIANT * /*pvarResult*/, EXCEPINFO * pexcepinfo, unsigned int * puArgErr, OUString& name, Sequence const & params) { HRESULT ret= S_OK; @@ -1044,7 +1044,7 @@ HRESULT InterfaceOleWrapper_Impl::doSetProperty( DISPPARAMS * /*pdispparams*/, V return ret; } -HRESULT InterfaceOleWrapper_Impl::InvokeGeneral( DISPID dispidMember, unsigned short wFlags, +HRESULT InterfaceOleWrapper::InvokeGeneral( DISPID dispidMember, unsigned short wFlags, DISPPARAMS * pdispparams, VARIANT * pvarResult, EXCEPINFO * pexcepinfo, unsigned int * /*puArgErr*/, bool& bHandled) { @@ -1135,7 +1135,7 @@ HRESULT InterfaceOleWrapper_Impl::InvokeGeneral( DISPID dispidMember, unsigned s if (!createUnoTypeWrapper(arg.bstrVal, pvarResult)) { - writeExcepinfo(pexcepinfo, "[automation bridge] InterfaceOleWrapper_Impl::InvokeGeneral\n" + writeExcepinfo(pexcepinfo, "[automation bridge] InterfaceOleWrapper::InvokeGeneral\n" "Could not initialize UnoTypeWrapper object!"); return DISP_E_EXCEPTION; } @@ -1148,14 +1148,14 @@ HRESULT InterfaceOleWrapper_Impl::InvokeGeneral( DISPID dispidMember, unsigned s } catch(const Exception & e) { - OUString message= "InterfaceOleWrapper_Impl::InvokeGeneral : \n" + + OUString message= "InterfaceOleWrapper::InvokeGeneral : \n" + e.Message; writeExcepinfo(pexcepinfo, message); ret = DISP_E_EXCEPTION; } catch( ... ) { - OUString message= "InterfaceOleWrapper_Impl::InvokeGeneral : \n" + OUString message= "InterfaceOleWrapper::InvokeGeneral : \n" "Unexpected exception"; writeExcepinfo(pexcepinfo, message); ret = DISP_E_EXCEPTION; @@ -1163,12 +1163,12 @@ HRESULT InterfaceOleWrapper_Impl::InvokeGeneral( DISPID dispidMember, unsigned s return ret; } -STDMETHODIMP InterfaceOleWrapper_Impl::GetDispID(BSTR /*bstrName*/, DWORD /*grfdex*/, DISPID __RPC_FAR* /*pid*/) +STDMETHODIMP InterfaceOleWrapper::GetDispID(BSTR /*bstrName*/, DWORD /*grfdex*/, DISPID __RPC_FAR* /*pid*/) { return ResultFromScode(E_NOTIMPL); } -STDMETHODIMP InterfaceOleWrapper_Impl::InvokeEx( +STDMETHODIMP InterfaceOleWrapper::InvokeEx( /* [in] */ DISPID /*id*/, /* [in] */ LCID /*lcid*/, /* [in] */ WORD /*wFlags*/, @@ -1180,19 +1180,19 @@ STDMETHODIMP InterfaceOleWrapper_Impl::InvokeEx( return ResultFromScode(E_NOTIMPL); } -STDMETHODIMP InterfaceOleWrapper_Impl::DeleteMemberByName( +STDMETHODIMP InterfaceOleWrapper::DeleteMemberByName( /* [in] */ BSTR /*bstr*/, /* [in] */ DWORD /*grfdex*/) { return ResultFromScode(E_NOTIMPL); } -STDMETHODIMP InterfaceOleWrapper_Impl::DeleteMemberByDispID(DISPID /*id*/) +STDMETHODIMP InterfaceOleWrapper::DeleteMemberByDispID(DISPID /*id*/) { return ResultFromScode(E_NOTIMPL); } -STDMETHODIMP InterfaceOleWrapper_Impl::GetMemberProperties( +STDMETHODIMP InterfaceOleWrapper::GetMemberProperties( /* [in] */ DISPID /*id*/, /* [in] */ DWORD /*grfdexFetch*/, /* [out] */ DWORD __RPC_FAR* /*pgrfdex*/) @@ -1200,14 +1200,14 @@ STDMETHODIMP InterfaceOleWrapper_Impl::GetMemberProperties( return ResultFromScode(E_NOTIMPL); } -STDMETHODIMP InterfaceOleWrapper_Impl::GetMemberName( +STDMETHODIMP InterfaceOleWrapper::GetMemberName( /* [in] */ DISPID /*id*/, /* [out] */ BSTR __RPC_FAR* /*pbstrName*/) { return ResultFromScode(E_NOTIMPL); } -STDMETHODIMP InterfaceOleWrapper_Impl::GetNextDispID( +STDMETHODIMP InterfaceOleWrapper::GetNextDispID( /* [in] */ DWORD /*grfdex*/, /* [in] */ DISPID /*id*/, /* [out] */ DISPID __RPC_FAR* /*pid*/) @@ -1215,7 +1215,7 @@ STDMETHODIMP InterfaceOleWrapper_Impl::GetNextDispID( return ResultFromScode(E_NOTIMPL); } -STDMETHODIMP InterfaceOleWrapper_Impl::GetNameSpaceParent( +STDMETHODIMP InterfaceOleWrapper::GetNameSpaceParent( /* [out] */ IUnknown __RPC_FAR *__RPC_FAR* /*ppunk*/) { return ResultFromScode(E_NOTIMPL); @@ -1225,7 +1225,7 @@ STDMETHODIMP InterfaceOleWrapper_Impl::GetNameSpaceParent( UnoObjectWrapperRemoteOpt::UnoObjectWrapperRemoteOpt( Reference const & aFactory, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass): -InterfaceOleWrapper_Impl( aFactory, unoWrapperClass, comWrapperClass), +InterfaceOleWrapper( aFactory, unoWrapperClass, comWrapperClass), m_currentId(1) { @@ -1600,7 +1600,7 @@ static HRESULT mapCannotConvertException(const CannotConvertException &e, unsign // object. If a client asks the object for DISPID_VALUE and this // function returned VT_DISPATCH then the IDispatch of the same // object is being returned. -// See InterfaceOleWrapper_Impl::Invoke, InterfaceOleWrapper_Impl::m_defaultValueType +// See InterfaceOleWrapper::Invoke, InterfaceOleWrapper::m_defaultValueType VARTYPE getVarType( const Any& value) { VARTYPE ret= VT_EMPTY; diff --git a/extensions/source/ole/unoobjw.hxx b/extensions/source/ole/unoobjw.hxx index 94d49f51fd21..d2fa9c4ac8cc 100644 --- a/extensions/source/ole/unoobjw.hxx +++ b/extensions/source/ole/unoobjw.hxx @@ -46,23 +46,6 @@ using namespace cppu; using namespace com::sun::star::bridge; using namespace com::sun::star::script; -struct hash_IUnknown_Impl -{ - size_t operator()(const IUnknown* p) const - { - return reinterpret_cast(p); - } -}; - -struct equal_to_IUnknown_Impl -{ - bool operator()(const IUnknown* s1, const IUnknown* s2) const - { - return s1 == s2; - } -}; - - struct MemberInfo { MemberInfo() : flags(0), name() {} @@ -90,16 +73,16 @@ typedef std::unordered_map MemberInfo > IdToMemberInfoMap; -class InterfaceOleWrapper_Impl : public WeakImplHelper, - public IDispatchEx, - public UnoConversionUtilities, - public IUnoObjectWrapper +class InterfaceOleWrapper : public WeakImplHelper, + public IDispatchEx, + public UnoConversionUtilities, + public IUnoObjectWrapper { public: - InterfaceOleWrapper_Impl(Reference const & xFactory, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass); - ~InterfaceOleWrapper_Impl() override; + InterfaceOleWrapper(Reference const & xFactory, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass); + ~InterfaceOleWrapper() override; /* IUnknown methods */ STDMETHOD(QueryInterface)(REFIID riid, LPVOID FAR * ppvObj) override; @@ -205,7 +188,7 @@ protected: // and put a wrapped object on index null. The array object tries // to detect the default value. The wrapped object must then return // its own IDispatch* otherwise we cannot access it within the script. - // see InterfaceOleWrapper_Impl::Invoke + // see InterfaceOleWrapper::Invoke VARTYPE m_defaultValueType; }; @@ -223,7 +206,7 @@ protected: in a MemberInfo structure hold by a IdToMemberInfoMap stl map. *****************************************************************************/ -class UnoObjectWrapperRemoteOpt: public InterfaceOleWrapper_Impl +class UnoObjectWrapperRemoteOpt: public InterfaceOleWrapper { public: UnoObjectWrapperRemoteOpt( Reference const & aFactory, sal_uInt8 unoWrapperClass, sal_uInt8 comWrapperClass);