loplugin:refcounting in ucb

Change-Id: I5dd8295648aadafc7dfc73121354aa5f602c2c79
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111633
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

8 files changed, 15 insertions, 14 deletions

diff --git a/ucb/source/ucp/ftp/ftpcontent.cxx b/ucb/source/ucp/ftp/ftpcontent.cxx
index bac2579ffa54..7a6716ee06a0 100644
--- a/ucb/source/ucp/ftp/ftpcontent.cxx
+++ b/ucb/source/ucp/ftp/ftpcontent.cxx
@@ -208,7 +208,7 @@ ResultSetFactory::ResultSetFactory(const Reference<XComponentContext >&  rxConte
 }
 
 
-ResultSetBase* ResultSetFactory::createResultSet()
+rtl::Reference<ResultSetBase> ResultSetFactory::createResultSet()
 {
     return new ResultSetI(m_xContext,
                           m_xProvider,
diff --git a/ucb/source/ucp/ftp/ftpresultsetfactory.hxx b/ucb/source/ucp/ftp/ftpresultsetfactory.hxx
index 24102b870d5f..e230a04d316b 100644
--- a/ucb/source/ucp/ftp/ftpresultsetfactory.hxx
+++ b/ucb/source/ucp/ftp/ftpresultsetfactory.hxx
@@ -29,6 +29,7 @@
 #include "ftpdirp.hxx"
 #include <com/sun/star/uno/XComponentContext.hpp>
 #include <com/sun/star/beans/Property.hpp>
+#include <rtl/ref.hxx>
 #include <vector>
 
 namespace ftp {
@@ -43,7 +44,7 @@ public:
                       const css::uno::Sequence<css::beans::Property>& seq,
                       const std::vector<FTPDirentry>& dirvec);
 
-    ResultSetBase* createResultSet();
+    rtl::Reference<ResultSetBase> createResultSet();
 private:
     css::uno::Reference< css::uno::XComponentContext >        m_xContext;
     css::uno::Reference< css::ucb::XContentProvider >         m_xProvider;
diff --git a/ucb/source/ucp/hierarchy/hierarchycontent.cxx b/ucb/source/ucp/hierarchy/hierarchycontent.cxx
index e3cc82594f37..5afd6cc5bb3b 100644
--- a/ucb/source/ucp/hierarchy/hierarchycontent.cxx
+++ b/ucb/source/ucp/hierarchy/hierarchycontent.cxx
@@ -80,7 +80,7 @@ using namespace hierarchy_ucp;
 
 
 // static ( "virtual" ctor )
-HierarchyContent* HierarchyContent::create(
+rtl::Reference<HierarchyContent> HierarchyContent::create(
             const uno::Reference< uno::XComponentContext >& rxContext,
             HierarchyContentProvider* pProvider,
             const uno::Reference< ucb::XContentIdentifier >& Identifier )
@@ -95,7 +95,7 @@ HierarchyContent* HierarchyContent::create(
 
 
 // static ( "virtual" ctor )
-HierarchyContent* HierarchyContent::create(
+rtl::Reference<HierarchyContent> HierarchyContent::create(
             const uno::Reference< uno::XComponentContext >& rxContext,
             HierarchyContentProvider* pProvider,
             const uno::Reference< ucb::XContentIdentifier >& Identifier,
diff --git a/ucb/source/ucp/hierarchy/hierarchycontent.hxx b/ucb/source/ucp/hierarchy/hierarchycontent.hxx
index 3e17ac35e643..c830c5772b77 100644
--- a/ucb/source/ucp/hierarchy/hierarchycontent.hxx
+++ b/ucb/source/ucp/hierarchy/hierarchycontent.hxx
@@ -183,14 +183,14 @@ private:
 
 public:
     // Create existing content. Fail, if not already exists.
-    static HierarchyContent* create(
+    static rtl::Reference<HierarchyContent> create(
             const css::uno::Reference< css::uno::XComponentContext >& rxContext,
             HierarchyContentProvider* pProvider,
             const css::uno::Reference<
                 css::ucb::XContentIdentifier >& Identifier );
 
     // Create new content. Fail, if already exists.
-    static HierarchyContent* create(
+    static rtl::Reference<HierarchyContent> create(
             const css::uno::Reference< css::uno::XComponentContext >& rxContext,
             HierarchyContentProvider* pProvider,
             const css::uno::Reference< css::ucb::XContentIdentifier >& Identifier,
diff --git a/ucb/source/ucp/package/pkgcontent.cxx b/ucb/source/ucp/package/pkgcontent.cxx
index a3944757ec48..dfda010f2f63 100644
--- a/ucb/source/ucp/package/pkgcontent.cxx
+++ b/ucb/source/ucp/package/pkgcontent.cxx
@@ -140,7 +140,7 @@ ContentProperties::getCreatableContentsInfo( PackageUri const & rUri ) const
 
 
 // static ( "virtual" ctor )
-Content* Content::create(
+rtl::Reference<Content> Content::create(
             const uno::Reference< uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const uno::Reference< ucb::XContentIdentifier >& Identifier )
@@ -192,7 +192,7 @@ Content* Content::create(
 
 
 // static ( "virtual" ctor )
-Content* Content::create(
+rtl::Reference<Content> Content::create(
             const uno::Reference< uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const uno::Reference< ucb::XContentIdentifier >& Identifier,
diff --git a/ucb/source/ucp/package/pkgcontent.hxx b/ucb/source/ucp/package/pkgcontent.hxx
index 5bbd483ff175..fe5f67f535b8 100644
--- a/ucb/source/ucp/package/pkgcontent.hxx
+++ b/ucb/source/ucp/package/pkgcontent.hxx
@@ -200,13 +200,13 @@ private:
 
 public:
     // Create existing content. Fail, if not already exists.
-    static Content* create(
+    static rtl::Reference<Content> create(
             const css::uno::Reference< css::uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const css::uno::Reference< css::ucb::XContentIdentifier >& Identifier );
 
     // Create new content. Fail, if already exists.
-    static Content* create(
+    static rtl::Reference<Content> create(
             const css::uno::Reference< css::uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const css::uno::Reference< css::ucb::XContentIdentifier >& Identifier,
diff --git a/ucb/source/ucp/tdoc/tdoc_content.cxx b/ucb/source/ucp/tdoc/tdoc_content.cxx
index 03c0300b4b09..34c6ea2b31df 100644
--- a/ucb/source/ucp/tdoc/tdoc_content.cxx
+++ b/ucb/source/ucp/tdoc/tdoc_content.cxx
@@ -103,7 +103,7 @@ static ContentType lcl_getContentType( std::u16string_view rType )
 
 
 // static ( "virtual" ctor )
-Content* Content::create(
+rtl::Reference<Content> Content::create(
             const uno::Reference< uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const uno::Reference< ucb::XContentIdentifier >& Identifier )
@@ -120,7 +120,7 @@ Content* Content::create(
 
 
 // static ( "virtual" ctor )
-Content* Content::create(
+rtl::Reference<Content> Content::create(
             const uno::Reference< uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const uno::Reference< ucb::XContentIdentifier >& Identifier,
diff --git a/ucb/source/ucp/tdoc/tdoc_content.hxx b/ucb/source/ucp/tdoc/tdoc_content.hxx
index c8cf050f655c..08bedba75cff 100644
--- a/ucb/source/ucp/tdoc/tdoc_content.hxx
+++ b/ucb/source/ucp/tdoc/tdoc_content.hxx
@@ -201,13 +201,13 @@ private:
 
 public:
     // Create existing content. Fail, if not already exists.
-    static Content* create(
+    static rtl::Reference<Content> create(
             const css::uno::Reference< css::uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const css::uno::Reference< css::ucb::XContentIdentifier >& Identifier );
 
     // Create new content. Fail, if already exists.
-    static Content* create(
+    static rtl::Reference<Content> create(
             const css::uno::Reference< css::uno::XComponentContext >& rxContext,
             ContentProvider* pProvider,
             const css::uno::Reference< css::ucb::XContentIdentifier >& Identifier,