Use IsInArrayContext() instead of (bMatrixFormula || pCur->IsInForceArray())

Change-Id: I50debd5be2e0d4db0b5c57676d03ec33a2c38737
Reviewed-on: https://gerrit.libreoffice.org/58128Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins

sc/source/core/inc/interpre.hxx

@@ -1024,13 +1024,13 @@ inline bool ScInterpreter::IsInArrayContext() const
 
 inline void ScInterpreter::MatrixJumpConditionToMatrix()
 {
-    if (bMatrixFormula || pCur->IsInForceArray())
+    if (IsInArrayContext())
         ConvertMatrixJumpConditionToMatrix();
 }
 
 inline bool ScInterpreter::MatrixParameterConversion()
 {
-    if ( (bMatrixFormula || pCur->IsInForceArray() || ScParameterClassification::HasForceArray( pCur->GetOpCode())) &&
+    if ( (IsInArrayContext() || ScParameterClassification::HasForceArray( pCur->GetOpCode())) &&
             !pJumpMatrix && sp > 0 )
         return ConvertMatrixParameters();
     return false;

sc/source/core/tool/interpr4.cxx

@@ -1495,7 +1495,7 @@ bool ScInterpreter::ConvertMatrixParameters()
                             // has ForceArray or ReferenceOrForceArray
                             // parameter *somewhere else*) pick a normal
                             // position dependent implicit intersection later.
-                            (eType != formula::ParamClass::Value || bMatrixFormula || pCur->IsInForceArray()))
+                            (eType != formula::ParamClass::Value || IsInArrayContext()))
                     {
                         SCCOL nCol1, nCol2;
                         SCROW nRow1, nRow2;

sc/source/core/tool/interpr6.cxx

@@ -464,7 +464,7 @@ void IterateMatrix(
 size_t ScInterpreter::GetRefListArrayMaxSize( short nParamCount )
 {
     size_t nSize = 0;
-    if (bMatrixFormula || pCur->IsInForceArray())
+    if (IsInArrayContext())
     {
         for (short i=1; i <= nParamCount; ++i)
         {

sc/source/core/tool/interpr7.cxx

@@ -47,7 +47,7 @@ void ScInterpreter::ScFilterXML()
         // In array/matrix context node elements' results are to be
         // subsequently stored. Check this before obtaining any argument from
         // the stack so the stack type can be used.
-        if (pJumpMatrix || bMatrixFormula || pCur->IsInForceArray())
+        if (pJumpMatrix || IsInArrayContext())
         {
             if (pJumpMatrix)
             {