[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