diff options
author | Michael Weghorn <m.weghorn@posteo.de> | 2018-08-30 12:59:06 +0200 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2018-08-31 17:53:52 +0200 |
commit | db376075d564e5a834a5db6de4c7e381159c66be (patch) | |
tree | 87b58956910a803608a3a92ff39263dd025ab70a | |
parent | 2fffe4a50b6f541c1e43cf13a3a475caf94d203b (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.cxx | 2 |
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) |