diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2015-03-31 13:12:34 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2015-03-31 13:12:34 +0200 |
commit | 581c5075bbbb8068e2285c3392b712f41083cfb4 (patch) | |
tree | 47888b492e4439348add7b342c29bb1e7e044fd7 /comphelper/source | |
parent | b57f8a945117a8e32d36549a24c87377c54a798e (diff) |
Reduce to static_cast any reinterpret_cast from void pointers
Change-Id: I062024d4768374e319b6c67ed8c7abcc89f9d552
Diffstat (limited to 'comphelper/source')
-rw-r--r-- | comphelper/source/misc/anytostring.cxx | 2 | ||||
-rw-r--r-- | comphelper/source/misc/types.cxx | 28 | ||||
-rw-r--r-- | comphelper/source/property/propertycontainerhelper.cxx | 6 |
3 files changed, 18 insertions, 18 deletions
diff --git a/comphelper/source/misc/anytostring.cxx b/comphelper/source/misc/anytostring.cxx index 50dc18547411..99221361ed24 100644 --- a/comphelper/source/misc/anytostring.cxx +++ b/comphelper/source/misc/anytostring.cxx @@ -196,7 +196,7 @@ void appendValue( OUStringBuffer & buf, break; } case typelib_TypeClass_TYPE: - buf.append( (*reinterpret_cast< + buf.append( (*static_cast< typelib_TypeDescriptionReference * const * >(val) )->pTypeName ); break; diff --git a/comphelper/source/misc/types.cxx b/comphelper/source/misc/types.cxx index bb060c4ea912..4d9a074a9a34 100644 --- a/comphelper/source/misc/types.cxx +++ b/comphelper/source/misc/types.cxx @@ -167,7 +167,7 @@ template<class TYPE> bool tryCompare(const void* _pData, const Any& _rValue, bool& _bIdentical, TYPE& _rOut) { bool bSuccess = _rValue >>= _rOut; - _bIdentical = bSuccess && (_rOut == *reinterpret_cast<const TYPE*>(_pData)); + _bIdentical = bSuccess && (_rOut == *static_cast<const TYPE*>(_pData)); return bSuccess; } @@ -191,13 +191,13 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) // beides AnyWerte if (_rValue.getValueType().getTypeClass() == TypeClass_ANY) bRes = compare_impl( - reinterpret_cast<const Any*>(pData)->getValueType(), - reinterpret_cast<const Any*>(pData)->getValue(), - *reinterpret_cast<const Any*>(_rValue.getValue())); + static_cast<const Any*>(pData)->getValueType(), + static_cast<const Any*>(pData)->getValue(), + *static_cast<const Any*>(_rValue.getValue())); else bRes = compare_impl( - reinterpret_cast<const Any*>(pData)->getValueType(), - reinterpret_cast<const Any*>(pData)->getValue(), + static_cast<const Any*>(pData)->getValueType(), + static_cast<const Any*>(pData)->getValue(), _rValue); } else if ( (_rType.getTypeClass() == TypeClass_VOID) @@ -261,7 +261,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) { sal_Int32 nAsInt32 = 0; bConversionSuccess = ::cppu::enum2int(nAsInt32, _rValue); - bRes = bConversionSuccess && (nAsInt32== *reinterpret_cast<const sal_Int32*>(pData)); + bRes = bConversionSuccess && (nAsInt32== *static_cast<const sal_Int32*>(pData)); break; } case TypeClass_LONG: @@ -327,7 +327,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) bConversionSuccess = _rValue >>= aTemp; if (bConversionSuccess) { - const Sequence<sal_Int8>& rLeftSeq = *reinterpret_cast<const Sequence<sal_Int8>*>(pData); + const Sequence<sal_Int8>& rLeftSeq = *static_cast<const Sequence<sal_Int8>*>(pData); const Sequence<sal_Int8>& rRightSeq = aTemp; bRes = rLeftSeq.getLength() == rRightSeq.getLength() && memcmp(rLeftSeq.getConstArray(), rRightSeq.getConstArray(), rLeftSeq.getLength()) == 0; @@ -339,7 +339,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) bConversionSuccess = _rValue >>= aTemp; if (bConversionSuccess) { - const Sequence<sal_uInt8>& rLeftSeq = *reinterpret_cast<const Sequence<sal_uInt8>*>(pData); + const Sequence<sal_uInt8>& rLeftSeq = *static_cast<const Sequence<sal_uInt8>*>(pData); const Sequence<sal_uInt8>& rRightSeq = aTemp; bRes = rLeftSeq.getLength() == rRightSeq.getLength() && memcmp(rLeftSeq.getConstArray(), rRightSeq.getConstArray(), rLeftSeq.getLength()) == 0; @@ -351,7 +351,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) bConversionSuccess = _rValue >>= aTemp; if (bConversionSuccess) { - const Sequence<sal_Int16>& rLeftSeq = *reinterpret_cast<const Sequence<sal_Int16>*>(pData); + const Sequence<sal_Int16>& rLeftSeq = *static_cast<const Sequence<sal_Int16>*>(pData); const Sequence<sal_Int16>& rRightSeq = aTemp; bRes = rLeftSeq.getLength() == rRightSeq.getLength() && memcmp(rLeftSeq.getConstArray(), rRightSeq.getConstArray(), rLeftSeq.getLength()*sizeof(sal_Int16)) == 0; @@ -363,7 +363,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) bConversionSuccess = _rValue >>= aTemp; if (bConversionSuccess) { - const Sequence<sal_uInt16>& rLeftSeq = *reinterpret_cast<const Sequence<sal_uInt16>*>(pData); + const Sequence<sal_uInt16>& rLeftSeq = *static_cast<const Sequence<sal_uInt16>*>(pData); const Sequence<sal_uInt16>& rRightSeq = aTemp; bRes = rLeftSeq.getLength() == rRightSeq.getLength() && memcmp(rLeftSeq.getConstArray(), rRightSeq.getConstArray(), rLeftSeq.getLength()*sizeof(sal_uInt16)) == 0; @@ -375,7 +375,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) bConversionSuccess = _rValue >>= aTemp; if (bConversionSuccess) { - const Sequence<sal_Int32>& rLeftSeq = *reinterpret_cast<const Sequence<sal_Int32>*>(pData); + const Sequence<sal_Int32>& rLeftSeq = *static_cast<const Sequence<sal_Int32>*>(pData); const Sequence<sal_Int32>& rRightSeq = aTemp; bRes = rLeftSeq.getLength() == rRightSeq.getLength() && memcmp(rLeftSeq.getConstArray(), rRightSeq.getConstArray(), rLeftSeq.getLength()*sizeof(sal_Int32)) == 0; @@ -387,7 +387,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) bConversionSuccess = _rValue >>= aTemp; if (bConversionSuccess) { - const Sequence<sal_uInt32>& rLeftSeq = *reinterpret_cast<const Sequence<sal_uInt32>*>(pData); + const Sequence<sal_uInt32>& rLeftSeq = *static_cast<const Sequence<sal_uInt32>*>(pData); const Sequence<sal_uInt32>& rRightSeq = aTemp; bRes = rLeftSeq.getLength() == rRightSeq.getLength() && memcmp(rLeftSeq.getConstArray(), rRightSeq.getConstArray(), rLeftSeq.getLength()*sizeof(sal_uInt32)) == 0; @@ -399,7 +399,7 @@ bool compare_impl(const Type& _rType, const void* pData, const Any& _rValue) bConversionSuccess = _rValue >>= aTemp; if (bConversionSuccess) { - const Sequence< OUString >& rLeftSeq = *reinterpret_cast<const Sequence< OUString>*>(pData); + const Sequence< OUString >& rLeftSeq = *static_cast<const Sequence< OUString>*>(pData); const Sequence< OUString >& rRightSeq = aTemp; sal_Int32 nSeqLen = rLeftSeq.getLength(); bRes = ( nSeqLen == rRightSeq.getLength() ); diff --git a/comphelper/source/property/propertycontainerhelper.cxx b/comphelper/source/property/propertycontainerhelper.cxx index 6e10abb7e827..4cb185a32da1 100644 --- a/comphelper/source/property/propertycontainerhelper.cxx +++ b/comphelper/source/property/propertycontainerhelper.cxx @@ -288,7 +288,7 @@ bool OPropertyContainerHelper::convertFastPropertyValue( pPropContainer = &(*aIter); } else - pPropContainer = reinterpret_cast<Any*>(aPos->aLocation.pDerivedClassMember); + pPropContainer = static_cast<Any*>(aPos->aLocation.pDerivedClassMember); // check if the new value differs from the current one if (!pPropContainer->hasValue() || !aNewRequestedValue.hasValue()) @@ -386,7 +386,7 @@ bool OPropertyContainerHelper::setFastPropertyValue(sal_Int32 _nHandle, const An break; case PropertyDescription::ltDerivedClassAnyType: - *reinterpret_cast< Any* >(aPos->aLocation.pDerivedClassMember) = _rValue; + *static_cast< Any* >(aPos->aLocation.pDerivedClassMember) = _rValue; break; case PropertyDescription::ltDerivedClassRealType: @@ -427,7 +427,7 @@ void OPropertyContainerHelper::getFastPropertyValue(Any& _rValue, sal_Int32 _nHa _rValue = m_aHoldProperties[aPos->aLocation.nOwnClassVectorIndex]; break; case PropertyDescription::ltDerivedClassAnyType: - _rValue = *reinterpret_cast<Any*>(aPos->aLocation.pDerivedClassMember); + _rValue = *static_cast<Any*>(aPos->aLocation.pDerivedClassMember); break; case PropertyDescription::ltDerivedClassRealType: _rValue.setValue(aPos->aLocation.pDerivedClassMember, aPos->aProperty.Type); |