Kaydet (Commit) 55df5192 authored tarafından Caolán McNamara's avatar Caolán McNamara

coverity#1441442 Unintended sign extension

Change-Id: Id0286f34ccc387dd148e42b83e8f62aacfb54219
Reviewed-on: https://gerrit.libreoffice.org/63535
Tested-by: Jenkins
Reviewed-by: 's avatarCaolán McNamara <caolanm@redhat.com>
Tested-by: 's avatarCaolán McNamara <caolanm@redhat.com>
üst 9ccdf4d8
......@@ -99,7 +99,7 @@ inline void set_flag( Type& rnBitField, Type nMask, bool bSet = true )
template< typename Type, typename InsertType >
void insert_value( Type& rnBitField, InsertType nValue, sal_uInt8 nStartBit, sal_uInt8 nBitCount )
{
unsigned long nMask = ((1UL << nBitCount) - 1);
unsigned int nMask = ((1U << nBitCount) - 1);
Type nNewValue = static_cast< Type >( nValue & nMask );
rnBitField = (rnBitField & ~(nMask << nStartBit)) | (nNewValue << nStartBit);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment