[PATCH] Fix for fdo43460 Part XX getLength() to isEmpty()
Olivier Hallot
olivier.hallot at alta.org.br
Mon Dec 26 08:20:50 PST 2011
Part XX
Module
framework
---
.../accelerators/acceleratorconfiguration.cxx | 36 +++++++-------
.../moduleacceleratorconfiguration.cxx | 2 +-
framework/source/accelerators/presethandler.cxx | 4 +-
framework/source/accelerators/storageholder.cxx | 2 +-
framework/source/classes/fwktabwindow.cxx | 2 +-
framework/source/classes/menumanager.cxx | 8 ++--
framework/source/dispatch/closedispatcher.cxx | 2 +-
framework/source/dispatch/dispatchprovider.cxx | 4 +-
framework/source/dispatch/menudispatcher.cxx | 4 +-
framework/source/dispatch/servicehandler.cxx | 2 +-
framework/source/fwe/classes/addonmenu.cxx | 8 ++--
framework/source/fwe/classes/addonsoptions.cxx | 16 +++---
framework/source/fwe/classes/bmkmenu.cxx | 4 +-
.../source/fwe/helper/actiontriggerhelper.cxx | 2 +-
framework/source/fwe/helper/titlehelper.cxx | 10 ++--
framework/source/fwe/xml/menudocumenthandler.cxx | 22 ++++----
.../source/fwe/xml/statusbardocumenthandler.cxx | 4 +-
.../source/fwe/xml/toolboxdocumenthandler.cxx | 18 +++---
framework/source/fwe/xml/xmlnamespaces.cxx | 8 ++--
framework/source/fwi/helper/mischelper.cxx | 4 +-
framework/source/helper/persistentwindowstate.cxx | 4 +-
framework/source/helper/titlebarupdate.cxx | 2 +-
framework/source/jobs/helponstartup.cxx | 8 ++--
framework/source/jobs/jobexecutor.cxx | 4 +-
framework/source/jobs/joburl.cxx | 24 ++++----
framework/source/jobs/shelljob.cxx | 2 +-
framework/source/layoutmanager/helpers.cxx | 2 +-
framework/source/layoutmanager/layoutmanager.cxx | 4 +-
.../source/layoutmanager/toolbarlayoutmanager.cxx | 24 ++++----
framework/source/loadenv/loadenv.cxx | 24 ++++----
framework/source/loadenv/targethelper.cxx | 4 +-
framework/source/recording/dispatchrecorder.cxx | 2 +-
framework/source/services/autorecovery.cxx | 54 ++++++++++----------
framework/source/services/backingwindow.cxx | 6 +-
framework/source/services/desktop.cxx | 2 +-
framework/source/services/dispatchhelper.cxx | 2 +-
framework/source/services/frame.cxx | 2 +-
framework/source/services/license.cxx | 4 +-
framework/source/services/modulemanager.cxx | 2 +-
framework/source/services/pathsettings.cxx | 6 +-
framework/source/services/substitutepathvars.cxx | 10 ++--
framework/source/services/taskcreatorsrv.cxx | 4 +-
framework/source/services/urltransformer.cxx | 12 ++--
.../source/uiconfiguration/imagemanagerimpl.cxx | 2 +-
.../source/uiconfiguration/moduleuicfgsupplier.cxx | 2 +-
.../moduleuiconfigurationmanager.cxx | 6 +-
.../uiconfiguration/uiconfigurationmanager.cxx | 4 +-
.../uiconfiguration/windowstateconfiguration.cxx | 2 +-
.../source/uielement/addonstoolbarmanager.cxx | 6 +-
.../source/uielement/buttontoolbarcontroller.cxx | 2 +-
.../source/uielement/complextoolbarcontroller.cxx | 8 ++--
.../source/uielement/fontsizemenucontroller.cxx | 2 +-
.../source/uielement/generictoolbarcontroller.cxx | 2 +-
.../source/uielement/macrosmenucontroller.cxx | 2 +-
framework/source/uielement/menubarmanager.cxx | 18 +++---
framework/source/uielement/menubarmerger.cxx | 4 +-
framework/source/uielement/newmenucontroller.cxx | 8 ++--
.../uielement/spinfieldtoolbarcontroller.cxx | 10 ++--
framework/source/uielement/statusbarmanager.cxx | 2 +-
framework/source/uielement/toolbarmanager.cxx | 14 +++---
framework/source/uielement/toolbarmerger.cxx | 2 +-
.../source/uielement/toolbarsmenucontroller.cxx | 10 ++--
.../source/uielement/uicommanddescription.cxx | 4 +-
.../source/uifactory/addonstoolboxfactory.cxx | 4 +-
.../source/uifactory/factoryconfiguration.cxx | 4 +-
framework/source/uifactory/menubarfactory.cxx | 2 +-
.../source/uifactory/toolbarcontrollerfactory.cxx | 6 +-
.../source/uifactory/uielementfactorymanager.cxx | 4 +-
.../uifactory/windowcontentfactorymanager.cxx | 10 ++--
.../source/xml/acceleratorconfigurationreader.cxx | 2 +-
framework/source/xml/imagesdocumenthandler.cxx | 2 +-
71 files changed, 257 insertions(+), 257 deletions(-)
diff --git a/framework/source/accelerators/acceleratorconfiguration.cxx b/framework/source/accelerators/acceleratorconfiguration.cxx
index f6f1c88..25f1627 100644
--- a/framework/source/accelerators/acceleratorconfiguration.cxx
+++ b/framework/source/accelerators/acceleratorconfiguration.cxx
@@ -186,7 +186,7 @@ void SAL_CALL XMLBasedAcceleratorConfiguration::setKeyEvent(const css::awt::KeyE
static_cast< ::cppu::OWeakObject* >(this),
0);
- if (!sCommand.getLength())
+ if (sCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -226,7 +226,7 @@ css::uno::Sequence< css::awt::KeyEvent > SAL_CALL XMLBasedAcceleratorConfigurati
css::container::NoSuchElementException,
css::uno::RuntimeException )
{
- if (!sCommand.getLength())
+ if (sCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -263,7 +263,7 @@ css::uno::Sequence< css::uno::Any > SAL_CALL XMLBasedAcceleratorConfiguration::g
for (i=0; i<c; ++i)
{
const ::rtl::OUString& rCommand = lCommandList[i];
- if (!rCommand.getLength())
+ if (rCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -292,7 +292,7 @@ void SAL_CALL XMLBasedAcceleratorConfiguration::removeCommandFromAllKeyEvents(co
css::container::NoSuchElementException,
css::uno::RuntimeException )
{
- if (!sCommand.getLength())
+ if (sCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -643,7 +643,7 @@ AcceleratorCache& XMLBasedAcceleratorConfiguration::impl_getCFG(sal_Bool bWriteA
::rtl::OUString sISOLocale;
xProp->getPropertyValue(LOCALE_KEY) >>= sISOLocale;
- if (!sISOLocale.getLength())
+ if (sISOLocale.isEmpty())
return ::comphelper::Locale::EN_US();
return ::comphelper::Locale(sISOLocale);
}
@@ -757,7 +757,7 @@ void SAL_CALL XCUBasedAcceleratorConfiguration::setKeyEvent(const css::awt::KeyE
static_cast< ::cppu::OWeakObject* >(this),
0);
- if (!sCommand.getLength())
+ if (sCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -844,7 +844,7 @@ void SAL_CALL XCUBasedAcceleratorConfiguration::removeKeyEvent(const css::awt::K
if (rPrimaryCache.hasKey(aKeyEvent))
{
::rtl::OUString sDelCommand = rPrimaryCache.getCommandByKey(aKeyEvent);
- if (sDelCommand.getLength() > 0)
+ if (!sDelCommand.isEmpty())
{
::rtl::OUString sOriginalCommand = rPrimaryCache.getCommandByKey(aKeyEvent);
if (rSecondaryCache.hasCommand(sOriginalCommand))
@@ -861,7 +861,7 @@ void SAL_CALL XCUBasedAcceleratorConfiguration::removeKeyEvent(const css::awt::K
else
{
::rtl::OUString sDelCommand = rSecondaryCache.getCommandByKey(aKeyEvent);
- if (sDelCommand.getLength() > 0)
+ if (!sDelCommand.isEmpty())
rSecondaryCache.removeKey(aKeyEvent);
}
@@ -874,7 +874,7 @@ css::uno::Sequence< css::awt::KeyEvent > SAL_CALL XCUBasedAcceleratorConfigurati
css::container::NoSuchElementException,
css::uno::RuntimeException )
{
- if (!sCommand.getLength())
+ if (sCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -938,7 +938,7 @@ css::uno::Sequence< css::uno::Any > SAL_CALL XCUBasedAcceleratorConfiguration::g
for (i=0; i<c; ++i)
{
const ::rtl::OUString& rCommand = lCommandList[i];
- if (!rCommand.getLength())
+ if (rCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -971,7 +971,7 @@ void SAL_CALL XCUBasedAcceleratorConfiguration::removeCommandFromAllKeyEvents(co
css::container::NoSuchElementException,
css::uno::RuntimeException )
{
- if (!sCommand.getLength())
+ if (sCommand.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Empty command strings are not allowed here.")),
static_cast< ::cppu::OWeakObject* >(this),
@@ -1249,7 +1249,7 @@ void SAL_CALL XCUBasedAcceleratorConfiguration::changesOccurred(const css::util:
{
::rtl::OUString sModule;
sKey = ::utl::extractFirstFromConfigurationPath(sPath, &sPath);
- if (( sKey.getLength() > 0 ) && ( sPath.getLength() > 0 ))
+ if (( !sKey.isEmpty() ) && ( !sPath.isEmpty() ))
reloadChanged(sPrimarySecondary, sGlobalModules, sModule, sKey);
}
else if ( sGlobalModules.equals(CFG_ENTRY_MODULES) )
@@ -1257,7 +1257,7 @@ void SAL_CALL XCUBasedAcceleratorConfiguration::changesOccurred(const css::util:
::rtl::OUString sModule = ::utl::extractFirstFromConfigurationPath(sPath, &sPath);
sKey = ::utl::extractFirstFromConfigurationPath(sPath, &sPath);
- if (( sKey.getLength() > 0 ) && ( sPath.getLength() > 0 ))
+ if (( !sKey.isEmpty() ) && ( !sPath.isEmpty() ))
{
reloadChanged(sPrimarySecondary, sGlobalModules, sModule, sKey);
}
@@ -1328,7 +1328,7 @@ void XCUBasedAcceleratorConfiguration::impl_ts_load( sal_Bool bPreferred, const
::rtl::OUString sLocale = *pFound;
::rtl::OUString sCommand;
xCommand->getByName(sLocale) >>= sCommand;
- if (sCommand.getLength()<1)
+ if (sCommand.isEmpty())
continue;
css::awt::KeyEvent aKeyEvent;
@@ -1349,7 +1349,7 @@ void XCUBasedAcceleratorConfiguration::impl_ts_load( sal_Bool bPreferred, const
sToken[k] = sKey.getToken(0, '_', nIndex);
::rtl::OUString sTest = sToken[k];
- if (sToken[k].getLength() < 1)
+ if (sToken[k].isEmpty())
{
bValid = sal_False;
break;
@@ -1604,14 +1604,14 @@ void XCUBasedAcceleratorConfiguration::reloadChanged( const ::rtl::OUString& sPr
if (sPrimarySecondary.equals(CFG_ENTRY_PRIMARY))
{
- if (sCommand.getLength() ==0)
+ if (sCommand.isEmpty())
m_aPrimaryReadCache.removeKey(aKeyEvent);
else
m_aPrimaryReadCache.setKeyCommandPair(aKeyEvent, sCommand);
}
else if (sPrimarySecondary.equals(CFG_ENTRY_SECONDARY))
{
- if (sCommand.getLength() ==0)
+ if (sCommand.isEmpty())
m_aSecondaryReadCache.removeKey(aKeyEvent);
else
m_aSecondaryReadCache.setKeyCommandPair(aKeyEvent, sCommand);
@@ -1685,7 +1685,7 @@ AcceleratorCache& XCUBasedAcceleratorConfiguration::impl_getCFG(sal_Bool bPrefer
::rtl::OUString sISOLocale;
xProp->getPropertyValue(LOCALE_KEY) >>= sISOLocale;
- if (!sISOLocale.getLength())
+ if (sISOLocale.isEmpty())
return ::comphelper::Locale::EN_US();
return ::comphelper::Locale(sISOLocale);
}
diff --git a/framework/source/accelerators/moduleacceleratorconfiguration.cxx b/framework/source/accelerators/moduleacceleratorconfiguration.cxx
index 3b96453..9c3fb8f 100644
--- a/framework/source/accelerators/moduleacceleratorconfiguration.cxx
+++ b/framework/source/accelerators/moduleacceleratorconfiguration.cxx
@@ -115,7 +115,7 @@ void SAL_CALL ModuleAcceleratorConfiguration::initialize(const css::uno::Sequenc
m_sModule = lArgs.getUnpackedValueOrDefault(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("ModuleIdentifier")), ::rtl::OUString());
m_sLocale = lArgs.getUnpackedValueOrDefault(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Locale")) , ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("x-default")));
- if (!m_sModule.getLength())
+ if (m_sModule.isEmpty())
throw css::uno::RuntimeException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("The module dependend accelerator configuration service was initialized with an empty module identifier!")),
static_cast< ::cppu::OWeakObject* >(this));
diff --git a/framework/source/accelerators/presethandler.cxx b/framework/source/accelerators/presethandler.cxx
index 28e270d..d276b48 100644
--- a/framework/source/accelerators/presethandler.cxx
+++ b/framework/source/accelerators/presethandler.cxx
@@ -759,7 +759,7 @@ void PresetHandler::addStorageListener(IStorageListener* pListener)
aReadLock.unlock();
// <- SAFE ----------------------------------
- if (!sRelPath.getLength())
+ if (sRelPath.isEmpty())
return;
switch(eCfgType)
@@ -789,7 +789,7 @@ void PresetHandler::removeStorageListener(IStorageListener* pListener)
aReadLock.unlock();
// <- SAFE ----------------------------------
- if (!sRelPath.getLength())
+ if (sRelPath.isEmpty())
return;
switch(eCfgType)
diff --git a/framework/source/accelerators/storageholder.cxx b/framework/source/accelerators/storageholder.cxx
index 7bb720e..d60e18d 100644
--- a/framework/source/accelerators/storageholder.cxx
+++ b/framework/source/accelerators/storageholder.cxx
@@ -591,7 +591,7 @@ css::uno::Reference< css::io::XStream > StorageHolder::openSubStreamWithFallback
sNormedPath += sNormedPath.copy(1);
// "/" => "" || "" => "" ?
- if (sNormedPath.getLength() < 1)
+ if (sNormedPath.isEmpty())
return ::rtl::OUString();
// "bla" => "bla/"
diff --git a/framework/source/classes/fwktabwindow.cxx b/framework/source/classes/fwktabwindow.cxx
index 34e8b7a..3497e78 100644
--- a/framework/source/classes/fwktabwindow.cxx
+++ b/framework/source/classes/fwktabwindow.cxx
@@ -361,7 +361,7 @@ FwkTabPage* FwkTabWindow::AddTabPage( sal_Int32 nIndex, const uno::Sequence< bea
m_TabList.push_back( pEntry );
sal_uInt16 nIdx = static_cast< sal_uInt16 >( nIndex );
m_aTabCtrl.InsertPage( nIdx, sTitle );
- if ( sToolTip.getLength() > 0 )
+ if ( !sToolTip.isEmpty() )
m_aTabCtrl.SetHelpText( nIdx, sToolTip );
if ( xImage.is() )
m_aTabCtrl.SetPageImage( nIdx, Image( xImage ) );
diff --git a/framework/source/classes/menumanager.cxx b/framework/source/classes/menumanager.cxx
index 404f3c2..e2a08de 100644
--- a/framework/source/classes/menumanager.cxx
+++ b/framework/source/classes/menumanager.cxx
@@ -261,7 +261,7 @@ MenuManager::MenuManager(
MenuConfiguration::Attributes* pMenuAttributes =
(MenuConfiguration::Attributes*)pMenu->GetUserValue( nItemId );
- if ( pMenuAttributes && pMenuAttributes->aImageId.getLength() > 0 )
+ if ( pMenuAttributes && !pMenuAttributes->aImageId.isEmpty() )
{
// Retrieve image id from menu attributes
aImage = GetImageFromURL( rFrame, aImageId, false );
@@ -833,7 +833,7 @@ IMPL_LINK( MenuManager, Activate, Menu *, pMenu )
pMenuItemHandler->nItemId > END_ITEMID_WINDOWLIST )
{
::rtl::OUString aItemCommand = pMenu->GetItemCommand( pMenuItemHandler->nItemId );
- if ( !aItemCommand.getLength() )
+ if ( aItemCommand.isEmpty() )
{
const static ::rtl::OUString aSlotString( RTL_CONSTASCII_USTRINGPARAM( "slot:" ));
aItemCommand = aSlotString;
@@ -986,7 +986,7 @@ sal_uInt16 MenuManager::FillItemCommand(::rtl::OUString& _rItemCommand,Menu* _pM
sal_uInt16 nItemId = _pMenu->GetItemId( _nIndex );
_rItemCommand = _pMenu->GetItemCommand( nItemId );
- if ( !_rItemCommand.getLength() )
+ if ( _rItemCommand.isEmpty() )
{
const static ::rtl::OUString aSlotString( RTL_CONSTASCII_USTRINGPARAM( "slot:" ));
_rItemCommand = aSlotString;
@@ -1023,7 +1023,7 @@ void MenuManager::FillMenuImages(Reference< XFrame >& _xFrame,Menu* _pMenu,sal_B
if ( pMenuAttributes )
aImageId = pMenuAttributes->aImageId; // Retrieve image id from menu attributes
- if ( aImageId.getLength() > 0 )
+ if ( !aImageId.isEmpty() )
{
Image aImage = GetImageFromURL( _xFrame, aImageId, false );
if ( !!aImage )
diff --git a/framework/source/dispatch/closedispatcher.cxx b/framework/source/dispatch/closedispatcher.cxx
index c7ea2a3..067ea42 100644
--- a/framework/source/dispatch/closedispatcher.cxx
+++ b/framework/source/dispatch/closedispatcher.cxx
@@ -608,7 +608,7 @@ css::uno::Reference< css::frame::XFrame > CloseDispatcher::static_impl_searchRig
if (sTarget.equalsIgnoreAsciiCaseAscii("_self"))
return xFrame;
- OSL_ENSURE((sTarget.getLength() < 1), "CloseDispatch used for unexpected target. Magic things will happen now .-)");
+ OSL_ENSURE((sTarget.isEmpty()), "CloseDispatch used for unexpected target. Magic things will happen now .-)");
css::uno::Reference< css::frame::XFrame > xTarget = xFrame;
while(sal_True)
diff --git a/framework/source/dispatch/dispatchprovider.cxx b/framework/source/dispatch/dispatchprovider.cxx
index d54ffee..51633f6 100644
--- a/framework/source/dispatch/dispatchprovider.cxx
+++ b/framework/source/dispatch/dispatchprovider.cxx
@@ -257,7 +257,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryDeskt
if (
(sTargetFrameName==SPECIALTARGET_SELF) ||
(sTargetFrameName==SPECIALTARGET_TOP ) ||
- (sTargetFrameName.getLength()<1 )
+ (sTargetFrameName.isEmpty() )
)
{
xDispatcher = implts_searchProtocolHandler(aURL);
@@ -408,7 +408,7 @@ css::uno::Reference< css::frame::XDispatch > DispatchProvider::implts_queryFrame
else
if (
(sTargetName==SPECIALTARGET_SELF) ||
- (sTargetName.getLength()<1 )
+ (sTargetName.isEmpty() )
)
{
// There exist a hard coded interception for special URLs.
diff --git a/framework/source/dispatch/menudispatcher.cxx b/framework/source/dispatch/menudispatcher.cxx
index 5a80472..df6f90c 100644
--- a/framework/source/dispatch/menudispatcher.cxx
+++ b/framework/source/dispatch/menudispatcher.cxx
@@ -414,7 +414,7 @@ sal_Bool MenuDispatcher::impldbg_checkParameter_addStatusListener( const uno::
if (
( &xControl == NULL ) ||
( &aURL == NULL ) ||
- ( aURL.Complete.getLength() < 1 )
+ ( aURL.Complete.isEmpty() )
)
{
bOK = sal_False ;
@@ -435,7 +435,7 @@ sal_Bool MenuDispatcher::impldbg_checkParameter_removeStatusListener( const u
if (
( &xControl == NULL ) ||
( &aURL == NULL ) ||
- ( aURL.Complete.getLength() < 1 )
+ ( aURL.Complete.isEmpty() )
)
{
bOK = sal_False ;
diff --git a/framework/source/dispatch/servicehandler.cxx b/framework/source/dispatch/servicehandler.cxx
index f9b8d5b..51de3d4 100644
--- a/framework/source/dispatch/servicehandler.cxx
+++ b/framework/source/dispatch/servicehandler.cxx
@@ -292,7 +292,7 @@ css::uno::Reference< css::uno::XInterface > ServiceHandler::implts_dispatch( con
sServiceName = sServiceAndArguments;
}
- if (!sServiceName.getLength())
+ if (sServiceName.isEmpty())
return css::uno::Reference< css::uno::XInterface >();
// If a service doesnt support an optional job executor interface - he can't get
diff --git a/framework/source/fwe/classes/addonmenu.cxx b/framework/source/fwe/classes/addonmenu.cxx
index 69d3a79..1afe4c2 100644
--- a/framework/source/fwe/classes/addonmenu.cxx
+++ b/framework/source/fwe/classes/addonmenu.cxx
@@ -269,8 +269,8 @@ void AddonMenuManager::MergeAddonPopupMenus( const Reference< XFrame >& rFrame,
aImageId,
aContext,
aAddonSubMenu );
- if ( aTitle.getLength() > 0 &&
- aURL.getLength() > 0 &&
+ if ( !aTitle.isEmpty() &&
+ !aURL.isEmpty() &&
aAddonSubMenu.getLength() > 0 &&
AddonMenuManager::IsCorrectContext( rModel, aContext ))
{
@@ -321,7 +321,7 @@ void AddonMenuManager::BuildMenu( PopupMenu* pCurrent
{
GetMenuEntry( aAddonMenuDefinition[i], aTitle, aURL, aTarget, aImageId, aContext, aAddonSubMenu );
- if ( !IsCorrectContext( rModel, aContext ) || ( !aTitle.getLength() && !aURL.getLength() ))
+ if ( !IsCorrectContext( rModel, aContext ) || ( aTitle.isEmpty() && aURL.isEmpty() ))
continue;
if ( aURL == ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "private:separator" )))
@@ -420,7 +420,7 @@ sal_Bool AddonMenuManager::IsCorrectContext( const Reference< XModel >& rModel,
}
}
- return ( aContext.getLength() == 0 );
+ return ( aContext.isEmpty() );
}
}
diff --git a/framework/source/fwe/classes/addonsoptions.cxx b/framework/source/fwe/classes/addonsoptions.cxx
index 84d6769..d5e4561 100644
--- a/framework/source/fwe/classes/addonsoptions.cxx
+++ b/framework/source/fwe/classes/addonsoptions.cxx
@@ -864,7 +864,7 @@ sal_Bool AddonsOptions_Impl::ReadImages( ImageManager& aImageManager )
// An user-defined image entry must have an URL. As "ImageIdentifier" has a higher priority
// we also check if we already have an images association.
if (( aAddonImageItemNodeValues[0] >>= aURL ) &&
- aURL.getLength() > 0 &&
+ !aURL.isEmpty() &&
!HasAssociatedImages( aURL ))
{
::rtl::OUStringBuffer aBuf( aImagesItemNode );
@@ -1091,7 +1091,7 @@ sal_Bool AddonsOptions_Impl::ReadMenuItem( const ::rtl::OUString& aMenuNodeName,
Sequence< Any > aMenuItemNodePropValues;
aMenuItemNodePropValues = GetProperties( GetPropertyNamesMenuItem( aAddonMenuItemTreeNode ) );
- if (( aMenuItemNodePropValues[ OFFSET_MENUITEM_TITLE ] >>= aStrValue ) && aStrValue.getLength() > 0 )
+ if (( aMenuItemNodePropValues[ OFFSET_MENUITEM_TITLE ] >>= aStrValue ) && !aStrValue.isEmpty() )
{
aMenuItem[ OFFSET_MENUITEM_TITLE ].Value <<= aStrValue;
@@ -1122,7 +1122,7 @@ sal_Bool AddonsOptions_Impl::ReadMenuItem( const ::rtl::OUString& aMenuNodeName,
aMenuItem[ OFFSET_MENUITEM_SUBMENU ].Value <<= aSubMenuSeq;
bResult = sal_True;
}
- else if (( aMenuItemNodePropValues[ OFFSET_MENUITEM_URL ] >>= aStrValue ) && aStrValue.getLength() > 0 )
+ else if (( aMenuItemNodePropValues[ OFFSET_MENUITEM_URL ] >>= aStrValue ) && !aStrValue.isEmpty() )
{
// A simple menu item => read the other properties;
::rtl::OUString aMenuImageId;
@@ -1166,7 +1166,7 @@ sal_Bool AddonsOptions_Impl::ReadPopupMenu( const ::rtl::OUString& aPopupMenuNod
aPopupMenuNodePropValues = GetProperties( GetPropertyNamesPopupMenu( aAddonPopupMenuTreeNode ) );
if (( aPopupMenuNodePropValues[ OFFSET_POPUPMENU_TITLE ] >>= aStrValue ) &&
- aStrValue.getLength() > 0 )
+ !aStrValue.isEmpty() )
{
aPopupMenu[ OFFSET_POPUPMENU_TITLE ].Value <<= aStrValue;
@@ -1230,7 +1230,7 @@ sal_Bool AddonsOptions_Impl::ReadToolBarItem( const ::rtl::OUString& aToolBarIte
aToolBarItemNodePropValues = GetProperties( GetPropertyNamesToolBarItem( aAddonToolBarItemTreeNode ) );
// A toolbar item must have a command URL
- if (( aToolBarItemNodePropValues[ OFFSET_TOOLBARITEM_URL ] >>= aURL ) && aURL.getLength() > 0 )
+ if (( aToolBarItemNodePropValues[ OFFSET_TOOLBARITEM_URL ] >>= aURL ) && !aURL.isEmpty() )
{
if ( aURL.equals( SEPARATOR_URL ))
{
@@ -1245,7 +1245,7 @@ sal_Bool AddonsOptions_Impl::ReadToolBarItem( const ::rtl::OUString& aToolBarIte
bResult = sal_True;
}
- else if (( aToolBarItemNodePropValues[ OFFSET_TOOLBARITEM_TITLE ] >>= aTitle ) && aTitle.getLength() > 0 )
+ else if (( aToolBarItemNodePropValues[ OFFSET_TOOLBARITEM_TITLE ] >>= aTitle ) && !aTitle.isEmpty() )
{
// A normal toolbar item must also have title => read the other properties;
::rtl::OUString aImageId;
@@ -1384,7 +1384,7 @@ void AddonsOptions_Impl::ReadAndAssociateImages( const ::rtl::OUString& aURL, co
const char* aExtArray[MAX_NUM_IMAGES] = { "_16", "_26" };
const char* pBmpExt = ".bmp";
- if ( aImageId.getLength() == 0 )
+ if ( aImageId.isEmpty() )
return;
bool bImageFound = true;
@@ -1466,7 +1466,7 @@ AddonsOptions_Impl::ImageEntry* AddonsOptions_Impl::ReadImageData( const ::rtl::
// has a higher priority.
aPropertyData[i] >>= aImageURL;
- if ( aImageURL.getLength() > 0 )
+ if ( !aImageURL.isEmpty() )
{
Image aImage;
Image aImageNoScale;
diff --git a/framework/source/fwe/classes/bmkmenu.cxx b/framework/source/fwe/classes/bmkmenu.cxx
index 1bb977c..8b8ac60 100644
--- a/framework/source/fwe/classes/bmkmenu.cxx
+++ b/framework/source/fwe/classes/bmkmenu.cxx
@@ -158,7 +158,7 @@ void BmkMenu::Initialize()
{
GetMenuEntry( aDynamicMenuEntries[i], aTitle, aURL, aTargetFrame, aImageId );
- if ( !aTitle.getLength() && !aURL.getLength() )
+ if ( aTitle.isEmpty() && aURL.isEmpty() )
continue;
if ( aURL == ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "private:separator" )))
@@ -170,7 +170,7 @@ void BmkMenu::Initialize()
if ( bShowMenuImages )
{
- if ( aImageId.getLength() > 0 )
+ if ( !aImageId.isEmpty() )
{
Image aImage = GetImageFromURL( m_xFrame, aImageId, false );
if ( !!aImage )
diff --git a/framework/source/fwe/helper/actiontriggerhelper.cxx b/framework/source/fwe/helper/actiontriggerhelper.cxx
index f098c02..6c1cc69 100644
--- a/framework/source/fwe/helper/actiontriggerhelper.cxx
+++ b/framework/source/fwe/helper/actiontriggerhelper.cxx
@@ -275,7 +275,7 @@ Reference< XPropertySet > CreateActionTrigger( sal_uInt16 nItemId, const Menu* p
OUString aCommandURL = pMenu->GetItemCommand( nItemId );
- if ( aCommandURL.getLength() == 0 )
+ if ( aCommandURL.isEmpty() )
{
aCommandURL = OUString( RTL_CONSTASCII_USTRINGPARAM( "slot:" ));
aCommandURL += OUString::valueOf( (sal_Int32)nItemId );
diff --git a/framework/source/fwe/helper/titlehelper.cxx b/framework/source/fwe/helper/titlehelper.cxx
index 3a4bdd3..07ff57d 100644
--- a/framework/source/fwe/helper/titlehelper.cxx
+++ b/framework/source/fwe/helper/titlehelper.cxx
@@ -121,7 +121,7 @@ void TitleHelper::setOwner(const css::uno::Reference< css::uno::XInterface >& xO
return m_sTitle;
// Title seams to be up-to-date. Return it directly.
- if (m_sTitle.getLength() > 0)
+ if (!m_sTitle.isEmpty())
return m_sTitle;
// Title seams to be unused till now ... do bootstraping
@@ -377,7 +377,7 @@ void TitleHelper::impl_updateTitleForModel (const css::uno::Reference< css::fram
if (xURLProvider.is())
sURL = xURLProvider->getLocation ();
- if (sURL.getLength () > 0)
+ if (!sURL.isEmpty())
{
sTitle = impl_convertURL2Title(sURL);
if (nLeasedNumber != css::frame::UntitledNumbersConst::INVALID_NUMBER)
@@ -588,7 +588,7 @@ void TitleHelper::impl_appendModuleName (::rtl::OUStringBuffer& sTitle)
// An UIname property is an optional value !
// So please add it to the title in case it does realy exists only.
- if (sUIName.getLength() > 0)
+ if (!sUIName.isEmpty())
{
sTitle.appendAscii (" " );
sTitle.append (sUIName);
@@ -690,10 +690,10 @@ void TitleHelper::impl_setSubTitle (const css::uno::Reference< css::frame::XTitl
if (aURL.hasExtension(INetURLObject::LAST_SEGMENT))
sTitle = aURL.getName(INetURLObject::LAST_SEGMENT, sal_True, INetURLObject::DECODE_WITH_CHARSET);
- if ( ! sTitle.getLength() )
+ if ( sTitle.isEmpty() )
sTitle = aURL.GetHostPort(INetURLObject::DECODE_WITH_CHARSET);
- if ( ! sTitle.getLength() )
+ if ( sTitle.isEmpty() )
sTitle = aURL.GetURLNoPass(INetURLObject::DECODE_WITH_CHARSET);
}
diff --git a/framework/source/fwe/xml/menudocumenthandler.cxx b/framework/source/fwe/xml/menudocumenthandler.cxx
index 2195ac2..16e96a5 100644
--- a/framework/source/fwe/xml/menudocumenthandler.cxx
+++ b/framework/source/fwe/xml/menudocumenthandler.cxx
@@ -410,7 +410,7 @@ throw( SAXException, RuntimeException )
do
{
::rtl::OUString aToken = aTemp.getToken( 0, '+', nIndex );
- if ( aToken.getLength() > 0 )
+ if ( !aToken.isEmpty() )
{
if ( aToken.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( ATTRIBUTE_ITEMSTYLE_TEXT ) ) )
nItemBits |= ::com::sun::star::ui::ItemStyle::TEXT;
@@ -424,7 +424,7 @@ throw( SAXException, RuntimeException )
}
}
- if ( aCommandId.getLength() > 0 )
+ if ( !aCommandId.isEmpty() )
{
Sequence< PropertyValue > aSubMenuProp( 6 );
initPropertyCommon( aSubMenuProp, aCommandId, aHelpId, aLabel, nItemBits );
@@ -645,7 +645,7 @@ throw( SAXException, RuntimeException )
do
{
::rtl::OUString aToken = aTemp.getToken( 0, '+', nIndex );
- if ( aToken.getLength() > 0 )
+ if ( !aToken.isEmpty() )
{
if ( aToken.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( ATTRIBUTE_ITEMSTYLE_TEXT ) ) )
nItemBits |= ::com::sun::star::ui::ItemStyle::TEXT;
@@ -660,7 +660,7 @@ throw( SAXException, RuntimeException )
}
- if ( aCommandId.getLength() > 0 )
+ if ( !aCommandId.isEmpty() )
{
Sequence< PropertyValue > aSubMenuProp( 6 );
initPropertyCommon( aSubMenuProp, aCommandId, aHelpId, aLabel, nItemBits );
@@ -702,7 +702,7 @@ throw( SAXException, RuntimeException )
do
{
::rtl::OUString aToken = aTemp.getToken( 0, '+', nIndex );
- if ( aToken.getLength() > 0 )
+ if ( !aToken.isEmpty() )
{
if ( aToken.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( ATTRIBUTE_ITEMSTYLE_TEXT ) ) )
nItemBits |= ::com::sun::star::ui::ItemStyle::TEXT;
@@ -717,7 +717,7 @@ throw( SAXException, RuntimeException )
}
- if ( aCommandId.getLength() > 0 )
+ if ( !aCommandId.isEmpty() )
{
Sequence< PropertyValue > aMenuItem( 6 );
initPropertyCommon( aMenuItem, aCommandId, aHelpId, aLabel, nItemBits );
@@ -885,7 +885,7 @@ throw ( SAXException, RuntimeException )
WriteMenuItem( aCommandURL, aLabel, aHelpURL, nItemBits );
bSeparator = sal_False;
}
- else if (( aCommandURL.getLength() > 0 ) && !AddonPopupMenu::IsCommandURLPrefix ( aCommandURL ))
+ else if (( !aCommandURL.isEmpty() ) && !AddonPopupMenu::IsCommandURLPrefix ( aCommandURL ))
{
::comphelper::AttributeList* pListMenu = new ::comphelper::AttributeList;
Reference< XAttributeList > xListMenu( (XAttributeList *)pListMenu , UNO_QUERY );
@@ -919,7 +919,7 @@ throw ( SAXException, RuntimeException )
{
if ( nType == ::com::sun::star::ui::ItemType::DEFAULT )
{
- if ( aCommandURL.getLength() > 0 )
+ if ( !aCommandURL.isEmpty() )
{
bSeparator = sal_False;
WriteMenuItem( aCommandURL, aLabel, aHelpURL, nItemBits );
@@ -946,14 +946,14 @@ void OWriteMenuDocumentHandler::WriteMenuItem( const ::rtl::OUString& aCommandUR
m_aAttributeType,
aCommandURL );
- if ( aHelpURL.getLength() > 0 )
+ if ( !aHelpURL.isEmpty() )
{
pList->AddAttribute( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_NS_HELPID )),
m_aAttributeType,
aHelpURL );
}
- if (( aLabel.getLength() > 0 ) && !( aCommandURL.copy( CMD_PROTOCOL_SIZE ).equalsAsciiL(RTL_CONSTASCII_STRINGPARAM(CMD_PROTOCOL)) ))
+ if (( !aLabel.isEmpty() ) && !( aCommandURL.copy( CMD_PROTOCOL_SIZE ).equalsAsciiL(RTL_CONSTASCII_STRINGPARAM(CMD_PROTOCOL)) ))
{
pList->AddAttribute( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_NS_LABEL )),
m_aAttributeType,
@@ -968,7 +968,7 @@ void OWriteMenuDocumentHandler::WriteMenuItem( const ::rtl::OUString& aCommandUR
{
if ( nStyle & pStyle->nBit )
{
- if ( aValue.getLength() )
+ if ( !aValue.isEmpty() )
aValue = aValue.concat( rtl::OUString( RTL_CONSTASCII_USTRINGPARAM("+") ) );
aValue += rtl::OUString::createFromAscii( pStyle->attrName );
}
diff --git a/framework/source/fwe/xml/statusbardocumenthandler.cxx b/framework/source/fwe/xml/statusbardocumenthandler.cxx
index dbb1e77..3da7cc6 100644
--- a/framework/source/fwe/xml/statusbardocumenthandler.cxx
+++ b/framework/source/fwe/xml/statusbardocumenthandler.cxx
@@ -584,7 +584,7 @@ void OWriteStatusBarDocumentHandler::WriteStatusBarDocument() throw
nStyle,
nWidth );
- if ( aCommandURL.getLength() > 0 )
+ if ( !aCommandURL.isEmpty() )
WriteStatusBarItem( aCommandURL, aHelpURL, nOffset, nStyle, nWidth );
}
}
@@ -610,7 +610,7 @@ throw ( SAXException, RuntimeException )
::comphelper::AttributeList* pList = new ::comphelper::AttributeList;
Reference< XAttributeList > xList( (XAttributeList *) pList , UNO_QUERY );
- if ( m_aAttributeURL.getLength() == 0 )
+ if (m_aAttributeURL.isEmpty() )
{
m_aAttributeURL = m_aXMLXlinkNS;
m_aAttributeURL += ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_URL ));
diff --git a/framework/source/fwe/xml/toolboxdocumenthandler.cxx b/framework/source/fwe/xml/toolboxdocumenthandler.cxx
index 4d77bf4..58ee88f 100644
--- a/framework/source/fwe/xml/toolboxdocumenthandler.cxx
+++ b/framework/source/fwe/xml/toolboxdocumenthandler.cxx
@@ -267,7 +267,7 @@ throw( SAXException, RuntimeException )
}
}
- if ( aUIName.getLength() > 0 )
+ if ( !aUIName.isEmpty() )
{
// Try to set UI name as a container property
Reference< XPropertySet > xPropSet( m_rItemContainer, UNO_QUERY );
@@ -387,7 +387,7 @@ throw( SAXException, RuntimeException )
do
{
::rtl::OUString aToken = aTemp.getToken( 0, ' ', nIndex );
- if ( aToken.getLength() > 0 )
+ if ( !aToken.isEmpty() )
{
sal_Int32 nHashCode = aToken.hashCode();
if ( nHashCode == m_nHashCode_Style_Radio )
@@ -426,7 +426,7 @@ throw( SAXException, RuntimeException )
throw SAXException( aErrorMessage, Reference< XInterface >(), Any() );
}
- if ( aCommandURL.getLength() > 0 )
+ if ( !aCommandURL.isEmpty() )
{
Sequence< PropertyValue > aToolbarItemProp( 7 );
aToolbarItemProp[0].Name = m_aCommandURL;
@@ -714,7 +714,7 @@ void OWriteToolBoxDocumentHandler::WriteToolBoxDocument() throw
m_aAttributeType,
::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( XMLNS_XLINK )) );
- if ( aUIName.getLength() > 0 )
+ if ( !aUIName.isEmpty() )
pList->AddAttribute( m_aXMLToolbarNS + ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_UINAME )),
m_aAttributeType,
aUIName );
@@ -775,7 +775,7 @@ throw ( SAXException, RuntimeException )
::comphelper::AttributeList* pList = new ::comphelper::AttributeList;
Reference< XAttributeList > xList( (XAttributeList *) pList , UNO_QUERY );
- if ( m_aAttributeURL.getLength() == 0 )
+ if ( m_aAttributeURL.isEmpty() )
{
m_aAttributeURL = m_aXMLXlinkNS;
m_aAttributeURL += ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_URL ));
@@ -784,7 +784,7 @@ throw ( SAXException, RuntimeException )
// save required attribute (URL)
pList->AddAttribute( m_aAttributeURL, m_aAttributeType, rCommandURL );
- if ( rLabel.getLength() > 0 )
+ if ( !rLabel.isEmpty() )
{
pList->AddAttribute( m_aXMLToolbarNS + ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_TEXT )),
m_aAttributeType,
@@ -798,14 +798,14 @@ throw ( SAXException, RuntimeException )
::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_BOOLEAN_FALSE )) );
}
- if ( rHelpURL.getLength() > 0 )
+ if ( !rHelpURL.isEmpty() )
{
pList->AddAttribute( m_aXMLToolbarNS + ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_HELPID )),
m_aAttributeType,
rHelpURL );
}
- if ( rTooltip.getLength() > 0 )
+ if ( !rTooltip.isEmpty() )
{
pList->AddAttribute( m_aXMLToolbarNS + ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( ATTRIBUTE_TOOLTIP )),
m_aAttributeType,
@@ -821,7 +821,7 @@ throw ( SAXException, RuntimeException )
{
if ( nStyle & pStyle->nBit )
{
- if ( aValue.getLength() )
+ if ( !aValue.isEmpty() )
aValue = aValue.concat( rtl::OUString( RTL_CONSTASCII_USTRINGPARAM(" ") ) );
aValue += rtl::OUString::createFromAscii( pStyle->attrName );
}
diff --git a/framework/source/fwe/xml/xmlnamespaces.cxx b/framework/source/fwe/xml/xmlnamespaces.cxx
index 9b1c493..8010494 100644
--- a/framework/source/fwe/xml/xmlnamespaces.cxx
+++ b/framework/source/fwe/xml/xmlnamespaces.cxx
@@ -76,7 +76,7 @@ void XMLNamespaces::addNamespace( const ::rtl::OUString& aName, const ::rtl::OUS
}
}
- if ( aValue.getLength() == 0 && aNamespaceName.getLength() > 0 )
+ if ( aValue.isEmpty() && !aNamespaceName.isEmpty() )
{
// namespace should be reseted - as xml draft states this is only allowed
// for the default namespace - check and throw exception if check fails
@@ -85,7 +85,7 @@ void XMLNamespaces::addNamespace( const ::rtl::OUString& aName, const ::rtl::OUS
}
else
{
- if ( aNamespaceName.getLength() == 0 )
+ if ( aNamespaceName.isEmpty() )
m_aDefaultNamespace = aValue;
else
{
@@ -142,7 +142,7 @@ void XMLNamespaces::addNamespace( const ::rtl::OUString& aName, const ::rtl::OUS
else
aNamespace = m_aDefaultNamespace;
- if ( aNamespace.getLength() > 0 )
+ if ( !aNamespace.isEmpty() )
{
aElementName = aNamespace;
aElementName += ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("^"));
@@ -169,7 +169,7 @@ void XMLNamespaces::addNamespace( const ::rtl::OUString& aName, const ::rtl::OUS
::rtl::OUString XMLNamespaces::getNamespaceValue( const ::rtl::OUString& aNamespace ) const throw( SAXException )
{
- if ( aNamespace.getLength() == 0 )
+ if ( aNamespace.isEmpty() )
return m_aDefaultNamespace;
else
{
diff --git a/framework/source/fwi/helper/mischelper.cxx b/framework/source/fwi/helper/mischelper.cxx
index e1f139c..8a50e94 100644
--- a/framework/source/fwi/helper/mischelper.cxx
+++ b/framework/source/fwi/helper/mischelper.cxx
@@ -120,7 +120,7 @@ uno::Reference< linguistic2::XLanguageGuessing > LanguageGuessingHelper::GetGues
{
try
{
- if ( aCmdURL.getLength() > 0 )
+ if ( !aCmdURL.isEmpty() )
{
rtl::OUString aStr;
Sequence< PropertyValue > aPropSeq;
@@ -185,7 +185,7 @@ void FillLangItems( std::set< OUString > &rLangItems,
//4--guessed language
uno::Reference< linguistic2::XLanguageGuessing > xLangGuesser( rLangGuessHelper.GetGuesser() );
- if ( xLangGuesser.is() && rGuessedTextLang.getLength() > 0)
+ if ( xLangGuesser.is() && !rGuessedTextLang.isEmpty())
{
::com::sun::star::lang::Locale aLocale(xLangGuesser->guessPrimaryLanguage( rGuessedTextLang, 0, rGuessedTextLang.getLength()) );
LanguageType nLang = MsLangId::convertLocaleToLanguageWithFallback( aLocale );
diff --git a/framework/source/helper/persistentwindowstate.cxx b/framework/source/helper/persistentwindowstate.cxx
index 5318c8a..c6ff406 100644
--- a/framework/source/helper/persistentwindowstate.cxx
+++ b/framework/source/helper/persistentwindowstate.cxx
@@ -149,7 +149,7 @@ void SAL_CALL PersistentWindowState::frameAction(const css::frame::FrameActionEv
// unknown module -> no configuration available!
::rtl::OUString sModuleName = PersistentWindowState::implst_identifyModule(xSMGR, xFrame);
- if (!sModuleName.getLength())
+ if (sModuleName.isEmpty())
return;
switch(aEvent.Action)
@@ -312,7 +312,7 @@ void PersistentWindowState::implst_setWindowStateOnWindow(const css::uno::Refere
{
if (
(!xWindow.is() ) ||
- ( sWindowState.getLength() < 1)
+ ( sWindowState.isEmpty())
)
return;
diff --git a/framework/source/helper/titlebarupdate.cxx b/framework/source/helper/titlebarupdate.cxx
index 408f147..d027a1a 100644
--- a/framework/source/helper/titlebarupdate.cxx
+++ b/framework/source/helper/titlebarupdate.cxx
@@ -282,7 +282,7 @@ void TitleBarUpdate::impl_updateApplicationID(const css::uno::Reference< css::fr
// Note: If we could retrieve a module id ... everything is OK.
// UIName and Icon ID are optional values !
- ::sal_Bool bSuccess = (rInfo.sID.getLength () > 0);
+ ::sal_Bool bSuccess = !rInfo.sID.isEmpty();
return bSuccess;
}
catch(const css::uno::Exception&)
diff --git a/framework/source/jobs/helponstartup.cxx b/framework/source/jobs/helponstartup.cxx
index fb06ab0..c897cce 100644
--- a/framework/source/jobs/helponstartup.cxx
+++ b/framework/source/jobs/helponstartup.cxx
@@ -165,7 +165,7 @@ css::uno::Any SAL_CALL HelpOnStartup::execute(const css::uno::Sequence< css::bea
// Attention: We are bound to events for openeing any document inside the office.
// That includes e.g. the help module itself. But we have to do nothing then!
- if (!sModule.getLength())
+ if (sModule.isEmpty())
return css::uno::Any();
// check current state of the help module
@@ -177,7 +177,7 @@ css::uno::Any SAL_CALL HelpOnStartup::execute(const css::uno::Sequence< css::bea
sal_Bool bShowIt = sal_False;
// a)
- if (!sCurrentHelpURL.getLength())
+ if (sCurrentHelpURL.isEmpty())
bShowIt = sal_True;
else
// b)
@@ -188,7 +188,7 @@ css::uno::Any SAL_CALL HelpOnStartup::execute(const css::uno::Sequence< css::bea
{
// retrieve the help URL for the detected application module
::rtl::OUString sModuleDependendHelpURL = its_checkIfHelpEnabledAndGetURL(sModule);
- if (sModuleDependendHelpURL.getLength())
+ if (!sModuleDependendHelpURL.isEmpty())
{
// Show this help page.
// Note: The help window brings itself to front ...
@@ -321,7 +321,7 @@ void SAL_CALL HelpOnStartup::disposing(const css::lang::EventObject& aEvent)
//-----------------------------------------------
::sal_Bool HelpOnStartup::its_isHelpUrlADefaultOne(const ::rtl::OUString& sHelpURL)
{
- if (!sHelpURL.getLength())
+ if (sHelpURL.isEmpty())
return sal_False;
// SAFE ->
diff --git a/framework/source/jobs/jobexecutor.cxx b/framework/source/jobs/jobexecutor.cxx
index 415c4db..22e1da1 100644
--- a/framework/source/jobs/jobexecutor.cxx
+++ b/framework/source/jobs/jobexecutor.cxx
@@ -319,7 +319,7 @@ void SAL_CALL JobExecutor::elementInserted( const css::container::ContainerEvent
if (aEvent.Accessor >>= sValue)
{
::rtl::OUString sEvent = ::utl::extractFirstFromConfigurationPath(sValue);
- if (sEvent.getLength() > 0)
+ if (!sEvent.isEmpty())
{
OUStringList::iterator pEvent = m_lEvents.find(sEvent);
if (pEvent == m_lEvents.end())
@@ -334,7 +334,7 @@ void SAL_CALL JobExecutor::elementRemoved ( const css::container::ContainerEvent
if (aEvent.Accessor >>= sValue)
{
::rtl::OUString sEvent = ::utl::extractFirstFromConfigurationPath(sValue);
- if (sEvent.getLength() > 0)
+ if (!sEvent.isEmpty())
{
OUStringList::iterator pEvent = m_lEvents.find(sEvent);
if (pEvent != m_lEvents.end())
diff --git a/framework/source/jobs/joburl.cxx b/framework/source/jobs/joburl.cxx
index de45910..adedb5f 100644
--- a/framework/source/jobs/joburl.cxx
+++ b/framework/source/jobs/joburl.cxx
@@ -93,7 +93,7 @@ JobURL::JobURL( /*IN*/ const ::rtl::OUString& sURL )
// check for "event="
if (
(JobURL::implst_split(sToken,JOBURL_EVENT_STR,JOBURL_EVENT_LEN,sPartValue,sPartArguments)) &&
- (sPartValue.getLength()>0 )
+ (!sPartValue.isEmpty() )
)
{
// set the part value
@@ -105,7 +105,7 @@ JobURL::JobURL( /*IN*/ const ::rtl::OUString& sURL )
// check for "alias="
if (
(JobURL::implst_split(sToken,JOBURL_ALIAS_STR,JOBURL_ALIAS_LEN,sPartValue,sPartArguments)) &&
- (sPartValue.getLength()>0 )
+ (!sPartValue.isEmpty() )
)
{
// set the part value
@@ -117,7 +117,7 @@ JobURL::JobURL( /*IN*/ const ::rtl::OUString& sURL )
// check for "service="
if (
(JobURL::implst_split(sToken,JOBURL_SERVICE_STR,JOBURL_SERVICE_LEN,sPartValue,sPartArguments)) &&
- (sPartValue.getLength()>0 )
+ (!sPartValue.isEmpty() )
)
{
// set the part value
@@ -419,7 +419,7 @@ void JobURL::impldbg_checkURL( /*IN*/ const sal_Char* pURL ,
bOK = (
(aURL.isValid() ) &&
(aURL.getEvent(sEvent) ) &&
- (sEvent.getLength()>0 ) &&
+ (!sEvent.isEmpty() ) &&
(sEvent.compareToAscii(pExpectedEvent)==0)
);
@@ -440,9 +440,9 @@ void JobURL::impldbg_checkURL( /*IN*/ const sal_Char* pURL ,
{
bOK = (
(!aURL.getEvent(sEvent) ) &&
- (sEvent.getLength()==0 ) &&
+ (sEvent.isEmpty() ) &&
(!aURL.getEventArgs(sEventArgs)) &&
- (sEventArgs.getLength()==0 )
+ (sEventArgs.isEmpty() )
);
}
@@ -455,7 +455,7 @@ void JobURL::impldbg_checkURL( /*IN*/ const sal_Char* pURL ,
bOK = (
(aURL.isValid() ) &&
(aURL.getAlias(sAlias) ) &&
- (sAlias.getLength()>0 ) &&
+ (!sAlias.isEmpty() ) &&
(sAlias.compareToAscii(pExpectedAlias)==0)
);
@@ -476,9 +476,9 @@ void JobURL::impldbg_checkURL( /*IN*/ const sal_Char* pURL ,
{
bOK = (
(!aURL.getAlias(sAlias) ) &&
- (sAlias.getLength()==0 ) &&
+ (sAlias.isEmpty() ) &&
(!aURL.getAliasArgs(sAliasArgs)) &&
- (sAliasArgs.getLength()==0 )
+ (sAliasArgs.isEmpty() )
);
}
@@ -491,7 +491,7 @@ void JobURL::impldbg_checkURL( /*IN*/ const sal_Char* pURL ,
bOK = (
(aURL.isValid() ) &&
(aURL.getService(sService) ) &&
- (sService.getLength()>0 ) &&
+ (!sService.isEmpty() ) &&
(sService.compareToAscii(pExpectedService)==0)
);
@@ -512,9 +512,9 @@ void JobURL::impldbg_checkURL( /*IN*/ const sal_Char* pURL ,
{
bOK = (
(!aURL.getService(sService) ) &&
- (sService.getLength()==0 ) &&
+ (sService.isEmpty() ) &&
(!aURL.getServiceArgs(sServiceArgs)) &&
- (sServiceArgs.getLength()==0 )
+ (sServiceArgs.isEmpty() )
);
}
diff --git a/framework/source/jobs/shelljob.cxx b/framework/source/jobs/shelljob.cxx
index 1ce3266..1910d7a 100644
--- a/framework/source/jobs/shelljob.cxx
+++ b/framework/source/jobs/shelljob.cxx
@@ -123,7 +123,7 @@ css::uno::Any SAL_CALL ShellJob::execute(const css::uno::Sequence< css::beans::N
// Command is required as minimum.
// If it does not exists ... we cant do our job.
// Deactivate such miss configured job silently .-)
- if (sRealCommand.getLength() < 1)
+ if (sRealCommand.isEmpty())
return ShellJob::impl_generateAnswer4Deactivation();
// do it
diff --git a/framework/source/layoutmanager/helpers.cxx b/framework/source/layoutmanager/helpers.cxx
index ad4ef23..b1ddea9 100644
--- a/framework/source/layoutmanager/helpers.cxx
+++ b/framework/source/layoutmanager/helpers.cxx
@@ -103,7 +103,7 @@ bool isHorizontalDockingArea( const sal_Int32 nDockArea )
{
aToolbarName = rtl::OStringToOUString( pToolBox->GetHelpId(), RTL_TEXTENCODING_UTF8 );
sal_Int32 i = aToolbarName.lastIndexOf( ':' );
- if (( aToolbarName.getLength() > 0 ) && ( i > 0 ) && (( i+ 1 ) < aToolbarName.getLength() ))
+ if (( !aToolbarName.isEmpty() ) && ( i > 0 ) && (( i+ 1 ) < aToolbarName.getLength() ))
aToolbarName = aToolbarName.copy( i+1 ); // Remove ".HelpId:" protocol from toolbar name
else
aToolbarName = ::rtl::OUString();
diff --git a/framework/source/layoutmanager/layoutmanager.cxx b/framework/source/layoutmanager/layoutmanager.cxx
index 1598537..0020a2e 100644
--- a/framework/source/layoutmanager/layoutmanager.cxx
+++ b/framework/source/layoutmanager/layoutmanager.cxx
@@ -295,7 +295,7 @@ void LayoutManager::implts_reset( sal_Bool bAttached )
}
catch( const Exception& ) {}
- if ( aModuleIdentifier.getLength() && aOldModuleIdentifier != aModuleIdentifier )
+ if ( !aModuleIdentifier.isEmpty() && aOldModuleIdentifier != aModuleIdentifier )
{
Reference< XModuleUIConfigurationManagerSupplier > xModuleCfgSupplier;
if ( xServiceManager.is() )
@@ -1706,7 +1706,7 @@ throw (uno::RuntimeException)
}
else if ( aElementType.equalsIgnoreAsciiCaseAscii( UIRESOURCETYPE_TOOLBAR ) && m_bVisible )
{
- bool bComponentAttached( m_aModuleIdentifier.getLength() > 0 );
+ bool bComponentAttached( !m_aModuleIdentifier.isEmpty() );
uno::Reference< uno::XInterface > xThis( m_xToolbarManager, uno::UNO_QUERY );
ToolbarLayoutManager* pToolbarManager = m_pToolbarManager;
aWriteLock.unlock();
diff --git a/framework/source/layoutmanager/toolbarlayoutmanager.cxx b/framework/source/layoutmanager/toolbarlayoutmanager.cxx
index d14803e..3f5db68 100644
--- a/framework/source/layoutmanager/toolbarlayoutmanager.cxx
+++ b/framework/source/layoutmanager/toolbarlayoutmanager.cxx
@@ -821,7 +821,7 @@ long ToolbarLayoutManager::childWindowEvent( VclSimpleEvent* pEvent )
aCommand = pToolBox->GetItemCommand( nId );
}
- if (( aToolbarName.getLength() > 0 ) && ( aCommand.getLength() > 0 ))
+ if (( !aToolbarName.isEmpty() ) && ( !aCommand.isEmpty() ))
{
ReadGuard aReadLock( m_aLock );
::std::vector< uno::Reference< ui::XUIFunctionListener > > aListenerArray;
@@ -863,7 +863,7 @@ long ToolbarLayoutManager::childWindowEvent( VclSimpleEvent* pEvent )
if ( pToolBox )
{
::rtl::OUString aToolbarName = retrieveToolbarNameFromHelpURL( pToolBox );
- if ( aToolbarName.getLength() > 0 )
+ if ( !aToolbarName.isEmpty() )
{
::rtl::OUStringBuffer aBuf(100);
aBuf.appendAscii( "private:resource/toolbar/" );
@@ -1024,13 +1024,13 @@ void ToolbarLayoutManager::implts_createAddonsToolBars()
::rtl::OUString aGenericAddonTitle = implts_generateGenericAddonToolbarTitle( i+1 );
- if ( aElement.m_aName.getLength() > 0 )
+ if ( !aElement.m_aName.isEmpty() )
{
// Reuse a local entry so we are able to use the latest
// UI changes for this document.
implts_setElementData( aElement, xDockWindow );
aElement.m_xUIElement = xUIElement;
- if ( aElement.m_aUIName.getLength() == 0 )
+ if ( aElement.m_aUIName.isEmpty() )
{
aElement.m_aUIName = aGenericAddonTitle;
implts_writeWindowStateData( aElement );
@@ -1043,7 +1043,7 @@ void ToolbarLayoutManager::implts_createAddonsToolBars()
aNewToolbar.m_bFloating = true;
implts_readWindowStateData( aAddonToolBarName, aNewToolbar );
implts_setElementData( aNewToolbar, xDockWindow );
- if ( aNewToolbar.m_aUIName.getLength() == 0 )
+ if ( aNewToolbar.m_aUIName.isEmpty() )
{
aNewToolbar.m_aUIName = aGenericAddonTitle;
implts_writeWindowStateData( aNewToolbar );
@@ -1198,20 +1198,20 @@ void ToolbarLayoutManager::implts_createCustomToolBars( const uno::Sequence< uno
}
// Only create custom toolbars. Their name have to start with "custom_"!
- if ( aTbxResName.getLength() > 0 && aTbxResName.indexOf( m_aCustomTbxPrefix ) != -1 )
+ if ( !aTbxResName.isEmpty() && aTbxResName.indexOf( m_aCustomTbxPrefix ) != -1 )
implts_createCustomToolBar( aTbxResName, aTbxTitle );
}
}
void ToolbarLayoutManager::implts_createCustomToolBar( const rtl::OUString& aTbxResName, const rtl::OUString& aTitle )
{
- if ( aTbxResName.getLength() > 0 )
+ if ( !aTbxResName.isEmpty() )
{
bool bNotify( false );
uno::Reference< ui::XUIElement > xUIElement;
implts_createToolBar( aTbxResName, bNotify, xUIElement );
- if ( aTitle.getLength() != 0 && xUIElement.is() )
+ if ( !aTitle.isEmpty() && xUIElement.is() )
{
SolarMutexGuard aGuard;
@@ -1336,7 +1336,7 @@ void ToolbarLayoutManager::implts_createToolBar( const ::rtl::OUString& aName, b
WriteGuard aWriteLock( m_aLock );
UIElement& rElement = impl_findToolbar( aName );
- if ( rElement.m_aName.getLength() > 0 )
+ if ( !rElement.m_aName.isEmpty() )
{
// Reuse a local entry so we are able to use the latest
// UI changes for this document.
@@ -3744,7 +3744,7 @@ throw (uno::RuntimeException)
aReadLock.unlock();
UIElement aUIDockingElement = implts_findToolbar( e.Source );
- bool bWinFound( aUIDockingElement.m_aName.getLength() > 0 );
+ bool bWinFound( !aUIDockingElement.m_aName.isEmpty() );
uno::Reference< awt::XWindow > xWindow( e.Source, uno::UNO_QUERY );
if ( bWinFound && xWindow.is() )
@@ -3805,7 +3805,7 @@ throw (uno::RuntimeException)
if ( !bDockingInProgress )
{
aUIDockingElement = implts_findToolbar( e.Source );
- bool bWinFound = ( aUIDockingElement.m_aName.getLength() > 0 );
+ bool bWinFound = ( !aUIDockingElement.m_aName.isEmpty() );
if ( bWinFound && xWindow.is() )
{
@@ -3941,7 +3941,7 @@ throw (uno::RuntimeException)
aWriteLock.unlock();
// destroy element
- if ( aName.getLength() > 0 )
+ if ( !aName.isEmpty() )
{
implts_writeWindowStateData( aUIElement );
destroyToolbar( aName );
diff --git a/framework/source/loadenv/loadenv.cxx b/framework/source/loadenv/loadenv.cxx
index 090b6aa..dc5736f 100644
--- a/framework/source/loadenv/loadenv.cxx
+++ b/framework/source/loadenv/loadenv.cxx
@@ -295,7 +295,7 @@ void LoadEnv::initializeLoading(const ::rtl::OUString&
// BTW: Split URL and JumpMark ...
// Because such mark is an explicit value of the media descriptor!
- if (m_aURL.Mark.getLength())
+ if (!m_aURL.Mark.isEmpty())
m_lMediaDescriptor[::comphelper::MediaDescriptor::PROP_JUMPMARK()] <<= m_aURL.Mark;
// By the way: remove the old and deprecated value "FileName" from the descriptor!
@@ -650,7 +650,7 @@ LoadEnv::EContentType LoadEnv::classifyContent(const ::rtl::OUString&
// can detect such protocol schemata too :-)
if(
- (!sURL.getLength() ) ||
+ (sURL.isEmpty() ) ||
(ProtocolCheck::isProtocol(sURL,ProtocolCheck::E_UNO )) ||
(ProtocolCheck::isProtocol(sURL,ProtocolCheck::E_SLOT )) ||
(ProtocolCheck::isProtocol(sURL,ProtocolCheck::E_MACRO )) ||
@@ -804,7 +804,7 @@ void LoadEnv::impl_detectTypeAndFilter()
sType = xDetect->queryTypeByDescriptor(lDescriptor, sal_True); /*TODO should deep detection be able for enable/disable it from outside? */
// no valid content -> loading not possible
- if (!sType.getLength())
+ if (sType.isEmpty())
throw LoadEnvException(LoadEnvException::ID_UNSUPPORTED_CONTENT);
// SAFE ->
@@ -827,7 +827,7 @@ void LoadEnv::impl_detectTypeAndFilter()
// => We must try(!) to detect the right filter for this load request.
// On the other side ... if no filter is available .. ignore it.
// Then the type information must be enough.
- if (!sFilter.getLength())
+ if (sFilter.isEmpty())
{
// no -> try to find a preferred filter for the detected type.
// Dont forget to updatet he media descriptor.
@@ -836,7 +836,7 @@ void LoadEnv::impl_detectTypeAndFilter()
{
::comphelper::SequenceAsHashMap lTypeProps(xTypeCont->getByName(sType));
sFilter = lTypeProps.getUnpackedValueOrDefault(TYPEPROP_PREFERREDFILTER, ::rtl::OUString());
- if (sFilter.getLength())
+ if (!sFilter.isEmpty())
{
// SAFE ->
aWriteLock.lock();
@@ -858,7 +858,7 @@ void LoadEnv::impl_detectTypeAndFilter()
// has to know, what he is doing .-)
sal_Bool bIsOwnTemplate = sal_False;
- if (sFilter.getLength())
+ if (!sFilter.isEmpty())
{
css::uno::Reference< css::container::XNameAccess > xFilterCont(xSMGR->createInstance(SERVICENAME_FILTERFACTORY), css::uno::UNO_QUERY_THROW);
try
@@ -892,7 +892,7 @@ sal_Bool LoadEnv::impl_handleContent()
// the type must exist inside the descriptor ... otherwhise this class is implemented wrong :-)
::rtl::OUString sType = m_lMediaDescriptor.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_TYPENAME(), ::rtl::OUString());
- if (!sType.getLength())
+ if (sType.isEmpty())
throw LoadEnvException(LoadEnvException::ID_INVALID_MEDIADESCRIPTOR);
// convert media descriptor and URL to right format for later interface call!
@@ -1196,7 +1196,7 @@ css::uno::Reference< css::uno::XInterface > LoadEnv::impl_searchLoader()
// We need this type information to locate an registered frame loader
// Without such information we can't work!
::rtl::OUString sType = m_lMediaDescriptor.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_TYPENAME(), ::rtl::OUString());
- if (!sType.getLength())
+ if (sType.isEmpty())
throw LoadEnvException(LoadEnvException::ID_INVALID_MEDIADESCRIPTOR);
// try to locate any interested frame loader
@@ -1240,7 +1240,7 @@ css::uno::Reference< css::uno::XInterface > LoadEnv::impl_searchLoader()
void LoadEnv::impl_jumpToMark(const css::uno::Reference< css::frame::XFrame >& xFrame,
const css::util::URL& aURL )
{
- if (! aURL.Mark.getLength())
+ if (aURL.Mark.isEmpty())
return;
css::uno::Reference< css::frame::XDispatchProvider > xProvider(xFrame, css::uno::UNO_QUERY);
@@ -1401,7 +1401,7 @@ css::uno::Reference< css::frame::XFrame > LoadEnv::impl_searchAlreadyLoaded()
// Now we are sure, that this task includes the searched document.
// It's time to activate it. As special feature we try to jump internally
// if an optional jumpmark is given too.
- if (m_aURL.Mark.getLength())
+ if (!m_aURL.Mark.isEmpty())
impl_jumpToMark(xResult, m_aURL);
// bring it to front and make sure it's visible...
@@ -1763,7 +1763,7 @@ void LoadEnv::impl_applyPersistentWindowState(const css::uno::Reference< css::aw
::rtl::OUString sFilter = m_lMediaDescriptor.getUnpackedValueOrDefault(
::comphelper::MediaDescriptor::PROP_FILTERNAME(),
::rtl::OUString());
- if (!sFilter.getLength())
+ if (sFilter.isEmpty())
return;
css::uno::Reference< css::lang::XMultiServiceFactory > xSMGR = m_xSMGR;
@@ -1792,7 +1792,7 @@ void LoadEnv::impl_applyPersistentWindowState(const css::uno::Reference< css::aw
// Do nothing, if no configuration entry exists!
::rtl::OUString sWindowState ;
::comphelper::ConfigurationHelper::readRelativeKey(xModuleCfg, sModule, OFFICEFACTORY_PROPNAME_WINDOWATTRIBUTES) >>= sWindowState;
- if (sWindowState.getLength())
+ if (!sWindowState.isEmpty())
{
// SOLAR SAFE ->
SolarMutexGuard aSolarGuard;
diff --git a/framework/source/loadenv/targethelper.cxx b/framework/source/loadenv/targethelper.cxx
index bfc6104..95f5151 100644
--- a/framework/source/loadenv/targethelper.cxx
+++ b/framework/source/loadenv/targethelper.cxx
@@ -40,7 +40,7 @@ sal_Bool TargetHelper::matchSpecialTarget(const ::rtl::OUString& sCheckTarget ,
{
case E_SELF :
return (
- (!sCheckTarget.getLength() ) ||
+ (sCheckTarget.isEmpty() ) ||
(sCheckTarget.equals(SPECIALTARGET_SELF))
);
@@ -77,7 +77,7 @@ sal_Bool TargetHelper::isValidNameForFrame(const ::rtl::OUString& sName)
// some special targets are realy special ones :-)
// E.g. the are realy used to locate one frame inside the frame tree.
if (
- (!sName.getLength() ) ||
+ (sName.isEmpty() ) ||
(TargetHelper::matchSpecialTarget(sName, E_HELPTASK)) ||
(TargetHelper::matchSpecialTarget(sName, E_BEAMER) )
)
diff --git a/framework/source/recording/dispatchrecorder.cxx b/framework/source/recording/dispatchrecorder.cxx
index 5f95784..6c33051 100644
--- a/framework/source/recording/dispatchrecorder.cxx
+++ b/framework/source/recording/dispatchrecorder.cxx
@@ -253,7 +253,7 @@ void SAL_CALL DispatchRecorder::AppendToBuffer( css::uno::Any aValue, ::rtl::OUS
aValue >>= sVal;
// encode non printable characters or '"' by using the CHR$ function
- if ( sVal.getLength() )
+ if ( !sVal.isEmpty() )
{
const sal_Unicode* pChars = sVal.getStr();
sal_Bool bInString = sal_False;
diff --git a/framework/source/services/autorecovery.cxx b/framework/source/services/autorecovery.cxx
index 2b43178..7bfccac 100644
--- a/framework/source/services/autorecovery.cxx
+++ b/framework/source/services/autorecovery.cxx
@@ -1121,7 +1121,7 @@ void AutoRecovery::implts_readConfig()
//-----------------------------------------------
void AutoRecovery::implts_specifyDefaultFilterAndExtension(AutoRecovery::TDocumentInfo& rInfo)
{
- if (!rInfo.AppModule.getLength())
+ if (rInfo.AppModule.isEmpty())
{
throw css::uno::RuntimeException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Cant find out the default filter and its extension, if no application module is known!")),
@@ -1183,7 +1183,7 @@ void AutoRecovery::implts_specifyDefaultFilterAndExtension(AutoRecovery::TDocume
void AutoRecovery::implts_specifyAppModuleAndFactory(AutoRecovery::TDocumentInfo& rInfo)
{
ENSURE_OR_THROW2(
- rInfo.AppModule.getLength() || rInfo.Document.is(),
+ !rInfo.AppModule.isEmpty() || rInfo.Document.is(),
"Cant find out the application module nor its factory URL, if no application module (or a suitable) document is known!",
*this );
@@ -1196,7 +1196,7 @@ void AutoRecovery::implts_specifyAppModuleAndFactory(AutoRecovery::TDocumentInfo
css::uno::Reference< css::frame::XModuleManager > xManager (xSMGR->createInstance(SERVICENAME_MODULEMANAGER), css::uno::UNO_QUERY_THROW);
css::uno::Reference< css::container::XNameAccess > xModuleConfig(xManager , css::uno::UNO_QUERY_THROW);
- if (!rInfo.AppModule.getLength())
+ if (rInfo.AppModule.isEmpty())
rInfo.AppModule = xManager->identify(rInfo.Document);
::comphelper::SequenceAsHashMap lModuleDescription(xModuleConfig->getByName(rInfo.AppModule));
@@ -1223,9 +1223,9 @@ void AutoRecovery::implts_collectActiveViewNames( AutoRecovery::TDocumentInfo& i
::rtl::OUString sViewName;
if ( xController.is() )
sViewName = xController->getViewControllerName();
- OSL_ENSURE( sViewName.getLength(), "AutoRecovery::implts_collectActiveViewNames: (no XController2 ->) no view name -> no recovery of this view!" );
+ OSL_ENSURE( !sViewName.isEmpty(), "AutoRecovery::implts_collectActiveViewNames: (no XController2 ->) no view name -> no recovery of this view!" );
- if ( sViewName.getLength() )
+ if ( !sViewName.isEmpty() )
aViewNames.push_back( sViewName );
}
}
@@ -1235,9 +1235,9 @@ void AutoRecovery::implts_collectActiveViewNames( AutoRecovery::TDocumentInfo& i
::rtl::OUString sViewName;
if ( xController.is() )
sViewName = xController->getViewControllerName();
- OSL_ENSURE( sViewName.getLength(), "AutoRecovery::implts_collectActiveViewNames: (no XController2 ->) no view name -> no recovery of this view!" );
+ OSL_ENSURE( !sViewName.isEmpty(), "AutoRecovery::implts_collectActiveViewNames: (no XController2 ->) no view name -> no recovery of this view!" );
- if ( sViewName.getLength() )
+ if ( !sViewName.isEmpty() )
aViewNames.push_back( sViewName );
}
@@ -1726,8 +1726,8 @@ void AutoRecovery::implts_registerDocument(const css::uno::Reference< css::frame
// Its not realy a full featured office document. It doesnt provide an URL, any filter, a factory URL etcpp.
// TODO file bug to Basci IDE developers. They must remove the office document API from its service.
if (
- (!aNew.OrgURL.getLength() ) &&
- (!aNew.FactoryURL.getLength())
+ (aNew.OrgURL.isEmpty() ) &&
+ (aNew.FactoryURL.isEmpty())
)
{
OSL_FAIL( "AutoRecovery::implts_registerDocument: this should not happen anymore!" );
@@ -1937,7 +1937,7 @@ void AutoRecovery::implts_markDocumentAsSaved(const css::uno::Reference< css::fr
else
{
rInfo.Title = lDescriptor.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_TITLE() , ::rtl::OUString());
- if (!rInfo.Title.getLength())
+ if (rInfo.Title.isEmpty())
rInfo.Title = lDescriptor.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_DOCUMENTTITLE(), ::rtl::OUString());
}
@@ -2025,7 +2025,7 @@ void lc_removeLockFile(AutoRecovery::TDocumentInfo& rInfo)
{
css::uno::Reference< css::frame::XStorable > xStore(rInfo.Document, css::uno::UNO_QUERY_THROW);
::rtl::OUString aURL = xStore->getLocation();
- if ( aURL.getLength() )
+ if ( !aURL.isEmpty() )
{
::svt::DocumentLockFile aLockFile( aURL );
aLockFile.RemoveFile();
@@ -2333,12 +2333,12 @@ void AutoRecovery::implts_saveOneDoc(const ::rtl::OUString&
// stored with password
::comphelper::MediaDescriptor lNewArgs;
::rtl::OUString sPassword = lOldArgs.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_PASSWORD(), ::rtl::OUString());
- if (sPassword.getLength())
+ if (!sPassword.isEmpty())
lNewArgs[::comphelper::MediaDescriptor::PROP_PASSWORD()] <<= sPassword;
// Further it must be saved using the default file format of that application.
// Otherwhise we will some data lost.
- if (rInfo.DefaultFilter.getLength())
+ if (!rInfo.DefaultFilter.isEmpty())
lNewArgs[::comphelper::MediaDescriptor::PROP_FILTERNAME()] <<= rInfo.DefaultFilter;
// prepare frame/document/mediadescriptor in a way, that it uses OUR progress .-)
@@ -2517,19 +2517,19 @@ AutoRecovery::ETimerType AutoRecovery::implts_openDocs(const DispatchParams& aPa
if (!bBackupWasTried)
sLoadBackupURL = rInfo.OldTempURL;
- if (rInfo.OrgURL.getLength())
+ if (!rInfo.OrgURL.isEmpty())
{
sLoadOriginalURL = rInfo.OrgURL;
}
else
- if (rInfo.TemplateURL.getLength())
+ if (!rInfo.TemplateURL.isEmpty())
{
sLoadOriginalURL = rInfo.TemplateURL;
lDescriptor[::comphelper::MediaDescriptor::PROP_ASTEMPLATE()] <<= sal_True;
lDescriptor[::comphelper::MediaDescriptor::PROP_TEMPLATENAME()] <<= rInfo.TemplateURL;
}
else
- if (rInfo.FactoryURL.getLength())
+ if (!rInfo.FactoryURL.isEmpty())
{
sLoadOriginalURL = rInfo.FactoryURL;
lDescriptor[::comphelper::MediaDescriptor::PROP_ASTEMPLATE()] <<= sal_True;
@@ -2538,14 +2538,14 @@ AutoRecovery::ETimerType AutoRecovery::implts_openDocs(const DispatchParams& aPa
// A "Salvaged" item must exists every time. The core can make something special then for recovery.
// Of course it should be the real file name of the original file, in case we load the temp. backup here.
::rtl::OUString sURL;
- if (sLoadBackupURL.getLength())
+ if (!sLoadBackupURL.isEmpty())
{
sURL = sLoadBackupURL;
rInfo.DocumentState |= AutoRecovery::E_TRY_LOAD_BACKUP;
lDescriptor[::comphelper::MediaDescriptor::PROP_SALVAGEDFILE()] <<= sLoadOriginalURL;
}
else
- if (sLoadOriginalURL.getLength())
+ if (!sLoadOriginalURL.isEmpty())
{
sURL = sLoadOriginalURL;
rInfo.DocumentState |= AutoRecovery::E_TRY_LOAD_ORIGINAL;
@@ -2570,7 +2570,7 @@ AutoRecovery::ETimerType AutoRecovery::implts_openDocs(const DispatchParams& aPa
{
rInfo.DocumentState &= ~AutoRecovery::E_TRY_LOAD_BACKUP;
rInfo.DocumentState &= ~AutoRecovery::E_TRY_LOAD_ORIGINAL;
- if (sLoadBackupURL.getLength())
+ if (!sLoadBackupURL.isEmpty())
{
rInfo.DocumentState |= AutoRecovery::E_INCOMPLETE;
eTimer = AutoRecovery::E_CALL_ME_BACK;
@@ -2591,7 +2591,7 @@ AutoRecovery::ETimerType AutoRecovery::implts_openDocs(const DispatchParams& aPa
continue;
}
- if (rInfo.RealFilter.getLength())
+ if (!rInfo.RealFilter.isEmpty())
{
::comphelper::MediaDescriptor lPatchDescriptor(rInfo.Document->getArgs());
lPatchDescriptor[::comphelper::MediaDescriptor::PROP_FILTERNAME()] <<= rInfo.RealFilter;
@@ -2780,7 +2780,7 @@ void AutoRecovery::implts_generateNewTempURL(const ::rtl::OUString&
// into the configuration and dont create any recovery file on disk.
// We use the title of the document to make it unique.
::rtl::OUStringBuffer sUniqueName;
- if (rInfo.OrgURL.getLength())
+ if (!rInfo.OrgURL.isEmpty())
{
css::uno::Reference< css::util::XURLTransformer > xParser(xSMGR->createInstance(SERVICENAME_URLTRANSFORMER), css::uno::UNO_QUERY);
css::util::URL aURL;
@@ -2789,7 +2789,7 @@ void AutoRecovery::implts_generateNewTempURL(const ::rtl::OUString&
sUniqueName.append(aURL.Name);
}
else
- if (rInfo.FactoryURL.getLength())
+ if (!rInfo.FactoryURL.isEmpty())
sUniqueName.appendAscii("untitled");
sUniqueName.appendAscii("_");
@@ -2933,7 +2933,7 @@ css::frame::FeatureStateEvent AutoRecovery::implst_createFeatureStateEvent(
aInfo.put( CFG_ENTRY_PROP_ORIGINALURL, pInfo->OrgURL );
aInfo.put( CFG_ENTRY_PROP_FACTORYURL, pInfo->FactoryURL );
aInfo.put( CFG_ENTRY_PROP_TEMPLATEURL, pInfo->TemplateURL );
- aInfo.put( CFG_ENTRY_PROP_TEMPURL, pInfo->OldTempURL.getLength() ? pInfo->OldTempURL : pInfo->NewTempURL );
+ aInfo.put( CFG_ENTRY_PROP_TEMPURL, pInfo->OldTempURL.isEmpty() ? pInfo->NewTempURL : pInfo->OldTempURL );
aInfo.put( CFG_ENTRY_PROP_MODULE, pInfo->AppModule );
aInfo.put( CFG_ENTRY_PROP_TITLE, pInfo->Title );
aInfo.put( CFG_ENTRY_PROP_VIEWNAMES, pInfo->ViewNames );
@@ -3181,13 +3181,13 @@ void AutoRecovery::implts_backupWorkingEntry(const DispatchParams& aParams)
::rtl::OUString sSourceURL;
// Prefer temp file. It contains the changes against the original document!
- if (rInfo.OldTempURL.getLength())
+ if (!rInfo.OldTempURL.isEmpty())
sSourceURL = rInfo.OldTempURL;
else
- if (rInfo.NewTempURL.getLength())
+ if (!rInfo.NewTempURL.isEmpty())
sSourceURL = rInfo.NewTempURL;
else
- if (rInfo.OrgURL.getLength())
+ if (!rInfo.OrgURL.isEmpty())
sSourceURL = rInfo.OrgURL;
else
continue; // nothing real to save! An unmodified but new created document.
@@ -3628,7 +3628,7 @@ void AutoRecovery::impl_flushALLConfigChanges()
//-----------------------------------------------
void AutoRecovery::st_impl_removeFile(const ::rtl::OUString& sURL)
{
- if ( ! sURL.getLength())
+ if ( sURL.isEmpty())
return;
try
diff --git a/framework/source/services/backingwindow.cxx b/framework/source/services/backingwindow.cxx
index abc85eb..a28f80f 100644
--- a/framework/source/services/backingwindow.cxx
+++ b/framework/source/services/backingwindow.cxx
@@ -322,7 +322,7 @@ void BackingWindow::prepareRecentFileMenu()
maRecentFiles.push_back( LoadRecentFile() );
maRecentFiles.back().aTargetURL = aURL;
- sal_Int32 nArgs = aFilterOpt.getLength() ? 4 : 3;
+ sal_Int32 nArgs = aFilterOpt.isEmpty() ? 3 : 4;
Sequence< PropertyValue >& rArgsList( maRecentFiles.back().aArgSeq );
rArgsList.realloc( nArgs );
@@ -330,7 +330,7 @@ void BackingWindow::prepareRecentFileMenu()
rArgsList[nArgs].Name = rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "FilterName" ));
rArgsList[nArgs].Value = makeAny( aFilter );
- if( aFilterOpt.getLength() )
+ if( !aFilterOpt.isEmpty() )
{
nArgs--;
rArgsList[nArgs].Name = rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "FilterOptions" ));
@@ -546,7 +546,7 @@ void BackingWindow::initControls()
{
comphelper::SequenceAsHashMap aEntryItems( *pNewMenu );
rtl::OUString sURL( aEntryItems.getUnpackedValueOrDefault( sURLKey, rtl::OUString() ) );
- if ( sURL.getLength() )
+ if ( !sURL.isEmpty() )
aFileNewAppsAvailable.insert( sURL );
}
diff --git a/framework/source/services/desktop.cxx b/framework/source/services/desktop.cxx
index c2b6d51..c6d414c 100644
--- a/framework/source/services/desktop.cxx
+++ b/framework/source/services/desktop.cxx
@@ -1051,7 +1051,7 @@ css::uno::Reference< css::frame::XFrame > SAL_CALL Desktop::findFrame( const ::r
else
if (
( sTargetFrameName==SPECIALTARGET_SELF ) ||
- ( sTargetFrameName.getLength()<1 )
+ ( sTargetFrameName.isEmpty() )
)
{
xTarget = this;
diff --git a/framework/source/services/dispatchhelper.cxx b/framework/source/services/dispatchhelper.cxx
index fda1ef9..690d39c 100644
--- a/framework/source/services/dispatchhelper.cxx
+++ b/framework/source/services/dispatchhelper.cxx
@@ -121,7 +121,7 @@ css::uno::Any SAL_CALL DispatchHelper::executeDispatch(
// check for valid parameters
if (
(!xDispatchProvider.is()) ||
- (sURL.getLength()<1 )
+ (sURL.isEmpty() )
)
{
return css::uno::Any();
diff --git a/framework/source/services/frame.cxx b/framework/source/services/frame.cxx
index 4c4108f..cd60a6c 100644
--- a/framework/source/services/frame.cxx
+++ b/framework/source/services/frame.cxx
@@ -834,7 +834,7 @@ css::uno::Reference< css::frame::XFrame > SAL_CALL Frame::findFrame( const ::rtl
else
if (
( sTargetFrameName==SPECIALTARGET_SELF ) ||
- ( sTargetFrameName.getLength()<1 )
+ ( sTargetFrameName.isEmpty() )
)
{
xTarget = this;
diff --git a/framework/source/services/license.cxx b/framework/source/services/license.cxx
index 304d35b..3b2ef61 100644
--- a/framework/source/services/license.cxx
+++ b/framework/source/services/license.cxx
@@ -267,11 +267,11 @@ css::uno::Any SAL_CALL License::execute(const css::uno::Sequence< css::beans::Na
ResMgr* pResMgr = ResMgr::SearchCreateResMgr("fwe", aLocale);
aLangString = aLocale.Language;
- if ( aLocale.Country.getLength() != 0 )
+ if ( !aLocale.Country.isEmpty() )
{
aLangString += ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("-"));
aLangString += aLocale.Country;
- if ( aLocale.Variant.getLength() != 0 )
+ if ( !aLocale.Variant.isEmpty() )
{
aLangString += ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("-"));
aLangString += aLocale.Variant;
diff --git a/framework/source/services/modulemanager.cxx b/framework/source/services/modulemanager.cxx
index d195405..4cec298 100644
--- a/framework/source/services/modulemanager.cxx
+++ b/framework/source/services/modulemanager.cxx
@@ -172,7 +172,7 @@ ModuleManager::~ModuleManager()
if (xWindow.is())
sModule = implts_identify(xWindow);
- if (sModule.getLength() < 1)
+ if (sModule.isEmpty())
throw css::frame::UnknownModuleException(
::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Cant find suitable module for the given component.")),
static_cast< ::cppu::OWeakObject* >(this));
diff --git a/framework/source/services/pathsettings.cxx b/framework/source/services/pathsettings.cxx
index 45d3c8f15..d94df43 100644
--- a/framework/source/services/pathsettings.cxx
+++ b/framework/source/services/pathsettings.cxx
@@ -184,7 +184,7 @@ void SAL_CALL PathSettings::changesOccurred(const css::util::ChangesEvent& aEven
aChange.Accessor >>= sChanged;
::rtl::OUString sPath = ::utl::extractFirstFromConfigurationPath(sChanged);
- if (sPath.getLength())
+ if (!sPath.isEmpty())
{
PathSettings::EChangeOp eOp = impl_updatePath(sPath, sal_True);
if (
@@ -678,7 +678,7 @@ void PathSettings::impl_subst(PathSettings::PathInfo& aPath ,
lTemp.push_back(*pIt);
}
- if (rPath.sWritePath.getLength() > 0)
+ if (!rPath.sWritePath.isEmpty())
lTemp.push_back(rPath.sWritePath);
::rtl::OUStringBuffer sPathVal(256);
@@ -703,7 +703,7 @@ OUStringList PathSettings::impl_convertOldStyle2Path(const ::rtl::OUString& sOld
do
{
::rtl::OUString sToken = sOldStylePath.getToken(0, ';', nToken);
- if (sToken.getLength())
+ if (!sToken.isEmpty())
lList.push_back(sToken);
}
while(nToken >= 0);
diff --git a/framework/source/services/substitutepathvars.cxx b/framework/source/services/substitutepathvars.cxx
index 21f5535..8398809 100644
--- a/framework/source/services/substitutepathvars.cxx
+++ b/framework/source/services/substitutepathvars.cxx
@@ -722,7 +722,7 @@ rtl::OUString SubstitutePathVariables::GetWorkPath() const
}
// fallback in case config layer does not return an useable work dir value.
- if (aWorkPath.getLength() < 1)
+ if (aWorkPath.isEmpty())
aWorkPath = GetWorkVariableValue();
return aWorkPath;
@@ -748,7 +748,7 @@ rtl::OUString SubstitutePathVariables::GetWorkVariableValue() const
// fallback to $HOME in case platform dependend config layer does not return
// an usuable work dir value.
- if (aWorkPath.getLength() < 1)
+ if (aWorkPath.isEmpty())
{
osl::Security aSecurity;
aSecurity.getHomeDir( aWorkPath );
@@ -785,7 +785,7 @@ rtl::OUString SubstitutePathVariables::GetPathVariableValue() const
do
{
::rtl::OUString sToken = aPathList.getToken(0, SAL_PATHSEPARATOR, nToken);
- if (sToken.getLength())
+ if (!sToken.isEmpty())
{
osl::FileBase::getFileURLFromSystemPath( sToken, aTmp );
if ( bAppendSep )
@@ -1017,10 +1017,10 @@ throw ( RuntimeException )
if ( osl::FileBase::getFileURLFromSystemPath( rURL, aTemp ) == osl::FileBase::E_None )
{
aTemp = ConvertOSLtoUCBURL( aTemp );
- if ( aTemp.getLength() )
+ if ( !aTemp.isEmpty() )
{
aURL = INetURLObject( aTemp ).GetMainURL( INetURLObject::NO_DECODE );
- if( !aURL.getLength() )
+ if( aURL.isEmpty() )
return rURL;
}
else
diff --git a/framework/source/services/taskcreatorsrv.cxx b/framework/source/services/taskcreatorsrv.cxx
index 715436a..a687352 100644
--- a/framework/source/services/taskcreatorsrv.cxx
+++ b/framework/source/services/taskcreatorsrv.cxx
@@ -180,7 +180,7 @@ css::uno::Reference< css::uno::XInterface > SAL_CALL TaskCreatorService::createI
// to create it's previes frames. They must do it manually by using WindowDescriptor+Toolkit!
css::uno::Reference< css::frame::XDesktop > xDesktop(xParentFrame, css::uno::UNO_QUERY);
::sal_Bool bTopLevelDocumentWindow = (
- (sRightName.getLength () < 1) &&
+ (sRightName.isEmpty()) &&
(
(! xParentFrame.is() ) ||
( xDesktop.is() )
@@ -323,7 +323,7 @@ css::uno::Reference< css::frame::XFrame > TaskCreatorService::implts_createFrame
}
// Set it's API name (if there is one from outside)
- if (sName.getLength())
+ if (!sName.isEmpty())
xNewFrame->setName( sName );
return xNewFrame;
diff --git a/framework/source/services/urltransformer.cxx b/framework/source/services/urltransformer.cxx
index b008bb0..fc29f19 100644
--- a/framework/source/services/urltransformer.cxx
+++ b/framework/source/services/urltransformer.cxx
@@ -160,7 +160,7 @@ sal_Bool SAL_CALL URLTransformer::parseStrict( URL& aURL ) throw( RuntimeExcepti
{
// Safe impossible cases.
if (( &aURL == NULL ) ||
- ( aURL.Complete.getLength() < 1 ) )
+ ( aURL.Complete.isEmpty() ) )
{
return sal_False;
}
@@ -214,7 +214,7 @@ sal_Bool SAL_CALL URLTransformer::parseSmart( URL& aURL
{
// Safe impossible cases.
if (( &aURL == NULL ) ||
- ( aURL.Complete.getLength() < 1 ) )
+ ( aURL.Complete.isEmpty() ) )
{
return sal_False;
}
@@ -279,7 +279,7 @@ sal_Bool SAL_CALL URLTransformer::assemble( URL& aURL ) throw( RuntimeException
::rtl::OUStringBuffer aCompletePath( aURL.Path );
// Concat the name if it is provided, just support a final slash
- if ( aURL.Name.getLength() > 0 )
+ if ( !aURL.Name.isEmpty() )
{
sal_Int32 nIndex = aURL.Path.lastIndexOf( sal_Unicode('/') );
if ( nIndex == ( aURL.Path.getLength() -1 ))
@@ -312,7 +312,7 @@ sal_Bool SAL_CALL URLTransformer::assemble( URL& aURL ) throw( RuntimeException
// Return "URL is assembled".
return sal_True;
}
- else if ( aURL.Protocol.getLength() > 0 )
+ else if ( !aURL.Protocol.isEmpty() )
{
// Minimal support for unknown protocols
::rtl::OUStringBuffer aBuffer( aURL.Protocol );
@@ -333,7 +333,7 @@ sal_Bool SAL_CALL URLTransformer::assemble( URL& aURL ) throw( RuntimeException
{
// Safe impossible cases.
if (( &aURL == NULL ) ||
- ( aURL.Complete.getLength() < 1 ) ||
+ ( aURL.Complete.isEmpty() ) ||
(( bWithPassword != sal_True ) &&
( bWithPassword != sal_False ) ) )
{
@@ -345,7 +345,7 @@ sal_Bool SAL_CALL URLTransformer::assemble( URL& aURL ) throw( RuntimeException
sal_Bool bParseResult = parseSmart( aTestURL, aTestURL.Protocol );
if ( bParseResult )
{
- if ( !bWithPassword && aTestURL.Password.getLength() > 0 )
+ if ( !bWithPassword && !aTestURL.Password.isEmpty() )
{
// Exchange password text with other placeholder string
aTestURL.Password = ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("<******>"));
diff --git a/framework/source/uiconfiguration/imagemanagerimpl.cxx b/framework/source/uiconfiguration/imagemanagerimpl.cxx
index 86c3c97..7426360 100644
--- a/framework/source/uiconfiguration/imagemanagerimpl.cxx
+++ b/framework/source/uiconfiguration/imagemanagerimpl.cxx
@@ -195,7 +195,7 @@ void CmdImageList::impl_fillCommandToImageNameMap()
SERVICENAME_UICOMMANDDESCRIPTION ),
UNO_QUERY );
- if ( m_aModuleIdentifier.getLength() > 0 )
+ if ( !m_aModuleIdentifier.isEmpty() )
{
// If we have a module identifier - use to retrieve the command image name list from it.
// Otherwise we will use the global command image list
diff --git a/framework/source/uiconfiguration/moduleuicfgsupplier.cxx b/framework/source/uiconfiguration/moduleuicfgsupplier.cxx
index 47e2a9b..241a3d5 100644
--- a/framework/source/uiconfiguration/moduleuicfgsupplier.cxx
+++ b/framework/source/uiconfiguration/moduleuicfgsupplier.cxx
@@ -227,7 +227,7 @@ throw ( NoSuchElementException, RuntimeException)
sShort = ::rtl::OUString();
}
- if (!sShort.getLength())
+ if (sShort.isEmpty())
throw NoSuchElementException();
PropertyValue aArg;
Sequence< Any > aArgs( 2 );
diff --git a/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx b/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx
index 7977f8d..fc7252a 100644
--- a/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx
+++ b/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx
@@ -272,7 +272,7 @@ void ModuleUIConfigurationManager::impl_preloadUIElementTypeList( Layer eLayer,
OUString aExtension( aUIElementNames[n].copy( nIndex+1 ));
OUString aUIElementName( aUIElementNames[n].copy( 0, nIndex ));
- if (( aUIElementName.getLength() > 0 ) &&
+ if (( !aUIElementName.isEmpty() ) &&
( aExtension.equalsIgnoreAsciiCaseAsciiL( "xml", 3 )))
{
aUIElementData.aResourceURL = aResURLPrefix + aUIElementName;
@@ -302,7 +302,7 @@ void ModuleUIConfigurationManager::impl_requestUIElementData( sal_Int16 nElement
UIElementType& rElementTypeData = m_aUIElements[eLayer][nElementType];
Reference< XStorage > xElementTypeStorage = rElementTypeData.xStorage;
- if ( xElementTypeStorage.is() && aUIElementData.aName.getLength() )
+ if ( xElementTypeStorage.is() && !aUIElementData.aName.isEmpty() )
{
try
{
@@ -839,7 +839,7 @@ void SAL_CALL ModuleUIConfigurationManager::initialize( const Sequence< Any >& a
else if ( i == ::com::sun::star::ui::UIElementType::STATUSBAR )
aResourceType = PresetHandler::RESOURCETYPE_STATUSBAR();
- if ( aResourceType.getLength() > 0 )
+ if ( !aResourceType.isEmpty() )
{
m_pStorageHandler[i] = new PresetHandler( m_xServiceManager );
m_pStorageHandler[i]->connectToResource( PresetHandler::E_MODULES,
diff --git a/framework/source/uiconfiguration/uiconfigurationmanager.cxx b/framework/source/uiconfiguration/uiconfigurationmanager.cxx
index 66369b8..2e29184 100644
--- a/framework/source/uiconfiguration/uiconfigurationmanager.cxx
+++ b/framework/source/uiconfiguration/uiconfigurationmanager.cxx
@@ -215,7 +215,7 @@ void UIConfigurationManager::impl_preloadUIElementTypeList( sal_Int16 nElementTy
rtl::OUString aExtension( aUIElementNames[n].copy( nIndex+1 ));
rtl::OUString aUIElementName( aUIElementNames[n].copy( 0, nIndex ));
- if (( aUIElementName.getLength() > 0 ) &&
+ if (( !aUIElementName.isEmpty() ) &&
( aExtension.equalsIgnoreAsciiCaseAsciiL( "xml", 3 )))
{
aUIElementData.aResourceURL = aResURLPrefix + aUIElementName;
@@ -240,7 +240,7 @@ void UIConfigurationManager::impl_requestUIElementData( sal_Int16 nElementType,
UIElementType& rElementTypeData = m_aUIElements[nElementType];
Reference< XStorage > xElementTypeStorage = rElementTypeData.xStorage;
- if ( xElementTypeStorage.is() && aUIElementData.aName.getLength() )
+ if ( xElementTypeStorage.is() && !aUIElementData.aName.isEmpty() )
{
try
{
diff --git a/framework/source/uiconfiguration/windowstateconfiguration.cxx b/framework/source/uiconfiguration/windowstateconfiguration.cxx
index 88e0a3c..1f09742 100644
--- a/framework/source/uiconfiguration/windowstateconfiguration.cxx
+++ b/framework/source/uiconfiguration/windowstateconfiguration.cxx
@@ -1403,7 +1403,7 @@ WindowStateConfiguration::WindowStateConfiguration( const Reference< XMultiServi
}
}
- if ( aWindowStateFileStr.getLength() > 0 )
+ if ( !aWindowStateFileStr.isEmpty() )
{
// Create first mapping ModuleIdentifier ==> Window state configuration file
m_aModuleToFileHashMap.insert( ModuleToWindowStateFileMap::value_type( aModuleIdentifier, aWindowStateFileStr ));
diff --git a/framework/source/uielement/addonstoolbarmanager.cxx b/framework/source/uielement/addonstoolbarmanager.cxx
index 5d136aa..6ae3d09 100644
--- a/framework/source/uielement/addonstoolbarmanager.cxx
+++ b/framework/source/uielement/addonstoolbarmanager.cxx
@@ -125,10 +125,10 @@ AddonsToolBarManager::~AddonsToolBarManager()
static sal_Bool IsCorrectContext( const ::rtl::OUString& rModuleIdentifier, const ::rtl::OUString& aContextList )
{
- if ( aContextList.getLength() == 0 )
+ if ( aContextList.isEmpty() )
return sal_True;
- if ( rModuleIdentifier.getLength() > 0 )
+ if ( !rModuleIdentifier.isEmpty() )
{
sal_Int32 nIndex = aContextList.indexOf( rModuleIdentifier );
return ( nIndex >= 0 );
@@ -145,7 +145,7 @@ static Image RetrieveImage( Reference< com::sun::star::frame::XFrame >& rFrame,
{
Image aImage;
- if ( aImageId.getLength() > 0 )
+ if ( !aImageId.isEmpty() )
{
aImage = framework::AddonsOptions().GetImageFromURL( aImageId, bBigImage );
if ( !!aImage )
diff --git a/framework/source/uielement/buttontoolbarcontroller.cxx b/framework/source/uielement/buttontoolbarcontroller.cxx
index 554a9e7..4f8c655 100644
--- a/framework/source/uielement/buttontoolbarcontroller.cxx
+++ b/framework/source/uielement/buttontoolbarcontroller.cxx
@@ -245,7 +245,7 @@ throw (::com::sun::star::uno::RuntimeException)
if ( m_bInitialized &&
m_xFrame.is() &&
m_xServiceManager.is() &&
- m_aCommandURL.getLength() )
+ !m_aCommandURL.isEmpty() )
{
if ( !m_xURLTransformer.is() )
{
diff --git a/framework/source/uielement/complextoolbarcontroller.cxx b/framework/source/uielement/complextoolbarcontroller.cxx
index 94b3986..a70dd5a 100644
--- a/framework/source/uielement/complextoolbarcontroller.cxx
+++ b/framework/source/uielement/complextoolbarcontroller.cxx
@@ -136,7 +136,7 @@ throw ( RuntimeException )
if ( m_bInitialized &&
m_xFrame.is() &&
m_xServiceManager.is() &&
- m_aCommandURL.getLength() )
+ !m_aCommandURL.isEmpty() )
{
xURLTransformer = m_xURLTransformer;
xDispatch = getDispatchFromCommand( m_aCommandURL );
@@ -146,7 +146,7 @@ throw ( RuntimeException )
}
}
- if ( xDispatch.is() && aTargetURL.Complete.getLength() > 0 )
+ if ( xDispatch.is() && !aTargetURL.Complete.isEmpty() )
{
// Execute dispatch asynchronously
ExecuteInfo* pExecuteInfo = new ExecuteInfo;
@@ -321,7 +321,7 @@ uno::Reference< frame::XDispatch > ComplexToolbarController::getDispatchFromComm
{
uno::Reference< frame::XDispatch > xDispatch;
- if ( m_bInitialized && m_xFrame.is() && m_xServiceManager.is() && aCommand.getLength() )
+ if ( m_bInitialized && m_xFrame.is() && m_xServiceManager.is() && !aCommand.isEmpty() )
{
URLToDispatchMap::const_iterator pIter = m_aListenerMap.find( aCommand );
if ( pIter != m_aListenerMap.end() )
@@ -335,7 +335,7 @@ uno::Reference< frame::XDispatch > ComplexToolbarController::getDispatchFromComm
const ::com::sun::star::util::URL& ComplexToolbarController::getInitializedURL()
{
- if ( m_aURL.Complete.getLength() == 0 )
+ if ( m_aURL.Complete.isEmpty() )
{
m_aURL.Complete = m_aCommandURL;
m_xURLTransformer->parseStrict( m_aURL );
diff --git a/framework/source/uielement/fontsizemenucontroller.cxx b/framework/source/uielement/fontsizemenucontroller.cxx
index fadf338..2de8e8d 100644
--- a/framework/source/uielement/fontsizemenucontroller.cxx
+++ b/framework/source/uielement/fontsizemenucontroller.cxx
@@ -166,7 +166,7 @@ void FontSizeMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& r
// try to retrieve printer name of document
aPrinterName = retrievePrinterName( m_xFrame );
- if ( aPrinterName.getLength() > 0 )
+ if ( !aPrinterName.isEmpty() )
{
pInfoPrinter = new Printer( aPrinterName );
if ( pInfoPrinter && pInfoPrinter->GetDevFontCount() > 0 )
diff --git a/framework/source/uielement/generictoolbarcontroller.cxx b/framework/source/uielement/generictoolbarcontroller.cxx
index 08db404..defc407 100644
--- a/framework/source/uielement/generictoolbarcontroller.cxx
+++ b/framework/source/uielement/generictoolbarcontroller.cxx
@@ -165,7 +165,7 @@ throw ( RuntimeException )
if ( m_bInitialized &&
m_xFrame.is() &&
m_xServiceManager.is() &&
- m_aCommandURL.getLength() )
+ !m_aCommandURL.isEmpty() )
{
xURLTransformer = Reference< XURLTransformer >( m_xServiceManager->createInstance(
rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "com.sun.star.util.URLTransformer" ))),
diff --git a/framework/source/uielement/macrosmenucontroller.cxx b/framework/source/uielement/macrosmenucontroller.cxx
index 7da9718..568383d 100644
--- a/framework/source/uielement/macrosmenucontroller.cxx
+++ b/framework/source/uielement/macrosmenucontroller.cxx
@@ -169,7 +169,7 @@ IMPL_STATIC_LINK_NOINSTANCE( MacrosMenuController, ExecuteHdl_Impl, ExecuteInfo*
String MacrosMenuController::RetrieveLabelFromCommand( const String& aCmdURL )
{
- sal_Bool bModuleIdentified = m_aModuleIdentifier.getLength() != 0;
+ sal_Bool bModuleIdentified = !m_aModuleIdentifier.isEmpty();
return framework::RetrieveLabelFromCommand(aCmdURL,m_xServiceManager,m_xUICommandLabels,m_xFrame,m_aModuleIdentifier,bModuleIdentified,"Label");
}
diff --git a/framework/source/uielement/menubarmanager.cxx b/framework/source/uielement/menubarmanager.cxx
index 1ce4776..4de7ea4 100644
--- a/framework/source/uielement/menubarmanager.cxx
+++ b/framework/source/uielement/menubarmanager.cxx
@@ -782,8 +782,8 @@ void MenuBarManager::CheckAndAddMenuExtension( Menu* pMenu )
// retrieve menu extension item
MenuExtensionItem aMenuItem( GetMenuExtension() );
- if (( aMenuItem.aURL.getLength() > 0 ) &&
- ( aMenuItem.aLabel.getLength() > 0 ))
+ if (( !aMenuItem.aURL.isEmpty() ) &&
+ ( !aMenuItem.aLabel.isEmpty() ))
{
// remove all old window list entries from menu
sal_uInt16 nNewItemId( 0 );
@@ -925,7 +925,7 @@ IMPL_LINK( MenuBarManager, Activate, Menu *, pMenu )
Reference< XDispatch > xMenuItemDispatch;
::rtl::OUString aItemCommand = pMenu->GetItemCommand( pMenuItemHandler->nItemId );
- if ( !aItemCommand.getLength() )
+ if ( aItemCommand.isEmpty() )
{
aItemCommand = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "slot:" ));
aItemCommand += ::rtl::OUString::valueOf( (sal_Int32)pMenuItemHandler->nItemId );
@@ -1281,7 +1281,7 @@ void MenuBarManager::FillMenuManager( Menu* pMenu, const Reference< XFrame >& rF
sal_uInt16 nItemId = FillItemCommand(aItemCommand,pMenu, i );
// Set module identifier when provided from outside
- if ( rModuleIdentifier.getLength() > 0 )
+ if ( !rModuleIdentifier.isEmpty() )
{
m_aModuleIdentifier = rModuleIdentifier;
m_bModuleIdentified = sal_True;
@@ -1290,7 +1290,7 @@ void MenuBarManager::FillMenuManager( Menu* pMenu, const Reference< XFrame >& rF
if (( pMenu->IsMenuBar() || bAccessibilityEnabled ) &&
( pMenu->GetItemText( nItemId ).Len() == 0 ))
{
- if ( aItemCommand.getLength() > 0 )
+ if ( !aItemCommand.isEmpty() )
pMenu->SetItemText( nItemId, RetrieveLabelFromCommand( aItemCommand ));
}
@@ -1425,7 +1425,7 @@ void MenuBarManager::FillMenuManager( Menu* pMenu, const Reference< XFrame >& rF
MenuConfiguration::Attributes* pMenuAttributes =
(MenuConfiguration::Attributes*)pMenu->GetUserValue( nItemId );
- if ( pMenuAttributes && pMenuAttributes->aImageId.getLength() > 0 )
+ if ( pMenuAttributes && !pMenuAttributes->aImageId.isEmpty() )
{
// Retrieve image id from menu attributes
aImage = GetImageFromURL( m_xFrame, aImageId, false );
@@ -1646,7 +1646,7 @@ void MenuBarManager::RetrieveImageManagers()
}
Reference< XModuleManager > xModuleManager;
- if ( m_aModuleIdentifier.getLength() == 0 )
+ if ( m_aModuleIdentifier.isEmpty() )
xModuleManager.set( getServiceFactory()->createInstance( SERVICENAME_MODULEMANAGER ), UNO_QUERY_THROW );
try
@@ -1794,7 +1794,7 @@ void MenuBarManager::FillMenu(
}
// Use help command to transport module identifier
- if ( aModuleIdentifier.getLength() > 0 )
+ if ( !aModuleIdentifier.isEmpty() )
pMenu->SetHelpCommand( nId, aModuleIdentifier );
++nId;
@@ -2009,7 +2009,7 @@ sal_uInt16 MenuBarManager::FillItemCommand(::rtl::OUString& _rItemCommand,Menu*
sal_uInt16 nItemId = _pMenu->GetItemId( _nIndex );
_rItemCommand = _pMenu->GetItemCommand( nItemId );
- if ( !_rItemCommand.getLength() )
+ if ( _rItemCommand.isEmpty() )
{
const static ::rtl::OUString aSlotString( RTL_CONSTASCII_USTRINGPARAM( "slot:" ));
_rItemCommand = aSlotString;
diff --git a/framework/source/uielement/menubarmerger.cxx b/framework/source/uielement/menubarmerger.cxx
index 30a6a62..4dd727c 100644
--- a/framework/source/uielement/menubarmerger.cxx
+++ b/framework/source/uielement/menubarmerger.cxx
@@ -74,7 +74,7 @@ namespace framework
*/
bool MenuBarMerger::IsCorrectContext( const ::rtl::OUString& rContext, const ::rtl::OUString& rModuleIdentifier )
{
- return (( rContext.getLength() == 0 ) || ( rContext.indexOf( rModuleIdentifier ) >= 0 ));
+ return (( rContext.isEmpty() ) || ( rContext.indexOf( rModuleIdentifier ) >= 0 ));
}
void MenuBarMerger::RetrieveReferencePath(
@@ -88,7 +88,7 @@ void MenuBarMerger::RetrieveReferencePath(
do
{
::rtl::OUString aToken = rReferencePathString.getToken( 0, aDelimiter, nIndex );
- if ( aToken.getLength() > 0 )
+ if ( !aToken.isEmpty() )
rReferencePath.push_back( aToken );
}
while ( nIndex >= 0 );
diff --git a/framework/source/uielement/newmenucontroller.cxx b/framework/source/uielement/newmenucontroller.cxx
index dad7a19..741bb78 100644
--- a/framework/source/uielement/newmenucontroller.cxx
+++ b/framework/source/uielement/newmenucontroller.cxx
@@ -110,7 +110,7 @@ void NewMenuController::setMenuImages( PopupMenu* pPopupMenu, sal_Bool bSetImage
if ( pInfo != m_aAddInfoForItem.end() )
aImageId = pInfo->second.aImageId; // Retrieve image id for menu item
- if ( aImageId.getLength() > 0 )
+ if ( !aImageId.isEmpty() )
{
aImage = GetImageFromURL( xFrame, aImageId, false );
if ( !!aImage )
@@ -143,7 +143,7 @@ void NewMenuController::determineAndSetNewDocAccel( PopupMenu* pPopupMenu, const
sal_Bool bFound( sal_False );
rtl::OUString aCommand;
- if ( m_aEmptyDocURL.getLength() > 0 )
+ if ( !m_aEmptyDocURL.isEmpty() )
{
// Search for the empty document URL
@@ -167,7 +167,7 @@ void NewMenuController::determineAndSetNewDocAccel( PopupMenu* pPopupMenu, const
{
// Search for the default module name
rtl::OUString aDefaultModuleName( SvtModuleOptions().GetDefaultModuleName() );
- if ( aDefaultModuleName.getLength() > 0 )
+ if ( !aDefaultModuleName.isEmpty() )
{
for ( sal_uInt32 i = 0; i < sal_uInt32( nCount ); i++ )
{
@@ -507,7 +507,7 @@ void NewMenuController::impl_setPopupMenu()
m_bModuleIdentified = sal_True;
Reference< XNameAccess > xNameAccess( xModuleManager, UNO_QUERY );
- if (( m_aModuleIdentifier.getLength() > 0 ) && xNameAccess.is() )
+ if (( !m_aModuleIdentifier.isEmpty() ) && xNameAccess.is() )
{
Sequence< PropertyValue > aSeq;
diff --git a/framework/source/uielement/spinfieldtoolbarcontroller.cxx b/framework/source/uielement/spinfieldtoolbarcontroller.cxx
index 8a941ae..6245d01 100644
--- a/framework/source/uielement/spinfieldtoolbarcontroller.cxx
+++ b/framework/source/uielement/spinfieldtoolbarcontroller.cxx
@@ -477,7 +477,7 @@ void SpinfieldToolbarController::executeControlCommand( const ::com::sun::star::
}
// Check values and set members
- if ( aValue.getLength() > 0 )
+ if ( !aValue.isEmpty() )
{
m_bFloat = bFloatValue;
m_nValue = aValue.toDouble();
@@ -486,17 +486,17 @@ void SpinfieldToolbarController::executeControlCommand( const ::com::sun::star::
m_pSpinfieldControl->SetText( aOutString );
notifyTextChanged( aOutString );
}
- if ( aMax.getLength() > 0 )
+ if ( !aMax.isEmpty() )
{
m_nMax = aMax.toDouble();
m_bMaxSet = true;
}
- if ( aMin.getLength() > 0 )
+ if ( !aMin.isEmpty() )
{
m_nMin = aMin.toDouble();
m_bMinSet = true;
}
- if ( aStep.getLength() > 0 )
+ if ( !aStep.isEmpty() )
m_nStep = aStep.toDouble();
}
@@ -525,7 +525,7 @@ bool SpinfieldToolbarController::impl_getValue(
rtl::OUString SpinfieldToolbarController::impl_formatOutputString( double fValue )
{
- if ( m_aOutFormat.getLength() == 0 )
+ if ( m_aOutFormat.isEmpty() )
{
if ( m_bFloat )
return rtl::OUString::valueOf( fValue );
diff --git a/framework/source/uielement/statusbarmanager.cxx b/framework/source/uielement/statusbarmanager.cxx
index 029cc52..df2508b 100644
--- a/framework/source/uielement/statusbarmanager.cxx
+++ b/framework/source/uielement/statusbarmanager.cxx
@@ -519,7 +519,7 @@ void StatusBarManager::FillStatusBar( const uno::Reference< container::XIndexAcc
}
}
- if (( nType == ::com::sun::star::ui::ItemType::DEFAULT ) && ( aCommandURL.getLength() > 0 ))
+ if (( nType == ::com::sun::star::ui::ItemType::DEFAULT ) && ( !aCommandURL.isEmpty() ))
{
rtl::OUString aString( RetrieveLabelFromCommand( aCommandURL ));
sal_uInt16 nItemBits( impl_convertItemStyleToItemBits( nStyle ));
diff --git a/framework/source/uielement/toolbarmanager.cxx b/framework/source/uielement/toolbarmanager.cxx
index 3c2896e..5b0cfe5 100644
--- a/framework/source/uielement/toolbarmanager.cxx
+++ b/framework/source/uielement/toolbarmanager.cxx
@@ -842,7 +842,7 @@ uno::Sequence< beans::PropertyValue > ToolBarManager::GetPropsForCommand( const
m_bModuleIdentified = sal_True;
m_aModuleIdentifier = xModuleManager->identify( xIfac );
- if ( m_aModuleIdentifier.getLength() > 0 )
+ if ( !m_aModuleIdentifier.isEmpty() )
{
Reference< XNameAccess > xNameAccess( m_xServiceManager->createInstance( SERVICENAME_UICOMMANDDESCRIPTION ), UNO_QUERY );
if ( xNameAccess.is() )
@@ -852,7 +852,7 @@ uno::Sequence< beans::PropertyValue > ToolBarManager::GetPropsForCommand( const
if ( m_xUICommandLabels.is() )
{
- if ( rCmdURL.getLength() > 0 )
+ if ( !rCmdURL.isEmpty() )
m_xUICommandLabels->getByName( rCmdURL ) >>= aPropSeq;
}
}
@@ -1036,7 +1036,7 @@ void ToolBarManager::CreateControllers()
if ( xSubToolBar.is() && xSubToolBar->opensSubToolbar() )
{
rtl::OUString aSubToolBarName = xSubToolBar->getSubToolbarName();
- if ( aSubToolBarName.getLength() != 0 )
+ if ( !aSubToolBarName.isEmpty() )
{
SubToolBarToSubToolBarControllerMap::iterator pIter =
m_aSubToolBarControllerMap.find( aSubToolBarName );
@@ -1333,7 +1333,7 @@ void ToolBarManager::FillToolbar( const Reference< XIndexAccess >& rItemContaine
aProp[i].Value >>= nStyle;
}
- if (( nType == ::com::sun::star::ui::ItemType::DEFAULT ) && ( aCommandURL.getLength() > 0 ))
+ if (( nType == ::com::sun::star::ui::ItemType::DEFAULT ) && ( !aCommandURL.isEmpty() ))
{
::rtl::OUString aString( RetrieveLabelFromCommand( aCommandURL ));
@@ -1342,7 +1342,7 @@ void ToolBarManager::FillToolbar( const Reference< XIndexAccess >& rItemContaine
m_aMenuMap[ nId ] = aMenuDesc;
m_pToolBar->InsertItem( nId, aString, nItemBits );
m_pToolBar->SetItemCommand( nId, aCommandURL );
- if ( aTooltip.getLength() )
+ if ( !aTooltip.isEmpty() )
{
m_pToolBar->SetQuickHelpText( nId, aTooltip );
}
@@ -1360,7 +1360,7 @@ void ToolBarManager::FillToolbar( const Reference< XIndexAccess >& rItemContaine
m_pToolBar->SetQuickHelpText( nId, sQuickHelp );
}
- if ( aLabel.getLength() > 0 )
+ if ( !aLabel.isEmpty() )
{
m_pToolBar->SetItemText( nId, aLabel );
}
@@ -1493,7 +1493,7 @@ void ToolBarManager::FillToolbar( const Reference< XIndexAccess >& rItemContaine
{
rtl::OUString aUIName;
xPropSet->getPropertyValue( rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "UIName" ))) >>= aUIName;
- if ( aUIName.getLength() > 0 )
+ if ( !aUIName.isEmpty() )
m_pToolBar->SetText( aUIName );
}
catch (const Exception&)
diff --git a/framework/source/uielement/toolbarmerger.cxx b/framework/source/uielement/toolbarmerger.cxx
index 7e0b39f..21d7d6a 100644
--- a/framework/source/uielement/toolbarmerger.cxx
+++ b/framework/source/uielement/toolbarmerger.cxx
@@ -116,7 +116,7 @@ bool ToolBarMerger::IsCorrectContext(
const ::rtl::OUString& rContext,
const ::rtl::OUString& rModuleIdentifier )
{
- return (( rContext.getLength() == 0 ) || ( rContext.indexOf( rModuleIdentifier ) >= 0 ));
+ return (( rContext.isEmpty() ) || ( rContext.indexOf( rModuleIdentifier ) >= 0 ));
}
/**
diff --git a/framework/source/uielement/toolbarsmenucontroller.cxx b/framework/source/uielement/toolbarsmenucontroller.cxx
index a31f328..3a5ed2f 100644
--- a/framework/source/uielement/toolbarsmenucontroller.cxx
+++ b/framework/source/uielement/toolbarsmenucontroller.cxx
@@ -176,7 +176,7 @@ void ToolbarsMenuController::addCommand(
sal_uInt16 nItemId = m_xPopupMenu->getItemCount()+1;
rtl::OUString aLabel;
- if ( rLabel.getLength() == 0 )
+ if ( rLabel.isEmpty() )
aLabel = getUINameFromCommand( rCommandURL );
else
aLabel = rLabel;
@@ -297,7 +297,7 @@ static void fillHashMap( const Sequence< Sequence< ::com::sun::star::beans::Prop
pProperties[j].Value >>= aUIName;
}
- if ( aResourceURL.getLength() > 0 &&
+ if ( !aResourceURL.isEmpty() &&
rHashMap.find( aResourceURL ) == rHashMap.end() )
rHashMap.insert( ToolbarHashMap::value_type( aResourceURL, aUIName ));
}
@@ -322,7 +322,7 @@ Sequence< Sequence< com::sun::star::beans::PropertyValue > > ToolbarsMenuControl
xPropSet->getPropertyValue( rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "ResourceURL" ))) >>= aResName;
if (( nType == ::com::sun::star::ui::UIElementType::TOOLBAR ) &&
- ( aResName.getLength() > 0 ))
+ ( !aResName.isEmpty() ))
{
ToolBarInfo aToolBarInfo;
@@ -411,7 +411,7 @@ void ToolbarsMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& r
sal_Bool bHideFromMenu( sal_False );
sal_Bool bContextSensitive( sal_False );
sal_Bool bVisible( sal_False );
- if ( aUIName.getLength() == 0 &&
+ if ( aUIName.isEmpty() &&
m_xPersistentWindowState.is() )
{
try
@@ -444,7 +444,7 @@ void ToolbarsMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >& r
}
- if (( aUIName.getLength() > 0 ) && ( !bHideFromMenu ))
+ if (( !aUIName.isEmpty() ) && ( !bHideFromMenu ))
{
ToolBarEntry aTbEntry;
aTbEntry.aUIName = aUIName;
diff --git a/framework/source/uielement/uicommanddescription.cxx b/framework/source/uielement/uicommanddescription.cxx
index 7c1ad85..a14d593 100644
--- a/framework/source/uielement/uicommanddescription.cxx
+++ b/framework/source/uielement/uicommanddescription.cxx
@@ -341,7 +341,7 @@ Any ConfigurationAccess_UICommand::getSequenceFromCache( const ::rtl::OUString&
Sequence< PropertyValue > aPropSeq( 4 );
aPropSeq[0].Name = m_aPropLabel;
- aPropSeq[0].Value = pIter->second.aContextLabel.getLength() ?
+ aPropSeq[0].Value = !pIter->second.aContextLabel.isEmpty() ?
makeAny( pIter->second.aContextLabel ): makeAny( pIter->second.aLabel );
aPropSeq[1].Name = m_aPropName;
aPropSeq[1].Value <<= pIter->second.aCommandName;
@@ -738,7 +738,7 @@ throw (::com::sun::star::container::NoSuchElementException, ::com::sun::star::la
}
}
}
- else if ( m_aPrivateResourceURL.getLength() && aName.indexOf( m_aPrivateResourceURL ) == 0 )
+ else if ( !m_aPrivateResourceURL.isEmpty() && aName.indexOf( m_aPrivateResourceURL ) == 0 )
{
// special keys to retrieve information about a set of commands
return m_xGenericUICommands->getByName( aName );
diff --git a/framework/source/uifactory/addonstoolboxfactory.cxx b/framework/source/uifactory/addonstoolboxfactory.cxx
index 601bff0..aa024d5 100644
--- a/framework/source/uifactory/addonstoolboxfactory.cxx
+++ b/framework/source/uifactory/addonstoolboxfactory.cxx
@@ -91,10 +91,10 @@ AddonsToolBoxFactory::~AddonsToolBoxFactory()
static sal_Bool IsCorrectContext( const ::rtl::OUString& rModuleIdentifier, const rtl::OUString& aContextList )
{
- if ( aContextList.getLength() == 0 )
+ if ( aContextList.isEmpty() )
return sal_True;
- if ( rModuleIdentifier.getLength() > 0 )
+ if ( !rModuleIdentifier.isEmpty() )
{
sal_Int32 nIndex = aContextList.indexOf( rModuleIdentifier );
return ( nIndex >= 0 );
diff --git a/framework/source/uifactory/factoryconfiguration.cxx b/framework/source/uifactory/factoryconfiguration.cxx
index cce667a..cbfb934 100644
--- a/framework/source/uifactory/factoryconfiguration.cxx
+++ b/framework/source/uifactory/factoryconfiguration.cxx
@@ -113,7 +113,7 @@ rtl::OUString ConfigurationAccess_ControllerFactory::getServiceFromCommandModule
if ( pIter != m_aMenuControllerMap.end() )
return pIter->second.m_aImplementationName;
- else if ( rModule.getLength() )
+ else if ( !rModule.isEmpty() )
{
// Try to detect if we have a generic popup menu controller
pIter = m_aMenuControllerMap.find( getHashKeyFromStrings( rCommandURL, rtl::OUString() ));
@@ -134,7 +134,7 @@ rtl::OUString ConfigurationAccess_ControllerFactory::getValueFromCommandModule(
if ( pIter != m_aMenuControllerMap.end() )
return pIter->second.m_aValue;
- else if ( rModule.getLength() )
+ else if ( !rModule.isEmpty() )
{
// Try to detect if we have a generic popup menu controller
pIter = m_aMenuControllerMap.find( getHashKeyFromStrings( rCommandURL, rtl::OUString() ));
diff --git a/framework/source/uifactory/menubarfactory.cxx b/framework/source/uifactory/menubarfactory.cxx
index 47c0bce..7e2de9c 100644
--- a/framework/source/uifactory/menubarfactory.cxx
+++ b/framework/source/uifactory/menubarfactory.cxx
@@ -167,7 +167,7 @@ void MenuBarFactory::CreateUIElement(const ::rtl::OUString& ResourceURL
if ( !bHasSettings )
{
rtl::OUString aModuleIdentifier = _xModuleManager->identify( Reference< XInterface >( xFrame, UNO_QUERY ));
- if ( aModuleIdentifier.getLength() )
+ if ( !aModuleIdentifier.isEmpty() )
{
Reference< ::com::sun::star::ui::XModuleUIConfigurationManagerSupplier > xModuleCfgSupplier(
_xServiceManager->createInstance( SERVICENAME_MODULEUICONFIGURATIONMANAGERSUPPLIER ), UNO_QUERY );
diff --git a/framework/source/uifactory/toolbarcontrollerfactory.cxx b/framework/source/uifactory/toolbarcontrollerfactory.cxx
index c3a2365..2369605 100644
--- a/framework/source/uifactory/toolbarcontrollerfactory.cxx
+++ b/framework/source/uifactory/toolbarcontrollerfactory.cxx
@@ -119,7 +119,7 @@ throw (Exception, RuntimeException)
}
rtl::OUString aServiceName = m_pConfigAccess->getServiceFromCommandModule( aServiceSpecifier, rtl::OUString() );
- if ( aServiceName.getLength() > 0 )
+ if ( !aServiceName.isEmpty() )
return m_xServiceManager->createInstance( aServiceName );
else
return Reference< XInterface >();
@@ -189,7 +189,7 @@ throw (Exception, RuntimeException)
// SAFE
- if ( aServiceName.getLength() > 0 )
+ if ( !aServiceName.isEmpty() )
return xServiceManager->createInstanceWithArguments( aServiceName, aNewArgs );
else
return Reference< XInterface >();
@@ -216,7 +216,7 @@ throw (::com::sun::star::uno::RuntimeException)
m_pConfigAccess->readConfigurationData();
}
- return ( m_pConfigAccess->getServiceFromCommandModule( aCommandURL, aModuleName ).getLength() > 0 );
+ return ( !m_pConfigAccess->getServiceFromCommandModule( aCommandURL, aModuleName ).isEmpty() );
}
void SAL_CALL ToolbarControllerFactory::registerController(
diff --git a/framework/source/uifactory/uielementfactorymanager.cxx b/framework/source/uifactory/uielementfactorymanager.cxx
index 4d19c33..a673ba6 100644
--- a/framework/source/uifactory/uielementfactorymanager.cxx
+++ b/framework/source/uifactory/uielementfactorymanager.cxx
@@ -199,7 +199,7 @@ Sequence< Sequence< PropertyValue > > ConfigurationAccess_FactoryManager::getFac
while ( pIter != m_aFactoryManagerMap.end() )
{
rtl::OUString aFactory = pIter->first;
- if ( aFactory.getLength() > 0 )
+ if ( !aFactory.isEmpty() )
{
sal_Int32 nToken = 0;
Sequence< PropertyValue > aSeq( 1 );
@@ -507,7 +507,7 @@ throw ( RuntimeException )
rtl::OUString aServiceSpecifier = m_pConfigAccess->getFactorySpecifierFromTypeNameModule( aType, aName, aModuleId );
aLock.unlock();
- if ( aServiceSpecifier.getLength() )
+ if ( !aServiceSpecifier.isEmpty() )
return Reference< XUIElementFactory >( xSManager->createInstance( aServiceSpecifier ), UNO_QUERY );
else
return Reference< XUIElementFactory >();
diff --git a/framework/source/uifactory/windowcontentfactorymanager.cxx b/framework/source/uifactory/windowcontentfactorymanager.cxx
index 4004737..f9a841d 100644
--- a/framework/source/uifactory/windowcontentfactorymanager.cxx
+++ b/framework/source/uifactory/windowcontentfactorymanager.cxx
@@ -112,7 +112,7 @@ void WindowContentFactoryManager::RetrieveTypeNameFromResourceURL( const rtl::OU
do
{
::rtl::OUString sToken = aTmpStr.getToken( 0, '/', nToken);
- if ( sToken.getLength() )
+ if ( !sToken.isEmpty() )
{
if ( nPart == 0 )
aType = sToken;
@@ -178,9 +178,9 @@ throw (uno::Exception, uno::RuntimeException)
}
RetrieveTypeNameFromResourceURL( aResourceURL, aType, aName );
- if ( aType.getLength() > 0 &&
- aName.getLength() > 0 &&
- aModuleId.getLength() > 0 )
+ if ( !aType.isEmpty() &&
+ !aName.isEmpty() &&
+ !aModuleId.isEmpty() )
{
::rtl::OUString aImplementationName;
uno::Reference< uno::XInterface > xHolder( static_cast<cppu::OWeakObject*>(this), uno::UNO_QUERY );
@@ -197,7 +197,7 @@ throw (uno::Exception, uno::RuntimeException)
}
aImplementationName = m_pConfigAccess->getFactorySpecifierFromTypeNameModule( aType, aName, aModuleId );
- if ( aImplementationName.getLength() > 0 )
+ if ( !aImplementationName.isEmpty() )
{
aLock.unlock();
// UNSAFE
diff --git a/framework/source/xml/acceleratorconfigurationreader.cxx b/framework/source/xml/acceleratorconfigurationreader.cxx
index 42227261..588d6a4 100644
--- a/framework/source/xml/acceleratorconfigurationreader.cxx
+++ b/framework/source/xml/acceleratorconfigurationreader.cxx
@@ -176,7 +176,7 @@ void SAL_CALL AcceleratorConfigurationReader::startElement(const ::rtl::OUString
// validate command and key event.
if (
- (!sCommand.getLength()) ||
+ (sCommand.isEmpty()) ||
(aEvent.KeyCode == 0 )
)
{
diff --git a/framework/source/xml/imagesdocumenthandler.cxx b/framework/source/xml/imagesdocumenthandler.cxx
index 21f1e4f..deade3d 100644
--- a/framework/source/xml/imagesdocumenthandler.cxx
+++ b/framework/source/xml/imagesdocumenthandler.cxx
@@ -252,7 +252,7 @@ throw( SAXException, RuntimeException )
{
::rtl::OUString aColor = xAttribs->getValueByIndex( n );
- if ( aColor.getLength() > 0 )
+ if ( !aColor.isEmpty() )
{
if ( aColor.getStr()[0] == '#' )
{
--
1.7.5.4
--------------070101020607010304030602--
More information about the LibreOffice
mailing list