From b44cbb26efe1d0b0950b1e1613e131b506dc3876 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 20 Jan 2015 12:38:10 +0200 Subject: new loplugin: change virtual methods to non-virtual Where we can prove that the virtual method is never overriden. In the case of pure-virtual methods, we remove the method entirely. Sometimes this leads to entire methods and fields being eliminated. Change-Id: I138ef81c95f115dbd8c023a83cfc7e9d5d6d14ae --- xmloff/inc/txtvfldi.hxx | 4 ++-- xmloff/source/core/DocumentSettingsContext.cxx | 2 +- xmloff/source/core/DomExport.cxx | 20 ++++++++++---------- xmloff/source/forms/elementimport.hxx | 6 +++--- 4 files changed, 16 insertions(+), 16 deletions(-) (limited to 'xmloff') diff --git a/xmloff/inc/txtvfldi.hxx b/xmloff/inc/txtvfldi.hxx index 276dc9b303f6..3bd5b92e7d4a 100644 --- a/xmloff/inc/txtvfldi.hxx +++ b/xmloff/inc/txtvfldi.hxx @@ -87,11 +87,11 @@ public: virtual ~XMLValueImportHelper(); /// process attribute values - virtual void ProcessAttribute( sal_uInt16 nAttrToken, + void ProcessAttribute( sal_uInt16 nAttrToken, const OUString& sAttrValue ); /// prepare XTextField for insertion into document - virtual void PrepareField( + void PrepareField( const ::com::sun::star::uno::Reference< ::com::sun::star::beans::XPropertySet> & xPropertySet); diff --git a/xmloff/source/core/DocumentSettingsContext.cxx b/xmloff/source/core/DocumentSettingsContext.cxx index 5e3fc42739e2..15e94ec5b4cb 100644 --- a/xmloff/source/core/DocumentSettingsContext.cxx +++ b/xmloff/source/core/DocumentSettingsContext.cxx @@ -162,7 +162,7 @@ public: virtual void EndElement() SAL_OVERRIDE; - virtual void ManipulateConfigItem(); + void ManipulateConfigItem(); }; class XMLConfigItemSetContext : public XMLConfigBaseContext diff --git a/xmloff/source/core/DomExport.cxx b/xmloff/source/core/DomExport.cxx index 87cc9393952e..f25e7c9fae76 100644 --- a/xmloff/source/core/DomExport.cxx +++ b/xmloff/source/core/DomExport.cxx @@ -63,16 +63,16 @@ public: virtual ~DomVisitor() {} virtual void element( const Reference& ) {} virtual void character( const Reference& ) {} - virtual void attribute( const Reference& ) {} - virtual void cdata( const Reference& ) {} - virtual void comment( const Reference& ) {} - virtual void documentFragment( const Reference& ) {} - virtual void document( const Reference& ) {} - virtual void documentType( const Reference& ) {} - virtual void entity( const Reference& ) {} - virtual void entityReference( const Reference& ) {} - virtual void notation( const Reference& ) {} - virtual void processingInstruction( const Reference& ) {} + void attribute( const Reference& ) {} + void cdata( const Reference& ) {} + void comment( const Reference& ) {} + void documentFragment( const Reference& ) {} + void document( const Reference& ) {} + void documentType( const Reference& ) {} + void entity( const Reference& ) {} + void entityReference( const Reference& ) {} + void notation( const Reference& ) {} + void processingInstruction( const Reference& ) {} virtual void endElement( const Reference& ) {} }; diff --git a/xmloff/source/forms/elementimport.hxx b/xmloff/source/forms/elementimport.hxx index b9f0c5466353..4d2592aadd8c 100644 --- a/xmloff/source/forms/elementimport.hxx +++ b/xmloff/source/forms/elementimport.hxx @@ -257,13 +257,13 @@ namespace xmloff virtual void doRegisterCellValueBinding( const OUString& _rBoundCellAddress ); /** register the given XForms binding */ - virtual void doRegisterXFormsValueBinding( const OUString& ); + void doRegisterXFormsValueBinding( const OUString& ); /** register the given XForms list binding */ - virtual void doRegisterXFormsListBinding( const OUString& ); + void doRegisterXFormsListBinding( const OUString& ); /** register the given XForms submission */ - virtual void doRegisterXFormsSubmission( const OUString& ); + void doRegisterXFormsSubmission( const OUString& ); protected: -- cgit v1.2.3