SvxUnoXPropertyTable is never passed a null ptr

and make the type of the constructor functions more precise to avoid
unnecessary UNO_QUERY at the call site

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

7 files changed, 7 insertions, 16 deletions

diff --git a/svx/source/xoutdev/xtabbtmp.cxx b/svx/source/xoutdev/xtabbtmp.cxx
index fd1b7dfdadb7..178ba8ab2db4 100644
--- a/svx/source/xoutdev/xtabbtmp.cxx
+++ b/svx/source/xoutdev/xtabbtmp.cxx
@@ -35,8 +35,7 @@ XBitmapEntry* XBitmapList::GetBitmap(tools::Long nIndex) const
 
 uno::Reference< container::XNameContainer > XBitmapList::createInstance()
 {
-    return uno::Reference< container::XNameContainer >(
-        SvxUnoXBitmapTable_createInstance( this ), uno::UNO_QUERY );
+    return SvxUnoXBitmapTable_createInstance( *this );
 }
 
 bool XBitmapList::Create()
diff --git a/svx/source/xoutdev/xtabcolr.cxx b/svx/source/xoutdev/xtabcolr.cxx
index 67e963ce92a6..e952d6f8a44f 100644
--- a/svx/source/xoutdev/xtabcolr.cxx
+++ b/svx/source/xoutdev/xtabcolr.cxx
@@ -54,9 +54,7 @@ XColorEntry* XColorList::GetColor(tools::Long nIndex) const
 
 uno::Reference< container::XNameContainer > XColorList::createInstance()
 {
-    return uno::Reference< container::XNameContainer >(
-        SvxUnoXColorTable_createInstance( this ),
-        uno::UNO_QUERY );
+    return SvxUnoXColorTable_createInstance( *this );
 }
 
 bool XColorList::Create()
diff --git a/svx/source/xoutdev/xtabdash.cxx b/svx/source/xoutdev/xtabdash.cxx
index b0dcced37322..da5ec20a3bdf 100644
--- a/svx/source/xoutdev/xtabdash.cxx
+++ b/svx/source/xoutdev/xtabdash.cxx
@@ -60,8 +60,7 @@ XDashEntry* XDashList::GetDash(tools::Long nIndex) const
 
 uno::Reference< container::XNameContainer > XDashList::createInstance()
 {
-    return uno::Reference< container::XNameContainer >(
-        SvxUnoXDashTable_createInstance( this ), uno::UNO_QUERY );
+    return SvxUnoXDashTable_createInstance( *this );
 }
 
 bool XDashList::Create()
diff --git a/svx/source/xoutdev/xtabgrdt.cxx b/svx/source/xoutdev/xtabgrdt.cxx
index 94bda61ffa2e..3d2acb574c3c 100644
--- a/svx/source/xoutdev/xtabgrdt.cxx
+++ b/svx/source/xoutdev/xtabgrdt.cxx
@@ -60,9 +60,7 @@ XGradientEntry* XGradientList::GetGradient(tools::Long nIndex) const
 
 uno::Reference< container::XNameContainer > XGradientList::createInstance()
 {
-    return uno::Reference< container::XNameContainer >(
-        SvxUnoXGradientTable_createInstance( this ),
-        uno::UNO_QUERY );
+    return SvxUnoXGradientTable_createInstance( *this );
 }
 
 bool XGradientList::Create()
diff --git a/svx/source/xoutdev/xtabhtch.cxx b/svx/source/xoutdev/xtabhtch.cxx
index 44824cb08289..4fc97727b0cc 100644
--- a/svx/source/xoutdev/xtabhtch.cxx
+++ b/svx/source/xoutdev/xtabhtch.cxx
@@ -59,8 +59,7 @@ XHatchEntry* XHatchList::GetHatch(tools::Long nIndex) const
 
 uno::Reference< container::XNameContainer > XHatchList::createInstance()
 {
-    return uno::Reference< container::XNameContainer >(
-        SvxUnoXHatchTable_createInstance( this ), uno::UNO_QUERY );
+    return SvxUnoXHatchTable_createInstance( *this );
 }
 
 bool XHatchList::Create()
diff --git a/svx/source/xoutdev/xtablend.cxx b/svx/source/xoutdev/xtablend.cxx
index 50fc240a60ab..fcca059601e5 100644
--- a/svx/source/xoutdev/xtablend.cxx
+++ b/svx/source/xoutdev/xtablend.cxx
@@ -53,8 +53,7 @@ XLineEndEntry* XLineEndList::GetLineEnd(tools::Long nIndex) const
 
 uno::Reference< container::XNameContainer > XLineEndList::createInstance()
 {
-    return uno::Reference< container::XNameContainer >(
-        SvxUnoXLineEndTable_createInstance( this ), uno::UNO_QUERY );
+    return SvxUnoXLineEndTable_createInstance( *this );
 }
 
 bool XLineEndList::Create()
diff --git a/svx/source/xoutdev/xtabptrn.cxx b/svx/source/xoutdev/xtabptrn.cxx
index 8d461167a87a..29697880f07f 100644
--- a/svx/source/xoutdev/xtabptrn.cxx
+++ b/svx/source/xoutdev/xtabptrn.cxx
@@ -37,8 +37,7 @@ XBitmapEntry* XPatternList::GetBitmap(tools::Long nIndex) const
 
 uno::Reference< container::XNameContainer > XPatternList::createInstance()
 {
-    return uno::Reference< container::XNameContainer >(
-        SvxUnoXBitmapTable_createInstance( this ), uno::UNO_QUERY );
+    return SvxUnoXBitmapTable_createInstance( *this );
 }
 
 bool XPatternList::Create()