[Libreoffice-commits] core.git: sc/source

[Libreoffice Gerrit user]

 sc/source/core/opencl/formulagroupcl.cxx |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

New commits:
commit c706f61c076995e04038eef6c7fc2942f8e1fb58
Author:     Luboš Luňák <l.lunak at collabora.com>
AuthorDate: Fri Nov 30 12:52:53 2018 +0100
Commit:     Luboš Luňák <l.lunak at collabora.com>
CommitDate: Thu Jan 24 20:34:51 2019 +0100

    say in log message which opcode is not handled by OpenCL
    
    Change-Id: I836969c777382a3a7dce23733ebfc331391bf4b4
    Reviewed-on: https://gerrit.libreoffice.org/65477
    Tested-by: Jenkins
    Reviewed-by: Luboš Luňák <l.lunak at collabora.com>

diff --git a/sc/source/core/opencl/formulagroupcl.cxx b/sc/source/core/opencl/formulagroupcl.cxx
index 60b4ef6e7d3b..357daa62a07b 100644
--- a/sc/source/core/opencl/formulagroupcl.cxx
+++ b/sc/source/core/opencl/formulagroupcl.cxx
@@ -30,6 +30,8 @@
 #include "op_spreadsheet.hxx"
 #include "op_addin.hxx"
 
+#include <com/sun/star/sheet/FormulaLanguage.hpp>
+
 // FIXME: The idea that somebody would bother to (now and then? once a year? once a month?) manually
 // edit a source file and change the value of some #defined constant and run some ill-defined
 // "correctness test" is of course ludicrous. Either things are checked in normal unit tests, in
@@ -3758,7 +3760,9 @@ DynamicKernelSoPArguments::DynamicKernelSoPArguments(const ScCalcConfig& config,
                 break;
 
             default:
-                throw UnhandledToken("unhandled opcode", __FILE__, __LINE__);
+                throw UnhandledToken(OUString("unhandled opcode "
+                    + formula::FormulaCompiler().GetOpCodeMap(com::sun::star::sheet::FormulaLanguage::ENGLISH)->getSymbol(opc)
+                    + "(" + OUString::number(opc) + ")").toUtf8().getStr(), __FILE__, __LINE__);
         }
     }
 }