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 /scaddins | |
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 'scaddins')
-rw-r--r-- | scaddins/source/analysis/analysishelper.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scaddins/source/analysis/analysishelper.cxx b/scaddins/source/analysis/analysishelper.cxx index 9bd869bc8c95..3d05ac068921 100644 --- a/scaddins/source/analysis/analysishelper.cxx +++ b/scaddins/source/analysis/analysishelper.cxx @@ -538,7 +538,7 @@ double GetYearFrac( sal_Int32 nNullDate, sal_Int32 nStartDate, sal_Int32 nEndDat { // return average of days in year between nDate1 and nDate2, inclusive sal_Int32 nDayCount = 0; - for ( sal_Int16 i = nYear1; i <= nYear2; i++ ) + for ( sal_uInt16 i = nYear1; i <= nYear2; i++ ) nDayCount += ( IsLeapYear( i ) ? 366 : 365 ); nDaysInYear = static_cast<double>(nDayCount) / static_cast<double>( nYear2 - nYear1 + 1 ); |