diff options
| author | Xisco Fauli <xiscofauli@libreoffice.org> | 2024-10-21 12:17:58 +0200 |
|---|---|---|
| committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2024-10-21 16:43:10 +0200 |
| commit | 5cee433ef2d94e53764406f289af352750e03026 (patch) | |
| tree | 2953fefd299183354f530c51ac65ef935f825baa | |
| parent | c49ec77db3c99b6b45f70bee18244f399bd78a68 (diff) | |
tdf#163486 sw: PVS: identical sub-expressions
Since
commit 22dac22fe256b28b78c8e0783f2625aee0f75ace
Author: Winfried Donkers <winfrieddonkers@libreoffice.org>
Date: Thu Mar 27 13:33:15 2014 +0100
fdo#73147 add Excel2010 functions NETWORKDAYS.INTL and WORKDAY.INTL
and later changed in
commit 80f28e377db0ca04da57eed77d42f44cbdea363d
Author: Noel Grandin <noel.grandin@collabora.co.uk>
Date: Fri May 29 11:36:45 2020 +0200
loplugin:simplifybool in oox..sd
V501 There are identical sub-expressions 'nRef >= nMax' to the left and to the right of the '||' operator.
Change-Id: I78a36012dad89afa9980fcfd62772873817d871b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175327
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
Tested-by: Jenkins
| -rw-r--r-- | sc/source/core/tool/interpr2.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/core/tool/interpr2.cxx b/sc/source/core/tool/interpr2.cxx index f1443b9c5109..d57fbd22b49a 100644 --- a/sc/source/core/tool/interpr2.cxx +++ b/sc/source/core/tool/interpr2.cxx @@ -601,7 +601,7 @@ void ScInterpreter::ScWorkday_MS() while ( nRef < nMax && nSortArray.at( nRef ) < nDate ) nRef++; - if ( nRef >= nMax || nSortArray.at( nRef ) != nDate || nRef >= nMax ) + if ( nRef >= nMax || nSortArray.at( nRef ) != nDate ) nDays--; } } |