summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Weghorn <m.weghorn@posteo.de>2018-08-30 12:59:06 +0200
committerEike Rathke <erack@redhat.com>2018-08-31 17:53:52 +0200
commitdb376075d564e5a834a5db6de4c7e381159c66be (patch)
tree87b58956910a803608a3a92ff39263dd025ab70a
parent2fffe4a50b6f541c1e43cf13a3a475caf94d203b (diff)
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@cib.de> (cherry picked from commit 03b361673d04f31e0602d2988e4df26eaeb53a34) Reviewed-on: https://gerrit.libreoffice.org/59872 Reviewed-by: Eike Rathke <erack@redhat.com>
-rw-r--r--sc/source/core/data/document.cxx2
1 files changed, 1 insertions, 1 deletions
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)