diff options
author | Tamás Zolnai <tamas.zolnai@collabora.com> | 2018-10-31 17:20:46 +0100 |
---|---|---|
committer | Tamás Zolnai <tamas.zolnai@collabora.com> | 2018-11-01 13:02:26 +0100 |
commit | 26ccd00bc96c585b7065af0dcce246b5bfaae5e1 (patch) | |
tree | 960e163c0041c813e10f567995d648dc9332449d /filter | |
parent | 0610c4439edf95b4cc4e16ee625b60ba606f7528 (diff) |
clang-tidy: (WIP) bugprone-too-small-loop-variable findings
Change-Id: Iaa255b39928ac45dec1ed37e368c149d6027f561
Reviewed-on: https://gerrit.libreoffice.org/62701
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Diffstat (limited to 'filter')
-rw-r--r-- | filter/source/graphicfilter/ipict/shape.cxx | 2 | ||||
-rw-r--r-- | filter/source/msfilter/escherex.cxx | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/filter/source/graphicfilter/ipict/shape.cxx b/filter/source/graphicfilter/ipict/shape.cxx index f9bae2acd4eb..3bdd022bb3e7 100644 --- a/filter/source/graphicfilter/ipict/shape.cxx +++ b/filter/source/graphicfilter/ipict/shape.cxx @@ -79,7 +79,7 @@ namespace PictReaderShape { long origPt[4][2] = { { orig.X(), orig.Y() }, { orig.X()+pSize.Width(), orig.Y() }, { orig.X()+pSize.Width(), orig.Y()+pSize.Height() }, { orig.X(), orig.Y()+pSize.Height() }}; - long origAvoid = dir[0] > 0 ? (dir[1] > 0 ? 2 : 1) : (dir[1] > 0 ? 3 : 0); + int origAvoid = dir[0] > 0 ? (dir[1] > 0 ? 2 : 1) : (dir[1] > 0 ? 3 : 0); long destPt[4][2] = { { dest.X(), dest.Y() }, { dest.X()+pSize.Width(), dest.Y() }, { dest.X()+pSize.Width(), dest.Y()+pSize.Height() }, { dest.X(), dest.Y()+pSize.Height() }}; diff --git a/filter/source/msfilter/escherex.cxx b/filter/source/msfilter/escherex.cxx index 03e69c96c716..c158129fb6a5 100644 --- a/filter/source/msfilter/escherex.cxx +++ b/filter/source/msfilter/escherex.cxx @@ -4577,7 +4577,8 @@ sal_uInt32 EscherConnectorListEntry::GetConnectorRule( bool bFirst ) if(0 != aPolyPoly.Count()) { - sal_Int16 a, b, nIndex = 0; + sal_Int16 nIndex = 0; + sal_uInt16 a, b; sal_uInt32 nDistance = 0xffffffff; for ( a = 0; a < aPolyPoly.Count(); a++ ) |