sc/source/core/data/document.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit 9858b7004856a632778dde0dc7edddb14b673194 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 20:37:28 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. Reviewed-on: https://gerrit.libreoffice.org/59798 Tested-by: Jenkins Reviewed-by: Katarina Behrens <katarina.behr...@cib.de> (cherry picked from commit 03b361673d04f31e0602d2988e4df26eaeb53a34) Conflicts: sc/source/core/data/document.cxx Backported. Change-Id: I7b2447e059dc03444c56e7fc0b113e1b618dd510 Reviewed-on: https://gerrit.libreoffice.org/59878 Reviewed-by: Eike Rathke <er...@redhat.com> Tested-by: Jenkins diff --git a/sc/source/core/data/document.cxx b/sc/source/core/data/document.cxx index 2d1834d57cd9..941657c0545b 100644 --- a/sc/source/core/data/document.cxx +++ b/sc/source/core/data/document.cxx @@ -6725,7 +6725,7 @@ bool ScDocument::ContainsNotesInRange( const ScRangeList& rRange ) const for( size_t i = 0; i < rRange.size(); ++i) { const ScRange* pRange = rRange[i]; - for( SCTAB nTab = pRange->aStart.Tab(); nTab < pRange->aEnd.Tab(); ++nTab ) + for( SCTAB nTab = pRange->aStart.Tab(); nTab <= pRange->aEnd.Tab(); ++nTab ) { bool bContainsNote = maTabs[nTab]->ContainsNotesInRange( *pRange ); if(bContainsNote) _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits