From 9858b7004856a632778dde0dc7edddb14b673194 Mon Sep 17 00:00:00 2001 From: Michael Weghorn Date: Thu, 30 Aug 2018 12:59:06 +0200 Subject: 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 (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 Tested-by: Jenkins --- sc/source/core/data/document.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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) -- cgit v1.2.3