Pass context and resource string down to boost::locale separately

because this is often on a hot path, and we can avoid the splitting and
joining of strings like this.

Change-Id: Ia36047209368ca53431178c2e8723a18cfe8260a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119220
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 4 insertions, 4 deletions

diff --git a/fpicker/source/aqua/resourceprovider.mm b/fpicker/source/aqua/resourceprovider.mm
index 4daa24b9ff07..34b0359a231d 100644
--- a/fpicker/source/aqua/resourceprovider.mm
+++ b/fpicker/source/aqua/resourceprovider.mm
@@ -42,7 +42,7 @@ namespace {
 struct Entry
 {
     sal_Int32 ctrlId;
-    const char* resId;
+    TranslateId resId;
 };
 
 }
@@ -71,9 +71,9 @@ Entry const CtrlIdToResIdTable[] = {
 
 const sal_Int32 SIZE_TABLE = SAL_N_ELEMENTS( CtrlIdToResIdTable );
 
-static const char* CtrlIdToResId(sal_Int32 aControlId)
+static TranslateId CtrlIdToResId(sal_Int32 aControlId)
 {
-    const char *pResId = nullptr;
+    TranslateId pResId;
 
     for ( sal_Int32 i = 0; i < SIZE_TABLE; i++ )
     {
@@ -94,7 +94,7 @@ namespace CResourceProvider_Impl
         OUString aResString;
 
         // translate the control id to a resource id
-        const char* pResId = CtrlIdToResId(aId);
+        TranslateId pResId = CtrlIdToResId(aId);
         if (pResId)
             aResString = FpsResId(pResId);