diff options
| author | Winfried Donkers <winfrieddonkers@libreoffice.org> | 2017-01-01 11:57:34 +0100 |
|---|---|---|
| committer | Eike Rathke <erack@redhat.com> | 2017-01-03 21:31:10 +0000 |
| commit | 25625f18a282209235214674f2b81604ecd067ad (patch) | |
| tree | dfeb4f383abe260c1aa121ec9128308641e758a7 /sc | |
| parent | facb4e69e4c75839df89969206403d2e97146dbd (diff) | |
tdf#105026 Remove wrong constraint in PERMUTATIONA.
With permutations with repetitions, argument selection may
be larger than argument elements.
Probably inadvertently copied from PERMUT; has been there from
pre-git.
Change-Id: Icdee18eca647fb278f140630e45844684a1e3046
Reviewed-on: https://gerrit.libreoffice.org/32569
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/core/tool/interpr3.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/core/tool/interpr3.cxx b/sc/source/core/tool/interpr3.cxx index c0ac545df0f5..1d90d22b4b3a 100644 --- a/sc/source/core/tool/interpr3.cxx +++ b/sc/source/core/tool/interpr3.cxx @@ -1209,7 +1209,7 @@ void ScInterpreter::ScPermutationA() { double k = ::rtl::math::approxFloor(GetDouble()); double n = ::rtl::math::approxFloor(GetDouble()); - if (n < 0.0 || k < 0.0 || k > n) + if (n < 0.0 || k < 0.0) PushIllegalArgument(); else PushDouble(pow(n,k)); |