From a48f22db7dd1bfeff62007a655566ddaad79aebd Mon Sep 17 00:00:00 2001 From: Tor Lillqvist Date: Mon, 12 Feb 2018 16:13:10 +0200 Subject: [PATCH] Drop _Impl class name suffix where pointless I.e. where there is no other LibreOffice, or system, or 3rd-party library thing with just the un-suffixed name that they would be a pimpl of, etc. Why bother calling something InterfaceOleWrapper_Impl when there is nothing that would be called InterfaceOleWrapper? It just makes the code seem more complicated and harder to read. An "implementation" of something is what everything that is not fully abstract *is*. We don't suffix the name of every non-abstract class with _Impl elsewhere either. Also drop two unused structs. Change-Id: I2755e6da78d8e4ded5291688dc90510b5f8b498b Reviewed-on: https://gerrit.libreoffice.org/54411 Reviewed-by: Noel Grandin Tested-by: Jenkins Reviewed-by: Tor Lillqvist --- extensions/source/ole/oleobjw.cxx | 144 +++++++++--------- extensions/source/ole/oleobjw.hxx | 10 +- extensions/source/ole/servprov.cxx | 128 ++++++++-------- extensions/source/ole/servprov.hxx | 53 ++++--- extensions/source/ole/servreg.cxx | 8 +- .../source/ole/unoconversionutilities.hxx | 4 +- extensions/source/ole/unoobjw.cxx | 122 +++++++-------- extensions/source/ole/unoobjw.hxx | 33 +--- 8 files changed, 242 insertions(+), 260 deletions(-) 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);