[Libreoffice-commits] core.git: basic/source
Eike Rathke
erack at redhat.com
Tue Dec 5 14:21:35 UTC 2017
basic/source/classes/sb.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit d915cb417befeb4a2aa1913e4b91d8cc5297c3ec
Author: Eike Rathke <erack at redhat.com>
Date: Tue Dec 5 15:19:16 2017 +0100
Resolves: tdf#114231 do not mask ErrCode with GetRest()
Regression from
commit 7ca950ec744b7af1d15724ec2abc296573a641e4
Date: Wed Aug 23 19:25:02 2017 +0200
no need to use ERRCODE_RES_MASK here
the relevant usage sites already call GetRest() before comparing
which exactly is the reason that it didn't work anymore.
Old StringArray ItemList resources stored only 16-bit values,
hence ERRCODE_RES_MASK was used to mask the ErrCode values in the
resource for which code had to mask ERRCODE_RES_MASK as well to
compare values. Now the full ErrCode is stored, so code must not
use GetRest() on a value to compare against, or use GetRest() on
both values (which theoretically could lead to ambiguities, but
probably doesn't in resources that are restricted to one module).
Change-Id: I835e47424bb008bc680dc4f8c502c9558397db36
diff --git a/basic/source/classes/sb.cxx b/basic/source/classes/sb.cxx
index b2aaae37cdab..294f7ee5744c 100644
--- a/basic/source/classes/sb.cxx
+++ b/basic/source/classes/sb.cxx
@@ -1573,7 +1573,7 @@ void StarBASIC::MakeErrorText( ErrCode nId, const OUString& aMsg )
const char* pErrorMsg = nullptr;
for (ErrMsgCode* pItem = RID_BASIC_START; pItem->second; ++pItem)
{
- if (ErrCode(nId.GetRest()) == pItem->second)
+ if (nId == pItem->second)
{
pErrorMsg = pItem->first;
break;
More information about the Libreoffice-commits
mailing list