sc/source/core/tool/interpr1.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 91cad2de121a10c470b30832ac28597bdffd7101
Author: Eike Rathke <er...@redhat.com>
Date:   Fri Aug 5 18:29:06 2016 +0200

    Compare() and CompareMat(): PopError() to propagate
    
    ... an already existing error instead of overriding with
    errIllegalParameter.
    
    Change-Id: Iebdbe1fbaba8aa1628eacc2a3ad3d193af7a42bf

diff --git a/sc/source/core/tool/interpr1.cxx b/sc/source/core/tool/interpr1.cxx
index e14aaea..bec54db 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;
         }
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to