diff options
author | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2016-11-01 11:44:15 +0100 |
---|---|---|
committer | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2016-11-01 18:44:20 +0000 |
commit | 8826771e8398063018935c38039c05faf4c993a3 (patch) | |
tree | f726abbbc99a3a1c39c726a47f5f121573ab1c8b | |
parent | 33bc51a60523f23a860534a39a096fecce192671 (diff) |
style fixes for cppcheck noExplicitConstructor
Change-Id: I2a690caea7656f2a18beb6d09f53154178a30f34
Reviewed-on: https://gerrit.libreoffice.org/30460
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
Tested-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
28 files changed, 35 insertions, 35 deletions
diff --git a/cui/source/options/connpoolsettings.hxx b/cui/source/options/connpoolsettings.hxx index 07b607730953..7ef6bc7f61df 100644 --- a/cui/source/options/connpoolsettings.hxx +++ b/cui/source/options/connpoolsettings.hxx @@ -37,7 +37,7 @@ namespace offapp bool bEnabled; sal_Int32 nTimeoutSeconds; - DriverPooling( const OUString& _rName ); + explicit DriverPooling( const OUString& _rName ); bool operator == (const DriverPooling& _rR) const; bool operator != (const DriverPooling& _rR) const { return !operator ==(_rR); } diff --git a/dbaccess/source/ui/tabledesign/TEditControl.hxx b/dbaccess/source/ui/tabledesign/TEditControl.hxx index 19efd28419eb..0c6a44d6e73d 100644 --- a/dbaccess/source/ui/tabledesign/TEditControl.hxx +++ b/dbaccess/source/ui/tabledesign/TEditControl.hxx @@ -73,7 +73,7 @@ namespace dbaui VclPtr<OTableEditorCtrl> m_pOwner; public: - ClipboardInvalidator(OTableEditorCtrl*); + explicit ClipboardInvalidator(OTableEditorCtrl*); ~ClipboardInvalidator(); void Stop(); diff --git a/extensions/source/bibliography/bibcont.hxx b/extensions/source/bibliography/bibcont.hxx index f8547538d222..028cfff8fc11 100644 --- a/extensions/source/bibliography/bibcont.hxx +++ b/extensions/source/bibliography/bibcont.hxx @@ -82,7 +82,7 @@ class BibBookContainer: public BibSplitWindow public: - BibBookContainer(vcl::Window* pParent ); + explicit BibBookContainer(vcl::Window* pParent ); virtual ~BibBookContainer() override; virtual void dispose() override; diff --git a/extensions/source/propctrlr/browserlistbox.hxx b/extensions/source/propctrlr/browserlistbox.hxx index 07f2d4869d1e..21710f213e45 100644 --- a/extensions/source/propctrlr/browserlistbox.hxx +++ b/extensions/source/propctrlr/browserlistbox.hxx @@ -102,7 +102,7 @@ namespace pcr void Resize() override; public: - OBrowserListBox( vcl::Window* pParent ); + explicit OBrowserListBox( vcl::Window* pParent ); virtual ~OBrowserListBox() override; virtual void dispose() override; diff --git a/extensions/source/propctrlr/propertyeditor.hxx b/extensions/source/propctrlr/propertyeditor.hxx index 03be07a963b7..a40c312b9893 100644 --- a/extensions/source/propctrlr/propertyeditor.hxx +++ b/extensions/source/propctrlr/propertyeditor.hxx @@ -69,7 +69,7 @@ namespace pcr void GetFocus() override; public: - OPropertyEditor (vcl::Window* pParent); + explicit OPropertyEditor (vcl::Window* pParent); virtual ~OPropertyEditor() override; virtual void dispose() override; diff --git a/extensions/source/propctrlr/usercontrol.hxx b/extensions/source/propctrlr/usercontrol.hxx index d45876434c68..4a5719ac2d0e 100644 --- a/extensions/source/propctrlr/usercontrol.hxx +++ b/extensions/source/propctrlr/usercontrol.hxx @@ -59,7 +59,7 @@ namespace pcr class OFormatSampleControl : public OFormatSampleControl_Base { public: - OFormatSampleControl( vcl::Window* pParent ); + explicit OFormatSampleControl( vcl::Window* pParent ); // XPropertyControl virtual css::uno::Any SAL_CALL getValue() throw (css::uno::RuntimeException, std::exception) override; @@ -128,7 +128,7 @@ namespace pcr class OFileUrlControl : public OFileUrlControl_Base { public: - OFileUrlControl( vcl::Window* pParent ); + explicit OFileUrlControl( vcl::Window* pParent ); // XPropertyControl virtual css::uno::Any SAL_CALL getValue() throw (css::uno::RuntimeException, std::exception) override; @@ -145,7 +145,7 @@ namespace pcr class OTimeDurationControl : public ONumericControl { public: - OTimeDurationControl( vcl::Window* pParent ); + explicit OTimeDurationControl( vcl::Window* pParent ); virtual ~OTimeDurationControl() override; // XPropertyControl diff --git a/fpicker/source/win32/filepicker/comptr.hxx b/fpicker/source/win32/filepicker/comptr.hxx index 5cc4f4d3f907..89fc85b34787 100644 --- a/fpicker/source/win32/filepicker/comptr.hxx +++ b/fpicker/source/win32/filepicker/comptr.hxx @@ -42,7 +42,7 @@ class ComPtr /** initialize com ptr with given interface. */ - ComPtr(T_INTERFACE* pInterface) + explicit ComPtr(T_INTERFACE* pInterface) { m_pInterface = pInterface; if (m_pInterface) @@ -62,7 +62,7 @@ class ComPtr /** initialize object by quering external object for the right interface. */ - ComPtr(IUnknown* pIUnknown) + explicit ComPtr(IUnknown* pIUnknown) { if (pIUnknown) pIUnknown->QueryInterface(P_IID, (void**)&m_pInterface); diff --git a/lotuswordpro/source/filter/explode.hxx b/lotuswordpro/source/filter/explode.hxx index 27fc4277da88..013bb6bc881c 100644 --- a/lotuswordpro/source/filter/explode.hxx +++ b/lotuswordpro/source/filter/explode.hxx @@ -67,7 +67,7 @@ public: HuffmanTreeNode * right; sal_uInt32 value; - HuffmanTreeNode(sal_uInt32 value = 0xffffffff) ; + explicit HuffmanTreeNode(sal_uInt32 value = 0xffffffff) ; ~HuffmanTreeNode() ; HuffmanTreeNode * InsertNode(sal_uInt32 nValue, const sal_Char * pInCode); HuffmanTreeNode * QueryNode(const sal_Char *pCode); diff --git a/package/source/xstor/ohierarchyholder.hxx b/package/source/xstor/ohierarchyholder.hxx index e8ae40d7216b..4887d49438cc 100644 --- a/package/source/xstor/ohierarchyholder.hxx +++ b/package/source/xstor/ohierarchyholder.hxx @@ -65,7 +65,7 @@ struct OHierarchyElement_Impl : public cppu::WeakImplHelper< css::embed::XTransa OWeakStorRefList_Impl m_aOpenStreams; public: - OHierarchyElement_Impl( const css::uno::Reference< css::embed::XStorage >& xStorage ) + explicit OHierarchyElement_Impl( const css::uno::Reference< css::embed::XStorage >& xStorage ) : m_rParent( nullptr ) , m_xOwnStorage( xStorage ) {} diff --git a/sc/source/core/data/documentimport.cxx b/sc/source/core/data/documentimport.cxx index d8ebcfed0a70..69f66cd4b080 100644 --- a/sc/source/core/data/documentimport.cxx +++ b/sc/source/core/data/documentimport.cxx @@ -500,7 +500,7 @@ class CellStoreInitializer sc::CellTextAttrStoreType::iterator miPos; SvtScriptType mnScriptNumeric; - Impl(const SvtScriptType nScriptNumeric) + explicit Impl(const SvtScriptType nScriptNumeric) : maAttrs(MAXROWCOUNT), miPos(maAttrs.begin()), mnScriptNumeric(nScriptNumeric) {} }; diff --git a/sc/source/core/tool/bulkdatahint.cxx b/sc/source/core/tool/bulkdatahint.cxx index bdd02152105d..506942a23faa 100644 --- a/sc/source/core/tool/bulkdatahint.cxx +++ b/sc/source/core/tool/bulkdatahint.cxx @@ -16,7 +16,7 @@ struct BulkDataHint::Impl ScDocument& mrDoc; const ColumnSpanSet* mpSpans; - Impl( ScDocument& rDoc ) : + explicit Impl( ScDocument& rDoc ) : mrDoc(rDoc), mpSpans(nullptr) {} }; diff --git a/sc/source/filter/xml/XMLTableShapeImportHelper.hxx b/sc/source/filter/xml/XMLTableShapeImportHelper.hxx index ceadd12b65b1..1bb7d70cccb1 100644 --- a/sc/source/filter/xml/XMLTableShapeImportHelper.hxx +++ b/sc/source/filter/xml/XMLTableShapeImportHelper.hxx @@ -34,7 +34,7 @@ class XMLTableShapeImportHelper : public XMLShapeImportHelper public: - XMLTableShapeImportHelper( ScXMLImport& rImp ); + explicit XMLTableShapeImportHelper( ScXMLImport& rImp ); virtual ~XMLTableShapeImportHelper() override; static void SetLayer(css::uno::Reference<css::drawing::XShape>& rShape, sal_Int16 nLayerID, const OUString& sType); diff --git a/sc/source/ui/sidebar/CellBorderStyleControl.hxx b/sc/source/ui/sidebar/CellBorderStyleControl.hxx index 28c5c16d0654..a288f1a3eaca 100644 --- a/sc/source/ui/sidebar/CellBorderStyleControl.hxx +++ b/sc/source/ui/sidebar/CellBorderStyleControl.hxx @@ -43,7 +43,7 @@ private: DECL_LINK(TB3SelectHdl, ToolBox *, void); public: - CellBorderStylePopup(SfxDispatcher* pDispatcher); + explicit CellBorderStylePopup(SfxDispatcher* pDispatcher); virtual ~CellBorderStylePopup() override; virtual void dispose() override; }; diff --git a/sc/source/ui/sidebar/CellLineStyleControl.hxx b/sc/source/ui/sidebar/CellLineStyleControl.hxx index ea5f090d1c02..399ff9a260be 100644 --- a/sc/source/ui/sidebar/CellLineStyleControl.hxx +++ b/sc/source/ui/sidebar/CellLineStyleControl.hxx @@ -44,7 +44,7 @@ class CellLineStylePopup : public FloatingWindow public: - CellLineStylePopup(SfxDispatcher* pDispatcher); + explicit CellLineStylePopup(SfxDispatcher* pDispatcher); void SetLineStyleSelect(sal_uInt16 out, sal_uInt16 in, sal_uInt16 dis); virtual ~CellLineStylePopup() override; virtual void dispose() override; diff --git a/sc/source/ui/sidebar/CellLineStyleValueSet.hxx b/sc/source/ui/sidebar/CellLineStyleValueSet.hxx index 3e87b310dda7..edc3c8a15ab1 100644 --- a/sc/source/ui/sidebar/CellLineStyleValueSet.hxx +++ b/sc/source/ui/sidebar/CellLineStyleValueSet.hxx @@ -33,7 +33,7 @@ private: sal_uInt16 nSelItem; OUString maStrUnit[CELL_LINE_STYLE_ENTRIES]; public: - CellLineStyleValueSet(vcl::Window* pParent); + explicit CellLineStyleValueSet(vcl::Window* pParent); virtual ~CellLineStyleValueSet() override; virtual void dispose() override; diff --git a/sd/source/ui/inc/optsitem.hxx b/sd/source/ui/inc/optsitem.hxx index cde1ed3eae17..200415af9e06 100644 --- a/sd/source/ui/inc/optsitem.hxx +++ b/sd/source/ui/inc/optsitem.hxx @@ -469,7 +469,7 @@ class SdOptionsGridItem : public SvxGridItem { public: - SdOptionsGridItem( SdOptions* pOpts ); + explicit SdOptionsGridItem( SdOptions* pOpts ); void SetOptions( SdOptions* pOpts ) const; }; @@ -563,7 +563,7 @@ class SD_DLLPUBLIC SdOptionsPrintItem : public SfxPoolItem public: explicit SdOptionsPrintItem(); - SdOptionsPrintItem( SdOptions* pOpts ); + explicit SdOptionsPrintItem( SdOptions* pOpts ); virtual SfxPoolItem* Clone( SfxItemPool *pPool = nullptr ) const override; virtual bool operator==( const SfxPoolItem& ) const override; diff --git a/sd/source/ui/unoidl/unocpres.hxx b/sd/source/ui/unoidl/unocpres.hxx index 493c5ddccac7..c04ebb7f6a62 100644 --- a/sd/source/ui/unoidl/unocpres.hxx +++ b/sd/source/ui/unoidl/unocpres.hxx @@ -50,7 +50,7 @@ private: public: SdXCustomPresentation() throw(); - SdXCustomPresentation( SdCustomShow* mpSdCustomShow) throw(); + explicit SdXCustomPresentation( SdCustomShow* mpSdCustomShow ) throw(); virtual ~SdXCustomPresentation() throw() override; // internal diff --git a/sd/source/ui/view/ViewShellManager.cxx b/sd/source/ui/view/ViewShellManager.cxx index a95e27d872a8..1d11b149f705 100644 --- a/sd/source/ui/view/ViewShellManager.cxx +++ b/sd/source/ui/view/ViewShellManager.cxx @@ -55,7 +55,7 @@ public: bool mbIsListenerAddedToWindow; ShellDescriptor (); - ShellDescriptor (ShellId nId); + explicit ShellDescriptor (ShellId nId); ShellDescriptor (const ShellDescriptor& rDescriptor); ShellDescriptor& operator= (const ShellDescriptor& rDescriptor); bool IsMainViewShell() const; diff --git a/sdext/source/minimizer/configurationaccess.hxx b/sdext/source/minimizer/configurationaccess.hxx index 72bc1034ba1e..3a23d22fd199 100644 --- a/sdext/source/minimizer/configurationaccess.hxx +++ b/sdext/source/minimizer/configurationaccess.hxx @@ -80,7 +80,7 @@ class ConfigurationAccess { public: - ConfigurationAccess( const css::uno::Reference< css::uno::XComponentContext >& rXFactory ); + explicit ConfigurationAccess( const css::uno::Reference< css::uno::XComponentContext >& rXFactory ); ~ConfigurationAccess(); void SaveConfiguration(); diff --git a/sfx2/source/dialog/dockwin.cxx b/sfx2/source/dialog/dockwin.cxx index 183600a4075d..76c41d2ac048 100644 --- a/sfx2/source/dialog/dockwin.cxx +++ b/sfx2/source/dialog/dockwin.cxx @@ -406,7 +406,7 @@ friend class SfxDockingWindow; bool bDockingPrevented; OString aWinState; - SfxDockingWindow_Impl(SfxDockingWindow *pBase); + explicit SfxDockingWindow_Impl(SfxDockingWindow *pBase); SfxChildAlignment GetLastAlignment() const { return eLastAlignment; } void SetLastAlignment(SfxChildAlignment eAlign) diff --git a/sfx2/source/notebookbar/BigToolBox.hxx b/sfx2/source/notebookbar/BigToolBox.hxx index 4e319437c7cf..69d237508459 100644 --- a/sfx2/source/notebookbar/BigToolBox.hxx +++ b/sfx2/source/notebookbar/BigToolBox.hxx @@ -37,7 +37,7 @@ namespace sfx2 { namespace notebookbar { class SFX2_DLLPUBLIC BigToolBox : public sfx2::sidebar::SidebarToolBox { public: - BigToolBox(vcl::Window* pParent); + explicit BigToolBox(vcl::Window* pParent); }; diff --git a/sfx2/source/notebookbar/NotebookbarToolBox.cxx b/sfx2/source/notebookbar/NotebookbarToolBox.cxx index 08223436e064..99dc666c8bb5 100644 --- a/sfx2/source/notebookbar/NotebookbarToolBox.cxx +++ b/sfx2/source/notebookbar/NotebookbarToolBox.cxx @@ -27,7 +27,7 @@ class NotebookbarToolBox : public sfx2::sidebar::SidebarToolBox { public: - NotebookbarToolBox(vcl::Window* pParentWindow) + explicit NotebookbarToolBox(vcl::Window* pParentWindow) : sfx2::sidebar::SidebarToolBox(pParentWindow) { ToolBoxButtonSize eSize = ToolBoxButtonSize::Small; diff --git a/shell/source/unix/sysshell/recently_used_file_handler.cxx b/shell/source/unix/sysshell/recently_used_file_handler.cxx index d2393bf65dfc..0d2aead07a79 100644 --- a/shell/source/unix/sysshell/recently_used_file_handler.cxx +++ b/shell/source/unix/sysshell/recently_used_file_handler.cxx @@ -305,7 +305,7 @@ namespace /* private */ { class recent_item_writer { public: - recent_item_writer( recently_used_file& file ) : + explicit recent_item_writer( recently_used_file& file ) : file_(file), items_written_(0) {} diff --git a/svtools/source/control/scriptedtext.cxx b/svtools/source/control/scriptedtext.cxx index c6a709d4c3dd..c0e26ab11230 100644 --- a/svtools/source/control/scriptedtext.cxx +++ b/svtools/source/control/scriptedtext.cxx @@ -64,7 +64,7 @@ private: public: /** This constructor sets an output device and fonts for all script types. */ - SvtScriptedTextHelper_Impl( + explicit SvtScriptedTextHelper_Impl( OutputDevice& _rOutDevice ); /** Copy constructor. */ SvtScriptedTextHelper_Impl( diff --git a/svtools/source/control/toolbarmenuimp.hxx b/svtools/source/control/toolbarmenuimp.hxx index af4ac490053e..7ad29acf1b1e 100644 --- a/svtools/source/control/toolbarmenuimp.hxx +++ b/svtools/source/control/toolbarmenuimp.hxx @@ -254,7 +254,7 @@ struct ToolbarMenu_Impl Link<ToolbarMenu*,void> maSelectHdl; - ToolbarMenu_Impl( ToolbarMenu& rMenu ); + explicit ToolbarMenu_Impl( ToolbarMenu& rMenu ); ~ToolbarMenu_Impl(); void setAccessible( ToolbarMenuAcc* pAccessible ); diff --git a/sw/source/filter/xml/xmlimpit.hxx b/sw/source/filter/xml/xmlimpit.hxx index d0be90e33cf2..91bbfaab18f6 100644 --- a/sw/source/filter/xml/xmlimpit.hxx +++ b/sw/source/filter/xml/xmlimpit.hxx @@ -38,7 +38,7 @@ protected: sal_uInt16 nUnknownWhich; public: - SvXMLImportItemMapper( SvXMLItemMapEntriesRef const & rMapEntries ); + explicit SvXMLImportItemMapper( SvXMLItemMapEntriesRef const & rMapEntries ); virtual ~SvXMLImportItemMapper(); /** fills the given itemset with the attributes in the given list */ diff --git a/sw/source/uibase/sidebar/PageMarginControl.hxx b/sw/source/uibase/sidebar/PageMarginControl.hxx index 050495db0706..6482097fc45b 100644 --- a/sw/source/uibase/sidebar/PageMarginControl.hxx +++ b/sw/source/uibase/sidebar/PageMarginControl.hxx @@ -44,7 +44,7 @@ namespace sw { namespace sidebar { class PageMarginControl : public SfxPopupWindow { public: - PageMarginControl( sal_uInt16 nId ); + explicit PageMarginControl( sal_uInt16 nId ); virtual ~PageMarginControl() override; virtual void dispose() override; diff --git a/xmlsecurity/source/pdfio/pdfdocument.cxx b/xmlsecurity/source/pdfio/pdfdocument.cxx index 2125749863df..b690b5d62f77 100644 --- a/xmlsecurity/source/pdfio/pdfdocument.cxx +++ b/xmlsecurity/source/pdfio/pdfdocument.cxx @@ -54,7 +54,7 @@ class PDFCommentElement : public PDFElement OString m_aComment; public: - PDFCommentElement(PDFDocument& rDoc); + explicit PDFCommentElement(PDFDocument& rDoc); bool Read(SvStream& rStream) override; }; @@ -187,7 +187,7 @@ class PDFStreamElement : public PDFElement sal_uInt64 m_nOffset; public: - PDFStreamElement(size_t nLength); + explicit PDFStreamElement(size_t nLength); bool Read(SvStream& rStream) override; sal_uInt64 GetOffset() const; }; @@ -234,7 +234,7 @@ public: class PDFBooleanElement : public PDFElement { public: - PDFBooleanElement(bool bValue); + explicit PDFBooleanElement(bool bValue); bool Read(SvStream& rStream) override; }; @@ -270,7 +270,7 @@ class PDFTrailerElement : public PDFElement std::map<OString, PDFElement*> m_aDictionary; public: - PDFTrailerElement(PDFDocument& rDoc); + explicit PDFTrailerElement(PDFDocument& rDoc); bool Read(SvStream& rStream) override; PDFElement* Lookup(const OString& rDictionaryKey); }; |