diff options
author | Caolán McNamara <caolanm@redhat.com> | 2015-05-27 10:53:55 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2015-05-27 13:54:54 +0100 |
commit | afe5672fbc87ae2bc7fb72fac4bb5b837b646dea (patch) | |
tree | 1e78a6dcce4c7313ab4d9109b53077e687909606 /svx/workben | |
parent | 77c47fc79c1c8096c19a81a0d320e75b84857144 (diff) |
cppcheck: noExplicitConstructor
Change-Id: I27c24d3284a8e0678fc5c041426b4a7e71cbd363
Diffstat (limited to 'svx/workben')
-rw-r--r-- | svx/workben/msview/msview.cxx | 9 | ||||
-rw-r--r-- | svx/workben/msview/xmlconfig.hxx | 16 |
2 files changed, 8 insertions, 17 deletions
diff --git a/svx/workben/msview/msview.cxx b/svx/workben/msview/msview.cxx index b03b14b1d585..aff2e4e12394 100644 --- a/svx/workben/msview/msview.cxx +++ b/svx/workben/msview/msview.cxx @@ -503,13 +503,10 @@ public: } }; - - - class AtomContainerTreeListBox : public SvTreeListBox { public: - AtomContainerTreeListBox( vcl::Window* pParent ); + explicit AtomContainerTreeListBox( vcl::Window* pParent ); ~AtomContainerTreeListBox(); void SetRootAtom( const Atom* pAtom ); @@ -705,14 +702,12 @@ void AtomContainerTreeListBox::InsertAtom( const Atom* pAtom, SvTreeListEntry* p } } - - extern void load_config( const OUString& rPath ); class PPTDocument { public: - PPTDocument( const OUString& rFilePath ); + explicit PPTDocument( const OUString& rFilePath ); ~PPTDocument(); Atom* getRootAtom() const; diff --git a/svx/workben/msview/xmlconfig.hxx b/svx/workben/msview/xmlconfig.hxx index b9696023cf28..cf563f70fc64 100644 --- a/svx/workben/msview/xmlconfig.hxx +++ b/svx/workben/msview/xmlconfig.hxx @@ -33,8 +33,8 @@ class ElementConfig public: ElementConfig() : mnType( ECT_HEXDUMP ) {} ElementConfig( const OUString& rName, ElementConfigType rType ) : maName( rName ), mnType( rType ) {} - ElementConfig( const OUString& rName ) : maName( rName ) {} - ElementConfig( ElementConfigType rType ) : mnType( rType ) {} + explicit ElementConfig( const OUString& rName ) : maName( rName ) {} + explicit ElementConfig( ElementConfigType rType ) : mnType( rType ) {} virtual OUString format( SvStream& rStream, sal_Size& nLength ) const; @@ -73,8 +73,8 @@ class ElementConfigContainer : public ElementConfig public: ElementConfigContainer() : ElementConfig( ETC_CONTAINER ) {} ElementConfigContainer( const OUString& rName, ElementConfigType rType ) : ElementConfig( rName, rType ) {} - ElementConfigContainer( const OUString& rName ) : ElementConfig( rName, ETC_CONTAINER ) {} - ElementConfigContainer( ElementConfigType rType ) : ElementConfig( rType ) {} + explicit ElementConfigContainer( const OUString& rName ) : ElementConfig( rName, ETC_CONTAINER ) {} + explicit ElementConfigContainer( ElementConfigType rType ) : ElementConfig( rType ) {} virtual OUString format( SvStream& rStream, sal_Size& nLength ) const; @@ -89,7 +89,7 @@ protected: class CaseElementConfig : public ElementConfigContainer { public: - CaseElementConfig( const OUString& rValue ) : maValue( rValue ) {} + explicit CaseElementConfig( const OUString& rValue ) : maValue( rValue ) {} const OUString& getValue() const { return maValue; } @@ -97,18 +97,14 @@ private: OUString maValue; }; - - class SwitchElementConfig : public ElementConfigContainer { public: - SwitchElementConfig( ElementConfigType rType ) : ElementConfigContainer( rType ) {} + explicit SwitchElementConfig( ElementConfigType rType ) : ElementConfigContainer( rType ) {} virtual OUString format( SvStream& rStream, sal_Size& nLength ) const; }; - - class AtomConfig : public ElementConfigContainer { public: |