diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-05-18 07:23:56 +0100 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-05-19 03:39:13 +0200 |
| commit | 1462bdc3abe60cee43cf6064663d646adbf13894 (patch) | |
| tree | 55bb94d66485c5a0a71013dcb0803a71a823d634 /sw/source | |
| parent | e1a8338876bd161de4e9d9a4b22d4bc5335f7cee (diff) | |
Don't do the string operation needlessly
Change-Id: Ic5831e56d4528dcc1ad78e00569ad1d58272aeeb
Reviewed-on: https://gerrit.libreoffice.org/54514
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'sw/source')
| -rw-r--r-- | sw/source/core/doc/DocumentFieldsManager.cxx | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/sw/source/core/doc/DocumentFieldsManager.cxx b/sw/source/core/doc/DocumentFieldsManager.cxx index f6bd75d036a4..b4b3c0b32d80 100644 --- a/sw/source/core/doc/DocumentFieldsManager.cxx +++ b/sw/source/core/doc/DocumentFieldsManager.cxx @@ -279,15 +279,17 @@ SwFieldType* DocumentFieldsManager::GetFieldType( { SwFieldType* pFieldType = (*mpFieldTypes)[i]; - OUString aFieldName( pFieldType->GetName() ); - if (bDbFieldMatching && nResId == SwFieldIds::Database) // #i51815# - aFieldName = aFieldName.replace(DB_DELIM, '.'); - - if( nResId == pFieldType->Which() && - rSCmp.isEqual( rName, aFieldName )) + if (nResId == pFieldType->Which()) { - pRet = pFieldType; - break; + OUString aFieldName( pFieldType->GetName() ); + if (bDbFieldMatching && nResId == SwFieldIds::Database) // #i51815# + aFieldName = aFieldName.replace(DB_DELIM, '.'); + + if (rSCmp.isEqual( rName, aFieldName )) + { + pRet = pFieldType; + break; + } } } return pRet; |