diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-12-02 00:59:58 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-12-02 05:31:10 +0100 |
| commit | 1f2d3b3f09d16033dae1f99f7739564a5f4c5482 (patch) | |
| tree | 292145abe33a2bac69d67fdd67cfc7a8527f6024 /basic | |
| parent | a9b13f2b7f0d2ab8bcf25a3110a06ababb4573f6 (diff) | |
tdf#120703 PVS: V560 A part of conditional expression is always true/false
Change-Id: Ia9c496ec93bedb13764969bf21c3f21d84caacef
Reviewed-on: https://gerrit.libreoffice.org/64413
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'basic')
| -rw-r--r-- | basic/source/sbx/sbxform.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/basic/source/sbx/sbxform.cxx b/basic/source/sbx/sbxform.cxx index 45a88b556546..ce394fa903b3 100644 --- a/basic/source/sbx/sbxform.cxx +++ b/basic/source/sbx/sbxform.cxx @@ -647,7 +647,7 @@ void SbxBasicFormater::ScanFormatString( double dNumber, // Remark: in Visual-Basic the first 0 turns on the 0 for // all the following # (up to the decimal point), // this behaviour is simulated here with the flag. - if( bGenerateThousandSeparator && ( c=='0' || nMaxDigit >= nDigitPos ) && nDigitPos > 0 && (nDigitPos % 3 == 0) ) + if (bGenerateThousandSeparator && c == '0' && nDigitPos > 0 && (nDigitPos % 3 == 0)) { sReturnStrg.append(cThousandSep); } |