strepsirrhini/office-gobmx
9
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix Cppcheck report: reduce scope of the variable 'fTemp'

Browse source 
Change-Id: I935de8e287f94f3219542dfe67f7bb4a05be14da
Reviewed-on: https://gerrit.libreoffice.org/29200
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
This commit is contained in:
Laurent Balland-Poirier 2016-09-22 22:34:15 +02:00 committed by Noel Grandin
parent b74d2433c8
commit 792da744f3
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
svl/source/numbers/zformat.cxx
View file

@ -2476,13 +2476,13 @@ bool SvNumberformat::ImpGetFractionOutput(double fNumber,
{
sal_uInt64 nBasis = ((sal_uInt64)floor( pow(10.0,rInfo.nCntExp))) - 1; // 9, 99, 999 ,...
sal_uInt64 nFracPrev = 1L, nDivPrev = 0, nFracNext, nDivNext, nPartialDenom;
double fRemainder = fNumber, fTemp;
double fRemainder = fNumber;
// Use continued fraction representation of fNumber
// See https://en.wikipedia.org/wiki/Continued_fraction#Best_rational_approximations
while ( fRemainder > 0.0 )
{
fTemp = 1.0 / fRemainder; // 64bits precision required when fRemainder is very weak
double fTemp = 1.0 / fRemainder; // 64bits precision required when fRemainder is very weak
nPartialDenom = (sal_uInt64) floor(fTemp); // due to floating point notation with double precision
fRemainder = fTemp - (double)nPartialDenom;
nDivNext = nPartialDenom * nDiv + nDivPrev;