From d3ce5b298e3f5cc58256ba80f5d3f361618c4820 Mon Sep 17 00:00:00 2001
From: Stephan Bergmann <sbergman@redhat.com>
Date: Fri, 16 Jan 2015 10:19:36 +0100
Subject: Fix getTypes() of OPropertySetHelper and derived

...broken with 60d60caf99a40ca0c3891bf230c5a1fdbae5f49c "Renamed XPropertySet2
to XPropertySetOption" et al

Change-Id: I684736ffafc4642548b7c24171cc52c1acb32252
---
 comphelper/source/property/propstate.cxx | 15 ++++++---------
 cppuhelper/source/propshlp.cxx           | 10 ++++------
 include/cppuhelper/propshlp.hxx          |  6 +++++-
 3 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/comphelper/source/property/propstate.cxx b/comphelper/source/property/propstate.cxx
index d8f53b5e4570..8f6b90759cd6 100644
--- a/comphelper/source/property/propstate.cxx
+++ b/comphelper/source/property/propstate.cxx
@@ -55,15 +55,12 @@ namespace comphelper
 
     css::uno::Sequence<css::uno::Type> OPropertyStateHelper::getTypes()
     {
-        ::com::sun::star::uno::Sequence< ::com::sun::star::uno::Type> aTypes(4);
-        ::com::sun::star::uno::Type* pTypes = aTypes.getArray();
-        // base class types
-        pTypes[0] = cppu::UnoType<com::sun::star::beans::XPropertySet>::get();
-        pTypes[1] = cppu::UnoType<com::sun::star::beans::XMultiPropertySet>::get();
-        pTypes[2] = cppu::UnoType<com::sun::star::beans::XFastPropertySet>::get();
-        // my own type
-        pTypes[3] = cppu::UnoType<com::sun::star::beans::XPropertyState>::get();
-        return aTypes;
+        return css::uno::Sequence<css::uno::Type>({
+            cppu::UnoType<css::beans::XPropertySet>::get(),
+            cppu::UnoType<css::beans::XMultiPropertySet>::get(),
+            cppu::UnoType<css::beans::XFastPropertySet>::get(),
+            cppu::UnoType<css::beans::XPropertySetOption>::get(),
+            cppu::UnoType<css::beans::XPropertyState>::get()});
     }
 
     OPropertyStateHelper::OPropertyStateHelper(
diff --git a/cppuhelper/source/propshlp.cxx b/cppuhelper/source/propshlp.cxx
index d19e65674839..509019cf043a 100644
--- a/cppuhelper/source/propshlp.cxx
+++ b/cppuhelper/source/propshlp.cxx
@@ -231,12 +231,10 @@ Any OPropertySetHelper2::queryInterface( const ::com::sun::star::uno::Type & rTy
 ::com::sun::star::uno::Sequence< ::com::sun::star::uno::Type > OPropertySetHelper::getTypes()
     throw (RuntimeException)
 {
-    Sequence< ::com::sun::star::uno::Type > aTypes( 4 );
-    aTypes[ 0 ] = XPropertySet::static_type();
-    aTypes[ 1 ] = XPropertySetOption::static_type();
-    aTypes[ 2 ] = XMultiPropertySet::static_type();
-    aTypes[ 3 ] = XFastPropertySet::static_type();
-    return aTypes;
+    return css::uno::Sequence<css::uno::Type>({
+        UnoType<css::beans::XPropertySet>::get(),
+        UnoType<css::beans::XMultiPropertySet>::get(),
+        UnoType<css::beans::XFastPropertySet>::get()});
 }
 
 // ComponentHelper
diff --git a/include/cppuhelper/propshlp.hxx b/include/cppuhelper/propshlp.hxx
index ec769e354ac8..8cf0159ad96a 100644
--- a/include/cppuhelper/propshlp.hxx
+++ b/include/cppuhelper/propshlp.hxx
@@ -686,7 +686,11 @@ protected:
 
 /**
    OPropertySetHelper plus XPropertySetOption
- */
+
+   @attention
+   The getTypes() inherited from OPropertysetHelper does not cover
+   XPropertySetOption.
+*/
 class CPPUHELPER_DLLPUBLIC OPropertySetHelper2 : public OPropertySetHelper,
                             public ::com::sun::star::beans::XPropertySetOption
 {
-- 
cgit