connectivity: various small cleanups in the Writer driver

Change-Id: I91735c3e92262fd37966c53f973d5eee1522cd61
Reviewed-on: https://gerrit.libreoffice.org/47741
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>

6 files changed, 28 insertions, 27 deletions

diff --git a/connectivity/source/inc/writer/WCatalog.hxx b/connectivity/source/inc/writer/WCatalog.hxx
index 6a12d2424b64..53ac36a5d838 100644
--- a/connectivity/source/inc/writer/WCatalog.hxx
+++ b/connectivity/source/inc/writer/WCatalog.hxx
@@ -34,7 +34,7 @@ public:
     void refreshTables() override;
 
 public:
-    OWriterCatalog(OWriterConnection* _pCon);
+    OWriterCatalog(OWriterConnection* pConnection);
 };
 
 } // namespace writer
diff --git a/connectivity/source/inc/writer/WConnection.hxx b/connectivity/source/inc/writer/WConnection.hxx
index 69e59409f7ef..99d28ddee45b 100644
--- a/connectivity/source/inc/writer/WConnection.hxx
+++ b/connectivity/source/inc/writer/WConnection.hxx
@@ -26,6 +26,7 @@
 #include <com/sun/star/frame/XTerminateListener.hpp>
 #include <com/sun/star/uno/DeploymentException.hpp>
 #include <unotools/closeveto.hxx>
+#include <o3tl/make_unique.hxx>
 
 namespace com
 {
@@ -80,7 +81,7 @@ class OWriterConnection : public file::OConnection
         {
             m_xDesktop = rDesktop;
             m_xDesktop->addTerminateListener(this);
-            m_pCloseListener.reset(new utl::CloseVeto(rCloseable, true));
+            m_pCloseListener = o3tl::make_unique<utl::CloseVeto>(rCloseable, true);
         }
 
         void stop()
@@ -93,22 +94,22 @@ class OWriterConnection : public file::OConnection
         }
 
         // XTerminateListener
-        virtual void SAL_CALL queryTermination(const css::lang::EventObject& /*rEvent*/) override
+        void SAL_CALL queryTermination(const css::lang::EventObject& /*rEvent*/) override
         {
         }
 
-        virtual void SAL_CALL notifyTermination(const css::lang::EventObject& /*rEvent*/) override
+        void SAL_CALL notifyTermination(const css::lang::EventObject& /*rEvent*/) override
         {
             stop();
         }
 
-        virtual void SAL_CALL disposing() override
+        void SAL_CALL disposing() override
         {
             stop();
             cppu::WeakComponentImplHelperBase::disposing();
         }
 
-        virtual void SAL_CALL disposing(const css::lang::EventObject& rEvent) override
+        void SAL_CALL disposing(const css::lang::EventObject& rEvent) override
         {
             const bool bShutDown = (rEvent.Source == m_xDesktop);
             if (bShutDown)
@@ -120,23 +121,23 @@ class OWriterConnection : public file::OConnection
 
 public:
     OWriterConnection(ODriver* _pDriver);
-    virtual ~OWriterConnection() override;
+    ~OWriterConnection() override;
 
-    virtual void construct(const OUString& _rUrl,
-                           const css::uno::Sequence< css::beans::PropertyValue >& _rInfo) override;
+    void construct(const OUString& rURL,
+                   const css::uno::Sequence< css::beans::PropertyValue >& rInfo) override;
 
     // XServiceInfo
     DECLARE_SERVICE_INFO();
 
     // OComponentHelper
-    virtual void SAL_CALL disposing() override;
+    void SAL_CALL disposing() override;
 
     // XConnection
-    virtual css::uno::Reference< css::sdbc::XDatabaseMetaData > SAL_CALL getMetaData() override;
-    virtual css::uno::Reference< css::sdbcx::XTablesSupplier > createCatalog() override;
-    virtual css::uno::Reference< css::sdbc::XStatement > SAL_CALL createStatement() override;
-    virtual css::uno::Reference< css::sdbc::XPreparedStatement > SAL_CALL prepareStatement(const OUString& sql) override;
-    virtual css::uno::Reference< css::sdbc::XPreparedStatement > SAL_CALL prepareCall(const OUString& sql) override;
+    css::uno::Reference< css::sdbc::XDatabaseMetaData > SAL_CALL getMetaData() override;
+    css::uno::Reference< css::sdbcx::XTablesSupplier > createCatalog() override;
+    css::uno::Reference< css::sdbc::XStatement > SAL_CALL createStatement() override;
+    css::uno::Reference< css::sdbc::XPreparedStatement > SAL_CALL prepareStatement(const OUString& sql) override;
+    css::uno::Reference< css::sdbc::XPreparedStatement > SAL_CALL prepareCall(const OUString& sql) override;
 
     // no interface methods
     css::uno::Reference< css::text::XTextDocument> const& acquireDoc();
diff --git a/connectivity/source/inc/writer/WDatabaseMetaData.hxx b/connectivity/source/inc/writer/WDatabaseMetaData.hxx
index 7d0bad7deb60..46f47b328e5e 100644
--- a/connectivity/source/inc/writer/WDatabaseMetaData.hxx
+++ b/connectivity/source/inc/writer/WDatabaseMetaData.hxx
@@ -29,10 +29,10 @@ namespace writer
 
 class OWriterDatabaseMetaData : public component::OComponentDatabaseMetaData
 {
-    virtual OUString SAL_CALL getURL() override;
-    virtual css::uno::Reference< css::sdbc::XResultSet > SAL_CALL getTables(const css::uno::Any& catalog, const OUString& schemaPattern, const OUString& tableNamePattern, const css::uno::Sequence< OUString >& types) override;
+    OUString SAL_CALL getURL() override;
+    css::uno::Reference< css::sdbc::XResultSet > SAL_CALL getTables(const css::uno::Any& catalog, const OUString& schemaPattern, const OUString& tableNamePattern, const css::uno::Sequence< OUString >& types) override;
 protected:
-    virtual ~OWriterDatabaseMetaData() override;
+    ~OWriterDatabaseMetaData() override;
 public:
     OWriterDatabaseMetaData(file::OConnection* pConnection);
 };
diff --git a/connectivity/source/inc/writer/WDriver.hxx b/connectivity/source/inc/writer/WDriver.hxx
index 614c8321d1a0..ea457290b74c 100644
--- a/connectivity/source/inc/writer/WDriver.hxx
+++ b/connectivity/source/inc/writer/WDriver.hxx
@@ -44,11 +44,11 @@ public:
     OUString SAL_CALL getImplementationName() override;
 
     // XDriver
-    virtual css::uno::Reference< css::sdbc::XConnection > SAL_CALL
+    css::uno::Reference< css::sdbc::XConnection > SAL_CALL
     connect(const OUString& url, const css::uno::Sequence<
             css::beans::PropertyValue >& info) override;
-    virtual sal_Bool SAL_CALL acceptsURL(const OUString& url) override;
-    virtual css::uno::Sequence< css::sdbc::DriverPropertyInfo > SAL_CALL getPropertyInfo(const OUString& url, const css::uno::Sequence< css::beans::PropertyValue >& info) override;
+    sal_Bool SAL_CALL acceptsURL(const OUString& url) override;
+    css::uno::Sequence< css::sdbc::DriverPropertyInfo > SAL_CALL getPropertyInfo(const OUString& url, const css::uno::Sequence< css::beans::PropertyValue >& info) override;
 };
 }
 }
diff --git a/connectivity/source/inc/writer/WTable.hxx b/connectivity/source/inc/writer/WTable.hxx
index aed3e5a8ca4c..b11bc1a27541 100644
--- a/connectivity/source/inc/writer/WTable.hxx
+++ b/connectivity/source/inc/writer/WTable.hxx
@@ -56,7 +56,7 @@ namespace connectivity
 {
 namespace writer
 {
-typedef component::OComponentTable OWriterTable_BASE;
+using OWriterTable_BASE = component::OComponentTable;
 class OWriterConnection;
 
 class OWriterTable :  public OWriterTable_BASE
@@ -80,12 +80,12 @@ public:
                  const OUString& Type
                 );
 
-    virtual bool fetchRow(OValueRefRow& _rRow, const OSQLColumns& _rCols, bool bRetrieveData) override;
+    bool fetchRow(OValueRefRow& _rRow, const OSQLColumns& _rCols, bool bRetrieveData) override;
 
-    virtual void SAL_CALL disposing() override;
+    void SAL_CALL disposing() override;
 
     // css::lang::XUnoTunnel
-    virtual sal_Int64 SAL_CALL getSomething(const css::uno::Sequence< sal_Int8 >& aIdentifier) override;
+    sal_Int64 SAL_CALL getSomething(const css::uno::Sequence< sal_Int8 >& rId) override;
     static css::uno::Sequence< sal_Int8 > getUnoTunnelImplementationId();
 
     void construct() override;
diff --git a/connectivity/source/inc/writer/WTables.hxx b/connectivity/source/inc/writer/WTables.hxx
index 3112623e5046..939178a90bd2 100644
--- a/connectivity/source/inc/writer/WTables.hxx
+++ b/connectivity/source/inc/writer/WTables.hxx
@@ -26,12 +26,12 @@ namespace connectivity
 {
 namespace writer
 {
-typedef file::OTables OWriterTables_BASE;
+using OWriterTables_BASE = file::OTables;
 
 class OWriterTables : public OWriterTables_BASE
 {
 protected:
-    virtual sdbcx::ObjectType createObject(const OUString& _rName) override;
+    sdbcx::ObjectType createObject(const OUString& rName) override;
 public:
     OWriterTables(const css::uno::Reference<css::sdbc::XDatabaseMetaData>& _rMetaData,::cppu::OWeakObject& _rParent, ::osl::Mutex& _rMutex,
                   const ::std::vector< OUString>& _rVector) : OWriterTables_BASE(_rMetaData,_rParent,_rMutex,_rVector)