diff options
| author | Winfried Donkers <winfrieddonkers@libreoffice.org> | 2016-04-04 08:04:35 +0200 |
|---|---|---|
| committer | jan iversen <jani@documentfoundation.org> | 2016-04-04 07:25:56 +0000 |
| commit | 62967a0afc596811fe308640acdd90a6df57185f (patch) | |
| tree | e6e10b22f940dd978d086799f3d57cbf82fb02a8 /formula | |
| parent | edc63d3abb5c4dae223f745f6384e7d0e5fc717e (diff) | |
get rid of a TODO comment
Checked if the < is correct. It is.
Calc function Choose has 1 index argument plus up to 30 value-arguments,
so the maximum argument count is 31.
FORMULA_MAXJUMPCOUNT is 32.
Change-Id: I0bc8cc122902848dadfc92d56b6f06fbec43e4be
Reviewed-on: https://gerrit.libreoffice.org/23794
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>
Diffstat (limited to 'formula')
| -rw-r--r-- | formula/source/core/api/FormulaCompiler.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/formula/source/core/api/FormulaCompiler.cxx b/formula/source/core/api/FormulaCompiler.cxx index 2560e39ca3aa..fbf800a1a7db 100644 --- a/formula/source/core/api/FormulaCompiler.cxx +++ b/formula/source/core/api/FormulaCompiler.cxx @@ -1490,7 +1490,7 @@ void FormulaCompiler::Factor() bLimitOk = (nJumpCount <= 3); break; case ocChoose: - bLimitOk = (nJumpCount < FORMULA_MAXJUMPCOUNT); /* TODO: check, really <, not <=? */ + bLimitOk = (nJumpCount < FORMULA_MAXJUMPCOUNT); break; case ocIfError: case ocIfNA: |