diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2017-11-27 13:26:48 +0200 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2017-11-28 08:53:02 +0100 |
| commit | cf2c667fbb41bf7d9e172d940c25abb78155efe2 (patch) | |
| tree | 823317cf3c05dd6a49d4002de55b7539c151fc06 /sfx2 | |
| parent | 8d0e8d5d291cbb98de6964eab2f629a405c7e813 (diff) | |
Don't copy the string
Change-Id: If50d952acfea0be86d36bc5b23a0f087eff520c8
Reviewed-on: https://gerrit.libreoffice.org/45325
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'sfx2')
| -rw-r--r-- | sfx2/source/appl/appuno.cxx | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/sfx2/source/appl/appuno.cxx b/sfx2/source/appl/appuno.cxx index ca9ca3a13794..a5bc1ea0b959 100644 --- a/sfx2/source/appl/appuno.cxx +++ b/sfx2/source/appl/appuno.cxx @@ -208,8 +208,8 @@ void TransformParameters( sal_uInt16 nSlotId, const uno::Sequence<beans::Propert sal_uInt16 nSubCount = pType->nAttribs; const beans::PropertyValue& rProp = pPropsVal[0]; - OUString aName = rProp.Name; - if ( nCount == 1 && aName == OUString( pSlot->pUnoName, strlen( pSlot->pUnoName ), RTL_TEXTENCODING_UTF8 ) ) + const OUString& rName = rProp.Name; + if ( nCount == 1 && rName == OUString( pSlot->pUnoName, strlen( pSlot->pUnoName ), RTL_TEXTENCODING_UTF8 ) ) { // there is only one parameter and its name matches the name of the property, // so it's either a simple property or a complex property in one single UNO struct @@ -225,7 +225,7 @@ void TransformParameters( sal_uInt16 nSlotId, const uno::Sequence<beans::Propert else if ( nSubCount == 0 ) { // for a simple property there can be only one parameter and its name *must* match - SAL_WARN("sfx.appl", "Property name does not match: " << aName); + SAL_WARN("sfx.appl", "Property name does not match: " << rName); } #endif else @@ -309,8 +309,8 @@ void TransformParameters( sal_uInt16 nSlotId, const uno::Sequence<beans::Propert for ( sal_Int32 n=0; n<nCount; n++ ) { const beans::PropertyValue& rProp = pPropsVal[n]; - OUString aName = rProp.Name; - if ( aName == OUString( rArg.pName, strlen(rArg.pName), RTL_TEXTENCODING_UTF8 ) ) + const OUString& rName = rProp.Name; + if ( rName == OUString( rArg.pName, strlen(rArg.pName), RTL_TEXTENCODING_UTF8 ) ) { #ifdef DBG_UTIL ++nFoundArgs; @@ -333,8 +333,8 @@ void TransformParameters( sal_uInt16 nSlotId, const uno::Sequence<beans::Propert for ( sal_Int32 n=0; n<nCount; n++ ) { const beans::PropertyValue& rProp = pPropsVal[n]; - OUString aName = rProp.Name; - if ( aName == OUString(rArg.pName, strlen(rArg.pName), RTL_TEXTENCODING_UTF8) ) + const OUString& rName = rProp.Name; + if ( rName == OUString(rArg.pName, strlen(rArg.pName), RTL_TEXTENCODING_UTF8) ) { bAsWholeItem = true; #ifdef DBG_UTIL @@ -404,15 +404,15 @@ void TransformParameters( sal_uInt16 nSlotId, const uno::Sequence<beans::Propert for ( sal_Int32 n=0; n<nCount; n++ ) { const beans::PropertyValue& rProp = pPropsVal[n]; - OUString aName = rProp.Name; - if ( aName == sFrame ) + const OUString& rName = rProp.Name; + if ( rName == sFrame ) { Reference< XFrame > xFrame; OSL_VERIFY( rProp.Value >>= xFrame ); rSet.Put( SfxUnoFrameItem( SID_FILLFRAME, xFrame ) ); } else - if ( aName == sHidden ) + if ( rName == sHidden ) { bool bVal = false; if (rProp.Value >>= bVal) |