diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2014-04-15 07:44:14 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2014-04-15 07:46:43 +0200 |
commit | fa0ffdbeb5a12b51c5d9202a5d7709d713a719f1 (patch) | |
tree | 4b742ff087e7f8a3a97d4faa4402cb1a924f52a5 | |
parent | c2ffc9e180d387359d9c465770db3d01b7e84951 (diff) |
cppu::OMultiTypeInterfaceContainerHelperVar hashImpl param is unused
...since f059134fab0f91b7a15594ad06536a13fbfae034 "INTEGRATION: CWS
cppuhelpshrink: #i72766# save memory in cppuhelper, using a vector instead of a
hash_map" (but a dummy "void" default is left in place for backwards
compatibility).
Change-Id: I3bb0af8158f34737d41c344464f3cf944e2891ed
21 files changed, 37 insertions, 177 deletions
diff --git a/comphelper/source/property/genericpropertyset.cxx b/comphelper/source/property/genericpropertyset.cxx index 55ce6cd84eab..6badb4639d82 100644 --- a/comphelper/source/property/genericpropertyset.cxx +++ b/comphelper/source/property/genericpropertyset.cxx @@ -58,7 +58,7 @@ namespace comphelper { private: GenericAnyMapImpl maAnyMap; - ::cppu::OMultiTypeInterfaceContainerHelperVar<OUString, OUStringHash> m_aListener; + cppu::OMultiTypeInterfaceContainerHelperVar<OUString> m_aListener; protected: virtual void _setPropertyValues( const PropertyMapEntry** ppEntries, const Any* pValues ) throw( UnknownPropertyException, PropertyVetoException, IllegalArgumentException, WrappedTargetException ) SAL_OVERRIDE; diff --git a/cppuhelper/qa/ifcontainer/cppu_ifcontainer.cxx b/cppuhelper/qa/ifcontainer/cppu_ifcontainer.cxx index 3efd35da8ee5..838a33a66c60 100644 --- a/cppuhelper/qa/ifcontainer/cppu_ifcontainer.cxx +++ b/cppuhelper/qa/ifcontainer/cppu_ifcontainer.cxx @@ -237,8 +237,8 @@ namespace cppu_ifcontainer void testOMultiTypeInterfaceContainerHelperVar() { - typedef ::cppu::OMultiTypeInterfaceContainerHelperVar< - const char*, rtl::CStringHash, rtl::CStringEqual> StrContainer; + typedef cppu::OMultiTypeInterfaceContainerHelperVar< + char const *, void, rtl::CStringEqual> StrContainer; const char *pTypes[nTests] = { diff --git a/dbaccess/source/core/inc/ContentHelper.hxx b/dbaccess/source/core/inc/ContentHelper.hxx index bc3f5955e748..0cd416b26d81 100644 --- a/dbaccess/source/core/inc/ContentHelper.hxx +++ b/dbaccess/source/core/inc/ContentHelper.hxx @@ -77,10 +77,8 @@ namespace dbaccess typedef ::boost::shared_ptr<OContentHelper_Impl> TContentPtr; - - typedef ::cppu::OMultiTypeInterfaceContainerHelperVar< OUString, - OUStringHash - > PropertyChangeListenerContainer; + typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + PropertyChangeListenerContainer; typedef ::comphelper::OBaseMutex OContentHelper_MBASE; typedef ::cppu::WeakComponentImplHelper9 < ::com::sun::star::ucb::XContent , ::com::sun::star::ucb::XCommandProcessor diff --git a/dbaccess/source/ui/inc/sbagrid.hxx b/dbaccess/source/ui/inc/sbagrid.hxx index ef9b4db2f991..7d14bb52246f 100644 --- a/dbaccess/source/ui/inc/sbagrid.hxx +++ b/dbaccess/source/ui/inc/sbagrid.hxx @@ -48,12 +48,6 @@ namespace dbaui bool operator() (const ::com::sun::star::util::URL& x, const ::com::sun::star::util::URL& y) const {return x.Complete == y.Complete ? true : false;} }; - struct SbaURLHash - { - sal_Int32 operator() (const ::com::sun::star::util::URL& x) const {return x.Complete.hashCode();} - }; - // SbaXGridControl - class SbaXStatusMultiplexer; class SbaXGridControl :public FmXGridControl @@ -103,7 +97,8 @@ namespace dbaui :public FmXGridPeer ,public ::com::sun::star::frame::XDispatch { - ::cppu::OMultiTypeInterfaceContainerHelperVar< ::com::sun::star::util::URL,SbaURLHash , SbaURLCompare> m_aStatusListeners; + cppu::OMultiTypeInterfaceContainerHelperVar< + css::util::URL, void, SbaURLCompare> m_aStatusListeners; public: SbaXGridPeer(const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >&); diff --git a/dbaccess/source/ui/inc/sbamultiplex.hxx b/dbaccess/source/ui/inc/sbamultiplex.hxx index ba23e758a0b8..4828bccd6c4c 100644 --- a/dbaccess/source/ui/inc/sbamultiplex.hxx +++ b/dbaccess/source/ui/inc/sbamultiplex.hxx @@ -185,7 +185,7 @@ namespace dbaui ,public listenerclass \ { \ typedef ::cppu::OMultiTypeInterfaceContainerHelperVar< \ - OUString, OUStringHash > ListenerContainerMap; \ + OUString > ListenerContainerMap; \ ListenerContainerMap m_aListeners; \ \ public: \ diff --git a/embeddedobj/source/general/intercept.cxx b/embeddedobj/source/general/intercept.cxx index 23d2cf8e8ea3..d64b49cf1a0a 100644 --- a/embeddedobj/source/general/intercept.cxx +++ b/embeddedobj/source/general/intercept.cxx @@ -32,13 +32,11 @@ using namespace ::com::sun::star; uno::Sequence< OUString > Interceptor::m_aInterceptedURL(IUL); class StatusChangeListenerContainer - : public ::cppu::OMultiTypeInterfaceContainerHelperVar< -OUString,OUStringHash> + : public cppu::OMultiTypeInterfaceContainerHelperVar<OUString> { public: StatusChangeListenerContainer( ::osl::Mutex& aMutex ) - : cppu::OMultiTypeInterfaceContainerHelperVar< - OUString,OUStringHash>(aMutex) + : cppu::OMultiTypeInterfaceContainerHelperVar<OUString>(aMutex) { } }; diff --git a/embedserv/source/embed/intercept.cxx b/embedserv/source/embed/intercept.cxx index 7db42b7dd9cb..750c74a12d40 100644 --- a/embedserv/source/embed/intercept.cxx +++ b/embedserv/source/embed/intercept.cxx @@ -33,13 +33,11 @@ using namespace ::com::sun::star; uno::Sequence< OUString > Interceptor::m_aInterceptedURL(IUL); class StatusChangeListenerContainer - : public ::cppu::OMultiTypeInterfaceContainerHelperVar< -OUString,OUStringHash> + : public cppu::OMultiTypeInterfaceContainerHelperVar<OUString> { public: StatusChangeListenerContainer( ::osl::Mutex& aMutex ) - : cppu::OMultiTypeInterfaceContainerHelperVar< - OUString,OUStringHash>(aMutex) + : cppu::OMultiTypeInterfaceContainerHelperVar<OUString>(aMutex) { } }; diff --git a/framework/inc/dispatch/menudispatcher.hxx b/framework/inc/dispatch/menudispatcher.hxx index 4fa20001ac05..977c3855745d 100644 --- a/framework/inc/dispatch/menudispatcher.hxx +++ b/framework/inc/dispatch/menudispatcher.hxx @@ -47,13 +47,8 @@ namespace framework{ -/*-************************************************************************************************************ - We must save information about our listener and URL for listening. - We implement this as a hashtable for strings. -*//*-*************************************************************************************************************/ - -typedef ::cppu::OMultiTypeInterfaceContainerHelperVar< OUString , - OUStringHash > IMPL_ListenerHashContainer; +typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + IMPL_ListenerHashContainer; /*-************************************************************************************************************ @short helper for desktop only(!) to create new tasks on demand for dispatches diff --git a/framework/inc/dispatch/popupmenudispatcher.hxx b/framework/inc/dispatch/popupmenudispatcher.hxx index 7c2c4e74283b..8c60489b6703 100644 --- a/framework/inc/dispatch/popupmenudispatcher.hxx +++ b/framework/inc/dispatch/popupmenudispatcher.hxx @@ -49,13 +49,8 @@ namespace framework{ -/*-************************************************************************************************************ - We must save information about our listener and URL for listening. - We implement this as a hashtable for strings. -*//*-*************************************************************************************************************/ - -typedef ::cppu::OMultiTypeInterfaceContainerHelperVar< OUString , - OUStringHash > IMPL_ListenerHashContainer; +typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + IMPL_ListenerHashContainer; /*-************************************************************************************************************ @short helper for desktop only(!) to create new tasks on demand for dispatches diff --git a/framework/inc/stdtypes.h b/framework/inc/stdtypes.h index 9accf59e5340..3d23bdda4101 100644 --- a/framework/inc/stdtypes.h +++ b/framework/inc/stdtypes.h @@ -152,13 +152,7 @@ typedef BaseHash< OUString > OUStringHashMap; */ typedef BaseHash< sal_Int32 > NameToHandleHash; -/** - Sometimes we need this template to implement listener container ... - and we need it at different positions ... - So it's better to declare it one times only! -*/ -typedef ::cppu::OMultiTypeInterfaceContainerHelperVar< OUString , - OUStringHash > ListenerHash; +typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> ListenerHash; } // namespace framework diff --git a/include/cppuhelper/interfacecontainer.h b/include/cppuhelper/interfacecontainer.h index fade8bb52517..7dfff29d43d6 100644 --- a/include/cppuhelper/interfacecontainer.h +++ b/include/cppuhelper/interfacecontainer.h @@ -296,7 +296,7 @@ inline void OInterfaceContainerHelper::notifyEach( void ( SAL_CALL ListenerT::*N @see OInterfaceIteratorHelper @see OInterfaceContainerHelper */ -template< class key , class hashImpl , class equalImpl = std::equal_to<key> > +template< class key, class hashImpl = void, class equalImpl = std::equal_to<key> > class OMultiTypeInterfaceContainerHelperVar { public: diff --git a/include/sfx2/unoctitm.hxx b/include/sfx2/unoctitm.hxx index 75d245721836..6e8d5729d8c4 100644 --- a/include/sfx2/unoctitm.hxx +++ b/include/sfx2/unoctitm.hxx @@ -70,13 +70,8 @@ public: void ReleaseBindings(); }; -struct SfxStatusDispatcher_Impl_hashType -{ - size_t operator()(const OUString& s) const - { return s.hashCode(); } -}; - -typedef ::cppu::OMultiTypeInterfaceContainerHelperVar< OUString, SfxStatusDispatcher_Impl_hashType > SfxStatusDispatcher_Impl_ListenerContainer ; +typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + SfxStatusDispatcher_Impl_ListenerContainer; class SfxStatusDispatcher : public ::cppu::WeakImplHelper1< css::frame::XNotifyingDispatch > { diff --git a/linguistic/source/lngopt.hxx b/linguistic/source/lngopt.hxx index 2500c392976a..93806dd4f405 100644 --- a/linguistic/source/lngopt.hxx +++ b/linguistic/source/lngopt.hxx @@ -66,24 +66,8 @@ public: GetActiveConvDics() const { return pData->aActiveConvDics; } }; - - -// uses templates from <cppuhelper/interfacecontainer.h> - - -// helper function call class -struct PropHashType_Impl -{ - size_t operator()(const sal_Int32 &s) const { return s; } -}; - -typedef cppu::OMultiTypeInterfaceContainerHelperVar - < - sal_Int32, - PropHashType_Impl - > OPropertyListenerContainerHelper; - - +typedef cppu::OMultiTypeInterfaceContainerHelperVar<sal_Int32> + OPropertyListenerContainerHelper; class LinguProps : public cppu::WeakImplHelper5 diff --git a/svx/source/unodraw/shapepropertynotifier.cxx b/svx/source/unodraw/shapepropertynotifier.cxx index 612586455660..63e23486301a 100644 --- a/svx/source/unodraw/shapepropertynotifier.cxx +++ b/svx/source/unodraw/shapepropertynotifier.cxx @@ -63,12 +63,8 @@ namespace svx typedef ::boost::unordered_map< ShapeProperty, PPropertyValueProvider, ShapePropertyHash > PropertyProviders; - typedef ::cppu::OMultiTypeInterfaceContainerHelperVar < OUString - , OUStringHash - > PropertyChangeListenerContainer; - - - //= IPropertyValueProvider + typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + PropertyChangeListenerContainer; IPropertyValueProvider::~IPropertyValueProvider() { diff --git a/sw/source/core/uibase/inc/unomailmerge.hxx b/sw/source/core/uibase/inc/unomailmerge.hxx index a0780f46af52..3aeaa0207963 100644 --- a/sw/source/core/uibase/inc/unomailmerge.hxx +++ b/sw/source/core/uibase/inc/unomailmerge.hxx @@ -56,20 +56,8 @@ namespace com { namespace sun { namespace star { }}} -// uses templates from <cppuhelper/interfacecontainer.h> -// and <unotools/configitem.hxx> - -// helper function call class -struct PropHashType_Impl -{ - size_t operator()(const sal_Int32 &s) const { return s; } -}; - -typedef cppu::OMultiTypeInterfaceContainerHelperVar - < - sal_Int32, - PropHashType_Impl - > OPropertyListenerContainerHelper; +typedef cppu::OMultiTypeInterfaceContainerHelperVar<sal_Int32> + OPropertyListenerContainerHelper; class SwNewDBMgr; class MailMergeExecuteFinalizer; diff --git a/ucb/source/cacher/contentresultsetwrapper.hxx b/ucb/source/cacher/contentresultsetwrapper.hxx index 9148a02030f9..7041cd5cb1b0 100644 --- a/ucb/source/cacher/contentresultsetwrapper.hxx +++ b/ucb/source/cacher/contentresultsetwrapper.hxx @@ -48,15 +48,8 @@ class ContentResultSetWrapper , public com::sun::star::sdbc::XRow { protected: - - - //class PropertyChangeListenerContainer_Impl. - - typedef cppu::OMultiTypeInterfaceContainerHelperVar - < OUString, OUStringHash > - PropertyChangeListenerContainer_Impl; - - // class ReacquireableGuard + typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + PropertyChangeListenerContainer_Impl; class ReacquireableGuard { diff --git a/ucb/source/core/ucbstore.cxx b/ucb/source/core/ucbstore.cxx index 07f4533edbf0..d6ba6e7fe1e2 100644 --- a/ucb/source/core/ucbstore.cxx +++ b/ucb/source/core/ucbstore.cxx @@ -1119,23 +1119,7 @@ Reference< XInterface > PropertySetRegistry::getConfigWriteAccess( return Reference< XInterface >(); } - - -// PropertyListeners_Impl. - - - -typedef OMultiTypeInterfaceContainerHelperVar -< - OUString, - OUStringHash -> PropertyListeners_Impl; - - - -// PersistentPropertySet_Impl. - - +typedef OMultiTypeInterfaceContainerHelperVar<OUString> PropertyListeners_Impl; struct PersistentPropertySet_Impl { diff --git a/ucb/source/sorter/sortresult.cxx b/ucb/source/sorter/sortresult.cxx index 8b6203499301..85eb0e84bf25 100644 --- a/ucb/source/sorter/sortresult.cxx +++ b/ucb/source/sorter/sortresult.cxx @@ -111,40 +111,8 @@ public: throw( RuntimeException, std::exception ) SAL_OVERRIDE; }; - - -// PropertyChangeListenerContainer_Impl. - - - -struct equalStr_Impl -{ - bool operator()( const OUString& s1, const OUString& s2 ) const - { - return !!( s1 == s2 ); - } -}; - -struct hashStr_Impl -{ - size_t operator()( const OUString& rName ) const - { - return rName.hashCode(); - } -}; - -typedef OMultiTypeInterfaceContainerHelperVar -< - OUString, - hashStr_Impl, - equalStr_Impl -> PropertyChangeListenerContainer_Impl; - - - -// class PropertyChangeListeners_Impl - - +typedef OMultiTypeInterfaceContainerHelperVar<OUString> + PropertyChangeListenerContainer_Impl; class PropertyChangeListeners_Impl : public PropertyChangeListenerContainer_Impl { diff --git a/ucb/source/ucp/file/bc.cxx b/ucb/source/ucp/file/bc.cxx index 130c8bd41450..de2f777b0a57 100644 --- a/ucb/source/ucp/file/bc.cxx +++ b/ucb/source/ucp/file/bc.cxx @@ -54,11 +54,8 @@ using namespace com::sun::star::ucb; #define THROW_WHERE "" #endif -// PropertyListeners - - -typedef cppu::OMultiTypeInterfaceContainerHelperVar< OUString,OUStringHash > -PropertyListeners_impl; +typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + PropertyListeners_impl; class fileaccess::PropertyListeners : public PropertyListeners_impl diff --git a/ucbhelper/source/provider/contenthelper.cxx b/ucbhelper/source/provider/contenthelper.cxx index f758b934162a..b99794b5bf8b 100644 --- a/ucbhelper/source/provider/contenthelper.cxx +++ b/ucbhelper/source/provider/contenthelper.cxx @@ -83,11 +83,8 @@ typedef boost::unordered_map > PropertiesEventListenerMap; -typedef cppu::OMultiTypeInterfaceContainerHelperVar -< - OUString, - OUStringHash -> PropertyChangeListeners; +typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + PropertyChangeListeners; struct ContentImplHelper_Impl { diff --git a/ucbhelper/source/provider/resultset.cxx b/ucbhelper/source/provider/resultset.cxx index 8c9fac6bc82a..421b6aa853f2 100644 --- a/ucbhelper/source/provider/resultset.cxx +++ b/ucbhelper/source/provider/resultset.cxx @@ -122,23 +122,8 @@ public: throw( uno::RuntimeException, std::exception ) SAL_OVERRIDE; }; - - -// PropertyChangeListenerContainer. - - - -typedef cppu::OMultiTypeInterfaceContainerHelperVar -< - OUString, - OUStringHash -> PropertyChangeListenerContainer; - - - -// class PropertyChangeListeners. - - +typedef cppu::OMultiTypeInterfaceContainerHelperVar<OUString> + PropertyChangeListenerContainer; class PropertyChangeListeners : public PropertyChangeListenerContainer { |