diff options
| author | Noel Grandin <noel@peralex.com> | 2013-10-25 17:35:42 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2013-11-04 10:11:08 +0200 |
| commit | 5285beeaa49f6678b471d472868c305c7d9da5f9 (patch) | |
| tree | a3dbd28995142ab16b448f28e95821115ef5408f /framework | |
| parent | aeb41c9b9b7559c6d87bf92807acdc0df9e104cc (diff) | |
remove redundant calls to OUString constructor in if expression
Convert code like:
if( aStr == OUString("xxxx") )
to this:
if( aStr == "xxxx" )
Change-Id: I8d201f048477731eff590fb988259ef0935c080c
Diffstat (limited to 'framework')
| -rw-r--r-- | framework/source/fwe/classes/addonmenu.cxx | 2 | ||||
| -rw-r--r-- | framework/source/fwe/classes/bmkmenu.cxx | 2 | ||||
| -rw-r--r-- | framework/source/uielement/menubarmerger.cxx | 2 | ||||
| -rw-r--r-- | framework/source/uielement/toolbarmanager.cxx | 8 |
4 files changed, 7 insertions, 7 deletions
diff --git a/framework/source/fwe/classes/addonmenu.cxx b/framework/source/fwe/classes/addonmenu.cxx index ab8e4001377b..3fb7b8a90914 100644 --- a/framework/source/fwe/classes/addonmenu.cxx +++ b/framework/source/fwe/classes/addonmenu.cxx @@ -298,7 +298,7 @@ void AddonMenuManager::BuildMenu( PopupMenu* pCurrent if ( !IsCorrectContext( rModel, aContext ) || ( aTitle.isEmpty() && aURL.isEmpty() )) continue; - if ( aURL == OUString( "private:separator" )) + if ( aURL == "private:separator" ) bInsertSeparator = sal_True; else { diff --git a/framework/source/fwe/classes/bmkmenu.cxx b/framework/source/fwe/classes/bmkmenu.cxx index b15c9f4f9908..78b02981db65 100644 --- a/framework/source/fwe/classes/bmkmenu.cxx +++ b/framework/source/fwe/classes/bmkmenu.cxx @@ -134,7 +134,7 @@ void BmkMenu::Initialize() if ( aTitle.isEmpty() && aURL.isEmpty() ) continue; - if ( aURL == OUString( "private:separator" )) + if ( aURL == "private:separator" ) InsertSeparator(); else { diff --git a/framework/source/uielement/menubarmerger.cxx b/framework/source/uielement/menubarmerger.cxx index 74170745d1d3..b814e770ff50 100644 --- a/framework/source/uielement/menubarmerger.cxx +++ b/framework/source/uielement/menubarmerger.cxx @@ -149,7 +149,7 @@ sal_uInt16 MenuBarMerger::FindMenuItem( const OUString& rCmd, Menu* pCurrMenu ) const sal_uInt16 nItemId = pCurrMenu->GetItemId( i ); if ( nItemId > 0 ) { - if ( rCmd == OUString( pCurrMenu->GetItemCommand( nItemId ))) + if ( rCmd == pCurrMenu->GetItemCommand( nItemId ) ) return i; } } diff --git a/framework/source/uielement/toolbarmanager.cxx b/framework/source/uielement/toolbarmanager.cxx index 87413661ce6c..db552106ef36 100644 --- a/framework/source/uielement/toolbarmanager.cxx +++ b/framework/source/uielement/toolbarmanager.cxx @@ -1037,10 +1037,10 @@ void ToolBarManager::CreateControllers() if (pController) { - if(aCommandURL == OUString( ".uno:SwitchXFormsDesignMode" ) || - aCommandURL == OUString( ".uno:ViewDataSourceBrowser" ) || - aCommandURL == OUString( ".uno:ParaLeftToRight" ) || - aCommandURL == OUString( ".uno:ParaRightToLeft" ) + if(aCommandURL == ".uno:SwitchXFormsDesignMode" || + aCommandURL == ".uno:ViewDataSourceBrowser" || + aCommandURL == ".uno:ParaLeftToRight" || + aCommandURL == ".uno:ParaRightToLeft" ) pController->setFastPropertyValue_NoBroadcast(1,makeAny(sal_True)); } |