Compare() and CompareMat(): PopError() to propagate

... an already existing error instead of overriding with
errIllegalParameter.

Change-Id: Iebdbe1fbaba8aa1628eacc2a3ad3d193af7a42bf

1 files changed, 2 insertions, 0 deletions

diff --git a/sc/source/core/tool/interpr1.cxx b/sc/source/core/tool/interpr1.cxx
index e14aaea6ad0e..bec54db19b56 100644
--- a/sc/source/core/tool/interpr1.cxx
+++ b/sc/source/core/tool/interpr1.cxx
@@ -885,6 +885,7 @@ double ScInterpreter::Compare( ScQueryOp eOp )
                 // mode. Which also means we'd have to change all places where
                 // it currently is handled along with svMatrix.
             default:
+                PopError();
                 SetError( errIllegalParameter);
             break;
         }
@@ -953,6 +954,7 @@ sc::RangeMatrix ScInterpreter::CompareMat( ScQueryOp eOp, sc::CompareOptions* pO
                     // errors are transported as DoubleError inside matrix
                 break;
             default:
+                PopError();
                 SetError( errIllegalParameter);
             break;
         }