summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaarten Bosmans <mkbosmans@gmail.com>2016-09-20 20:27:04 +0200
committerAndras Timar <andras.timar@collabora.com>2016-10-10 21:13:15 +0200
commit5ee0cf887301ea0e994e3ec7299f4958808fc2d8 (patch)
tree038fc0685efe38df1d625f7328f79dfba9be8474
parentdf53a5f695b37bdd3eef57203fc0a3f4c95dbeea (diff)
tdf#53698: Cache more than 1 item in NumberFormatCodeMapper
In zforlist.cxx the function SvNumberFormatter::GetPreviewStringGuess switches between the system locale and en_US in order to guess the formatting of a number. This results in poor performance, because in the created SvNumberFormat only the last used locale is cached. Caching more than 1 entries improves the load time for the document attached to tdf#53698 from 90s to 60s in case of non en_US locales. Change-Id: Id0eb4447dea6213015e2d958d8303a1e7892487a Reviewed-on: https://gerrit.libreoffice.org/29136 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk> (cherry picked from commit 450a2fd5e2dafd1a0c08e73ef85db978f6b1a927)
-rw-r--r--i18npool/inc/numberformatcode.hxx15
-rw-r--r--i18npool/source/numberformatcode/numberformatcode.cxx77
2 files changed, 37 insertions, 55 deletions
diff --git a/i18npool/inc/numberformatcode.hxx b/i18npool/inc/numberformatcode.hxx
index 9c7d78066d3e..48c64c24de70 100644
--- a/i18npool/inc/numberformatcode.hxx
+++ b/i18npool/inc/numberformatcode.hxx
@@ -29,6 +29,9 @@
#include <com/sun/star/uno/Sequence.hxx>
#include <com/sun/star/lang/XServiceInfo.hpp>
+#include <deque>
+#include <utility>
+
class NumberFormatCodeMapper : public cppu::WeakImplHelper
<
css::i18n::XNumberFormatCode,
@@ -55,19 +58,15 @@ public:
private:
osl::Mutex maMutex;
- css::lang::Locale aLocale;
- css::uno::Reference < css::uno::XComponentContext > mxContext;
- css::uno::Sequence< css::i18n::FormatElement > aFormatSeq;
- css::uno::Reference < css::i18n::XLocaleData4 > mxLocaleData;
- bool bFormatsValid;
+ css::uno::Reference < css::i18n::XLocaleData4 > m_xLocaleData;
+ typedef std::pair< css::lang::Locale, css::uno::Sequence< css::i18n::FormatElement > > FormatElementCacheItem;
+ std::deque < FormatElementCacheItem > m_aFormatElementCache;
- void setupLocale( const css::lang::Locale& rLocale );
- void getFormats( const css::lang::Locale& rLocale );
+ const css::uno::Sequence< css::i18n::FormatElement >& getFormats( const css::lang::Locale& rLocale );
static OUString mapElementTypeShortToString(sal_Int16 formatType);
static sal_Int16 mapElementTypeStringToShort(const OUString& formatType);
static OUString mapElementUsageShortToString(sal_Int16 formatUsage);
static sal_Int16 mapElementUsageStringToShort(const OUString& formatUsage);
- void createLocaleDataObject();
};
diff --git a/i18npool/source/numberformatcode/numberformatcode.cxx b/i18npool/source/numberformatcode/numberformatcode.cxx
index 4a5e14782cef..fe56556009d8 100644
--- a/i18npool/source/numberformatcode/numberformatcode.cxx
+++ b/i18npool/source/numberformatcode/numberformatcode.cxx
@@ -25,10 +25,8 @@
NumberFormatCodeMapper::NumberFormatCodeMapper(
const css::uno::Reference < css::uno::XComponentContext >& rxContext )
- :
- mxContext( rxContext ),
- bFormatsValid( false )
{
+ m_xLocaleData.set( css::i18n::LocaleData::create( rxContext ) );
}
@@ -45,10 +43,10 @@ NumberFormatCodeMapper::getDefault( sal_Int16 formatType, sal_Int16 formatUsage,
OUString elementUsage = mapElementUsageShortToString(formatUsage);
osl::MutexGuard g(maMutex);
- getFormats( rLocale );
+ const css::uno::Sequence< css::i18n::FormatElement > &aFormatSeq = getFormats( rLocale );
- for(sal_Int32 i = 0; i < aFormatSeq.getLength(); i++) {
- if(aFormatSeq[i].isDefault && aFormatSeq[i].formatType == elementType &&
+ for (sal_Int32 i = 0; i < aFormatSeq.getLength(); i++) {
+ if (aFormatSeq[i].isDefault && aFormatSeq[i].formatType == elementType &&
aFormatSeq[i].formatUsage == elementUsage) {
css::i18n::NumberFormatCode anumberFormatCode(formatType,
formatUsage,
@@ -69,10 +67,10 @@ css::i18n::NumberFormatCode SAL_CALL
NumberFormatCodeMapper::getFormatCode( sal_Int16 formatIndex, const css::lang::Locale& rLocale ) throw(css::uno::RuntimeException, std::exception)
{
osl::MutexGuard g(maMutex);
- getFormats( rLocale );
+ const css::uno::Sequence< css::i18n::FormatElement > &aFormatSeq = getFormats( rLocale );
- for(sal_Int32 i = 0; i < aFormatSeq.getLength(); i++) {
- if(aFormatSeq[i].formatIndex == formatIndex) {
+ for (sal_Int32 i = 0; i < aFormatSeq.getLength(); i++) {
+ if (aFormatSeq[i].formatIndex == formatIndex) {
css::i18n::NumberFormatCode anumberFormatCode(mapElementTypeStringToShort(aFormatSeq[i].formatType),
mapElementUsageStringToShort(aFormatSeq[i].formatUsage),
aFormatSeq[i].formatCode,
@@ -85,7 +83,6 @@ NumberFormatCodeMapper::getFormatCode( sal_Int16 formatIndex, const css::lang::L
}
css::i18n::NumberFormatCode defaultNumberFormatCode;
return defaultNumberFormatCode;
-
}
@@ -93,21 +90,21 @@ css::uno::Sequence< css::i18n::NumberFormatCode > SAL_CALL
NumberFormatCodeMapper::getAllFormatCode( sal_Int16 formatUsage, const css::lang::Locale& rLocale ) throw(css::uno::RuntimeException, std::exception)
{
osl::MutexGuard g(maMutex);
- getFormats( rLocale );
+ const css::uno::Sequence< css::i18n::FormatElement > &aFormatSeq = getFormats( rLocale );
sal_Int32 i, count;
count = 0;
- for(i = 0; i < aFormatSeq.getLength(); i++) {
+ for (i = 0; i < aFormatSeq.getLength(); i++) {
sal_Int16 elementUsage = mapElementUsageStringToShort(aFormatSeq[i].formatUsage);
- if( elementUsage == formatUsage)
+ if ( elementUsage == formatUsage )
count++;
}
css::uno::Sequence<css::i18n::NumberFormatCode> seq(count);
sal_Int32 j = 0;
- for(i = 0; i < aFormatSeq.getLength(); i++) {
+ for (i = 0; i < aFormatSeq.getLength(); i++) {
sal_Int16 elementUsage = mapElementUsageStringToShort(aFormatSeq[i].formatUsage);
- if( elementUsage == formatUsage) {
+ if ( elementUsage == formatUsage ) {
seq[j] = css::i18n::NumberFormatCode(mapElementTypeStringToShort(aFormatSeq[i].formatType),
formatUsage,
aFormatSeq[i].formatCode,
@@ -119,7 +116,6 @@ NumberFormatCodeMapper::getAllFormatCode( sal_Int16 formatUsage, const css::lang
}
}
return seq;
-
}
@@ -127,10 +123,10 @@ css::uno::Sequence< css::i18n::NumberFormatCode > SAL_CALL
NumberFormatCodeMapper::getAllFormatCodes( const css::lang::Locale& rLocale ) throw(css::uno::RuntimeException, std::exception)
{
osl::MutexGuard g(maMutex);
- getFormats( rLocale );
+ const css::uno::Sequence< css::i18n::FormatElement > &aFormatSeq = getFormats( rLocale );
css::uno::Sequence<css::i18n::NumberFormatCode> seq(aFormatSeq.getLength());
- for(sal_Int32 i = 0; i < aFormatSeq.getLength(); i++)
+ for (sal_Int32 i = 0; i < aFormatSeq.getLength(); i++)
{
seq[i] = css::i18n::NumberFormatCode(mapElementTypeStringToShort(aFormatSeq[i].formatType),
mapElementUsageStringToShort(aFormatSeq[i].formatUsage),
@@ -146,30 +142,26 @@ NumberFormatCodeMapper::getAllFormatCodes( const css::lang::Locale& rLocale ) th
// --- private implementation -----------------------------------------
-void NumberFormatCodeMapper::setupLocale( const css::lang::Locale& rLocale )
+const css::uno::Sequence< css::i18n::FormatElement >& NumberFormatCodeMapper::getFormats( const css::lang::Locale& rLocale )
{
- if ( aLocale.Country != rLocale.Country
- || aLocale.Language != rLocale.Language
- || aLocale.Variant != rLocale.Variant )
+ /* Find the FormatElement Sequence in the cache */
+ for (const FormatElementCacheItem& item : m_aFormatElementCache)
{
- bFormatsValid = false;
- aLocale = rLocale;
+ if ( item.first == rLocale )
+ return item.second;
}
-}
-
-void NumberFormatCodeMapper::getFormats( const css::lang::Locale& rLocale )
-{
- setupLocale( rLocale );
- if ( !bFormatsValid )
- {
- createLocaleDataObject();
- if( !mxLocaleData.is() )
- aFormatSeq = css::uno::Sequence< css::i18n::FormatElement > (0);
- else
- aFormatSeq = mxLocaleData->getAllFormats( aLocale );
- bFormatsValid = true;
- }
+ /* Not found; Get the FormatElement Sequence for the given Locale */
+ css::uno::Sequence< css::i18n::FormatElement > aFormatElementSequence;
+ if ( m_xLocaleData.is() )
+ aFormatElementSequence = m_xLocaleData->getAllFormats( rLocale );
+
+ /* Add the FormatElement Sequence to the cache */
+ const int FORMATELEMENTCACHE_SIZE = 3;
+ if ( m_aFormatElementCache.size() > FORMATELEMENTCACHE_SIZE )
+ m_aFormatElementCache.pop_front();
+ m_aFormatElementCache.emplace_back( rLocale, aFormatElementSequence );
+ return m_aFormatElementCache.back().second;
}
@@ -252,15 +244,6 @@ NumberFormatCodeMapper::mapElementUsageStringToShort(const OUString& formatUsage
}
-void
-NumberFormatCodeMapper::createLocaleDataObject() {
-
- if(mxLocaleData.is())
- return;
-
- mxLocaleData.set( css::i18n::LocaleData::create(mxContext) );
-}
-
OUString SAL_CALL
NumberFormatCodeMapper::getImplementationName()
throw( css::uno::RuntimeException, std::exception )