diff options
author | Winfried Donkers <winfrieddonkers@libreoffice.org> | 2016-04-29 07:25:23 +0200 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2016-04-29 09:11:32 +0000 |
commit | 47405e009500538732926739ae4b2636c0957238 (patch) | |
tree | 138e9d77db1483c7f9e902dfdfcd08c791e0625b | |
parent | 86b49add3a7ec24923cb7f95461465a8863ab5ad (diff) |
tdf#97831 follow up improve error handling
(And make code consistent with other Excel2016 functions.)
Change-Id: Id46b98fd1b7e8e036b6584ef382d1ff2171edab7
Reviewed-on: https://gerrit.libreoffice.org/24485
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>
-rw-r--r-- | sc/source/core/tool/interpr8.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sc/source/core/tool/interpr8.cxx b/sc/source/core/tool/interpr8.cxx index 321929a5422b..123581031909 100644 --- a/sc/source/core/tool/interpr8.cxx +++ b/sc/source/core/tool/interpr8.cxx @@ -1413,7 +1413,7 @@ void ScInterpreter::ScConcat_MS() { case svString: case svDouble: - aResBuf.append( PopString().getString() ); + aResBuf.append( GetString().getString() ); break; case svSingleRef : { @@ -1541,7 +1541,7 @@ void ScInterpreter::ScTextJoin_MS() { case svString: case svDouble: - aDelimiters.push_back( PopString().getString() ); + aDelimiters.push_back( GetString().getString() ); break; case svSingleRef : { @@ -1669,7 +1669,7 @@ void ScInterpreter::ScTextJoin_MS() case svString: case svDouble: { - OUString aStr = PopString().getString(); + OUString aStr = GetString().getString(); if ( !aStr.isEmpty() || !bSkipEmpty ) { if ( !bFirst ) |