From cbe3ae1894800a5fddbd598403be54f9495cc964 Mon Sep 17 00:00:00 2001 From: Luboš Luňák Date: Fri, 8 Jun 2018 21:14:19 +0200 Subject: properly go to next row when walking mdds container Since only elements from nOffset up to nEnd are iterated, then the difference is not just nEnd but (nEnd-nOffset). Without this, ScColumn::HandleRefArrayForParallelism() skips some cells and e.g. fdo#77750-1 asserts in ScFormulaCell::MaybeInterpret(). Change-Id: I4d595e9faff51f08bf074d549185c24aa00d8a45 Reviewed-on: https://gerrit.libreoffice.org/55488 Tested-by: Jenkins Reviewed-by: Kohei Yoshida Tested-by: Kohei Yoshida --- sc/source/core/data/column.cxx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'sc/source/core/data/column.cxx') diff --git a/sc/source/core/data/column.cxx b/sc/source/core/data/column.cxx index ae6813988293..d7c25de498de 100644 --- a/sc/source/core/data/column.cxx +++ b/sc/source/core/data/column.cxx @@ -184,7 +184,7 @@ sc::MatrixEdge ScColumn::GetBlockMatrixEdges( SCROW nRow1, SCROW nRow2, sc::Matr bOpen = false; // bottom edge closes } - nRow += nEnd; + nRow += nEnd - nOffset; } if (bOpen) nEdges |= MatrixEdge::Open; // not closed, matrix continues -- cgit v1.2.3