From dafd27829359201cab45462bae43baf19b369a14 Mon Sep 17 00:00:00 2001
From: Eike Rathke <erack@redhat.com>
Date: Sat, 21 Sep 2013 14:57:40 +0200
Subject: register separate theSystemLocale for faster access

Change-Id: I0fc5938bd2c0157471539217806ad7844d765e8d
---
 i18nlangtag/source/languagetag/languagetag.cxx | 83 +++++++++++++++++++++++---
 1 file changed, 75 insertions(+), 8 deletions(-)

(limited to 'i18nlangtag/source')

diff --git a/i18nlangtag/source/languagetag/languagetag.cxx b/i18nlangtag/source/languagetag/languagetag.cxx
index 4a3ba4dbfaab..9ad4155571a7 100644
--- a/i18nlangtag/source/languagetag/languagetag.cxx
+++ b/i18nlangtag/source/languagetag/languagetag.cxx
@@ -95,6 +95,7 @@ typedef ::std::map< LanguageType, LanguageTag::ImplPtr > MapLangID;
 struct theMapBcp47 : public rtl::Static< MapBcp47, theMapBcp47 > {};
 struct theMapLangID : public rtl::Static< MapLangID, theMapLangID > {};
 struct theDontKnow : public rtl::Static< LanguageTag::ImplPtr, theDontKnow > {};
+struct theSystemLocale : public rtl::Static< LanguageTag::ImplPtr, theSystemLocale > {};
 }
 
 
@@ -665,19 +666,49 @@ LanguageTag::ImplPtr LanguageTag::registerImpl() const
     // methods as they may create temporary LanguageTag instances. Only
     // LanguageTagImpl::convertToBcp47(Locale) is ok.
 
+    ImplPtr pImpl;
+
 #if OSL_DEBUG_LEVEL > 0
-    static size_t nAccesses = 0;
-    ++nAccesses;
-    SAL_INFO( "i18nlangtag", "LanguageTagImpl::registerImpl: " << nAccesses << " calls");
+    static size_t nCalls = 0;
+    ++nCalls;
+    SAL_INFO( "i18nlangtag", "LanguageTag::registerImpl: " << nCalls << " calls");
 #endif
 
-    ImplPtr pImpl;
+    // Do not register unresolved system locale, also force LangID if system
+    // and take the system locale shortcut if possible.
+    if (mbSystemLocale)
+    {
+        pImpl = theSystemLocale::get();
+        if (pImpl)
+        {
+#if OSL_DEBUG_LEVEL > 0
+            static size_t nCallsSystem = 0;
+            ++nCallsSystem;
+            SAL_INFO( "i18nlangtag", "LanguageTag::registerImpl: " << nCallsSystem << " system calls");
+#endif
+            return pImpl;
+        }
+        if (!mbInitializedLangID)
+        {
+            mnLangID = MsLangId::getRealLanguage( LANGUAGE_SYSTEM);
+            mbInitializedLangID = (mnLangID != LANGUAGE_SYSTEM);
+            SAL_WARN_IF( !mbInitializedLangID, "i18nlangtag", "LanguageTag::registerImpl: can't resolve system!");
+        }
+    }
 
-    // Do not register unresolved system locale, also force LangID if system.
-    if (mbSystemLocale && !mbInitializedLangID)
+    if (mbInitializedLangID)
     {
-        mnLangID = MsLangId::getRealLanguage( LANGUAGE_SYSTEM);
-        mbInitializedLangID = true;
+        // A great share are calls for a system equal locale.
+        pImpl = theSystemLocale::get();
+        if (pImpl && pImpl->mnLangID == mnLangID)
+        {
+#if OSL_DEBUG_LEVEL > 0
+            static size_t nCallsSystemEqual = 0;
+            ++nCallsSystemEqual;
+            SAL_INFO( "i18nlangtag", "LanguageTag::registerImpl: " << nCallsSystemEqual << " system equal LangID calls");
+#endif
+            return pImpl;
+        }
     }
 
     // Force Bcp47 if not LangID.
@@ -687,6 +718,27 @@ LanguageTag::ImplPtr LanguageTag::registerImpl() const
         mbInitializedBcp47 = !maBcp47.isEmpty();
     }
 
+    if (mbInitializedBcp47)
+    {
+        // A great share are calls for a system equal locale.
+        pImpl = theSystemLocale::get();
+        if (pImpl && pImpl->maBcp47 == maBcp47)
+        {
+#if OSL_DEBUG_LEVEL > 0
+            static size_t nCallsSystemEqual = 0;
+            ++nCallsSystemEqual;
+            SAL_INFO( "i18nlangtag", "LanguageTag::registerImpl: " << nCallsSystemEqual << " system equal BCP47 calls");
+#endif
+            return pImpl;
+        }
+    }
+
+#if OSL_DEBUG_LEVEL > 0
+    static size_t nCallsNonSystem = 0;
+    ++nCallsNonSystem;
+    SAL_INFO( "i18nlangtag", "LanguageTag::registerImpl: " << nCallsNonSystem << " non-system calls");
+#endif
+
     osl::MutexGuard aGuard( theMutex::get());
 
 #if OSL_DEBUG_LEVEL > 0
@@ -820,12 +872,27 @@ LanguageTag::ImplPtr LanguageTag::registerImpl() const
         if (!rDontKnow)
             rDontKnow.reset( new LanguageTagImpl( *this));
         pImpl = rDontKnow;
+#if OSL_DEBUG_LEVEL > 0
+        static size_t nCallsDontKnow = 0;
+        ++nCallsDontKnow;
+        SAL_INFO( "i18nlangtag", "LanguageTag::registerImpl: " << nCallsDontKnow << " DontKnow calls");
+#endif
     }
     else
     {
         SAL_WARN( "i18nlangtag", "LanguageTag::registerImpl: can't register for 0x" << ::std::hex << mnLangID );
         pImpl.reset( new LanguageTagImpl( *this));
     }
+
+    // If we reach here for mbSystemLocale we didn't have theSystemLocale
+    // above, so add it.
+    if (mbSystemLocale && mbInitializedLangID)
+    {
+        theSystemLocale::get() = pImpl;
+        SAL_INFO( "i18nlangtag", "LanguageTag::registerImpl: added system locale 0x"
+                << ::std::hex << pImpl->mnLangID << " '" << pImpl->maBcp47 << "'");
+    }
+
     return pImpl;
 }
 
-- 
cgit