Kaydet (Commit) 377cb34a authored tarafından Luboš Luňák's avatar Luboš Luňák

fix misplaced case labels

Jump opcodes such as ocIf are quite clearly not in the function range,
as can be seen in include/formula/opcode.hxx . This used to be harmless,
since originally ScTokenArray::CheckToken() didn't do anything in the default
case, but commit b366adcf changed it to disable vectorization for
unhandled opcodes.

Change-Id: Ia182f446f1da819e18309075aa00251674640c74
Reviewed-on: https://gerrit.libreoffice.org/55361Reviewed-by: 's avatarMichael Meeks <michael.meeks@collabora.com>
Reviewed-by: 's avatarEike Rathke <erack@redhat.com>
Tested-by: 's avatarJenkins <ci@libreoffice.org>
üst c6222fb6
......@@ -1388,12 +1388,6 @@ void ScTokenArray::CheckToken( const FormulaToken& r )
// We support vectorization for the following opcodes.
switch (eOp)
{
case ocIf:
case ocIfError:
case ocIfNA:
case ocChoose:
// Jump commands are now supported.
break;
case ocAverage:
case ocMin:
case ocMinA:
......@@ -1697,6 +1691,12 @@ void ScTokenArray::CheckToken( const FormulaToken& r )
case ocErrNum:
case ocErrNA:
break;
case ocIf:
case ocIfError:
case ocIfNA:
case ocChoose:
// Jump commands are now supported.
break;
}
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment