[Libreoffice-commits] core.git: Branch 'libreoffice-6-1' - sc/source

[Libreoffice Gerrit user]

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

New commits:
commit db376075d564e5a834a5db6de4c7e381159c66be
Author:     Michael Weghorn <m.weghorn at posteo.de>
AuthorDate: Thu Aug 30 12:59:06 2018 +0200
Commit:     Eike Rathke <erack at redhat.com>
CommitDate: Fri Aug 31 17:53:52 2018 +0200

    tdf#112454 Also consider last SCTAB in loop
    
    Since 'rRange.aEnd.Tab()' is valid, it needs to
    be included in the loop to get a valid result.
    
    Change-Id: I7b2447e059dc03444c56e7fc0b113e1b618dd510
    Reviewed-on: https://gerrit.libreoffice.org/59798
    Tested-by: Jenkins
    Reviewed-by: Katarina Behrens <Katarina.Behrens at cib.de>
    (cherry picked from commit 03b361673d04f31e0602d2988e4df26eaeb53a34)
    Reviewed-on: https://gerrit.libreoffice.org/59872
    Reviewed-by: Eike Rathke <erack at redhat.com>

diff --git a/sc/source/core/data/document.cxx b/sc/source/core/data/document.cxx
index 84caf96da812..f952fe5fdd13 100644
--- a/sc/source/core/data/document.cxx
+++ b/sc/source/core/data/document.cxx
@@ -6718,7 +6718,7 @@ bool ScDocument::ContainsNotesInRange( const ScRangeList& rRangeList ) const
     for( size_t i = 0; i < rRangeList.size(); ++i)
     {
         const ScRange & rRange = rRangeList[i];
-        for( SCTAB nTab = rRange.aStart.Tab(); nTab < rRange.aEnd.Tab(); ++nTab )
+        for( SCTAB nTab = rRange.aStart.Tab(); nTab <= rRange.aEnd.Tab(); ++nTab )
         {
             bool bContainsNote = maTabs[nTab]->ContainsNotesInRange( rRange );
             if(bContainsNote)