diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2019-03-03 19:00:34 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2019-03-03 23:20:35 +0100 |
| commit | 923c82a966d65e7b5a4b35573090567e93703eb5 (patch) | |
| tree | 888d8dc18e42a4be813523ead162c91e8b8acc32 | |
| parent | 5e73afe96a1a1f5444683c210fa31e1c30826675 (diff) | |
Avoid unnecessary conversion to sal_uInt16
(the negated value is ultimately passed to a short parameter of
SvxLRSpaceItem::SetTextFirstLineOfstValue. This change has no acutal effect on
the result, but silences Clang -fsanitize=implicit-signed-integer-truncation
warnings.
Change-Id: Ieefab13b9594fb6e19953c5f0ca22b36141ff986
Reviewed-on: https://gerrit.libreoffice.org/68648
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
| -rw-r--r-- | filter/source/msfilter/svdfppt.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/filter/source/msfilter/svdfppt.cxx b/filter/source/msfilter/svdfppt.cxx index 3cd0247d8fc5..ff31e39c2c48 100644 --- a/filter/source/msfilter/svdfppt.cxx +++ b/filter/source/msfilter/svdfppt.cxx @@ -6223,7 +6223,7 @@ void PPTParagraphObj::ApplyTo( SfxItemSet& rSet, boost::optional< sal_Int16 >& { SvxLRSpaceItem aLRSpaceItem( EE_PARA_LRSPACE ); sal_uInt16 nAbsLSpace = static_cast<sal_uInt16>( ( _nTextOfs * 2540 ) / 576 ); - sal_uInt16 nFirstLineOffset = nAbsLSpace - static_cast<sal_uInt16>( ( _nBulletOfs * 2540 ) / 576 ); + auto const nFirstLineOffset = nAbsLSpace - static_cast<sal_uInt16>( ( _nBulletOfs * 2540 ) / 576 ); aLRSpaceItem.SetLeft( nAbsLSpace ); aLRSpaceItem.SetTextFirstLineOfstValue( -nFirstLineOffset ); rSet.Put( aLRSpaceItem ); |