Kaydet (Commit) 00b30079 authored tarafından Eike Rathke's avatar Eike Rathke

Avoid side effects of additional Interpret() during iteration

... which could alter results if this formula cell again depends
on other formula cells in the circular reference(s).

Change-Id: I2b56902a71c0f16f9777507402ea004ae609bfbc
Reviewed-on: https://gerrit.libreoffice.org/56182Reviewed-by: 's avatarEike Rathke <erack@redhat.com>
Tested-by: 's avatarEike Rathke <erack@redhat.com>
üst b15244d5
...@@ -2598,6 +2598,11 @@ bool ScFormulaCell::IsMultilineResult() ...@@ -2598,6 +2598,11 @@ bool ScFormulaCell::IsMultilineResult()
bool ScFormulaCell::NeedsInterpret() const bool ScFormulaCell::NeedsInterpret() const
{ {
if (bIsIterCell)
// Shortcut to force return of current value and not enter Interpret()
// as we're looping over all iteration cells.
return false;
if (!IsDirtyOrInTableOpDirty()) if (!IsDirtyOrInTableOpDirty())
return false; return false;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment