[Libreoffice-commits] core.git: svx/source

Stephan Bergmann (via logerrit) logerrit at kemper.freedesktop.org
Tue Jun 9 07:16:24 UTC 2020


 svx/source/items/customshapeitem.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b505cd8635171e39d055b9a800f672a6313d8cea
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Mon Jun 8 22:44:40 2020 +0200
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Tue Jun 9 09:15:47 2020 +0200

    tdf#126025: Non-const GetPropertyValueByName must use non-const Sequence []
    
    ...which internally calls getArray to make sure it references a unique sequence.
    Regression introduced with 85f28ec44a2c169c91dac9346e9c71feda6a6cab "Clean up C-
    style casts from pointers to void".
    
    Change-Id: Id80b75f06322c65fc31040cc0b1892fe3a0aeaab
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/95862
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/svx/source/items/customshapeitem.cxx b/svx/source/items/customshapeitem.cxx
index 8626d893ed7e..57ec15e6b5e3 100644
--- a/svx/source/items/customshapeitem.cxx
+++ b/svx/source/items/customshapeitem.cxx
@@ -90,7 +90,7 @@ css::uno::Any* SdrCustomShapeGeometryItem::GetPropertyValueByName( const OUStrin
             PropertyPairHashMap::iterator aHashIter( aPropPairHashMap.find( PropertyPair( rSequenceName, rPropName ) ) );
             if ( aHashIter != aPropPairHashMap.end() )
             {
-                pRet = const_cast<css::uno::Any *>(&(*rSecSequence)[ (*aHashIter).second ].Value);
+                pRet = &const_cast<css::uno::Sequence<css::beans::PropertyValue> &>(*rSecSequence)[ (*aHashIter).second ].Value;
             }
         }
     }


More information about the Libreoffice-commits mailing list