sc/source/core/data/document.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit db376075d564e5a834a5db6de4c7e381159c66be Author: Michael Weghorn <m.wegh...@posteo.de> AuthorDate: Thu Aug 30 12:59:06 2018 +0200 Commit: Eike Rathke <er...@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.behr...@cib.de> (cherry picked from commit 03b361673d04f31e0602d2988e4df26eaeb53a34) Reviewed-on: https://gerrit.libreoffice.org/59872 Reviewed-by: Eike Rathke <er...@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) _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits