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/win32/resourceprovider.cxx b/fpicker/source/win32/resourceprovider.cxx
index 883edf36dff2..ff841b385b26 100644
--- a/fpicker/source/win32/resourceprovider.cxx
+++ b/fpicker/source/win32/resourceprovider.cxx
@@ -46,7 +46,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++ )
     {
@@ -93,7 +93,7 @@ namespace CResourceProvider
     {
         OUString aResOUString;
         // translate the control id to a resource id
-        const char *pResId = CtrlIdToResId(aId);
+        TranslateId pResId = CtrlIdToResId(aId);
         if (pResId)
             aResOUString = FpsResId(pResId);
         return aResOUString;