diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-07-21 10:13:17 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-07-21 13:59:33 +0100 |
| commit | 6554c1f363e5c7d9edbd2cb7b58d391dff3c06a4 (patch) | |
| tree | cbc4c39e7267080fcf081c0cb320360ef97a968d | |
| parent | 55125105e87fdbe444b114acea5c1b70d61987a1 (diff) | |
coverity#706101 Unintended sign extension
Change-Id: Ic044245aef233501c03e342084f5c27a1bee8d4e
| -rw-r--r-- | sw/source/core/doc/poolfmt.cxx | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/sw/source/core/doc/poolfmt.cxx b/sw/source/core/doc/poolfmt.cxx index 9b99b5d404c9..55d82233841d 100644 --- a/sw/source/core/doc/poolfmt.cxx +++ b/sw/source/core/doc/poolfmt.cxx @@ -1796,8 +1796,9 @@ SwNumRule* SwDoc::GetNumRuleFromPool( sal_uInt16 nId ) } else if ( eNumberFormatPositionAndSpaceMode == SvxNumberFormat::LABEL_ALIGNMENT ) { - aFmt.SetListtabPos( (n+1) * nOffs ); - aFmt.SetIndentAt( (n+1) * nOffs ); + long nPos = (n+1) * static_cast<long>(nOffs); + aFmt.SetListtabPos(nPos); + aFmt.SetIndentAt(nPos); } aFmt.SetStart( n+1 ); @@ -1952,8 +1953,9 @@ SwNumRule* SwDoc::GetNumRuleFromPool( sal_uInt16 nId ) } else if ( eNumberFormatPositionAndSpaceMode == SvxNumberFormat::LABEL_ALIGNMENT ) { - aFmt.SetListtabPos( nOffs2 + ((n-3) * nOffs) ); - aFmt.SetIndentAt( nOffs2 + ((n-3) * nOffs) ); + long nPos = nOffs2 + ((n-3) * static_cast<long>(nOffs)); + aFmt.SetListtabPos(nPos); + aFmt.SetIndentAt(nPos); } pNewRule->Set( n, aFmt ); @@ -2085,8 +2087,9 @@ SwNumRule* SwDoc::GetNumRuleFromPool( sal_uInt16 nId ) } else if ( eNumberFormatPositionAndSpaceMode == SvxNumberFormat::LABEL_ALIGNMENT ) { - aFmt.SetListtabPos( ((n & 1) +1) * nOffs ); - aFmt.SetIndentAt( ((n & 1) +1) * nOffs ); + long nPos = ((n & 1) +1) * static_cast<long>(nOffs); + aFmt.SetListtabPos(nPos); + aFmt.SetIndentAt(nPos); } pNewRule->Set( n, aFmt ); |