From 176d54fa5dd4aeda1cc77a1e46b7bfff0c07b8ce Mon Sep 17 00:00:00 2001
From: Jochen Nitschke <j.nitschke+logerrit@ok.de>
Date: Mon, 11 Jul 2016 23:37:57 +0200
Subject: chart2: inline local macros

make use of Sequence initializer_list constructor
and use makeAny to simplify further.
removed unused includes

Change-Id: I7d5b901a8bb375533c8ab945632dc8668806ca1a
Reviewed-on: https://gerrit.libreoffice.org/27173
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
---
 chart2/source/tools/OPropertySet.cxx    | 35 +++++++++------------------------
 chart2/source/view/main/DummyXShape.cxx | 24 +++++++---------------
 2 files changed, 16 insertions(+), 43 deletions(-)

diff --git a/chart2/source/tools/OPropertySet.cxx b/chart2/source/tools/OPropertySet.cxx
index 7ed2ce3a1428..0dcf03c0da1a 100644
--- a/chart2/source/tools/OPropertySet.cxx
+++ b/chart2/source/tools/OPropertySet.cxx
@@ -19,10 +19,7 @@
 
 #include "OPropertySet.hxx"
 #include "ImplOPropertySet.hxx"
-#include "ContainerHelper.hxx"
 #include <cppuhelper/queryinterface.hxx>
-#include <comphelper/servicehelper.hxx>
-#include <comphelper/sequence.hxx>
 
 #include <vector>
 #include <algorithm>
@@ -30,8 +27,6 @@
 
 using namespace ::com::sun::star;
 
-using ::com::sun::star::style::XStyleSupplier;
-
 using ::com::sun::star::uno::Reference;
 using ::com::sun::star::uno::Sequence;
 using ::com::sun::star::uno::Any;
@@ -85,34 +80,22 @@ Any SAL_CALL OPropertySet::queryInterface( const uno::Type& aType )
         static_cast< beans::XFastPropertySet * >( this ),
         static_cast< beans::XPropertyState * >( this ),
         static_cast< beans::XMultiPropertyStates * >( this ),
-        static_cast< XStyleSupplier * >( this ) );
+        static_cast< style::XStyleSupplier * >( this ) );
 }
 
-#define LCL_PROP_CPPUTYPE(t) (cppu::UnoType<t>::get())
-
 //  ____ XTypeProvider ____
 Sequence< uno::Type > SAL_CALL
     OPropertySet::getTypes()
     throw (uno::RuntimeException, std::exception)
 {
-    static Sequence< uno::Type > aTypeList;
-
-    MutexGuard aGuard( m_rMutex );
-
-    if( aTypeList.getLength() == 0 )
-    {
-        ::std::vector< uno::Type > aTypes;
-
-        aTypes.push_back( LCL_PROP_CPPUTYPE( lang::XTypeProvider ));
-        aTypes.push_back( LCL_PROP_CPPUTYPE( beans::XPropertySet ));
-        aTypes.push_back( LCL_PROP_CPPUTYPE( beans::XMultiPropertySet ));
-        aTypes.push_back( LCL_PROP_CPPUTYPE( beans::XFastPropertySet ));
-        aTypes.push_back( LCL_PROP_CPPUTYPE( beans::XPropertyState ));
-        aTypes.push_back( LCL_PROP_CPPUTYPE( beans::XMultiPropertyStates ));
-        aTypes.push_back( LCL_PROP_CPPUTYPE( XStyleSupplier ));
-
-        aTypeList = comphelper::containerToSequence( aTypes );
-    }
+    static const Sequence< uno::Type > aTypeList{
+        cppu::UnoType<lang::XTypeProvider>::get(),
+        cppu::UnoType<beans::XPropertySet>::get(),
+        cppu::UnoType<beans::XMultiPropertySet>::get(),
+        cppu::UnoType<beans::XFastPropertySet>::get(),
+        cppu::UnoType<beans::XPropertyState>::get(),
+        cppu::UnoType<beans::XMultiPropertyStates>::get(),
+        cppu::UnoType<style::XStyleSupplier>::get() };
 
     return aTypeList;
 }
diff --git a/chart2/source/view/main/DummyXShape.cxx b/chart2/source/view/main/DummyXShape.cxx
index e926557e3635..3f5baf76b21d 100644
--- a/chart2/source/view/main/DummyXShape.cxx
+++ b/chart2/source/view/main/DummyXShape.cxx
@@ -1012,33 +1012,23 @@ DummyChart* DummyChart::getRootShape()
     return this;
 }
 
-#define QUERYINT( xint ) \
-    if( rType == cppu::UnoType<xint>::get() ) \
-        aAny <<= uno::Reference< xint >(this)
-
-#define QUERY_INTERFACE( xint ) \
-    if( rType == cppu::UnoType<xint>::get() ) \
-        return uno::makeAny(uno::Reference<xint>(this));
-
 uno::Any SAL_CALL DummyXShapes::queryInterface( const uno::Type& rType )
     throw(uno::RuntimeException, std::exception)
 {
-    QUERY_INTERFACE( drawing::XShapes );
-    QUERY_INTERFACE( container::XIndexAccess );
+    if( rType == cppu::UnoType<drawing::XShapes>::get() )
+        return uno::makeAny(uno::Reference<drawing::XShapes>(this));
+    if( rType == cppu::UnoType<container::XIndexAccess>::get() )
+        return uno::makeAny(uno::Reference<container::XIndexAccess>(this));
     return DummyXShape::queryInterface(rType);
 }
 
 uno::Any SAL_CALL DummyXShapes::queryAggregation( const uno::Type & rType )
     throw(uno::RuntimeException, std::exception)
 {
-    uno::Any aAny;
-
-    //QUERYINT( drawing::XShapeGroup );
-    QUERYINT( drawing::XShapes );
+    if( rType == cppu::UnoType<drawing::XShapes>::get() )
+        return uno::makeAny(uno::Reference< drawing::XShapes >(this));
     else
         return DummyXShape::queryAggregation( rType );
-
-    return aAny;
 }
 
 void SAL_CALL DummyXShapes::acquire()
@@ -1083,7 +1073,7 @@ uno::Type SAL_CALL DummyXShapes::getElementType()
     return cppu::UnoType<drawing::XShape>::get();
 }
 
-sal_Bool SAL_CALL SAL_CALL DummyXShapes::hasElements()
+sal_Bool SAL_CALL DummyXShapes::hasElements()
     throw(uno::RuntimeException, std::exception)
 {
     return !maUNOShapes.empty();
-- 
cgit