diff options
author | Eike Rathke <erack@redhat.com> | 2015-12-01 14:09:02 +0100 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2015-12-01 14:12:20 +0100 |
commit | d46203b55d92185ab2c1d3ac79761100d26aaee3 (patch) | |
tree | ae8bfc3d1ffc25f22ab3c5677d255883d30949cb | |
parent | 87ad1705746f8e6a15ae60037a1dd8a163eeecd7 (diff) |
Resolves: tdf#95748 no group interpreter when calculations are running
Change-Id: I17283fc5e39da2d7222c302dd63522df69d0ad96
-rw-r--r-- | sc/source/core/data/formulacell.cxx | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/sc/source/core/data/formulacell.cxx b/sc/source/core/data/formulacell.cxx index b37fbaa32518..9079d560cbed 100644 --- a/sc/source/core/data/formulacell.cxx +++ b/sc/source/core/data/formulacell.cxx @@ -1551,15 +1551,24 @@ void ScFormulaCell::Interpret() } else { + // Do not attempt to interpret a group when calculations are already + // running, otherwise we may run into a circular reference hell. See + // tdf#95748 + if (rRecursionHelper.GetRecursionCount()) + InterpretTail( SCITP_NORMAL); + else + { #if DEBUG_CALCULATION - aDC.enterGroup(); -#endif - bool bGroupInterpreted = InterpretFormulaGroup(); -#if DEBUG_CALCULATION - aDC.leaveGroup(); + aDC.enterGroup(); + bool bGroupInterpreted = InterpretFormulaGroup(); + aDC.leaveGroup(); + if (!bGroupInterpreted) + InterpretTail( SCITP_NORMAL); +#else + if (!InterpretFormulaGroup()) + InterpretTail( SCITP_NORMAL); #endif - if (!bGroupInterpreted) - InterpretTail( SCITP_NORMAL); + } } // While leaving a recursion or iteration stack, insert its cells to the |