core.git: sc/source

[Xisco Fauli (via logerrit)]

 sc/source/core/tool/interpr2.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 5cee433ef2d94e53764406f289af352750e03026
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Mon Oct 21 12:17:58 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Mon Oct 21 16:43:10 2024 +0200

    tdf#163486 sw: PVS: identical sub-expressions
    
    Since
    commit 22dac22fe256b28b78c8e0783f2625aee0f75ace
    Author: Winfried Donkers <winfrieddonk...@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.gran...@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 <xiscofa...@libreoffice.org>
    Tested-by: Jenkins

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--;
                 }
             }