[Libreoffice-commits] .: extensions/source

[Noel Power]

 extensions/source/ole/unoconversionutilities.hxx |   16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

New commits:
commit 3fcb94311fd7dd40c05ca132e3a30a888316cbbe
Author: Noel Power <noel.po...@novell.com>
Date:   Wed Oct 26 17:28:51 2011 +0100

    fix handling of SAFEARRAY(s) returned as variant in olebridge (fdo#38441)

diff --git a/extensions/source/ole/unoconversionutilities.hxx 
b/extensions/source/ole/unoconversionutilities.hxx
index b6ac7f1..359009e 100644
--- a/extensions/source/ole/unoconversionutilities.hxx
+++ b/extensions/source/ole/unoconversionutilities.hxx
@@ -134,7 +134,7 @@ public:
     /** @exception com.sun.star.lang.IllegalArgumentException
         If rSeq does not contain a sequence then the exception is thrown.
     */
-    SAFEARRAY*  createUnoSequenceWrapper(const Any& rSeq);
+    VARTYPE createUnoSequenceWrapper(const Any& rSeq, SAFEARRAY*& pOutArray );
     /** @exception com.sun.star.lang.IllegalArgumentException
         If rSeq does not contain a sequence or elemtype has no proper value
         then the exception is thrown.
@@ -779,11 +779,12 @@ void UnoConversionUtilities<T>::anyToVariant(VARIANT* 
pVariant, const Any& rAny)
         }
         case TypeClass_SEQUENCE:        // sequence ??? SafeArray descriptor
         {
-            SAFEARRAY* pArray = createUnoSequenceWrapper(rAny);
-            if (pArray)
+            SAFEARRAY* pOutArray = NULL;
+            VARTYPE eArrayType = createUnoSequenceWrapper(rAny, pOutArray );
+            if (pOutArray)
             {
-                V_VT(pVariant) = VT_ARRAY | VT_VARIANT;
-                V_ARRAY(pVariant) = pArray;
+                V_VT(pVariant) = VT_ARRAY | eArrayType;
+                V_ARRAY(pVariant) = pOutArray;
             }
             else
             {
@@ -1295,9 +1296,8 @@ void  
UnoConversionUtilities<T>::getElementCountAndTypeOfSequence( const Any& rS
 
 
 template<class T>
-SAFEARRAY*  UnoConversionUtilities<T>::createUnoSequenceWrapper(const Any& 
rSeq)
+VARTYPE UnoConversionUtilities<T>::createUnoSequenceWrapper(const Any& rSeq,  
SAFEARRAY*& pArray)
 {
-    SAFEARRAY* pArray = NULL;
     sal_uInt32 n = 0;
 
     if( rSeq.getValueTypeClass() != TypeClass_SEQUENCE )
@@ -1361,7 +1361,7 @@ SAFEARRAY*  
UnoConversionUtilities<T>::createUnoSequenceWrapper(const Any& rSeq)
 
     TYPELIB_DANGER_RELEASE( pSeqElementDesc);
 
-    return pArray;
+    return eTargetElementType;
 }
 
 /* The argument rObj can contain
_______________________________________________
Libreoffice-commits mailing list
Libreoffice-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits