[Libreoffice-commits] core.git: sc/source

[Libreoffice Gerrit user]

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

New commits:
commit cadd6c595b250b3ce0025a2a09c160eaae483d5a
Author:     Eike Rathke <erack at redhat.com>
AuthorDate: Wed Aug 22 20:00:16 2018 +0200
Commit:     Eike Rathke <erack at redhat.com>
CommitDate: Wed Aug 22 23:46:21 2018 +0200

    Resolves: tdf#119402 +- typo sign flip
    
    Regression from
    
        commit 667aa6ade94a4dee441868d21db037e42c871932
        CommitDate: Wed Jul 19 20:13:58 2017 +0200
    
            Avoid temporary Date and normalization
    
    that introduced a sign flip
    from        +  5 - eDay
    to          + (5 + eDay)
    instead of  + (5 - eDay)
    
    Change-Id: I28194296b20552135e6abd24eb9eefe8634ce7c0
    Reviewed-on: https://gerrit.libreoffice.org/59461
    Reviewed-by: Eike Rathke <erack at redhat.com>
    Tested-by: Jenkins

diff --git a/sc/source/core/data/conditio.cxx b/sc/source/core/data/conditio.cxx
index 4d0ff28f0441..a7ef20a468d1 100644
--- a/sc/source/core/data/conditio.cxx
+++ b/sc/source/core/data/conditio.cxx
@@ -1619,7 +1619,7 @@ bool ScCondDateFormatEntry::IsValid( const ScAddress& rPos ) const
                 if( eDay != SUNDAY )
                 {
                     Date aBegin(rActDate - (1 + static_cast<sal_Int32>(eDay)));
-                    Date aEnd(rActDate + (5 + static_cast<sal_Int32>(eDay)));
+                    Date aEnd(rActDate + (5 - static_cast<sal_Int32>(eDay)));
                     return aCellDate.IsBetween( aBegin, aEnd );
                 }
                 else