From a941df0f89ab8222ebd22c697e23fb539a3730ed Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Mon, 8 Jun 2015 16:17:45 +0100 Subject: cppcheck: noExplicitConstructor Change-Id: Ie2ae923ad4c1a66e779711de6ff05328ef144dac --- comphelper/source/compare/AnyCompareFactory.cxx | 2 +- comphelper/source/container/namecontainer.cxx | 2 +- comphelper/source/misc/accessiblewrapper.cxx | 2 +- comphelper/source/misc/asyncnotification.cxx | 2 +- comphelper/source/misc/instancelocker.hxx | 2 +- comphelper/source/misc/officerestartmanager.hxx | 2 +- comphelper/source/misc/syntaxhighlight.cxx | 2 +- comphelper/source/misc/threadpool.cxx | 2 +- comphelper/source/officeinstdir/officeinstallationdirectories.hxx | 2 +- comphelper/source/property/MasterPropertySet.cxx | 2 +- comphelper/source/property/genericpropertyset.cxx | 2 +- comphelper/source/property/propagg.cxx | 2 +- comphelper/source/property/propertycontainerhelper.cxx | 2 +- 13 files changed, 13 insertions(+), 13 deletions(-) (limited to 'comphelper') diff --git a/comphelper/source/compare/AnyCompareFactory.cxx b/comphelper/source/compare/AnyCompareFactory.cxx index d641d6bb1887..b3d58700be3f 100644 --- a/comphelper/source/compare/AnyCompareFactory.cxx +++ b/comphelper/source/compare/AnyCompareFactory.cxx @@ -61,7 +61,7 @@ class AnyCompareFactory : public cppu::WeakImplHelper3< XAnyCompareFactory, XIni Locale m_Locale; public: - AnyCompareFactory( Reference< XComponentContext > xContext ) : m_xContext( xContext ) + explicit AnyCompareFactory( Reference< XComponentContext > xContext ) : m_xContext( xContext ) {} // XAnyCompareFactory diff --git a/comphelper/source/container/namecontainer.cxx b/comphelper/source/container/namecontainer.cxx index 7a273b24e5dd..c296d5373c97 100644 --- a/comphelper/source/container/namecontainer.cxx +++ b/comphelper/source/container/namecontainer.cxx @@ -40,7 +40,7 @@ namespace comphelper class NameContainer : public ::cppu::WeakImplHelper1< ::com::sun::star::container::XNameContainer >, private NameContainerImpl { public: - NameContainer( ::com::sun::star::uno::Type aType ); + explicit NameContainer( ::com::sun::star::uno::Type aType ); virtual ~NameContainer(); // XNameContainer diff --git a/comphelper/source/misc/accessiblewrapper.cxx b/comphelper/source/misc/accessiblewrapper.cxx index a73134a33663..21ebb167fdb0 100644 --- a/comphelper/source/misc/accessiblewrapper.cxx +++ b/comphelper/source/misc/accessiblewrapper.cxx @@ -40,7 +40,7 @@ namespace comphelper Reference< XEventListener > m_xListener; public: - RemoveEventListener( const Reference< XEventListener >& _rxListener ) + explicit RemoveEventListener( const Reference< XEventListener >& _rxListener ) :m_xListener( _rxListener ) { } diff --git a/comphelper/source/misc/asyncnotification.cxx b/comphelper/source/misc/asyncnotification.cxx index 3c15ce6f8f0e..85643e2b3cf4 100644 --- a/comphelper/source/misc/asyncnotification.cxx +++ b/comphelper/source/misc/asyncnotification.cxx @@ -61,7 +61,7 @@ namespace comphelper struct EqualProcessor : public ::std::unary_function< ProcessableEvent, bool > { const ::rtl::Reference< IEventProcessor >& rProcessor; - EqualProcessor( const ::rtl::Reference< IEventProcessor >& _rProcessor ) :rProcessor( _rProcessor ) { } + explicit EqualProcessor( const ::rtl::Reference< IEventProcessor >& _rProcessor ) :rProcessor( _rProcessor ) { } bool operator()( const ProcessableEvent& _rEvent ) { diff --git a/comphelper/source/misc/instancelocker.hxx b/comphelper/source/misc/instancelocker.hxx index 058190827f81..06908eff869a 100644 --- a/comphelper/source/misc/instancelocker.hxx +++ b/comphelper/source/misc/instancelocker.hxx @@ -55,7 +55,7 @@ class OInstanceLocker : public ::cppu::WeakImplHelper3< ::com::sun::star::lang:: bool m_bInitialized; public: - OInstanceLocker( const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& xContext ); + explicit OInstanceLocker( const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& xContext ); virtual ~OInstanceLocker(); static ::com::sun::star::uno::Sequence< OUString > SAL_CALL diff --git a/comphelper/source/misc/officerestartmanager.hxx b/comphelper/source/misc/officerestartmanager.hxx index 72e2238d2d4e..349288571f55 100644 --- a/comphelper/source/misc/officerestartmanager.hxx +++ b/comphelper/source/misc/officerestartmanager.hxx @@ -42,7 +42,7 @@ class OOfficeRestartManager : public ::cppu::WeakImplHelper3< ::com::sun::star:: bool m_bRestartRequested; public: - OOfficeRestartManager( const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& xContext ) + explicit OOfficeRestartManager( const ::com::sun::star::uno::Reference< ::com::sun::star::uno::XComponentContext >& xContext ) : m_xContext( xContext ) , m_bOfficeInitialized( false ) , m_bRestartRequested( false ) diff --git a/comphelper/source/misc/syntaxhighlight.cxx b/comphelper/source/misc/syntaxhighlight.cxx index 41dd9f2a070c..9f0eb7dfe862 100644 --- a/comphelper/source/misc/syntaxhighlight.cxx +++ b/comphelper/source/misc/syntaxhighlight.cxx @@ -273,7 +273,7 @@ class SyntaxHighlighter::Tokenizer public: HighlighterLanguage const aLanguage; - Tokenizer( HighlighterLanguage aLang ); + explicit Tokenizer( HighlighterLanguage aLang ); ~Tokenizer(); void getHighlightPortions(const OUString& rLine, diff --git a/comphelper/source/misc/threadpool.cxx b/comphelper/source/misc/threadpool.cxx index 77e9962c6e9c..a916ba90f7bd 100644 --- a/comphelper/source/misc/threadpool.cxx +++ b/comphelper/source/misc/threadpool.cxx @@ -23,7 +23,7 @@ class ThreadPool::ThreadWorker : public salhelper::Thread bool mbWorking; public: - ThreadWorker( ThreadPool *pPool ) : + explicit ThreadWorker( ThreadPool *pPool ) : salhelper::Thread("thread-pool"), mpPool( pPool ), mbWorking( false ) diff --git a/comphelper/source/officeinstdir/officeinstallationdirectories.hxx b/comphelper/source/officeinstdir/officeinstallationdirectories.hxx index fc9576d8eaaa..0d0ab574f976 100644 --- a/comphelper/source/officeinstdir/officeinstallationdirectories.hxx +++ b/comphelper/source/officeinstdir/officeinstallationdirectories.hxx @@ -43,7 +43,7 @@ struct mutex_holder class OfficeInstallationDirectories : public mutex_holder, public UnoImplBase { public: - OfficeInstallationDirectories( + explicit OfficeInstallationDirectories( const com::sun::star::uno::Reference< com::sun::star::uno::XComponentContext > & xCtx ); virtual ~OfficeInstallationDirectories(); diff --git a/comphelper/source/property/MasterPropertySet.cxx b/comphelper/source/property/MasterPropertySet.cxx index f5e9cba48387..87e4ba42c82f 100644 --- a/comphelper/source/property/MasterPropertySet.cxx +++ b/comphelper/source/property/MasterPropertySet.cxx @@ -33,7 +33,7 @@ class AutoOGuardArray std::unique_ptr< osl::Guard< comphelper::SolarMutex > > * mpGuardArray; public: - AutoOGuardArray( sal_Int32 nNumElements ); + explicit AutoOGuardArray( sal_Int32 nNumElements ); ~AutoOGuardArray(); std::unique_ptr< osl::Guard< comphelper::SolarMutex > > & operator[] ( sal_Int32 i ) { return mpGuardArray[i]; } diff --git a/comphelper/source/property/genericpropertyset.cxx b/comphelper/source/property/genericpropertyset.cxx index 2ddb513a7cc9..cf7af4ecfbbe 100644 --- a/comphelper/source/property/genericpropertyset.cxx +++ b/comphelper/source/property/genericpropertyset.cxx @@ -64,7 +64,7 @@ namespace comphelper virtual void _getPropertyValues( const PropertyMapEntry** ppEntries, Any* pValue ) throw( UnknownPropertyException, WrappedTargetException ) SAL_OVERRIDE; public: - GenericPropertySet( PropertySetInfo* pInfo ) throw(); + explicit GenericPropertySet( PropertySetInfo* pInfo ) throw(); virtual ~GenericPropertySet() throw(); // XInterface diff --git a/comphelper/source/property/propagg.cxx b/comphelper/source/property/propagg.cxx index 7abb03d31e59..9d4930c54e3b 100644 --- a/comphelper/source/property/propagg.cxx +++ b/comphelper/source/property/propagg.cxx @@ -356,7 +356,7 @@ namespace internal sal_Int32 m_nCurrentlyForwarding; public: - PropertyForwarder( OPropertySetAggregationHelper& _rAggregationHelper ); + explicit PropertyForwarder( OPropertySetAggregationHelper& _rAggregationHelper ); ~PropertyForwarder(); /** declares that the forwarder should be responsible for the given property diff --git a/comphelper/source/property/propertycontainerhelper.cxx b/comphelper/source/property/propertycontainerhelper.cxx index 8b8f7a691a5a..1d4a91197391 100644 --- a/comphelper/source/property/propertycontainerhelper.cxx +++ b/comphelper/source/property/propertycontainerhelper.cxx @@ -52,7 +52,7 @@ namespace struct PropertyDescriptionNameMatch : public ::std::unary_function< PropertyDescription, bool > { OUString m_rCompare; - PropertyDescriptionNameMatch( const OUString& _rCompare ) : m_rCompare( _rCompare ) { } + explicit PropertyDescriptionNameMatch( const OUString& _rCompare ) : m_rCompare( _rCompare ) { } bool operator() (const PropertyDescription& x ) const { -- cgit