summaryrefslogtreecommitdiff
path: root/framework
diff options
context:
space:
mode:
authorNoel Grandin <noel@peralex.com>2013-10-25 17:35:42 +0200
committerNoel Grandin <noel@peralex.com>2013-11-04 10:11:08 +0200
commit5285beeaa49f6678b471d472868c305c7d9da5f9 (patch)
treea3dbd28995142ab16b448f28e95821115ef5408f /framework
parentaeb41c9b9b7559c6d87bf92807acdc0df9e104cc (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.cxx2
-rw-r--r--framework/source/fwe/classes/bmkmenu.cxx2
-rw-r--r--framework/source/uielement/menubarmerger.cxx2
-rw-r--r--framework/source/uielement/toolbarmanager.cxx8
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));
}