Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Please find attached a partial fix for Easy Hack FDO43460

Part XXI
Module
oox
- -- 
Olivier Hallot
Founder, Board of Directors Member - The Document Foundation
LibreOffice translation leader for Brazilian Portuguese
+55-21-8822-8812
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJPAHP5AAoJEJp3R7nH3vLxOTMIAK3/trwnoFzVEhc4eZHhwAmO
NEbHhnzv6bWAVk0RF/ajoUkmdLWu3R+ztEiyEo2OPtJt0MJNVTENS9UsUF5xskWs
V5kSOQ/SobN0IVehrvDphRIpmdGTH1BL47vmi7E9Fp7Kwt7/mftl0xqTXEjiRspX
Rm6HNivu+clAAI/V3Mf9cBkOK/TNJC1403Vz672oY0T+EvyKN43T/GDHcmEgucDR
IvcPvpJ5WhOgReCCz3a+mOQd92njn788ZFSQnzeKQP40pps+dz+7c8pgbeWgIvaf
mrnxKXpwxU0B7UfnD0HVMxLDrlb2lDX3FrOoeFwvVSPOExlAQvjfZvvjYJHkJTM=
=k5DV
-----END PGP SIGNATURE-----
From f4333d8724fd0a47e7e8b29de03bd53d54294e7d Mon Sep 17 00:00:00 2001
From: Olivier Hallot <olivier.hallot@alta.org.br>
Date: Sun, 1 Jan 2012 12:51:35 -0200
Subject: [PATCH] Fix for fdo43460 Part XXV getLength() to isEmpty()

Part XXV
Modules
oox
---
 mysqlc/source/mysqlc_connection.cxx                |    8 ++--
 oox/inc/oox/dump/dumperbase.hxx                    |    2 +-
 oox/source/core/contexthandler2.cxx                |    2 +-
 oox/source/core/filterbase.cxx                     |   16 +++---
 oox/source/core/filterdetect.cxx                   |    2 +-
 oox/source/core/relations.cxx                      |    6 +-
 oox/source/core/relationshandler.cxx               |    2 +-
 oox/source/core/xmlfilterbase.cxx                  |   10 ++--
 oox/source/docprop/docprophandler.cxx              |    8 ++--
 oox/source/drawingml/chart/chartconverter.cxx      |    2 +-
 oox/source/drawingml/chart/chartspaceconverter.cxx |    6 +-
 oox/source/drawingml/chart/objectformatter.cxx     |    2 +-
 oox/source/drawingml/chart/seriesconverter.cxx     |    4 +-
 oox/source/drawingml/chart/titleconverter.cxx      |    4 +-
 oox/source/drawingml/customshapegeometry.cxx       |    6 +-
 oox/source/drawingml/diagram/diagram.cxx           |   32 ++++++------
 .../drawingml/diagram/diagramdefinitioncontext.cxx |    2 +-
 .../drawingml/diagram/diagramlayoutatoms.cxx       |    2 +-
 oox/source/drawingml/fillproperties.cxx            |    4 +-
 .../drawingml/fillpropertiesgroupcontext.cxx       |    2 +-
 oox/source/drawingml/graphicshapecontext.cxx       |    4 +-
 oox/source/drawingml/hyperlinkcontext.cxx          |   12 ++--
 oox/source/drawingml/lineproperties.cxx            |    2 +-
 oox/source/drawingml/shape.cxx                     |   14 +++---
 oox/source/drawingml/shapepropertymap.cxx          |   10 ++--
 oox/source/drawingml/table/tableproperties.cxx     |    2 +-
 oox/source/drawingml/textbodypropertiescontext.cxx |   16 +++---
 oox/source/drawingml/textcharacterproperties.cxx   |    2 +-
 oox/source/drawingml/textfont.cxx                  |    4 +-
 .../drawingml/textparagraphpropertiescontext.cxx   |    6 +-
 oox/source/drawingml/textrun.cxx                   |    2 +-
 oox/source/drawingml/texttabstoplistcontext.cxx    |    2 +-
 oox/source/dump/biffdumper.cxx                     |    4 +-
 oox/source/dump/dumperbase.cxx                     |   54 ++++++++++----------
 oox/source/dump/oledumper.cxx                      |    8 ++--
 oox/source/export/chartexport.cxx                  |    8 ++--
 oox/source/export/drawingml.cxx                    |   14 +++---
 oox/source/helper/attributelist.cxx                |   14 +++---
 oox/source/helper/graphichelper.cxx                |    4 +-
 oox/source/helper/modelobjecthelper.cxx            |    4 +-
 oox/source/helper/storagebase.cxx                  |   18 +++---
 oox/source/helper/textinputstream.cxx              |    2 +-
 oox/source/mathml/importutils.cxx                  |    2 +-
 oox/source/ole/axcontrol.cxx                       |   12 ++--
 oox/source/ole/axcontrolfragment.cxx               |    6 +-
 oox/source/ole/olehelper.cxx                       |    2 +-
 oox/source/ole/oleobjecthelper.cxx                 |    4 +-
 oox/source/ole/olestorage.cxx                      |    4 +-
 oox/source/ole/vbacontrol.cxx                      |   14 +++---
 oox/source/ole/vbahelper.cxx                       |    8 ++--
 oox/source/ole/vbamodule.cxx                       |    8 ++--
 oox/source/ole/vbaproject.cxx                      |   14 +++---
 oox/source/ppt/pptgraphicshapecontext.cxx          |    2 +-
 oox/source/ppt/pptimport.cxx                       |    2 +-
 oox/source/ppt/pptshape.cxx                        |    8 ++--
 oox/source/ppt/presentationfragmenthandler.cxx     |   10 ++--
 oox/source/ppt/slidefragmenthandler.cxx            |    4 +-
 oox/source/ppt/soundactioncontext.cxx              |    4 +-
 oox/source/ppt/timenode.cxx                        |    2 +-
 oox/source/ppt/timenodelistcontext.cxx             |   10 ++--
 oox/source/ppt/timetargetelementcontext.cxx        |    2 +-
 oox/source/vml/vmldrawing.cxx                      |   10 ++--
 oox/source/vml/vmlformatting.cxx                   |   10 ++--
 oox/source/vml/vmlinputstream.cxx                  |    2 +-
 oox/source/vml/vmlshape.cxx                        |   14 +++---
 oox/source/vml/vmlshapecontainer.cxx               |    4 +-
 oox/source/vml/vmlshapecontext.cxx                 |    6 +-
 oox/source/xls/addressconverter.cxx                |    2 +-
 oox/source/xls/autofilterbuffer.cxx                |   16 +++---
 oox/source/xls/condformatbuffer.cxx                |   12 ++--
 oox/source/xls/connectionsbuffer.cxx               |    6 +-
 oox/source/xls/defnamesbuffer.cxx                  |    4 +-
 oox/source/xls/drawingfragment.cxx                 |   16 +++---
 oox/source/xls/drawingmanager.cxx                  |    4 +-
 oox/source/xls/excelchartconverter.cxx             |    4 +-
 oox/source/xls/excelfilter.cxx                     |    2 +-
 oox/source/xls/excelhandlers.cxx                   |    2 +-
 oox/source/xls/excelvbaproject.cxx                 |    2 +-
 oox/source/xls/externallinkbuffer.cxx              |   32 ++++++------
 oox/source/xls/formulabase.cxx                     |   32 ++++++------
 oox/source/xls/formulaparser.cxx                   |   20 ++++----
 oox/source/xls/numberformatsbuffer.cxx             |    4 +-
 oox/source/xls/pagesettings.cxx                    |   18 +++---
 oox/source/xls/pivotcachebuffer.cxx                |   30 ++++++------
 oox/source/xls/pivotcachefragment.cxx              |    2 +-
 oox/source/xls/pivottablebuffer.cxx                |   14 +++---
 oox/source/xls/querytablebuffer.cxx                |    6 +-
 oox/source/xls/richstring.cxx                      |   12 ++--
 oox/source/xls/scenariobuffer.cxx                  |    2 +-
 oox/source/xls/sheetdatabuffer.cxx                 |    2 +-
 oox/source/xls/sheetdatacontext.cxx                |    4 +-
 oox/source/xls/stylesbuffer.cxx                    |   20 ++++----
 oox/source/xls/tablebuffer.cxx                     |    4 +-
 oox/source/xls/workbookfragment.cxx                |   16 +++---
 oox/source/xls/workbookhelper.cxx                  |    6 +-
 oox/source/xls/worksheetbuffer.cxx                 |    2 +-
 oox/source/xls/worksheetfragment.cxx               |    4 +-
 oox/source/xls/worksheethelper.cxx                 |   14 +++---
 98 files changed, 403 insertions(+), 403 deletions(-)

diff --git a/mysqlc/source/mysqlc_connection.cxx b/mysqlc/source/mysqlc_connection.cxx
index 7ef4678..9d9f1cc 100644
--- a/mysqlc/source/mysqlc_connection.cxx
+++ b/mysqlc/source/mysqlc_connection.cxx
@@ -139,17 +139,17 @@ void OConnection::construct(const OUString& url, const Sequence< 
PropertyValue >
     }
 
     token = url.getToken(0, '/', nIndex);
-    if (token.getLength()) {
+    if (!token.isEmpty()) {
         sal_Int32 nIndex1 = 0;
         OUString hostandport = token.getToken(0,':', nIndex1);
-        if (hostandport.getLength()) {
+        if (!hostandport.isEmpty()) {
             aHostName = hostandport;
             hostandport = token.getToken(0, ':', nIndex1);
-            if (hostandport.getLength() && nIndex1) {
+            if (!hostandport.isEmpty() && nIndex1) {
                 nPort = hostandport.toInt32();
             }
             token = url.getToken(0, '/', nIndex);
-            if (token.getLength() && nIndex) {
+            if (!token.isEmpty() && nIndex) {
                 aDbName = token;
             }
         }
diff --git a/oox/inc/oox/dump/dumperbase.hxx b/oox/inc/oox/dump/dumperbase.hxx
index 18207cf..8debfa5 100644
--- a/oox/inc/oox/dump/dumperbase.hxx
+++ b/oox/inc/oox/dump/dumperbase.hxx
@@ -955,7 +955,7 @@ template< typename ListType >
 ::boost::shared_ptr< ListType > SharedConfigData::createNameList( const ::rtl::OUString& rListName 
)
 {
     ::boost::shared_ptr< ListType > xList;
-    if( rListName.getLength() > 0 )
+    if( !rListName.isEmpty() )
     {
         xList.reset( new ListType( *this ) );
         setNameList( rListName, xList );
diff --git a/oox/source/core/contexthandler2.cxx b/oox/source/core/contexthandler2.cxx
index fc79afc..20c62f6 100644
--- a/oox/source/core/contexthandler2.cxx
+++ b/oox/source/core/contexthandler2.cxx
@@ -183,7 +183,7 @@ void ContextHandler2Helper::processCollectedChars()
         OUString aChars = rInfo.maChars.makeStringAndClear();
         if( mbEnableTrimSpace && rInfo.mbTrimSpaces )
             aChars = aChars.trim();
-        if( aChars.getLength() > 0 )
+        if( !aChars.isEmpty() )
             onCharacters( aChars );
     }
 }
diff --git a/oox/source/core/filterbase.cxx b/oox/source/core/filterbase.cxx
index 5e37d06e..c8acd4d 100644
--- a/oox/source/core/filterbase.cxx
+++ b/oox/source/core/filterbase.cxx
@@ -102,8 +102,8 @@ DocumentOpenedGuard::DocumentOpenedGuard( const OUString& rUrl )
 {
     UrlPool& rUrlPool = StaticUrlPool::get();
     ::osl::MutexGuard aGuard( rUrlPool.maMutex );
-    mbValid = (rUrl.getLength() == 0) || (rUrlPool.maUrls.count( rUrl ) == 0);
-    if( mbValid && (rUrl.getLength() > 0) )
+    mbValid = rUrl.isEmpty() || (rUrlPool.maUrls.count( rUrl ) == 0);
+    if( mbValid && !rUrl.isEmpty() )
     {
         rUrlPool.maUrls.insert( rUrl );
         maUrl = rUrl;
@@ -114,7 +114,7 @@ DocumentOpenedGuard::~DocumentOpenedGuard()
 {
     UrlPool& rUrlPool = StaticUrlPool::get();
     ::osl::MutexGuard aGuard( rUrlPool.maMutex );
-    if( maUrl.getLength() > 0 )
+    if( !maUrl.isEmpty() )
         rUrlPool.maUrls.erase( maUrl );
 }
 
@@ -334,7 +334,7 @@ OUString FilterBase::getAbsoluteUrl( const OUString& rUrl ) const
     /*  (1) convert all backslashes to slashes, and check that passed URL is
         not empty. */
     OUString aUrl = rUrl.replace( '\\', '/' );
-    if( aUrl.getLength() == 0 )
+    if( aUrl.isEmpty() )
         return aUrl;
 
     /*  (2) add 'file:///' to absolute Windows paths, e.g. convert
@@ -359,7 +359,7 @@ OUString FilterBase::getAbsoluteUrl( const OUString& rUrl ) const
     /*  (5) handle URLs relative to current drive, e.g. the URL '/path1/file1'
         relative to the base URL 'file:///C:/path2/file2' does not result in
         the expected 'file:///C:/path1/file1', but in 'file:///path1/file1'. */
-    if( (aUrl.getLength() >= 1) && (aUrl[ 0 ] == '/') &&
+    if( !aUrl.isEmpty() && (aUrl[ 0 ] == '/') &&
         mxImpl->maFileUrl.match( aFilePrefix ) &&
         lclIsDosDrive( mxImpl->maFileUrl, nFilePrefixLen ) )
     {
@@ -441,8 +441,8 @@ Sequence< NamedValue > FilterBase::requestEncryptionData( ::comphelper::IDocPass
 
 bool FilterBase::importBinaryData( StreamDataSequence& orDataSeq, const OUString& rStreamName )
 {
-    OSL_ENSURE( rStreamName.getLength() > 0, "FilterBase::importBinaryData - empty stream name" );
-    if( rStreamName.getLength() == 0 )
+    OSL_ENSURE( !rStreamName.isEmpty(), "FilterBase::importBinaryData - empty stream name" );
+    if( rStreamName.isEmpty() )
         return false;
 
     // try to open the stream (this may fail - do not assert)
@@ -517,7 +517,7 @@ sal_Bool SAL_CALL FilterBase::filter( const Sequence< PropertyValue >& rMediaDes
     sal_Bool bRet = sal_False;
     setMediaDescriptor( rMediaDescSeq );
     DocumentOpenedGuard aOpenedGuard( mxImpl->maFileUrl );
-    if( aOpenedGuard.isValid() || !mxImpl->maFileUrl.getLength() )
+    if( aOpenedGuard.isValid() || mxImpl->maFileUrl.isEmpty() )
     {
         mxImpl->initializeFilter();
         switch( mxImpl->meDirection )
diff --git a/oox/source/core/filterdetect.cxx b/oox/source/core/filterdetect.cxx
index c0bde4d..259b5c1 100644
--- a/oox/source/core/filterdetect.cxx
+++ b/oox/source/core/filterdetect.cxx
@@ -223,7 +223,7 @@ OUString FilterDetectDocHandler::getFilterNameFromContentType( const OUString& r
 void FilterDetectDocHandler::parseContentTypesDefault( const AttributeList& rAttribs )
 {
     // only if no overridden part name found
-    if( mrFilterName.getLength() == 0 )
+    if( mrFilterName.isEmpty() )
     {
         // check if target path ends with extension
         OUString aExtension = rAttribs.getString( XML_Extension, OUString() );
diff --git a/oox/source/core/relations.cxx b/oox/source/core/relations.cxx
index 3030ef4..f1e39d4 100644
--- a/oox/source/core/relations.cxx
+++ b/oox/source/core/relations.cxx
@@ -50,7 +50,7 @@ OUString lclRemoveFileName( const OUString& rPath )
 
 OUString lclAppendFileName( const OUString& rPath, const OUString& rFileName )
 {
-    return (rPath.getLength() == 0) ? rFileName :
+    return rPath.isEmpty() ? rFileName :
         OUStringBuffer( rPath ).append( sal_Unicode( '/' ) ).append( rFileName 
).makeStringAndClear();
 }
 
@@ -101,7 +101,7 @@ OUString Relations::getExternalTargetFromFirstType( const OUString& rType ) cons
 OUString Relations::getFragmentPathFromRelation( const Relation& rRelation ) const
 {
     // no target, no fragment path
-    if( rRelation.mbExternal || (rRelation.maTarget.getLength() == 0) )
+    if( rRelation.mbExternal || rRelation.maTarget.isEmpty() )
         return OUString();
 
     // absolute target: return it without leading slash (#i100978)
@@ -109,7 +109,7 @@ OUString Relations::getFragmentPathFromRelation( const Relation& rRelation ) con
         return rRelation.maTarget.copy( 1 );
 
     // empty fragment path: return target
-    if( maFragmentPath.getLength() == 0 )
+    if( maFragmentPath.isEmpty() )
         return rRelation.maTarget;
 
     // resolve relative target path according to base path
diff --git a/oox/source/core/relationshandler.cxx b/oox/source/core/relationshandler.cxx
index 021b73a..d64ec71 100644
--- a/oox/source/core/relationshandler.cxx
+++ b/oox/source/core/relationshandler.cxx
@@ -85,7 +85,7 @@ Reference< XFastContextHandler > RelationsFragment::createFastChildContext(
             aRelation.maId     = aAttribs.getString( XML_Id, OUString() );
             aRelation.maType   = aAttribs.getString( XML_Type, OUString() );
             aRelation.maTarget = aAttribs.getString( XML_Target, OUString() );
-            if( (aRelation.maId.getLength() > 0) && (aRelation.maType.getLength() > 0) && 
(aRelation.maTarget.getLength() > 0) )
+            if( !aRelation.maId.isEmpty() && !aRelation.maType.isEmpty() && 
!aRelation.maTarget.isEmpty() )
             {
                 sal_Int32 nTargetMode = aAttribs.getToken( XML_TargetMode, XML_Internal );
                 OSL_ENSURE( (nTargetMode == XML_Internal) || (nTargetMode == XML_External),
diff --git a/oox/source/core/xmlfilterbase.cxx b/oox/source/core/xmlfilterbase.cxx
index cf93e14..f5868bf 100644
--- a/oox/source/core/xmlfilterbase.cxx
+++ b/oox/source/core/xmlfilterbase.cxx
@@ -266,8 +266,8 @@ bool XmlFilterBase::importFragment( const ::rtl::Reference< FragmentHandler >& r
 
     // fragment handler must contain path to fragment stream
     OUString aFragmentPath = rxHandler->getFragmentPath();
-    OSL_ENSURE( aFragmentPath.getLength() > 0, "XmlFilterBase::importFragment - missing fragment 
path" );
-    if( aFragmentPath.getLength() == 0 )
+    OSL_ENSURE( !aFragmentPath.isEmpty(), "XmlFilterBase::importFragment - missing fragment path" 
);
+    if( aFragmentPath.isEmpty() )
         return false;
 
     // try to import binary streams (fragment extension must be '.bin')
@@ -343,8 +343,8 @@ Reference<XDocument> XmlFilterBase::importFragment( const ::rtl::OUString& aFrag
     Reference<XDocument> xRet;
 
     // path to fragment stream valid?
-    OSL_ENSURE( aFragmentPath.getLength() > 0, "XmlFilterBase::importFragment - empty fragment 
path" );
-    if( aFragmentPath.getLength() == 0 )
+    OSL_ENSURE( !aFragmentPath.isEmpty(), "XmlFilterBase::importFragment - empty fragment path" );
+    if( aFragmentPath.isEmpty() )
         return xRet;
 
     // try to open the fragment stream (this may fail - do not assert)
@@ -488,7 +488,7 @@ OUString XmlFilterBase::addRelation( const Reference< XOutputStream > xOutputStr
 static void
 writeElement( FSHelperPtr pDoc, sal_Int32 nXmlElement, const OUString& sValue )
 {
-    if( sValue.getLength() == 0 )
+    if( sValue.isEmpty() )
         return;
     pDoc->startElement( nXmlElement, FSEND );
     pDoc->write( sValue );
diff --git a/oox/source/docprop/docprophandler.cxx b/oox/source/docprop/docprophandler.cxx
index 242d83b..264ea9f 100644
--- a/oox/source/docprop/docprophandler.cxx
+++ b/oox/source/docprop/docprophandler.cxx
@@ -72,7 +72,7 @@ void OOXMLDocPropHandler::InitNew()
 // ------------------------------------------------
 void OOXMLDocPropHandler::AddCustomProperty( const uno::Any& aAny )
 {
-    if ( m_aCustomPropertyName.getLength() )
+    if ( !m_aCustomPropertyName.isEmpty() )
     {
         const uno::Reference< beans::XPropertyContainer > xUserProps =
             m_xDocProp->getUserDefinedProperties();
@@ -164,7 +164,7 @@ util::DateTime OOXMLDocPropHandler::GetDateTimeFromW3CDTF( const ::rtl::OUString
 // ------------------------------------------------
 uno::Sequence< ::rtl::OUString > OOXMLDocPropHandler::GetKeywordsSet( const ::rtl::OUString& 
aChars )
 {
-    if ( aChars.getLength() )
+    if ( !aChars.isEmpty() )
     {
         uno::Sequence< ::rtl::OUString > aResult( 20 );
         sal_Int32 nCounter = 0;
@@ -181,7 +181,7 @@ uno::Sequence< ::rtl::OUString > OOXMLDocPropHandler::GetKeywordsSet( const ::rt
                 case (sal_Unicode)'\t':
                     // this is a delimiter
                     // unfortunately I did not find any specification for the possible delimiters
-                    if ( aResult[nCounter].getLength() )
+                    if ( !aResult[nCounter].isEmpty() )
                     {
                         if ( nCounter >= aResult.getLength() )
                             aResult.realloc( nCounter + 10 );
@@ -246,7 +246,7 @@ void OOXMLDocPropHandler::UpdateDocStatistic( const ::rtl::OUString& aChars )
             break;
     }
 
-    if ( aName.getLength() )
+    if ( !aName.isEmpty() )
     {
         sal_Bool bFound = sal_False;
         sal_Int32 nLen = aSet.getLength();
diff --git a/oox/source/drawingml/chart/chartconverter.cxx 
b/oox/source/drawingml/chart/chartconverter.cxx
index 2614943..12d4812 100644
--- a/oox/source/drawingml/chart/chartconverter.cxx
+++ b/oox/source/drawingml/chart/chartconverter.cxx
@@ -148,7 +148,7 @@ Reference< XDataSequence > ChartConverter::createDataSequence( const Reference<
             aRangeRep = lclGenerateApiArray( aMatrix );
         }
 
-        if( aRangeRep.getLength() > 0 ) try
+        if( !aRangeRep.isEmpty() ) try
         {
             // create the data sequence
             xDataSeq = rxDataProvider->createDataSequenceByRangeRepresentation( aRangeRep );
diff --git a/oox/source/drawingml/chart/chartspaceconverter.cxx 
b/oox/source/drawingml/chart/chartspaceconverter.cxx
index 822ac52..bee3d7c 100644
--- a/oox/source/drawingml/chart/chartspaceconverter.cxx
+++ b/oox/source/drawingml/chart/chartspaceconverter.cxx
@@ -129,9 +129,9 @@ void ChartSpaceConverter::convertFromModel( const Reference< XShapes >& rxExtern
         /*  If the title model is missing, but the chart shows exactly one
             series, the series title is shown as chart title. */
         OUString aAutoTitle = aPlotAreaConv.getAutomaticTitle();
-        if( mrModel.mxTitle.is() || (aAutoTitle.getLength() > 0) )
+        if( mrModel.mxTitle.is() || !aAutoTitle.isEmpty() )
         {
-            if( aAutoTitle.getLength() == 0 )
+            if( aAutoTitle.isEmpty() )
                 aAutoTitle = CREATE_OUSTRING( "Chart Title" );
             Reference< XTitled > xTitled( getChartDocument(), UNO_QUERY_THROW );
             TitleConverter aTitleConv( *this, mrModel.mxTitle.getOrCreate() );
@@ -184,7 +184,7 @@ void ChartSpaceConverter::convertFromModel( const Reference< XShapes >& rxExtern
     }
 
     // embedded drawing shapes
-    if( mrModel.maDrawingPath.getLength() > 0 ) try
+    if( !mrModel.maDrawingPath.isEmpty() ) try
     {
         /*  Get the internal draw page of the chart document, if no external
             drawing page has been passed. */
diff --git a/oox/source/drawingml/chart/objectformatter.cxx 
b/oox/source/drawingml/chart/objectformatter.cxx
index 1b07ae3..7aca5b8 100644
--- a/oox/source/drawingml/chart/objectformatter.cxx
+++ b/oox/source/drawingml/chart/objectformatter.cxx
@@ -1141,7 +1141,7 @@ void ObjectFormatter::convertNumberFormat( PropertySet& rPropSet, const 
NumberFo
     if( mxData->mxNumFmts.is() )
     {
         sal_Int32 nPropId = bPercentFormat ? PROP_PercentageNumberFormat : PROP_NumberFormat;
-        if( rNumberFormat.mbSourceLinked || (rNumberFormat.maFormatCode.getLength() == 0) )
+        if( rNumberFormat.mbSourceLinked || rNumberFormat.maFormatCode.isEmpty() )
         {
             rPropSet.setAnyProperty( nPropId, Any() );
         }
diff --git a/oox/source/drawingml/chart/seriesconverter.cxx 
b/oox/source/drawingml/chart/seriesconverter.cxx
index 3ed07e6..0522376 100644
--- a/oox/source/drawingml/chart/seriesconverter.cxx
+++ b/oox/source/drawingml/chart/seriesconverter.cxx
@@ -374,7 +374,7 @@ Reference< XLabeledDataSequence > ErrorBarConverter::createLabeledDataSequence(
             }
         break;
     }
-    OSL_ENSURE( aRole.getLength() > 0, "ErrorBarConverter::createLabeledDataSequence - invalid 
error bar direction" );
+    OSL_ENSURE( !aRole.isEmpty(), "ErrorBarConverter::createLabeledDataSequence - invalid error 
bar direction" );
     return lclCreateLabeledDataSequence( *this, mrModel.maSources.get( eSourceType ).get(), aRole 
);
 }
 
@@ -422,7 +422,7 @@ void TrendlineConverter::convertFromModel( const Reference< XDataSeries >& rxDat
             case XML_power:     aServiceName = CREATE_OUSTRING( 
"com.sun.star.chart2.PotentialRegressionCurve" );   break;
             default:            OSL_FAIL( "TrendlineConverter::convertFromModel - unknown 
trendline type" );
         }
-        if( aServiceName.getLength() > 0 )
+        if( !aServiceName.isEmpty() )
         {
             Reference< XRegressionCurve > xRegCurve( createInstance( aServiceName ), 
UNO_QUERY_THROW );
             PropertySet aPropSet( xRegCurve );
diff --git a/oox/source/drawingml/chart/titleconverter.cxx 
b/oox/source/drawingml/chart/titleconverter.cxx
index 5699a0d..9ed69fe 100644
--- a/oox/source/drawingml/chart/titleconverter.cxx
+++ b/oox/source/drawingml/chart/titleconverter.cxx
@@ -109,11 +109,11 @@ Sequence< Reference< XFormattedString > > TextConverter::createStringSequence(
         if( mrModel.mxDataSeq.is() && !mrModel.mxDataSeq->maData.empty() )
             mrModel.mxDataSeq->maData.begin()->second >>= aString;
         // no linked string -> fall back to default string
-        if( aString.getLength() == 0 )
+        if( aString.isEmpty() )
             aString = rDefaultText;
 
         // create formatted string object
-        if( aString.getLength() > 0 )
+        if( !aString.isEmpty() )
         {
             Reference< XFormattedString > xFmtStr = appendFormattedString( aStringVec, aString, 
false );
             PropertySet aPropSet( xFmtStr );
diff --git a/oox/source/drawingml/customshapegeometry.cxx 
b/oox/source/drawingml/customshapegeometry.cxx
index cd75285..b5c7290 100644
--- a/oox/source/drawingml/customshapegeometry.cxx
+++ b/oox/source/drawingml/customshapegeometry.cxx
@@ -236,7 +236,7 @@ rtl::OUString GetFormulaParameter( const EnhancedCustomShapeParameter& rParamete
 static EnhancedCustomShapeParameter GetAdjCoordinate( CustomShapeProperties& 
rCustomShapeProperties, const::rtl::OUString& rValue, sal_Bool bNoSymbols = sal_True )
 {
     com::sun::star::drawing::EnhancedCustomShapeParameter aRet;
-    if ( rValue.getLength() )
+    if ( !rValue.isEmpty() )
     {
         sal_Bool    bConstant = sal_True;
         sal_Int32   nConstant = -1;
@@ -412,7 +412,7 @@ static EnhancedCustomShapeParameter GetAdjCoordinate( CustomShapeProperties& rCu
             sal_Unicode n = rValue[ 0 ];
             if ( ( n == '+' ) || ( n == '-' ) )
             {
-                if ( rValue.getLength() > 0 )
+                if ( !rValue.isEmpty() )
                     n = rValue[ 1 ];
             }
             if ( ( n >= '0' ) && ( n <= '9' ) )
@@ -483,7 +483,7 @@ static rtl::OUString convertToOOEquation( CustomShapeProperties& rCustomShapePro
     do
     {
         rtl::OUString aToken( rSource.getToken( 0, ' ', nIndex ) );
-        if ( aToken.getLength() )
+        if ( !aToken.isEmpty() )
             aTokens.push_back( aToken );
     }
     while ( nIndex >= 0 );
diff --git a/oox/source/drawingml/diagram/diagram.cxx b/oox/source/drawingml/diagram/diagram.cxx
index ad53443..91a475f 100644
--- a/oox/source/drawingml/diagram/diagram.cxx
+++ b/oox/source/drawingml/diagram/diagram.cxx
@@ -138,10 +138,10 @@ static sal_Int32 calcDepth( const rtl::OUString& rNodeName,
     const dgm::Connections::const_iterator aEndCxn( rCnx.end() );
     while( aCurrCxn != aEndCxn )
     {
-        if( aCurrCxn->msParTransId.getLength() &&
-            aCurrCxn->msSibTransId.getLength() &&
-            aCurrCxn->msSourceId.getLength() &&
-            aCurrCxn->msDestId.getLength() &&
+        if( !aCurrCxn->msParTransId.isEmpty() &&
+            !aCurrCxn->msSibTransId.isEmpty() &&
+            !aCurrCxn->msSourceId.isEmpty() &&
+            !aCurrCxn->msDestId.isEmpty() &&
             aCurrCxn->mnType != XML_presOf &&
             aCurrCxn->mnType != XML_presParOf &&
             rNodeName == aCurrCxn->msDestId )
@@ -176,7 +176,7 @@ void Diagram::build(  )
                << normalizeDotName(aCurrPoint->msModelId).getStr()
                << "[";
 
-        if( aCurrPoint->msPresentationLayoutName.getLength() )
+        if( !aCurrPoint->msPresentationLayoutName.isEmpty() )
             output << "label=\""
                    << rtl::OUStringToOString(
                        aCurrPoint->msPresentationLayoutName,
@@ -230,7 +230,7 @@ void Diagram::build(  )
 
         OSL_ENSURE(bInserted1,"Diagram::build(): non-unique point model id");
 
-        if( aCurrPoint->msPresentationLayoutName.getLength() )
+        if( !aCurrPoint->msPresentationLayoutName.isEmpty() )
         {
             DiagramData::PointsNameMap::value_type::second_type& rVec=
                 getData()->getPointsPresNameMap()[aCurrPoint->msPresentationLayoutName];
@@ -244,11 +244,11 @@ void Diagram::build(  )
     while( aCurrCxn != aEndCxn )
     {
 #if OSL_DEBUG_LEVEL > 1
-        if( aCurrCxn->msParTransId.getLength() ||
-            aCurrCxn->msSibTransId.getLength() )
+        if( !aCurrCxn->msParTransId.isEmpty() ||
+            !aCurrCxn->msSibTransId.isEmpty() )
         {
-            if( aCurrCxn->msSourceId.getLength() ||
-                aCurrCxn->msDestId.getLength() )
+            if( !aCurrCxn->msSourceId.isEmpty() ||
+                !aCurrCxn->msDestId.isEmpty() )
             {
                 output << "\t"
                        << normalizeDotName(aCurrCxn->msSourceId).getStr()
@@ -279,8 +279,8 @@ void Diagram::build(  )
                        << "\"];" << std::endl;
             }
         }
-        else if( aCurrCxn->msSourceId.getLength() ||
-                 aCurrCxn->msDestId.getLength() )
+        else if( !aCurrCxn->msSourceId.isEmpty() ||
+                 !aCurrCxn->msDestId.isEmpty() )
             output << "\t"
                    << normalizeDotName(aCurrCxn->msSourceId).getStr()
                    << " -> "
@@ -391,7 +391,7 @@ void loadDiagram( ShapePtr& pShape,
     pDiagram->setLayout( pLayout );
 
     // data
-    if( rDataModelPath.getLength() > 0 )
+    if( !rDataModelPath.isEmpty() )
     {
         rtl::Reference< core::FragmentHandler > xRef(
             new DiagramDataFragmentHandler( rFilter, rDataModelPath, pData ));
@@ -408,7 +408,7 @@ void loadDiagram( ShapePtr& pShape,
     }
 
     // layout
-    if( rLayoutPath.getLength() > 0 )
+    if( !rLayoutPath.isEmpty() )
     {
         rtl::Reference< core::FragmentHandler > xRef(
             new DiagramLayoutFragmentHandler( rFilter, rLayoutPath, pLayout ));
@@ -420,7 +420,7 @@ void loadDiagram( ShapePtr& pShape,
     }
 
     // style
-    if( rQStylePath.getLength() > 0 )
+    if( !rQStylePath.isEmpty() )
     {
         rtl::Reference< core::FragmentHandler > xRef(
             new DiagramQStylesFragmentHandler( rFilter, rQStylePath, pDiagram->getStyles() ));
@@ -432,7 +432,7 @@ void loadDiagram( ShapePtr& pShape,
     }
 
     // colors
-    if( rColorStylePath.getLength() > 0 )
+    if( !rColorStylePath.isEmpty() )
     {
         rtl::Reference< core::FragmentHandler > xRef(
             new ColorFragmentHandler( rFilter, rColorStylePath, pDiagram->getColors() ));
diff --git a/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx 
b/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx
index 22aa299..7333857 100644
--- a/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx
+++ b/oox/source/drawingml/diagram/diagramdefinitioncontext.cxx
@@ -48,7 +48,7 @@ DiagramDefinitionContext::DiagramDefinitionContext( ContextHandler& rParent,
     OSL_TRACE( "OOX: DiagramDefinitionContext::DiagramDefinitionContext()" );
     mpLayout->setDefStyle( xAttributes->getOptionalValue( XML_defStyle ) );
     OUString sValue = xAttributes->getOptionalValue( XML_minVer );
-    if( sValue.getLength() == 0 )
+    if( sValue.isEmpty() )
     {
         sValue = CREATE_OUSTRING( "http://schemas.openxmlformats.org/drawingml/2006/diagram"; );
     }
diff --git a/oox/source/drawingml/diagram/diagramlayoutatoms.cxx 
b/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
index 16b9420..5bad9f5 100644
--- a/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
+++ b/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
@@ -468,7 +468,7 @@ bool LayoutNode::setupShape( const ShapePtr& rShape, const Diagram& rDgm, sal_uI
         // TODO(Q1): apply styling & coloring - taking
         // layout node's styleLbl for both style & color
         // now, but docs are a bit unclear on this
-        if( msStyleLabel.getLength() )
+        if( !msStyleLabel.isEmpty() )
         {
             OSL_TRACE("setting style with label %s",
                       OUSTRING_TO_CSTR( msStyleLabel ) );
diff --git a/oox/source/drawingml/fillproperties.cxx b/oox/source/drawingml/fillproperties.cxx
index bf93612..ad6cf4b 100644
--- a/oox/source/drawingml/fillproperties.cxx
+++ b/oox/source/drawingml/fillproperties.cxx
@@ -310,7 +310,7 @@ void FillProperties::pushToPropMap( ShapePropertyMap& rPropMap,
 
                     OUString aGraphicUrl = rGraphicHelper.createGraphicObject( 
maBlipProps.mxGraphic );
                     // push bitmap or named bitmap to property map
-                    if( (aGraphicUrl.getLength() > 0) && rPropMap.setProperty( 
SHAPEPROP_FillBitmapUrl, aGraphicUrl ) )
+                    if( !aGraphicUrl.isEmpty() && rPropMap.setProperty( SHAPEPROP_FillBitmapUrl, 
aGraphicUrl ) )
                         eFillStyle = FillStyle_BITMAP;
 
                     // set other bitmap properties, if bitmap has been inserted into the map
@@ -404,7 +404,7 @@ void GraphicProperties::pushToPropMap( PropertyMap& rPropMap, const GraphicHelpe
         }
 
         OUString aGraphicUrl = rGraphicHelper.createGraphicObject( xGraphic );
-        if( aGraphicUrl.getLength() > 0 )
+        if( !aGraphicUrl.isEmpty() )
             rPropMap[ PROP_GraphicURL ] <<= aGraphicUrl;
 
         // cropping
diff --git a/oox/source/drawingml/fillpropertiesgroupcontext.cxx 
b/oox/source/drawingml/fillpropertiesgroupcontext.cxx
index 82a2f59..ef5cbc3 100644
--- a/oox/source/drawingml/fillpropertiesgroupcontext.cxx
+++ b/oox/source/drawingml/fillpropertiesgroupcontext.cxx
@@ -170,7 +170,7 @@ BlipContext::BlipContext( ContextHandler& rParent,
     {
         // internal picture URL
         OUString aFragmentPath = getFragmentPathFromRelId( aAttribs.getString( R_TOKEN( embed ), 
OUString() ) );
-        if( aFragmentPath.getLength() > 0 )
+        if( !aFragmentPath.isEmpty() )
             mrBlipProps.mxGraphic = getFilter().getGraphicHelper().importEmbeddedGraphic( 
aFragmentPath );
     }
     else if( aAttribs.hasAttribute( R_TOKEN( link ) ) )
diff --git a/oox/source/drawingml/graphicshapecontext.cxx 
b/oox/source/drawingml/graphicshapecontext.cxx
index 5f7385a..016fbf3 100644
--- a/oox/source/drawingml/graphicshapecontext.cxx
+++ b/oox/source/drawingml/graphicshapecontext.cxx
@@ -155,7 +155,7 @@ OleObjectGraphicDataContext::~OleObjectGraphicDataContext()
 {
     /*  Register the OLE shape at the VML drawing, this prevents that the
         related VML shape converts the OLE object by itself. */
-    if( mrOleObjectInfo.maShapeId.getLength() > 0 )
+    if( !mrOleObjectInfo.maShapeId.isEmpty() )
         if( ::oox::vml::Drawing* pVmlDrawing = getFilter().getVmlDrawing() )
             pVmlDrawing->registerOleObject( mrOleObjectInfo );
 }
@@ -182,7 +182,7 @@ Reference< XFastContextHandler > OleObjectGraphicDataContext::createFastChildCon
                 else
                 {
                     OUString aFragmentPath = getFragmentPathFromRelation( *pRelation );
-                    if( aFragmentPath.getLength() > 0 )
+                    if( !aFragmentPath.isEmpty() )
                         getFilter().importBinaryData( mrOleObjectInfo.maEmbeddedData, 
aFragmentPath );
                 }
             }
diff --git a/oox/source/drawingml/hyperlinkcontext.cxx b/oox/source/drawingml/hyperlinkcontext.cxx
index 1975561..06958aa 100644
--- a/oox/source/drawingml/hyperlinkcontext.cxx
+++ b/oox/source/drawingml/hyperlinkcontext.cxx
@@ -50,24 +50,24 @@ HyperLinkContext::HyperLinkContext( ContextHandler& rParent,
 {
     OUString sURL, sHref;
     OUString aRelId = xAttributes->getOptionalValue( R_TOKEN( id ) );
-    if ( aRelId.getLength() )
+    if ( !aRelId.isEmpty() )
     {
         OSL_TRACE("OOX: URI rId %s", ::rtl::OUStringToOString (aRelId, 
RTL_TEXTENCODING_UTF8).pData->buffer);
         sHref = getRelations().getExternalTargetFromRelId( aRelId );
-        if( sHref.getLength() > 0 )
+        if( !sHref.isEmpty() )
         {
             OSL_TRACE("OOX: URI href %s", ::rtl::OUStringToOString (sHref, 
RTL_TEXTENCODING_UTF8).pData->buffer);
             sURL = getFilter().getAbsoluteUrl( sHref );
         }
     }
     OUString sTooltip = xAttributes->getOptionalValue( R_TOKEN( tooltip ) );
-    if ( sTooltip.getLength() )
+    if ( !sTooltip.isEmpty() )
         maProperties[ PROP_Representation ] <<= sTooltip;
     OUString sFrame = xAttributes->getOptionalValue( R_TOKEN( tgtFrame ) );
-    if( sFrame.getLength() )
+    if( !sFrame.isEmpty() )
         maProperties[ PROP_TargetFrame ] <<= sFrame;
     OUString aAction = xAttributes->getOptionalValue( XML_action );
-    if ( aAction.getLength() )
+    if ( !aAction.isEmpty() )
     {
         // reserved values of the unrestricted string aAction:
         // ppaction://customshow?id=SHOW_ID             // custom presentation
@@ -140,7 +140,7 @@ HyperLinkContext::HyperLinkContext( ContextHandler& rParent,
             }
         }
     }
-    if ( sURL.getLength() )
+    if ( !sURL.isEmpty() )
         maProperties[ PROP_URL ] <<= sURL;
 
     // TODO unhandled
diff --git a/oox/source/drawingml/lineproperties.cxx b/oox/source/drawingml/lineproperties.cxx
index 4bfdcad..9676795 100644
--- a/oox/source/drawingml/lineproperties.cxx
+++ b/oox/source/drawingml/lineproperties.cxx
@@ -327,7 +327,7 @@ void lclPushMarkerProperties( ShapePropertyMap& rPropMap,
     }
 
     // push the properties (filled aNamedMarker.Name indicates valid marker)
-    if( aNamedMarker.Name.getLength() > 0 )
+    if( !aNamedMarker.Name.isEmpty() )
     {
         if( bLineEnd )
         {
diff --git a/oox/source/drawingml/shape.cxx b/oox/source/drawingml/shape.cxx
index f551b85..54d9527 100644
--- a/oox/source/drawingml/shape.cxx
+++ b/oox/source/drawingml/shape.cxx
@@ -210,12 +210,12 @@ void Shape::addShape(
     try
     {
         rtl::OUString sServiceName( msServiceName );
-        if( sServiceName.getLength() )
+        if( !sServiceName.isEmpty() )
         {
             basegfx::B2DHomMatrix aMatrix( aTransformation );
             Reference< XShape > xShape( createAndInsert( rFilterBase, sServiceName, pTheme, 
rxShapes, pShapeRect, sal_False, aMatrix ) );
 
-            if( pShapeMap && msId.getLength() )
+            if( pShapeMap && !msId.isEmpty() )
             {
                 (*pShapeMap)[ msId ] = shared_from_this();
             }
@@ -429,7 +429,7 @@ Reference< XShape > Shape::createAndInsert(
     Reference< XPropertySet > xSet( mxShape, UNO_QUERY );
     if( mxShape.is() && xSet.is() )
     {
-        if( msName.getLength() )
+        if( !msName.isEmpty() )
         {
             Reference< container::XNamed > xNamed( mxShape, UNO_QUERY );
             if( xNamed.is() )
@@ -618,13 +618,13 @@ OUString Shape::finalizeServiceName( XmlFilterBase& rFilter, const OUString& 
rSe
 
             // get the path to the representation graphic
             OUString aGraphicPath;
-            if( mxOleObjectInfo->maShapeId.getLength() > 0 )
+            if( !mxOleObjectInfo->maShapeId.isEmpty() )
                 if( ::oox::vml::Drawing* pVmlDrawing = rFilter.getVmlDrawing() )
                     if( const ::oox::vml::ShapeBase* pVmlShape = 
pVmlDrawing->getShapes().getShapeById( mxOleObjectInfo->maShapeId, true ) )
                         aGraphicPath = pVmlShape->getGraphicPath();
 
             // import and store the graphic
-            if( aGraphicPath.getLength() > 0 )
+            if( !aGraphicPath.isEmpty() )
             {
                 Reference< graphic::XGraphic > xGraphic = 
rFilter.getGraphicHelper().importEmbeddedGraphic( aGraphicPath );
                 if( xGraphic.is() )
@@ -644,8 +644,8 @@ void Shape::finalizeXShape( XmlFilterBase& rFilter, const Reference< XShapes >&
     {
         case FRAMETYPE_CHART:
         {
-            OSL_ENSURE( mxChartShapeInfo->maFragmentPath.getLength() > 0, "Shape::finalizeXShape - 
missing chart fragment" );
-            if( mxShape.is() && (mxChartShapeInfo->maFragmentPath.getLength() > 0) ) try
+            OSL_ENSURE( !mxChartShapeInfo->maFragmentPath.isEmpty(), "Shape::finalizeXShape - 
missing chart fragment" );
+            if( mxShape.is() && !mxChartShapeInfo->maFragmentPath.isEmpty() ) try
             {
                 // set the chart2 OLE class ID at the OLE shape
                 PropertySet aShapeProp( mxShape );
diff --git a/oox/source/drawingml/shapepropertymap.cxx b/oox/source/drawingml/shapepropertymap.cxx
index fe62523..f05153c 100644
--- a/oox/source/drawingml/shapepropertymap.cxx
+++ b/oox/source/drawingml/shapepropertymap.cxx
@@ -130,7 +130,7 @@ bool ShapePropertyMap::setAnyProperty( ShapePropertyId ePropId, const Any& rValu
 bool ShapePropertyMap::setLineMarker( sal_Int32 nPropId, const Any& rValue )
 {
     NamedValue aNamedMarker;
-    if( (rValue >>= aNamedMarker) && (aNamedMarker.Name.getLength() > 0) )
+    if( (rValue >>= aNamedMarker) && !aNamedMarker.Name.isEmpty() )
     {
         // push line marker explicitly
         if( !maShapePropInfo.mbNamedLineMarker )
@@ -154,7 +154,7 @@ bool ShapePropertyMap::setLineDash( sal_Int32 nPropId, const Any& rValue )
     if( rValue.has< LineDash >() )
     {
         OUString aDashName = mrModelObjHelper.insertLineDash( rValue.get< LineDash >() );
-        return (aDashName.getLength() > 0) && setProperty( nPropId, aDashName );
+        return !aDashName.isEmpty() && setProperty( nPropId, aDashName );
     }
 
     return false;
@@ -170,7 +170,7 @@ bool ShapePropertyMap::setFillGradient( sal_Int32 nPropId, const Any& rValue )
     if( rValue.has< Gradient >() )
     {
         OUString aGradientName = mrModelObjHelper.insertFillGradient( rValue.get< Gradient >() );
-        return (aGradientName.getLength() > 0) && setProperty( nPropId, aGradientName );
+        return !aGradientName.isEmpty() && setProperty( nPropId, aGradientName );
     }
 
     return false;
@@ -182,7 +182,7 @@ bool ShapePropertyMap::setGradientTrans( sal_Int32 nPropId, const Any& rValue )
     if( rValue.has< Gradient >() )
     {
         OUString aGradientName = mrModelObjHelper.insertTransGrandient( rValue.get< Gradient >() );
-        return ( aGradientName.getLength() > 0 ) && setProperty( nPropId, aGradientName );
+        return !aGradientName.isEmpty()  && setProperty( nPropId, aGradientName );
     }
 
     return false;
@@ -198,7 +198,7 @@ bool ShapePropertyMap::setFillBitmapUrl( sal_Int32 nPropId, const Any& rValue )
     if( rValue.has< OUString >() )
     {
         OUString aBitmapUrlName = mrModelObjHelper.insertFillBitmapUrl( rValue.get< OUString >() );
-        return (aBitmapUrlName.getLength() > 0) && setProperty( nPropId, aBitmapUrlName );
+        return !aBitmapUrlName.isEmpty() && setProperty( nPropId, aBitmapUrlName );
     }
 
     return false;
diff --git a/oox/source/drawingml/table/tableproperties.cxx 
b/oox/source/drawingml/table/tableproperties.cxx
index a6434c6..7fbd4bd 100644
--- a/oox/source/drawingml/table/tableproperties.cxx
+++ b/oox/source/drawingml/table/tableproperties.cxx
@@ -119,7 +119,7 @@ const TableStyle& TableProperties::getUsedTableStyle( const ::oox::core::XmlFilt
     else if ( rBase.getTableStyles() )
     {
         const std::vector< TableStyle >& rTableStyles( rBase.getTableStyles()->getTableStyles() );
-        const rtl::OUString aStyleId( getStyleId().getLength() ? getStyleId() : 
rBase.getTableStyles()->getDefaultStyleId() );
+        const rtl::OUString aStyleId( getStyleId().isEmpty() ? 
rBase.getTableStyles()->getDefaultStyleId() : getStyleId() );
         std::vector< TableStyle >::const_iterator aIter( rTableStyles.begin() );
         while( aIter != rTableStyles.end() )
         {
diff --git a/oox/source/drawingml/textbodypropertiescontext.cxx 
b/oox/source/drawingml/textbodypropertiescontext.cxx
index 3c77a35..ea84306 100644
--- a/oox/source/drawingml/textbodypropertiescontext.cxx
+++ b/oox/source/drawingml/textbodypropertiescontext.cxx
@@ -66,23 +66,23 @@ TextBodyPropertiesContext::TextBodyPropertiesContext( ContextHandler& rParent,
     // ST_Coordinate
     OUString sValue;
     sValue = xAttributes->getOptionalValue( XML_lIns );
-    if( sValue.getLength() ) {
-    sal_Int32 nLeftInset = ( sValue.getLength() != 0 ? GetCoordinate(  sValue ) : 91440 / 360 );
+    if( !sValue.isEmpty() ) {
+    sal_Int32 nLeftInset = ( !sValue.isEmpty() ? GetCoordinate(  sValue ) : 91440 / 360 );
     mrTextBodyProp.maPropertyMap[ PROP_TextLeftDistance ]  <<= static_cast< sal_Int32 >( 
nLeftInset );
     }
     sValue = xAttributes->getOptionalValue( XML_tIns );
-    if( sValue.getLength() ) {
-    sal_Int32 nTopInset  = ( sValue.getLength() != 0 ? GetCoordinate(  sValue ) : 91440 / 360 );
+    if( !sValue.isEmpty() ) {
+    sal_Int32 nTopInset  = ( !sValue.isEmpty() ? GetCoordinate(  sValue ) : 91440 / 360 );
     mrTextBodyProp.maPropertyMap[ PROP_TextUpperDistance ] <<= static_cast< sal_Int32 >( nTopInset 
);
     }
     sValue = xAttributes->getOptionalValue( XML_rIns );
-    if( sValue.getLength() ) {
-    sal_Int32 nRightInset  = ( sValue.getLength() != 0 ? GetCoordinate(  sValue ) : 91440 / 360 );
+    if( !sValue.isEmpty() ) {
+    sal_Int32 nRightInset  = ( !sValue.isEmpty() ? GetCoordinate(  sValue ) : 91440 / 360 );
     mrTextBodyProp.maPropertyMap[ PROP_TextRightDistance ] <<= static_cast< sal_Int32 >( 
nRightInset );
     }
     sValue = xAttributes->getOptionalValue( XML_bIns );
-    if( sValue.getLength() ) {
-    sal_Int32 nBottonInset = ( sValue.getLength() != 0 ? GetCoordinate(  sValue ) : 45720 / 360 );
+    if( !sValue.isEmpty() ) {
+    sal_Int32 nBottonInset = ( !sValue.isEmpty() ? GetCoordinate(  sValue ) : 45720 / 360 );
     mrTextBodyProp.maPropertyMap[ PROP_TextLowerDistance ] <<= static_cast< sal_Int32 >( 
nBottonInset );
     }
 
diff --git a/oox/source/drawingml/textcharacterproperties.cxx 
b/oox/source/drawingml/textcharacterproperties.cxx
index fb4b069..adfa2fe 100644
--- a/oox/source/drawingml/textcharacterproperties.cxx
+++ b/oox/source/drawingml/textcharacterproperties.cxx
@@ -101,7 +101,7 @@ void TextCharacterProperties::assignUsed( const TextCharacterProperties& rSource
     if( maCharColor.isUsed() )
         rPropMap[ PROP_CharColor ] <<= maCharColor.getColor( rFilter.getGraphicHelper() );
 
-    if( moLang.has() && (moLang.get().getLength() > 0) )
+    if( moLang.has() && !moLang.get().isEmpty() )
     {
         lang::Locale aLocale;
         sal_Int32 nSepPos = moLang.get().indexOf( sal_Unicode( '-' ), 0 );
diff --git a/oox/source/drawingml/textfont.cxx b/oox/source/drawingml/textfont.cxx
index 5cd0fe2..9503db8 100644
--- a/oox/source/drawingml/textfont.cxx
+++ b/oox/source/drawingml/textfont.cxx
@@ -77,7 +77,7 @@ void TextFont::setAttributes( const AttributeList& rAttribs )
 
 void TextFont::assignIfUsed( const TextFont& rTextFont )
 {
-    if( rTextFont.maTypeface.getLength() > 0 )
+    if( !rTextFont.maTypeface.isEmpty() )
         *this = rTextFont;
 }
 
@@ -94,7 +94,7 @@ bool TextFont::implGetFontData( OUString& rFontName, sal_Int16& rnFontPitch, sal
     rFontName = maTypeface;
     rnFontPitch = lclGetFontPitch( extractValue< sal_Int16 >( mnPitch, 0, 4 ) );
     rnFontFamily = lclGetFontFamily( extractValue< sal_Int16 >( mnPitch, 4, 4 ) );
-    return rFontName.getLength() > 0;
+    return !rFontName.isEmpty();
 }
 
 // ============================================================================
diff --git a/oox/source/drawingml/textparagraphpropertiescontext.cxx 
b/oox/source/drawingml/textparagraphpropertiescontext.cxx
index 0bee276..2164bf2e 100644
--- a/oox/source/drawingml/textparagraphpropertiescontext.cxx
+++ b/oox/source/drawingml/textparagraphpropertiescontext.cxx
@@ -100,7 +100,7 @@ TextParagraphPropertiesContext::TextParagraphPropertiesContext( ContextHandler&
     if ( xAttribs->hasAttribute( XML_indent ) )
     {
         sValue = xAttribs->getOptionalValue( XML_indent );
-        mrTextParagraphProperties.getFirstLineIndentation() = boost::optional< sal_Int32 >( 
sValue.getLength() == 0 ? 0 : GetCoordinate( sValue ) );
+        mrTextParagraphProperties.getFirstLineIndentation() = boost::optional< sal_Int32 >( 
sValue.isEmpty() ? 0 : GetCoordinate( sValue ) );
     }
 
   // ST_TextIndentLevelType
@@ -123,14 +123,14 @@ TextParagraphPropertiesContext::TextParagraphPropertiesContext( 
ContextHandler&
     if ( xAttribs->hasAttribute( XML_marL ) )
     {
         sValue = xAttribs->getOptionalValue( XML_marL );
-        mrTextParagraphProperties.getParaLeftMargin() = boost::optional< sal_Int32 >( 
sValue.getLength() == 0 ? 0 : GetCoordinate( sValue ) );
+        mrTextParagraphProperties.getParaLeftMargin() = boost::optional< sal_Int32 >( 
sValue.isEmpty() ? 0 : GetCoordinate( sValue ) );
     }
 
     // ParaRightMargin
     if ( xAttribs->hasAttribute( XML_marR ) )
     {
         sValue = xAttribs->getOptionalValue( XML_marR );
-        sal_Int32 nMarR  = ( sValue.getLength() == 0 ? 0 : GetCoordinate( sValue ) );
+        sal_Int32 nMarR  = sValue.isEmpty() ? 0 : GetCoordinate( sValue ) ;
         rPropertyMap[ PROP_ParaRightMargin ] <<= nMarR;
     }
 
diff --git a/oox/source/drawingml/textrun.cxx b/oox/source/drawingml/textrun.cxx
index 3c5b99c..93c9ca7 100644
--- a/oox/source/drawingml/textrun.cxx
+++ b/oox/source/drawingml/textrun.cxx
@@ -87,7 +87,7 @@ sal_Int32 TextRun::insertAt(
 
                 if ( !aTextCharacterProps.maSymbolFont.getFontData( aSymbolFontName, 
nSymbolFontPitch, nSymbolFontFamily, rFilterBase ) )
                     xText->insertString( xStart, getText(), sal_False );
-                else if ( getText().getLength() )
+                else if ( !getText().isEmpty() )
                 {   // !!#i113673<<<
                     sal_Int16 nLatinFontPitch = 0, nLatinFontFamily = 0;
                     aTextCharacterProps.maLatinFont.getFontData( aLatinFontName, nLatinFontPitch, 
nLatinFontFamily, rFilterBase );
diff --git a/oox/source/drawingml/texttabstoplistcontext.cxx 
b/oox/source/drawingml/texttabstoplistcontext.cxx
index 4e3340f..9aae592 100644
--- a/oox/source/drawingml/texttabstoplistcontext.cxx
+++ b/oox/source/drawingml/texttabstoplistcontext.cxx
@@ -70,7 +70,7 @@ namespace oox { namespace drawingml {
                 OUString sValue;
                 TabStop aTabStop;
                 sValue = xAttribs->getOptionalValue( XML_pos );
-                if( sValue.getLength() )
+                if( !sValue.isEmpty() )
                 {
                     aTabStop.Position = GetCoordinate( sValue );
                 }
diff --git a/oox/source/dump/biffdumper.cxx b/oox/source/dump/biffdumper.cxx
index 976ae64..6dbc2eb 100644
--- a/oox/source/dump/biffdumper.cxx
+++ b/oox/source/dump/biffdumper.cxx
@@ -2359,7 +2359,7 @@ void WorkbookStreamObject::implDumpRecordBody()
                 }
             }
             OUString aName = dumpString( "name", BIFF_STR_8BITLENGTH, BIFF_STR_8BITLENGTH );
-            if( (aName.getLength() > 0) && (aName[ 0 ] == 1) && (rStrm.getRemaining() >= 2) )
+            if( !aName.isEmpty() && (aName[ 0 ] == 1) && (rStrm.getRemaining() >= 2) )
                 getFormulaDumper().dumpNameFormula();
         }
         break;
@@ -4527,7 +4527,7 @@ RootStorageObject::RootStorageObject( const DumperBase& rParent )
 
 void RootStorageObject::implDumpStream( const Reference< XInputStream >& rxStrm, const OUString& 
rStrgPath, const OUString& rStrmName, const OUString& rSysFileName )
 {
-    if( (rStrgPath.getLength() == 0) && (rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( 
"Book" ) ) || rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Workbook" ) )) )
+    if( rStrgPath.isEmpty() && (rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Book" ))  || 
rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Workbook" ) )) )
         WorkbookStreamObject( *this, rxStrm, rSysFileName ).dump();
     else if( rStrgPath.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "_SX_DB" ) ) )
         PivotCacheStreamObject( *this, rxStrm, BIFF5, rSysFileName ).dump();
diff --git a/oox/source/dump/dumperbase.cxx b/oox/source/dump/dumperbase.cxx
index f162379..eacbba3 100644
--- a/oox/source/dump/dumperbase.cxx
+++ b/oox/source/dump/dumperbase.cxx
@@ -617,7 +617,7 @@ void StringHelper::appendEncString( OUStringBuffer& rStr, const OUString& rData,
 
 void StringHelper::appendToken( OUStringBuffer& rStr, const OUString& rToken, sal_Unicode cSep )
 {
-    if( (rStr.getLength() > 0) && (rToken.getLength() > 0) )
+    if( (rStr.getLength() > 0) && (!rToken.isEmpty()) )
         rStr.append( cSep );
     rStr.append( rToken );
 }
@@ -631,7 +631,7 @@ void StringHelper::appendToken( OUStringBuffer& rStr, sal_Int64 nToken, sal_Unic
 
 void StringHelper::prependToken( OUStringBuffer& rStr, const OUString& rToken, sal_Unicode cSep )
 {
-    if( (rStr.getLength() > 0) && (rToken.getLength() > 0) )
+    if( rStr.getLength() > 0 && !rToken.isEmpty() )
         rStr.insert( 0, cSep );
     rStr.insert( 0, rToken );
 }
@@ -878,7 +878,7 @@ bool StringHelper::convertStringToBool( const OUString& rData )
 OUStringPair StringHelper::convertStringToPair( const OUString& rString, sal_Unicode cSep )
 {
     OUStringPair aPair;
-    if( rString.getLength() > 0 )
+    if( !rString.isEmpty() )
     {
         sal_Int32 nEqPos = rString.indexOf( cSep );
         if( nEqPos < 0 )
@@ -903,7 +903,7 @@ void StringHelper::convertStringToStringList( OUStringVector& orVec, const OUStr
     while( (0 <= nPos) && (nPos < nLen) )
     {
         OUString aToken = getToken( aUnquotedData, nPos, OOX_DUMP_LF );
-        if( !bIgnoreEmpty || (aToken.getLength() > 0) )
+        if( !bIgnoreEmpty || !aToken.isEmpty() )
             orVec.push_back( aToken );
     }
 }
@@ -1058,13 +1058,13 @@ ConfigItemBase::LineType ConfigItemBase::readConfigLine(
         TextInputStream& rStrm, OUString& orKey, OUString& orData ) const
 {
     OUString aLine;
-    while( !rStrm.isEof() && (aLine.getLength() == 0) )
+    while( !rStrm.isEof() && aLine.isEmpty() )
     {
         aLine = rStrm.readLine();
-        if( (aLine.getLength() > 0) && (aLine[ 0 ] == OOX_DUMP_BOM) )
+        if( !aLine.isEmpty() && (aLine[ 0 ] == OOX_DUMP_BOM) )
             aLine = aLine.copy( 1 );
         aLine = StringHelper::trimSpaces( aLine );
-        if( aLine.getLength() > 0 )
+        if( !aLine.isEmpty() )
         {
             // ignore comments (starting with hash or semicolon)
             sal_Unicode cChar = aLine[ 0 ];
@@ -1076,7 +1076,7 @@ ConfigItemBase::LineType ConfigItemBase::readConfigLine(
     OUStringPair aPair = StringHelper::convertStringToPair( aLine );
     orKey = aPair.first;
     orData = aPair.second;
-    return ((orKey.getLength() > 0) && ((orData.getLength() > 0) || !orKey.equalsAsciiL( 
RTL_CONSTASCII_STRINGPARAM( "end" ) ))) ?
+    return (!orKey.isEmpty() && (!orData.isEmpty() || !orKey.equalsAsciiL( 
RTL_CONSTASCII_STRINGPARAM( "end" ) ))) ?
         LINETYPE_DATA : LINETYPE_END;
 }
 
@@ -1239,7 +1239,7 @@ void MultiList::setNamesFromVec( sal_Int64 nStartKey, const OUStringVector& 
rNam
 {
     sal_Int64 nKey = nStartKey;
     for( OUStringVector::const_iterator aIt = rNames.begin(), aEnd = rNames.end(); aIt != aEnd; 
++aIt, ++nKey )
-        if( !mbIgnoreEmpty || (aIt->getLength() > 0) )
+        if( !mbIgnoreEmpty || !aIt->isEmpty() )
             insertRawName( nKey, *aIt );
 }
 
@@ -1300,7 +1300,7 @@ OUString FlagsList::implGetName( const Config& /*rCfg*/, sal_Int64 nKey ) 
const
         if( !getFlag( mnIgnore, nMask ) )
         {
             const OUString& rFlagName = aIt->second;
-            bool bOnOff = (rFlagName.getLength() > 0) && (rFlagName[ 0 ] == ':');
+            bool bOnOff = !rFlagName.isEmpty() && rFlagName[ 0 ] == ':';
             bool bFlag = getFlag( nKey, nMask );
             if( bOnOff )
             {
@@ -1309,7 +1309,7 @@ OUString FlagsList::implGetName( const Config& /*rCfg*/, sal_Int64 nKey ) 
const
             }
             else
             {
-                bool bNegated = (rFlagName.getLength() > 0) && (rFlagName[ 0 ] == '!');
+                bool bNegated = !rFlagName.isEmpty() && rFlagName[ 0 ] == '!';
                 sal_Int32 nBothSep = bNegated ? rFlagName.indexOf( '!', 1 ) : -1;
                 if( bFlag )
                 {
@@ -1383,8 +1383,8 @@ void CombiList::implSetName( sal_Int64 nKey, const OUString& rName )
             {
                 OUStringPair aFilter = StringHelper::convertStringToPair( aPair.second, '~' );
                 ExtItemFormatKey aKey( nKey );
-                if( (aFilter.first.getLength() > 0) && StringHelper::convertStringToInt( 
aKey.maFilter.first, aFilter.first ) &&
-                    (aFilter.second.getLength() > 0) && StringHelper::convertStringToInt( 
aKey.maFilter.second, aFilter.second ) )
+                if( !aFilter.first.isEmpty() && StringHelper::convertStringToInt( 
aKey.maFilter.first, aFilter.first ) &&
+                    !aFilter.second.isEmpty() && StringHelper::convertStringToInt( 
aKey.maFilter.second, aFilter.second ) )
                 {
                     if( aKey.maFilter.first == 0 )
                         aKey.maFilter.second = 0;
@@ -1443,7 +1443,7 @@ OUString CombiList::implGetName( const Config& rCfg, sal_Int64 nKey ) const
                 default:;
             }
             StringHelper::appendToken( aItem, aValue.makeStringAndClear(), OOX_DUMP_ITEMSEP );
-            if( rItemFmt.maListName.getLength() > 0 )
+            if( !rItemFmt.maListName.isEmpty() )
             {
                 OUString aValueName = rCfg.getName( rItemFmt.maListName, static_cast< sal_Int64 >( 
nUValue ) );
                 StringHelper::appendToken( aItem, aValueName, OOX_DUMP_ITEMSEP );
@@ -1516,7 +1516,7 @@ SharedConfigData::SharedConfigData( const OUString& rFileName,
     mbPwCancelled( false )
 {
     OUString aFileUrl = InputOutputHelper::convertFileNameToUrl( rFileName );
-    if( aFileUrl.getLength() > 0 )
+    if( !aFileUrl.isEmpty() )
     {
         sal_Int32 nNamePos = InputOutputHelper::getFileNamePos( aFileUrl );
         maConfigPath = aFileUrl.copy( 0, nNamePos );
@@ -1541,7 +1541,7 @@ const OUString* SharedConfigData::getOption( const OUString& rKey ) const
 
 void SharedConfigData::setNameList( const OUString& rListName, const NameListRef& rxList )
 {
-    if( rListName.getLength() > 0 )
+    if( !rListName.isEmpty() )
         maNameLists[ rListName ] = rxList;
 }
 
@@ -1575,7 +1575,7 @@ Sequence< NamedValue > SharedConfigData::requestEncryptionData( 
::comphelper::ID
 
 bool SharedConfigData::implIsValid() const
 {
-    return mbLoaded && mxContext.is() && mxRootStrg.get() && (maSysFileName.getLength() > 0);
+    return mbLoaded && mxContext.is() && mxRootStrg.get() && !maSysFileName.isEmpty();
 }
 
 void SharedConfigData::implProcessConfigItemStr(
@@ -1642,7 +1642,7 @@ void SharedConfigData::createUnitConverter( const OUString& rData )
     if( aDataVec.size() >= 2 )
     {
         OUString aFactor = aDataVec[ 1 ];
-        bool bRecip = (aFactor.getLength() > 0) && (aFactor[ 0 ] == '/');
+        bool bRecip = !aFactor.isEmpty() && aFactor[ 0 ] == '/';
         if( bRecip )
             aFactor = aFactor.copy( 1 );
         double fFactor;
@@ -1688,13 +1688,13 @@ void Config::construct( const Config& rParent )
 
 void Config::construct( const sal_Char* pcEnvVar, const FilterBase& rFilter )
 {
-    if( rFilter.getFileUrl().getLength() > 0 )
+    if( !rFilter.getFileUrl().isEmpty() )
         construct( pcEnvVar, rFilter.getComponentContext(), rFilter.getStorage(), 
rFilter.getFileUrl(), rFilter.getMediaDescriptor() );
 }
 
 void Config::construct( const sal_Char* pcEnvVar, const Reference< XComponentContext >& rxContext, 
const StorageRef& rxRootStrg, const OUString& rSysFileName, MediaDescriptor& rMediaDesc )
 {
-    if( pcEnvVar && rxRootStrg.get() && (rSysFileName.getLength() > 0) )
+    if( pcEnvVar && rxRootStrg.get() && !rSysFileName.isEmpty() )
         if( const sal_Char* pcFileName = ::getenv( pcEnvVar ) )
             mxCfgData.reset( new SharedConfigData( OUString::createFromAscii( pcFileName ), 
rxContext, rxRootStrg, rSysFileName, rMediaDesc ) );
 }
@@ -1925,7 +1925,7 @@ void Output::endItem()
     if( mnItemLevel > 0 )
     {
         maLastItem = OUString( maLine.getStr() + mnLastItem );
-        if( (maLastItem.getLength() == 0) && (mnLastItem > 0) && (maLine[ mnLastItem - 1 ] == 
OOX_DUMP_ITEMSEP) )
+        if( maLastItem.isEmpty() && mnLastItem > 0 && maLine[ mnLastItem - 1 ] == OOX_DUMP_ITEMSEP 
)
             maLine.setLength( mnLastItem - 1 );
         --mnItemLevel;
     }
@@ -2188,7 +2188,7 @@ void StorageObjectBase::construct( const ObjectBase& rParent )
 
 bool StorageObjectBase::implIsValid() const
 {
-    return mxStrg.get() && (maSysPath.getLength() > 0) && ObjectBase::implIsValid();
+    return mxStrg.get() && !maSysPath.isEmpty() && ObjectBase::implIsValid();
 }
 
 void StorageObjectBase::implDump()
@@ -2286,7 +2286,7 @@ void StorageObjectBase::extractStorage( const StorageRef& rxStrg, const 
OUString
         return;
 
     // process preferred storages and streams in root storage first
-    if( rStrgPath.getLength() == 0 )
+    if( rStrgPath.isEmpty() )
         for( PreferredItemVector::iterator aIt = maPreferred.begin(), aEnd = maPreferred.end(); 
aIt != aEnd; ++aIt )
             extractItem( rxStrg, rStrgPath, aIt->maName, rSysPath, aIt->mbStorage, !aIt->mbStorage 
);
 
@@ -2296,7 +2296,7 @@ void StorageObjectBase::extractStorage( const StorageRef& rxStrg, const 
OUString
         // skip processed preferred items
         OUString aItemName = aIt.getName();
         bool bFound = false;
-        if( rStrgPath.getLength() == 0 )
+        if( rStrgPath.isEmpty() )
             for( PreferredItemVector::iterator aIIt = maPreferred.begin(), aIEnd = 
maPreferred.end(); !bFound && (aIIt != aIEnd); ++aIIt )
                 bFound = aIIt->maName == aItemName;
         if( !bFound )
@@ -2651,7 +2651,7 @@ sal_Unicode InputObjectBase::dumpChar( const String& rName, rtl_TextEncoding 
eTe
     sal_uInt8 nChar;
     *mxStrm >> nChar;
     OUString aChar = OStringToOUString( OString( static_cast< sal_Char >( nChar ) ), eTextEnc );
-    sal_Unicode cChar = (aChar.getLength() > 0) ? aChar[ 0 ] : 0;
+    sal_Unicode cChar = aChar.isEmpty() ? 0 : aChar[ 0 ];
     writeCharItem( rName( "char" ), cChar );
     return cChar;
 }
@@ -2915,7 +2915,7 @@ void TextLineStreamObject::implDumpText( TextInputStream& rTextStrm )
     while( !rTextStrm.isEof() )
     {
         OUString aLine = rTextStrm.readLine();
-        if( !rTextStrm.isEof() || (aLine.getLength() > 0) )
+        if( !rTextStrm.isEof() || !aLine.isEmpty() )
             implDumpLine( aLine, ++nLine );
     }
 }
@@ -3012,7 +3012,7 @@ void XmlStreamObject::implDumpText( TextInputStream& rTextStrm )
                     entirely. */
                 mxOut->writeString( aElem );
                 mxOut->newLine();
-                if( aText.trim().getLength() > 0 )
+                if( !aText.trim().isEmpty() )
                 {
                     mxOut->writeString( aText );
                     mxOut->newLine();
diff --git a/oox/source/dump/oledumper.cxx b/oox/source/dump/oledumper.cxx
index a7c0daf..8c5e5a0 100644
--- a/oox/source/dump/oledumper.cxx
+++ b/oox/source/dump/oledumper.cxx
@@ -1654,7 +1654,7 @@ void FormControlStreamObject::implDump()
     if( mbReadGuid )
         maProgId = cfg().getStringOption( dumpGuid(), OUString() );
 
-    if( (maProgId.getLength() > 0) && !mxStrm->isEof() )
+    if( !maProgId.isEmpty() && !mxStrm->isEof() )
     {
         if( maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Forms.CommandButton.1" ) ) )
             AxCommandButtonObject( *this ).dump();
@@ -2191,7 +2191,7 @@ void VbaDirStreamObject::implDumpRecordBody()
             dumpByteString( "description" );
         break;
         case 0x002B:
-            if( maCurrStream.getLength() > 0 )
+            if( !maCurrStream.isEmpty() )
                 mrVbaData.maStrmOffsets[ maCurrStream ] = mnCurrOffset;
             maCurrStream = OUString();
             mnCurrOffset = 0;
@@ -2283,7 +2283,7 @@ VbaStorageObject::VbaStorageObject( const ObjectBase& rParent, const 
StorageRef&
 
 void VbaStorageObject::implDumpStream( const Reference< XInputStream >& rxStrm, const OUString& 
rStrgPath, const OUString& rStrmName, const OUString& rSysFileName )
 {
-    if( (rStrgPath.getLength() == 0) && rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "dir" 
) ) )
+    if( rStrgPath.isEmpty() && rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "dir" ) ) )
         VbaDirStreamObject( *this, rxStrm, rSysFileName, mrVbaData ).dump();
     else if( mrVbaData.isModuleStream( rStrmName ) )
         VbaModuleStreamObject( *this, rxStrm, rSysFileName, mrVbaData, mrVbaData.getStreamOffset( 
rStrmName ) ).dump();
@@ -2317,7 +2317,7 @@ VbaProjectStorageObject::VbaProjectStorageObject( const ObjectBase& rParent, 
con
 
 void VbaProjectStorageObject::implDumpStream( const Reference< XInputStream >& rxStrm, const 
OUString& rStrgPath, const OUString& rStrmName, const OUString& rSysFileName )
 {
-    if( (rStrgPath.getLength() == 0) && rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( 
"PROJECT" ) ) )
+    if( rStrgPath.isEmpty() && rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "PROJECT" ) ) )
         TextLineStreamObject( *this, rxStrm, maVbaData.meTextEnc, rSysFileName ).dump();
     else
         OleStorageObject::implDumpStream( rxStrm, rStrgPath, rStrmName, rSysFileName );
diff --git a/oox/source/export/chartexport.cxx b/oox/source/export/chartexport.cxx
index e757b1e..9996d1e 100644
--- a/oox/source/export/chartexport.cxx
+++ b/oox/source/export/chartexport.cxx
@@ -383,7 +383,7 @@ OUString lcl_flattenStringSequence( const Sequence< OUString > & rSequence )
     bool bPrecedeWithSpace = false;
     for( sal_Int32 nIndex=0; nIndex<rSequence.getLength(); ++nIndex )
     {
-        if( rSequence[nIndex].getLength())
+        if( !rSequence[nIndex].isEmpty())
         {
             if( bPrecedeWithSpace )
                 aResult.append( static_cast< sal_Unicode >( ' ' ));
@@ -777,7 +777,7 @@ void ChartExport::InitRangeSegmentationProperties( const Reference< chart2::XCha
                 // range, where every row number is noe too large, so that older
                 // version can correctly read those files.
                 msChartAddress = (bBrokenRangeAvailable ? sBrokenRange : sCellRange);
-                if( msChartAddress.getLength() > 0 )
+                if( !msChartAddress.isEmpty() )
                 {
                     // convert format to XML-conform one
                     Reference< chart2::data::XRangeXMLConversion > xConversion( xDataProvider, 
uno::UNO_QUERY );
@@ -857,7 +857,7 @@ void ChartExport::_ExportContent()
                             //maExportHelper.SetTableNumberList( sTableNumberList );
 
                             // do not include own table if there are external addresses
-                            bIncludeTable = (sChartAddress.getLength() == 0);
+                            bIncludeTable = sChartAddress.isEmpty();
                         }
                         catch( beans::UnknownPropertyException & )
                         {
@@ -1059,7 +1059,7 @@ void ChartExport::exportTitle( Reference< XShape > xShape )
     {
         xPropSet->getPropertyValue( OUString( RTL_CONSTASCII_USTRINGPARAM( "String" ))) >>= sText;
     }
-    if( sText.getLength() == 0 )
+    if( sText.isEmpty() )
         return;
 
     FSHelperPtr pFS = GetFS();
diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx
index 745c6c5..cfe4bf0 100644
--- a/oox/source/export/drawingml.cxx
+++ b/oox/source/export/drawingml.cxx
@@ -620,7 +620,7 @@ void DrawingML::WriteBlipFill( Reference< XPropertySet > rXPropSet, String sURLP
 
         DBG(printf ("URL: %s\n", OUStringToOString( aURL, RTL_TEXTENCODING_UTF8 ).getStr() ));
 
-        if( !aURL.getLength() )
+        if( aURL.isEmpty() )
             return;
 
         mpFS->startElementNS( nXmlNamespace , XML_blipFill, FSEND );
@@ -762,7 +762,7 @@ void DrawingML::WriteRunProperties( Reference< XPropertySet > rRun, sal_Bool bIs
         mAny >>= eLocale;
 
         OUStringBuffer usLanguageBuffer = eLocale.Language;
-        if( eLocale.Country.getLength() ) {
+        if( !eLocale.Country.isEmpty() ) {
             usLanguageBuffer.appendAscii( "-" );
             usLanguageBuffer.append( eLocale.Country );
         }
@@ -774,7 +774,7 @@ void DrawingML::WriteRunProperties( Reference< XPropertySet > rRun, sal_Bool bIs
     mpFS->startElementNS( XML_a, XML_rPr,
                           XML_b, bold,
                           XML_i, italic,
-                          XML_lang, usLanguage.getLength() ? USS( usLanguage ) : NULL,
+                          XML_lang, usLanguage.isEmpty() ? NULL : USS( usLanguage ),
                           XML_sz, nSize == 1800 ? NULL : IS( nSize ),
                           XML_u, underline,
                           FSEND );
@@ -848,7 +848,7 @@ void DrawingML::WriteRunProperties( Reference< XPropertySet > rRun, sal_Bool bIs
     OUString sURL;
 
     mAny >>= sURL;
-    if( sURL.getLength() ) {
+    if( !sURL.isEmpty() ) {
         OUString sRelId = mpFB->addRelation( mpFS->getOutputStream(),
                               US( 
"http://schemas.openxmlformats.org/officeDocument/2006/relationships/hyperlink"; ),
                               sURL, true );
@@ -937,13 +937,13 @@ void DrawingML::WriteRun( Reference< XTextRange > rRun )
     sal_Bool bIsField = sal_False;
     OUString sText = rRun->getString();
 
-    if( sText.getLength() < 1) {
+    if( sText.isEmpty()) {
         Reference< XPropertySet > xPropSet( rRun, UNO_QUERY );
 
         try {
         if( !xPropSet.is() || !( xPropSet->getPropertyValue( S( "PlaceholderText" ) ) >>= sText ) )
             return;
-        if( sText.getLength() < 1 )
+        if( sText.isEmpty() )
             return;
         }
         catch (const Exception &) {
@@ -1100,7 +1100,7 @@ void DrawingML::WriteParagraphNumbering( Reference< XPropertySet > rXPropSet, 
sa
                 const char* pAutoNumType = GetAutoNumType( nNumberingType, bSDot, bPBehind, bPBoth 
);
 
                 if( nLevel >= 0 ) {
-                    if( aGraphicURL.getLength() > 0 ) {
+                    if( !aGraphicURL.isEmpty() ) {
                         OUString sRelId = WriteImage( aGraphicURL );
 
                         mpFS->startElementNS( XML_a, XML_buBlip, FSEND );
diff --git a/oox/source/helper/attributelist.cxx b/oox/source/helper/attributelist.cxx
index 767c571..2d74450 100644
--- a/oox/source/helper/attributelist.cxx
+++ b/oox/source/helper/attributelist.cxx
@@ -172,49 +172,49 @@ OptValue< OUString > AttributeList::getXString( sal_Int32 nAttrToken ) const
 OptValue< double > AttributeList::getDouble( sal_Int32 nAttrToken ) const
 {
     OUString aValue = mxAttribs->getOptionalValue( nAttrToken );
-    bool bValid = aValue.getLength() > 0;
+    bool bValid = !aValue.isEmpty();
     return OptValue< double >( bValid, bValid ? AttributeConversion::decodeDouble( aValue ) : 0.0 
);
 }
 
 OptValue< sal_Int32 > AttributeList::getInteger( sal_Int32 nAttrToken ) const
 {
     OUString aValue = mxAttribs->getOptionalValue( nAttrToken );
-    bool bValid = aValue.getLength() > 0;
+    bool bValid = !aValue.isEmpty();
     return OptValue< sal_Int32 >( bValid, bValid ? AttributeConversion::decodeInteger( aValue ) : 
0 );
 }
 
 OptValue< sal_uInt32 > AttributeList::getUnsigned( sal_Int32 nAttrToken ) const
 {
     OUString aValue = mxAttribs->getOptionalValue( nAttrToken );
-    bool bValid = aValue.getLength() > 0;
+    bool bValid = !aValue.isEmpty();
     return OptValue< sal_uInt32 >( bValid, AttributeConversion::decodeUnsigned( aValue ) );
 }
 
 OptValue< sal_Int64 > AttributeList::getHyper( sal_Int32 nAttrToken ) const
 {
     OUString aValue = mxAttribs->getOptionalValue( nAttrToken );
-    bool bValid = aValue.getLength() > 0;
+    bool bValid = !aValue.isEmpty();
     return OptValue< sal_Int64 >( bValid, bValid ? AttributeConversion::decodeHyper( aValue ) : 0 
);
 }
 
 OptValue< sal_Int32 > AttributeList::getIntegerHex( sal_Int32 nAttrToken ) const
 {
     OUString aValue = mxAttribs->getOptionalValue( nAttrToken );
-    bool bValid = aValue.getLength() > 0;
+    bool bValid = !aValue.isEmpty();
     return OptValue< sal_Int32 >( bValid, bValid ? AttributeConversion::decodeIntegerHex( aValue ) 
: 0 );
 }
 
 OptValue< sal_uInt32 > AttributeList::getUnsignedHex( sal_Int32 nAttrToken ) const
 {
     OUString aValue = mxAttribs->getOptionalValue( nAttrToken );
-    bool bValid = aValue.getLength() > 0;
+    bool bValid = !aValue.isEmpty();
     return OptValue< sal_uInt32 >( bValid, bValid ? AttributeConversion::decodeUnsignedHex( aValue 
) : 0 );
 }
 
 OptValue< sal_Int64 > AttributeList::getHyperHex( sal_Int32 nAttrToken ) const
 {
     OUString aValue = mxAttribs->getOptionalValue( nAttrToken );
-    bool bValid = aValue.getLength() > 0;
+    bool bValid = !aValue.isEmpty();
     return OptValue< sal_Int64 >( bValid, bValid ? AttributeConversion::decodeHyperHex( aValue ) : 
0 );
 }
 
diff --git a/oox/source/helper/graphichelper.cxx b/oox/source/helper/graphichelper.cxx
index 6ddb986..ddca88f 100644
--- a/oox/source/helper/graphichelper.cxx
+++ b/oox/source/helper/graphichelper.cxx
@@ -293,8 +293,8 @@ Reference< XGraphic > GraphicHelper::importGraphic( const StreamDataSequence& rG
 Reference< XGraphic > GraphicHelper::importEmbeddedGraphic( const OUString& rStreamName ) const
 {
     Reference< XGraphic > xGraphic;
-    OSL_ENSURE( rStreamName.getLength() > 0, "GraphicHelper::importEmbeddedGraphic - empty stream 
name" );
-    if( rStreamName.getLength() > 0 )
+    OSL_ENSURE( !rStreamName.isEmpty(), "GraphicHelper::importEmbeddedGraphic - empty stream name" 
);
+    if( !rStreamName.isEmpty() )
     {
         EmbeddedGraphicMap::const_iterator aIt = maEmbeddedGraphics.find( rStreamName );
         if( aIt == maEmbeddedGraphics.end() )
diff --git a/oox/source/helper/modelobjecthelper.cxx b/oox/source/helper/modelobjecthelper.cxx
index 2268bfa..f0c31f9 100644
--- a/oox/source/helper/modelobjecthelper.cxx
+++ b/oox/source/helper/modelobjecthelper.cxx
@@ -130,7 +130,7 @@ bool ModelObjectHelper::insertLineMarker( const OUString& rMarkerName, const Pol
 {
     OSL_ENSURE( rMarker.Coordinates.hasElements(), "ModelObjectHelper::insertLineMarker - line 
marker without coordinates" );
     if( rMarker.Coordinates.hasElements() )
-        return maMarkerContainer.insertObject( rMarkerName, Any( rMarker ), false ).getLength() > 
0;
+        return !maMarkerContainer.insertObject( rMarkerName, Any( rMarker ), false ).isEmpty();
     return false;
 }
 
@@ -151,7 +151,7 @@ OUString ModelObjectHelper::insertTransGrandient( const Gradient& rGradient )
 
 OUString ModelObjectHelper::insertFillBitmapUrl( const OUString& rGraphicUrl )
 {
-    if( rGraphicUrl.getLength() > 0 )
+    if( !rGraphicUrl.isEmpty() )
         return maBitmapUrlContainer.insertObject( maBitmapUrlNameBase, Any( rGraphicUrl ), true );
     return OUString();
 }
diff --git a/oox/source/helper/storagebase.cxx b/oox/source/helper/storagebase.cxx
index 7e5b2d0..38eacaf 100644
--- a/oox/source/helper/storagebase.cxx
+++ b/oox/source/helper/storagebase.cxx
@@ -110,7 +110,7 @@ bool StorageBase::isStorage() const
 
 bool StorageBase::isRootStorage() const
 {
-    return implIsStorage() && (maStorageName.getLength() == 0);
+    return implIsStorage() && maStorageName.isEmpty();
 }
 
 bool StorageBase::isReadOnly() const
@@ -151,9 +151,9 @@ StorageRef StorageBase::openSubStorage( const OUString& rStorageName, bool bCrea
     {
         OUString aElement, aRemainder;
         lclSplitFirstElement( aElement, aRemainder, rStorageName );
-        if( aElement.getLength() > 0 )
+        if( !aElement.isEmpty() )
             xSubStorage = getSubStorage( aElement, bCreateMissing );
-        if( xSubStorage.get() && (aRemainder.getLength() > 0) )
+        if( xSubStorage.get() && !aRemainder.isEmpty() )
             xSubStorage = xSubStorage->openSubStorage( aRemainder, bCreateMissing );
     }
     return xSubStorage;
@@ -164,9 +164,9 @@ Reference< XInputStream > StorageBase::openInputStream( const OUString& rStreamN
     Reference< XInputStream > xInStream;
     OUString aElement, aRemainder;
     lclSplitFirstElement( aElement, aRemainder, rStreamName );
-    if( aElement.getLength() > 0 )
+    if( !aElement.isEmpty() )
     {
-        if( aRemainder.getLength() > 0 )
+        if( !aRemainder.isEmpty() )
         {
             StorageRef xSubStorage = getSubStorage( aElement, false );
             if( xSubStorage.get() )
@@ -192,9 +192,9 @@ Reference< XOutputStream > StorageBase::openOutputStream( const OUString& rStrea
     {
         OUString aElement, aRemainder;
         lclSplitFirstElement( aElement, aRemainder, rStreamName );
-        if( aElement.getLength() > 0 )
+        if( !aElement.isEmpty() )
         {
-            if( aRemainder.getLength() > 0 )
+            if( !aRemainder.isEmpty() )
             {
                 StorageRef xSubStorage = getSubStorage( aElement, true );
                 if( xSubStorage.get() )
@@ -216,8 +216,8 @@ Reference< XOutputStream > StorageBase::openOutputStream( const OUString& rStrea
 void StorageBase::copyToStorage( StorageBase& rDestStrg, const OUString& rElementName )
 {
     OSL_ENSURE( rDestStrg.isStorage() && !rDestStrg.isReadOnly(), "StorageBase::copyToStorage - 
invalid destination" );
-    OSL_ENSURE( rElementName.getLength() > 0, "StorageBase::copyToStorage - invalid element name" 
);
-    if( rDestStrg.isStorage() && !rDestStrg.isReadOnly() && (rElementName.getLength() > 0) )
+    OSL_ENSURE( !rElementName.isEmpty(), "StorageBase::copyToStorage - invalid element name" );
+    if( rDestStrg.isStorage() && !rDestStrg.isReadOnly() && !rElementName.isEmpty() )
     {
         StorageRef xSubStrg = openSubStorage( rElementName, false );
         if( xSubStrg.get() )
diff --git a/oox/source/helper/textinputstream.cxx b/oox/source/helper/textinputstream.cxx
index e98ed86..f374cc9 100644
--- a/oox/source/helper/textinputstream.cxx
+++ b/oox/source/helper/textinputstream.cxx
@@ -181,7 +181,7 @@ OUString TextInputStream::readToChar( sal_Unicode cChar, bool bIncludeChar )
             a character that has been buffered in the previous call. */
         OUString aString = createFinalString( mxTextStrm->readString( aDelimiters, sal_False ) );
         // remove last character from string and remember it for next call
-        if( !bIncludeChar && (aString.getLength() > 0) && (aString[ aString.getLength() - 1 ] == 
cChar) )
+        if( !bIncludeChar && !aString.isEmpty() && (aString[ aString.getLength() - 1 ] == cChar) )
         {
             mcPendingChar = cChar;
             aString = aString.copy( 0, aString.getLength() - 1 );
diff --git a/oox/source/mathml/importutils.cxx b/oox/source/mathml/importutils.cxx
index 4a71d56..55eb1fa 100644
--- a/oox/source/mathml/importutils.cxx
+++ b/oox/source/mathml/importutils.cxx
@@ -155,7 +155,7 @@ sal_Unicode XmlStream::AttributeList::attribute( int token, sal_Unicode def ) co
     std::map< int, rtl::OUString >::const_iterator find = attrs.find( token );
     if( find != attrs.end())
     {
-        if( find->second.getLength() >= 1 )
+        if( !find->second.isEmpty() )
         {
 //            if( find->second.getLength() != 1 )
 //                fprintf( stderr, "Cannot convert \'%s\' to sal_Unicode, stripping.\n", CSTR( 
find->second ));
diff --git a/oox/source/ole/axcontrol.cxx b/oox/source/ole/axcontrol.cxx
index c9bde1f..d6a0c62 100644
--- a/oox/source/ole/axcontrol.cxx
+++ b/oox/source/ole/axcontrol.cxx
@@ -292,7 +292,7 @@ void ControlConverter::convertPicture( PropertyMap& rPropMap, const StreamDataSe
     if( rPicData.hasElements() )
     {
         OUString aGraphicUrl = mrGraphicHelper.importGraphicObject( rPicData );
-        if( aGraphicUrl.getLength() > 0 )
+        if( !aGraphicUrl.isEmpty() )
             rPropMap.setProperty( PROP_ImageURL, aGraphicUrl );
     }
 }
@@ -337,7 +337,7 @@ void ControlConverter::bindToSources( const Reference< XControlModel >& rxCtrlMo
         const OUString& rCtrlSource, const OUString& rRowSource, sal_Int32 nRefSheet ) const
 {
     // value binding
-    if( rCtrlSource.getLength() > 0 ) try
+    if( !rCtrlSource.isEmpty() ) try
     {
         // first check if the XBindableValue interface is supported
         Reference< XBindableValue > xBindable( rxCtrlModel, UNO_QUERY_THROW );
@@ -369,7 +369,7 @@ void ControlConverter::bindToSources( const Reference< XControlModel >& rxCtrlMo
     }
 
     // list entry source
-    if( rRowSource.getLength() > 0 ) try
+    if( !rRowSource.isEmpty() ) try
     {
         // first check if the XListEntrySink interface is supported
         Reference< XListEntrySink > xEntrySink( rxCtrlModel, UNO_QUERY_THROW );
@@ -899,7 +899,7 @@ void AxFontDataModel::exportBinaryModel( BinaryOutputStream& rOutStrm )
 void AxFontDataModel::convertProperties( PropertyMap& rPropMap, const ControlConverter& rConv ) 
const
 {
     // font name
-    if( maFontData.maFontName.getLength() > 0 )
+    if( !maFontData.maFontName.isEmpty() )
         rPropMap.setProperty( PROP_FontName, maFontData.maFontName );
 
     // font effects
@@ -2637,7 +2637,7 @@ OUString EmbeddedControl::getServiceName() const
 
 bool EmbeddedControl::convertProperties( const Reference< XControlModel >& rxCtrlModel, const 
ControlConverter& rConv ) const
 {
-    if( mxModel.get() && rxCtrlModel.is() && (maName.getLength() > 0) )
+    if( mxModel.get() && rxCtrlModel.is() && !maName.isEmpty() )
     {
         PropertyMap aPropMap;
         aPropMap.setProperty( PROP_Name, maName );
@@ -2658,7 +2658,7 @@ bool EmbeddedControl::convertProperties( const Reference< XControlModel >& 
rxCtr
 
 bool EmbeddedControl::convertFromProperties( const Reference< XControlModel >& rxCtrlModel, const 
ControlConverter& rConv )
 {
-    if( mxModel.get() && rxCtrlModel.is() && (maName.getLength() > 0) )
+    if( mxModel.get() && rxCtrlModel.is() && !maName.isEmpty() )
     {
         PropertySet aPropSet( rxCtrlModel );
         aPropSet.getProperty( maName, PROP_Name );
diff --git a/oox/source/ole/axcontrolfragment.cxx b/oox/source/ole/axcontrolfragment.cxx
index 7b985c5..53d21b3 100644
--- a/oox/source/ole/axcontrolfragment.cxx
+++ b/oox/source/ole/axcontrolfragment.cxx
@@ -83,7 +83,7 @@ ContextHandlerRef AxControlPropertyContext::onCreateContext( sal_Int32 nElement,
             if( nElement == AX_TOKEN( picture ) )
             {
                 OUString aPicturePath = getFragmentPathFromRelId( rAttribs.getString( R_TOKEN( id 
), OUString() ) );
-                if( aPicturePath.getLength() > 0 )
+                if( !aPicturePath.isEmpty() )
                 {
                     BinaryXInputStream aInStrm( getFilter().openInputStream( aPicturePath ), true 
);
                     mrModel.importPictureData( mnPropId, aInStrm );
@@ -117,7 +117,7 @@ ContextHandlerRef AxControlFragment::onCreateContext( sal_Int32 nElement, const
             case XML_persistStreamInit:
             {
                 OUString aFragmentPath = getFragmentPathFromRelId( rAttribs.getString( R_TOKEN( id 
), OUString() ) );
-                if( aFragmentPath.getLength() > 0 )
+                if( !aFragmentPath.isEmpty() )
                 {
                     BinaryXInputStream aInStrm( getFilter().openInputStream( aFragmentPath ), true 
);
                     if( !aInStrm.isEof() )
@@ -136,7 +136,7 @@ ContextHandlerRef AxControlFragment::onCreateContext( sal_Int32 nElement, const
             case XML_persistStorage:
             {
                 OUString aFragmentPath = getFragmentPathFromRelId( rAttribs.getString( R_TOKEN( id 
), OUString() ) );
-                if( aFragmentPath.getLength() > 0 )
+                if( !aFragmentPath.isEmpty() )
                 {
                     Reference< XInputStream > xStrgStrm = getFilter().openInputStream( 
aFragmentPath );
                     if( xStrgStrm.is() )
diff --git a/oox/source/ole/olehelper.cxx b/oox/source/ole/olehelper.cxx
index a04a2c6..012fc20 100644
--- a/oox/source/ole/olehelper.cxx
+++ b/oox/source/ole/olehelper.cxx
@@ -676,7 +676,7 @@ sal_Bool MSConvertOCXControls::ReadOCXStorage( SotStorageRef& xOleStg,
         {
             OUString aName = aNameStream.readNulUnicodeArray();
             Reference< XControlModel > xCtlModel( rxFormComp, UNO_QUERY );
-            if ( aName.getLength() && xCtlModel.is() )
+            if ( !aName.isEmpty() && xCtlModel.is() )
             {
                 PropertyMap aPropMap;
                 aPropMap.setProperty( PROP_Name, aName );
diff --git a/oox/source/ole/oleobjecthelper.cxx b/oox/source/ole/oleobjecthelper.cxx
index 30a1ce6..98c46aa 100644
--- a/oox/source/ole/oleobjecthelper.cxx
+++ b/oox/source/ole/oleobjecthelper.cxx
@@ -95,7 +95,7 @@ bool OleObjectHelper::importOleObject( PropertyMap& rPropMap, const OleObjectInf
     if( rOleObject.mbLinked )
     {
         // linked OLE object - set target URL
-        if( rOleObject.maTargetLink.getLength() > 0 )
+        if( !rOleObject.maTargetLink.isEmpty() )
         {
             rPropMap[ PROP_LinkURL ] <<= rOleObject.maTargetLink;
             bRet = true;
@@ -116,7 +116,7 @@ bool OleObjectHelper::importOleObject( PropertyMap& rPropMap, const OleObjectInf
             OUString aUrl = mxResolver->resolveEmbeddedObjectURL( aObjectId );
             OSL_ENSURE( aUrl.match( maEmbeddedObjScheme ), "OleObjectHelper::importOleObject - 
unexpected URL scheme" );
             OUString aPersistName = aUrl.copy( maEmbeddedObjScheme.getLength() );
-            if( aPersistName.getLength() > 0 )
+            if( !aPersistName.isEmpty() )
             {
                 rPropMap[ PROP_PersistName ] <<= aPersistName;
                 bRet = true;
diff --git a/oox/source/ole/olestorage.cxx b/oox/source/ole/olestorage.cxx
index 511b4ee..490fc6f 100755
--- a/oox/source/ole/olestorage.cxx
+++ b/oox/source/ole/olestorage.cxx
@@ -324,7 +324,7 @@ void OleStorage::implGetElementNames( ::std::vector< OUString >& orElementNames
 StorageRef OleStorage::implOpenSubStorage( const OUString& rElementName, bool bCreateMissing )
 {
     StorageRef xSubStorage;
-    if( mxStorage.is() && (rElementName.getLength() > 0) )
+    if( mxStorage.is() && !rElementName.isEmpty() )
     {
         try
         {
@@ -376,7 +376,7 @@ Reference< XInputStream > OleStorage::implOpenInputStream( const OUString& rElem
 Reference< XOutputStream > OleStorage::implOpenOutputStream( const OUString& rElementName )
 {
     Reference< XOutputStream > xOutStream;
-    if( mxStorage.is() && (rElementName.getLength() > 0) )
+    if( mxStorage.is() && !rElementName.isEmpty() )
         xOutStream.set( new OleOutputStream( mxContext, mxStorage, rElementName ) );
     return xOutStream;
 }
diff --git a/oox/source/ole/vbacontrol.cxx b/oox/source/ole/vbacontrol.cxx
index 048d4f8..0a4b91b 100644
--- a/oox/source/ole/vbacontrol.cxx
+++ b/oox/source/ole/vbacontrol.cxx
@@ -125,7 +125,7 @@ VbaControlNamesSet::VbaControlNamesSet() :
 void VbaControlNamesSet::insertName( const VbaFormControl& rControl )
 {
     OUString aName = rControl.getControlName();
-    if( aName.getLength() > 0 )
+    if( !aName.isEmpty() )
         maCtrlNames.insert( aName );
 }
 
@@ -455,8 +455,8 @@ bool VbaFormControl::convertProperties( const Reference< XControlModel >& rxCtrl
     if( rxCtrlModel.is() && mxSiteModel.get() && mxCtrlModel.get() )
     {
         const OUString& rCtrlName = mxSiteModel->getName();
-        OSL_ENSURE( rCtrlName.getLength() > 0, "VbaFormControl::convertProperties - control 
without name" );
-        if( rCtrlName.getLength() > 0 )
+        OSL_ENSURE( !rCtrlName.isEmpty(), "VbaFormControl::convertProperties - control without 
name" );
+        if( !rCtrlName.isEmpty() )
         {
             // convert all properties
             PropertyMap aPropMap;
@@ -785,7 +785,7 @@ bool lclEatKeyword( OUString& rCodeLine, const OUString& rKeyword )
     {
         rCodeLine = rCodeLine.copy( rKeyword.getLength() );
         // success, if code line ends after keyword, or if whitespace follows
-        return (rCodeLine.getLength() == 0) || lclEatWhitespace( rCodeLine );
+        return rCodeLine.isEmpty() || lclEatWhitespace( rCodeLine );
     }
     return false;
 }
@@ -835,11 +835,11 @@ void VbaUserForm::importForm(
 
     // remaining line is the form name
     OUString aFormName = aLine.trim();
-    OSL_ENSURE( aFormName.getLength() > 0, "VbaUserForm::importForm - missing form name" );
+    OSL_ENSURE( !aFormName.isEmpty(), "VbaUserForm::importForm - missing form name" );
     OSL_ENSURE( rModuleName.equalsIgnoreAsciiCase( aFormName ), "VbaUserForm::importFrameStream - 
form and module name mismatch" );
-    if( aFormName.getLength() == 0 )
+    if( aFormName.isEmpty() )
         aFormName = rModuleName;
-    if( aFormName.getLength() == 0 )
+    if( aFormName.isEmpty() )
         return;
     mxSiteModel.reset( new VbaSiteModel );
     mxSiteModel->importProperty( XML_Name, aFormName );
diff --git a/oox/source/ole/vbahelper.cxx b/oox/source/ole/vbahelper.cxx
index 47bbacb..e78ec21 100644
--- a/oox/source/ole/vbahelper.cxx
+++ b/oox/source/ole/vbahelper.cxx
@@ -45,9 +45,9 @@ using ::rtl::OUStringBuffer;
 /*static*/ OUString VbaHelper::getBasicScriptUrl(
         const OUString& rLibraryName, const OUString& rModuleName, const OUString& rMacroName )
 {
-    OSL_ENSURE( rLibraryName.getLength() > 0, "VbaHelper::getBasicScriptUrl - library name is 
empty" );
-    OSL_ENSURE( rModuleName.getLength() > 0, "VbaHelper::getBasicScriptUrl - module name is empty" 
);
-    OSL_ENSURE( rMacroName.getLength() > 0, "VbaHelper::getBasicScriptUrl - macro name is empty" );
+    OSL_ENSURE( !rLibraryName.isEmpty(), "VbaHelper::getBasicScriptUrl - library name is empty" );
+    OSL_ENSURE( !rModuleName.isEmpty(), "VbaHelper::getBasicScriptUrl - module name is empty" );
+    OSL_ENSURE( !rMacroName.isEmpty(), "VbaHelper::getBasicScriptUrl - macro name is empty" );
     const sal_Unicode cDot = '.';
     return OUStringBuffer().
         appendAscii( RTL_CONSTASCII_STRINGPARAM( "vnd.sun.star.script:" ) ).
@@ -78,7 +78,7 @@ using ::rtl::OUStringBuffer;
     {
         rKey = rKeyValue.copy( 0, nEqSignPos ).trim();
         rValue = rKeyValue.copy( nEqSignPos + 1 ).trim();
-        return (rKey.getLength() > 0) && (rValue.getLength() > 0);
+        return !rKey.isEmpty() && !rValue.isEmpty();
     }
     return false;
 }
diff --git a/oox/source/ole/vbamodule.cxx b/oox/source/ole/vbamodule.cxx
index cbd9744..d907051 100644
--- a/oox/source/ole/vbamodule.cxx
+++ b/oox/source/ole/vbamodule.cxx
@@ -197,8 +197,8 @@ void VbaModule::importDirRecords( BinaryInputStream& rDirStrm )
 #undef OOX_ENSURE_RECORDSIZE
         }
     }
-    OSL_ENSURE( maName.getLength() > 0, "VbaModule::importDirRecords - missing module name" );
-    OSL_ENSURE( maStreamName.getLength() > 0, "VbaModule::importDirRecords - missing module stream 
name" );
+    OSL_ENSURE( !maName.isEmpty(), "VbaModule::importDirRecords - missing module name" );
+    OSL_ENSURE( !maStreamName.isEmpty(), "VbaModule::importDirRecords - missing module stream 
name" );
     OSL_ENSURE( mnType != ModuleType::UNKNOWN, "VbaModule::importDirRecords - missing module type" 
);
     OSL_ENSURE( mnOffset < SAL_MAX_UINT32, "VbaModule::importDirRecords - missing module stream 
offset" );
 }
@@ -218,7 +218,7 @@ void VbaModule::createEmptyModule( const Reference< XNameContainer >& rxBasicLib
 OUString VbaModule::readSourceCode( StorageBase& rVbaStrg, const Reference< XNameContainer >& 
rxOleNameOverrides ) const
 {
     OUStringBuffer aSourceCode;
-    if( (maStreamName.getLength() > 0) && (mnOffset != SAL_MAX_UINT32) )
+    if( !maStreamName.isEmpty() && (mnOffset != SAL_MAX_UINT32) )
     {
         BinaryXInputStream aInStrm( rVbaStrg.openInputStream( maStreamName ), true );
         OSL_ENSURE( !aInStrm.isEof(), "VbaModule::readSourceCode - cannot open module stream" );
@@ -280,7 +280,7 @@ void VbaModule::extractOleOverrideFromAttr( const OUString& rAttribute, const Re
 void VbaModule::createModule( const OUString& rVBASourceCode,
         const Reference< XNameContainer >& rxBasicLib, const Reference< XNameAccess >& 
rxDocObjectNA ) const
 {
-    if( maName.getLength() == 0 )
+    if( maName.isEmpty() )
         return;
 
     // prepare the Basic module
diff --git a/oox/source/ole/vbaproject.cxx b/oox/source/ole/vbaproject.cxx
index 3685c2d..4237d2a 100644
--- a/oox/source/ole/vbaproject.cxx
+++ b/oox/source/ole/vbaproject.cxx
@@ -98,7 +98,7 @@ VbaFilterConfig::VbaFilterConfig( const Reference< XComponentContext >& rxContex
     OSL_ENSURE( rxContext.is(), "VbaFilterConfig::VbaFilterConfig - missing component context" );
     if( rxContext.is() ) try
     {
-        OSL_ENSURE( rConfigCompName.getLength() > 0, "VbaFilterConfig::VbaFilterConfig - invalid 
configuration component name" );
+        OSL_ENSURE( !rConfigCompName.isEmpty(), "VbaFilterConfig::VbaFilterConfig - invalid 
configuration component name" );
         OUString aConfigPackage = CREATE_OUSTRING( "org.openoffice.Office." ) + rConfigCompName;
         Reference< XMultiServiceFactory > xFactory( rxContext->getServiceManager(), 
UNO_QUERY_THROW );
         mxConfigAccess = ConfigurationHelper::openConfig( xFactory, aConfigPackage, 
ConfigurationHelper::E_READONLY );
@@ -133,7 +133,7 @@ bool VbaFilterConfig::isExportVba() const
 VbaMacroAttacherBase::VbaMacroAttacherBase( const OUString& rMacroName ) :
     maMacroName( rMacroName )
 {
-    OSL_ENSURE( maMacroName.getLength() > 0, "VbaMacroAttacherBase::VbaMacroAttacherBase - empty 
macro name" );
+    OSL_ENSURE( !maMacroName.isEmpty(), "VbaMacroAttacherBase::VbaMacroAttacherBase - empty macro 
name" );
 }
 
 VbaMacroAttacherBase::~VbaMacroAttacherBase()
@@ -231,7 +231,7 @@ bool VbaProject::hasDialog( const OUString& rDialogName ) const
 
 void VbaProject::addDummyModule( const OUString& rName, sal_Int32 nType )
 {
-    OSL_ENSURE( rName.getLength() > 0, "VbaProject::addDummyModule - missing module name" );
+    OSL_ENSURE( !rName.isEmpty(), "VbaProject::addDummyModule - missing module name" );
     maDummyModules[ rName ] = nType;
 }
 
@@ -335,8 +335,8 @@ void VbaProject::importVba( StorageBase& rVbaPrjStrg, const GraphicHelper& rGrap
             case VBA_ID_PROJECTNAME:
             {
                 OUString aPrjName = aRecStrm.readCharArrayUC( nRecSize, eTextEnc );
-                OSL_ENSURE( aPrjName.getLength() > 0, "VbaProject::importVba - invalid project 
name" );
-                if( aPrjName.getLength() > 0 )
+                OSL_ENSURE( !aPrjName.isEmpty(), "VbaProject::importVba - invalid project name" );
+                if( !aPrjName.isEmpty() )
                     maPrjName = aPrjName;
             }
             break;
@@ -348,7 +348,7 @@ void VbaProject::importVba( StorageBase& rVbaPrjStrg, const GraphicHelper& rGrap
             case VBA_ID_MODULENAME:
             {
                 OUString aName = aRecStrm.readCharArrayUC( nRecSize, eTextEnc );
-                OSL_ENSURE( aName.getLength() > 0, "VbaProject::importVba - invalid module name" );
+                OSL_ENSURE( !aName.isEmpty(), "VbaProject::importVba - invalid module name" );
                 OSL_ENSURE( !aModules.has( aName ), "VbaProject::importVba - multiple modules with 
the same name" );
                 VbaModuleMap::mapped_type& rxModule = aModules[ aName ];
                 rxModule.reset( new VbaModule( mxContext, mxDocModel, aName, eTextEnc, bExecutable 
) );
@@ -416,7 +416,7 @@ void VbaProject::importVba( StorageBase& rVbaPrjStrg, const GraphicHelper& rGrap
                 else if( aKey.equalsIgnoreAsciiCaseAsciiL( RTL_CONSTASCII_STRINGPARAM( "BaseClass" 
) ) )
                     nType = ModuleType::FORM;
 
-                if( (nType != ModuleType::UNKNOWN) && (aValue.getLength() > 0) )
+                if( (nType != ModuleType::UNKNOWN) && !aValue.isEmpty() )
                 {
                     OSL_ENSURE( aModules.has( aValue ), "VbaProject::importVba - module not found" 
);
                     if( VbaModule* pModule = aModules.get( aValue ).get() )
diff --git a/oox/source/ppt/pptgraphicshapecontext.cxx b/oox/source/ppt/pptgraphicshapecontext.cxx
index dd5820a..11a9a27 100644
--- a/oox/source/ppt/pptgraphicshapecontext.cxx
+++ b/oox/source/ppt/pptgraphicshapecontext.cxx
@@ -128,7 +128,7 @@ Reference< XFastContextHandler > PPTGraphicShapeContext::createFastChildContext(
         sal_Int32 nSubType( xAttribs->getOptionalValueToken( XML_type, XML_obj ) );
         mpShapePtr->setSubType( nSubType );
         OUString sIdx( xAttribs->getOptionalValue( XML_idx ) );
-        sal_Bool bHasIdx = sIdx.getLength() > 0;
+        sal_Bool bHasIdx = !sIdx.isEmpty();
         sal_Int32 nIdx = sIdx.toInt32();
         if( xAttribs->hasAttribute( XML_idx ) )
             mpShapePtr->setSubTypeIndex( nIdx );
diff --git a/oox/source/ppt/pptimport.cxx b/oox/source/ppt/pptimport.cxx
index e542e53..e64f89b 100644
--- a/oox/source/ppt/pptimport.cxx
+++ b/oox/source/ppt/pptimport.cxx
@@ -174,7 +174,7 @@ sal_Bool SAL_CALL PowerPointImport::filter( const Sequence< PropertyValue >& rDe
 
 const oox::drawingml::table::TableStyleListPtr PowerPointImport::getTableStyles()
 {
-    if ( !mpTableStyleList && maTableStyleListPath.getLength() )
+    if ( !mpTableStyleList && !maTableStyleListPath.isEmpty() )
     {
         mpTableStyleList = oox::drawingml::table::TableStyleListPtr( new 
oox::drawingml::table::TableStyleList() );
         importFragment( new oox::drawingml::table::TableStyleListFragmentHandler(
diff --git a/oox/source/ppt/pptshape.cxx b/oox/source/ppt/pptshape.cxx
index 8ea04c4..dfb4a38 100644
--- a/oox/source/ppt/pptshape.cxx
+++ b/oox/source/ppt/pptshape.cxx
@@ -133,7 +133,7 @@ void PPTShape::addShape(
     try
     {
         rtl::OUString sServiceName( msServiceName );
-        if( sServiceName.getLength() )
+        if( !sServiceName.isEmpty() )
         {
             oox::drawingml::TextListStylePtr aMasterTextListStyle;
             Reference< lang::XMultiServiceFactory > xServiceFact( rFilterBase.getModel(), 
UNO_QUERY_THROW );
@@ -274,7 +274,7 @@ void PPTShape::addShape(
                 OSL_TRACE("placeholder id: %s", pPlaceholder.get() ? 
rtl::OUStringToOString(pPlaceholder->getId(), RTL_TEXTENCODING_UTF8 ).getStr() : "not found");
             }
 
-            if ( sServiceName.getLength() )
+            if ( !sServiceName.isEmpty() )
             {
             // use style from master slide for placeholders only, otherwise use slide's style, 
which might be the default style from presentation
                 if ( !aMasterTextListStyle.get() )
@@ -301,7 +301,7 @@ void PPTShape::addShape(
                         rtl::OUString aTitleText;
                         Reference< XTextRange > xText( xShape, UNO_QUERY_THROW );
                         aTitleText = xText->getString();
-                        if ( aTitleText.getLength() && ( aTitleText.getLength() < 64 ) )    // 
just a magic value, but we don't want to set slide names which are too long
+                        if ( !aTitleText.isEmpty() && ( aTitleText.getLength() < 64 ) )    // just 
a magic value, but we don't want to set slide names which are too long
                         {
                             Reference< container::XNamed > xName( rSlidePersist.getPage(), 
UNO_QUERY_THROW );
                             xName->setName( aTitleText );
@@ -312,7 +312,7 @@ void PPTShape::addShape(
 
                     }
                 }
-                if( pShapeMap && msId.getLength() )
+                if( pShapeMap && !msId.isEmpty() )
                 {
                     (*pShapeMap)[ msId ] = shared_from_this();
                 }
diff --git a/oox/source/ppt/presentationfragmenthandler.cxx 
b/oox/source/ppt/presentationfragmenthandler.cxx
index 63d5f76..cac49d8 100644
--- a/oox/source/ppt/presentationfragmenthandler.cxx
+++ b/oox/source/ppt/presentationfragmenthandler.cxx
@@ -170,7 +170,7 @@ void PresentationFragmentHandler::finalizeImport()
                 xSlide = xDrawPages->insertNewByIndex( nSlide );
 
             OUString aSlideFragmentPath = getFragmentPathFromRelId( maSlidesVector[ nSlide ] );
-            if( aSlideFragmentPath.getLength() > 0 )
+            if( !aSlideFragmentPath.isEmpty() )
             {
                 SlidePersistPtr pMasterPersistPtr;
                 SlidePersistPtr pSlidePersistPtr( new SlidePersist( rFilter, sal_False, sal_False, 
xSlide,
@@ -180,12 +180,12 @@ void PresentationFragmentHandler::finalizeImport()
 
                 // importing the corresponding masterpage/layout
                 OUString aLayoutFragmentPath = 
xSlideFragmentHandler->getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( "slideLayout" 
) );
-                if ( aLayoutFragmentPath.getLength() > 0 )
+                if ( !aLayoutFragmentPath.isEmpty() )
                 {
                     // importing layout
                     RelationsRef xLayoutRelations = rFilter.importRelations( aLayoutFragmentPath );
                     OUString aMasterFragmentPath = xLayoutRelations->getFragmentPathFromFirstType( 
CREATE_OFFICEDOC_RELATION_TYPE( "slideMaster" ) );
-                    if( aMasterFragmentPath.getLength() )
+                    if( !aMasterFragmentPath.isEmpty() )
                     {
                         // check if the corresponding masterpage+layout has already been imported
                         std::vector< SlidePersistPtr >& rMasterPages( rFilter.getMasterPages() );
@@ -219,7 +219,7 @@ void PresentationFragmentHandler::finalizeImport()
 
                             // set the correct theme
                             OUString aThemeFragmentPath = 
xMasterFragmentHandler->getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( "theme" ) );
-                            if( aThemeFragmentPath.getLength() > 0 )
+                            if( !aThemeFragmentPath.isEmpty() )
                             {
                                 std::map< OUString, oox::drawingml::ThemePtr >& rThemes( 
rFilter.getThemes() );
                                 std::map< OUString, oox::drawingml::ThemePtr >::iterator aIter2( 
rThemes.find( aThemeFragmentPath ) );
@@ -268,7 +268,7 @@ void PresentationFragmentHandler::finalizeImport()
 
                 // now importing the notes page
                 OUString aNotesFragmentPath = xSlideFragmentHandler->getFragmentPathFromFirstType( 
CREATE_OFFICEDOC_RELATION_TYPE( "notesSlide" ) );
-                if( aNotesFragmentPath.getLength() > 0 )
+                if( !aNotesFragmentPath.isEmpty() )
                 {
                     Reference< XPresentationPage > xPresentationPage( xSlide, UNO_QUERY );
                     if ( xPresentationPage.is() )
diff --git a/oox/source/ppt/slidefragmenthandler.cxx b/oox/source/ppt/slidefragmenthandler.cxx
index 4652690..a27e5e3 100644
--- a/oox/source/ppt/slidefragmenthandler.cxx
+++ b/oox/source/ppt/slidefragmenthandler.cxx
@@ -64,7 +64,7 @@ SlideFragmentHandler::SlideFragmentHandler( XmlFilterBase& rFilter, const OUStri
 , meShapeLocation( eShapeLocation )
 {
     OUString aVMLDrawingFragmentPath = getFragmentPathFromFirstType( 
CREATE_OFFICEDOC_RELATION_TYPE( "vmlDrawing" ) );
-    if( aVMLDrawingFragmentPath.getLength() > 0 )
+    if( !aVMLDrawingFragmentPath.isEmpty() )
         getFilter().importFragment( new oox::vml::DrawingFragment(
             getFilter(), aVMLDrawingFragmentPath, *pPersistPtr->getDrawing() ) );
 }
@@ -177,7 +177,7 @@ void SlideFragmentHandler::finalizeImport()
         Reference< XDrawPage > xSlide( mpSlidePersistPtr->getPage() );
         PropertySet aSlideProp( xSlide );
         aSlideProp.setProperties( maSlideProperties );
-        if ( maSlideName.getLength() )
+        if ( !maSlideName.isEmpty() )
         {
             Reference< XNamed > xNamed( xSlide, UNO_QUERY );
             if( xNamed.is() )
diff --git a/oox/source/ppt/soundactioncontext.cxx b/oox/source/ppt/soundactioncontext.cxx
index 7b510bed..72e8165 100644
--- a/oox/source/ppt/soundactioncontext.cxx
+++ b/oox/source/ppt/soundactioncontext.cxx
@@ -67,7 +67,7 @@ namespace oox { namespace ppt {
             {
                 OUString url;
                 // TODO this is very wrong
-                if ( msSndName.getLength() != 0 )
+                if ( !msSndName.isEmpty() )
                 {
                     // try the builtIn version
                     url = msSndName;
@@ -83,7 +83,7 @@ namespace oox { namespace ppt {
                     url = msLink;
                 }
 #endif
-                if ( url.getLength() != 0 )
+                if ( !url.isEmpty() )
                 {
                     maSlideProperties[ PROP_Sound ] <<= url;
                     maSlideProperties[ PROP_SoundOn ] <<= sal_True;
diff --git a/oox/source/ppt/timenode.cxx b/oox/source/ppt/timenode.cxx
index fc28f72..776e2bf 100644
--- a/oox/source/ppt/timenode.cxx
+++ b/oox/source/ppt/timenode.cxx
@@ -232,7 +232,7 @@ namespace oox { namespace ppt {
         OSL_ENSURE( xNode.is(), "null node passed" );
 
         try {
-            if( msId.getLength() )
+            if( !msId.isEmpty() )
             {
                 pSlide->getAnimNodesMap()[ msId ] = xNode;
             }
diff --git a/oox/source/ppt/timenodelistcontext.cxx b/oox/source/ppt/timenodelistcontext.cxx
index ab250d9..fa312d5 100644
--- a/oox/source/ppt/timenodelistcontext.cxx
+++ b/oox/source/ppt/timenodelistcontext.cxx
@@ -557,17 +557,17 @@ namespace oox { namespace ppt {
                 }
                 OUString aStr;
                 aStr = xAttribs->getOptionalValue( XML_from );
-                if( aStr.getLength() )
+                if( !aStr.isEmpty() )
                 {
                     pNode->setFrom( makeAny( aStr ) );
                 }
                 aStr = xAttribs->getOptionalValue( XML_by );
-                if( aStr.getLength() )
+                if( !aStr.isEmpty() )
                 {
                     pNode->setBy( makeAny( aStr ) );
                 }
                 aStr = xAttribs->getOptionalValue( XML_to );
-                if( aStr.getLength() )
+                if( !aStr.isEmpty() )
                 {
                     pNode->setTo( makeAny( aStr ) );
                 }
@@ -596,7 +596,7 @@ namespace oox { namespace ppt {
 
                         OUString aTest;
                         iter->maValue >>= aTest;
-                        if( aTest.getLength() != 0 )
+                        if( !aTest.isEmpty() )
                         {
                             aValues[i] = iter->maValue;
                         }
@@ -886,7 +886,7 @@ namespace oox { namespace ppt {
                 // TODO
 //              OUString sPrList = xAttribs->getOptionalValue( XML_prLst );
 
-                if( sFilter.getLength() )
+                if( !sFilter.isEmpty() )
                 {
                     SlideTransition aFilter( sFilter );
                     aFilter.setMode( nDir == XML_out ? false : true );
diff --git a/oox/source/ppt/timetargetelementcontext.cxx 
b/oox/source/ppt/timetargetelementcontext.cxx
index 93cbcf7..2418430d 100644
--- a/oox/source/ppt/timetargetelementcontext.cxx
+++ b/oox/source/ppt/timetargetelementcontext.cxx
@@ -121,7 +121,7 @@ namespace oox { namespace ppt {
         {
             mpTarget->mnType = XML_inkTgt;
             OUString aId = rAttribs.getString( XML_spid, OUString() );
-            if( aId.getLength() )
+            if( !aId.isEmpty() )
             {
                 mpTarget->msValue = aId;
             }
diff --git a/oox/source/vml/vmldrawing.cxx b/oox/source/vml/vmldrawing.cxx
index 6483f20..8c6207f 100644
--- a/oox/source/vml/vmldrawing.cxx
+++ b/oox/source/vml/vmldrawing.cxx
@@ -132,15 +132,15 @@ void Drawing::registerBlockId( sal_Int32 nBlockId )
 
 void Drawing::registerOleObject( const OleObjectInfo& rOleObject )
 {
-    OSL_ENSURE( rOleObject.maShapeId.getLength() > 0, "Drawing::registerOleObject - missing OLE 
object shape id" );
+    OSL_ENSURE( !rOleObject.maShapeId.isEmpty(), "Drawing::registerOleObject - missing OLE object 
shape id" );
     OSL_ENSURE( maOleObjects.count( rOleObject.maShapeId ) == 0, "Drawing::registerOleObject - OLE 
object already registered" );
     maOleObjects.insert( OleObjectInfoMap::value_type( rOleObject.maShapeId, rOleObject ) );
 }
 
 void Drawing::registerControl( const ControlInfo& rControl )
 {
-    OSL_ENSURE( rControl.maShapeId.getLength() > 0, "Drawing::registerControl - missing form 
control shape id" );
-    OSL_ENSURE( rControl.maName.getLength() > 0, "Drawing::registerControl - missing form control 
name" );
+    OSL_ENSURE( !rControl.maShapeId.isEmpty(), "Drawing::registerControl - missing form control 
shape id" );
+    OSL_ENSURE( !rControl.maName.isEmpty(), "Drawing::registerControl - missing form control name" 
);
     OSL_ENSURE( maControls.count( rControl.maShapeId ) == 0, "Drawing::registerControl - form 
control already registered" );
     maControls.insert( ControlInfoMap::value_type( rControl.maShapeId, rControl ) );
 }
@@ -210,10 +210,10 @@ const ControlInfo* Drawing::getControlInfo( const OUString& rShapeId ) const
 Reference< XShape > Drawing::createAndInsertXShape( const OUString& rService,
         const Reference< XShapes >& rxShapes, const Rectangle& rShapeRect ) const
 {
-    OSL_ENSURE( rService.getLength() > 0, "Drawing::createAndInsertXShape - missing UNO shape 
service name" );
+    OSL_ENSURE( !rService.isEmpty(), "Drawing::createAndInsertXShape - missing UNO shape service 
name" );
     OSL_ENSURE( rxShapes.is(), "Drawing::createAndInsertXShape - missing XShapes container" );
     Reference< XShape > xShape;
-    if( (rService.getLength() > 0) && rxShapes.is() ) try
+    if( !rService.isEmpty() && rxShapes.is() ) try
     {
         Reference< XMultiServiceFactory > xModelFactory( mrFilter.getModelFactory(), UNO_SET_THROW 
);
         xShape.set( xModelFactory->createInstance( rService ), UNO_QUERY_THROW );
diff --git a/oox/source/vml/vmlformatting.cxx b/oox/source/vml/vmlformatting.cxx
index 9faeb04..1197829 100644
--- a/oox/source/vml/vmlformatting.cxx
+++ b/oox/source/vml/vmlformatting.cxx
@@ -81,7 +81,7 @@ bool lclExtractDouble( double& orfValue, sal_Int32& ornEndPos, const OUString& r
     {
         orValue1 = rValue.trim();
     }
-    return (orValue1.getLength() > 0) && (orValue2.getLength() > 0);
+    return !orValue1.isEmpty() && !orValue2.isEmpty();
 }
 
 /*static*/ bool ConversionHelper::decodeBool( const OUString& rValue )
@@ -93,7 +93,7 @@ bool lclExtractDouble( double& orfValue, sal_Int32& ornEndPos, const OUString& r
 
 /*static*/ double ConversionHelper::decodePercent( const OUString& rValue, double fDefValue )
 {
-    if( rValue.getLength() == 0 )
+    if( rValue.isEmpty() )
         return fDefValue;
 
     double fValue = 0.0;
@@ -115,7 +115,7 @@ bool lclExtractDouble( double& orfValue, sal_Int32& ornEndPos, const OUString& r
         const OUString& rValue, sal_Int32 nRefValue, bool bPixelX, bool bDefaultAsPixel )
 {
     // default for missing values is 0
-    if( rValue.getLength() == 0 )
+    if( rValue.isEmpty() )
         return 0;
 
     // TODO: according to spec, value may contain "auto"
@@ -164,7 +164,7 @@ bool lclExtractDouble( double& orfValue, sal_Int32& ornEndPos, const OUString& r
     {
         fValue *= nRefValue / 100.0;
     }
-    else if( bDefaultAsPixel || (aUnit.getLength() > 0) )   // default as EMU and no unit -> do 
nothing
+    else if( bDefaultAsPixel || !aUnit.isEmpty() )   // default as EMU and no unit -> do nothing
     {
         OSL_FAIL( "ConversionHelper::decodeMeasureToEmu - unknown measure unit" );
         fValue = nRefValue;
@@ -552,7 +552,7 @@ void FillModel::pushToPropMap( ShapePropertyMap& rPropMap, const GraphicHelper&
             case XML_tile:
             case XML_frame:
             {
-                if( moBitmapPath.has() && moBitmapPath.get().getLength() > 0 )
+                if( moBitmapPath.has() && !moBitmapPath.get().isEmpty() )
                 {
                     aFillProps.maBlipProps.mxGraphic = rGraphicHelper.importEmbeddedGraphic( 
moBitmapPath.get() );
                     if( aFillProps.maBlipProps.mxGraphic.is() )
diff --git a/oox/source/vml/vmlinputstream.cxx b/oox/source/vml/vmlinputstream.cxx
index bec1654..80d9030 100644
--- a/oox/source/vml/vmlinputstream.cxx
+++ b/oox/source/vml/vmlinputstream.cxx
@@ -392,7 +392,7 @@ OString InputStream::readToElementBegin() throw (IOException, RuntimeException)
 OString InputStream::readToElementEnd() throw (IOException, RuntimeException)
 {
     OString aText = OUStringToOString( mxTextStrm->readString( maClosingBracket, sal_False ), 
RTL_TEXTENCODING_ISO_8859_1 );
-    OSL_ENSURE( (aText.getLength() > 0) && (aText[ aText.getLength() - 1 ] == '>'), 
"InputStream::readToElementEnd - missing closing bracket of XML element" );
+    OSL_ENSURE( !aText.isEmpty() && (aText[ aText.getLength() - 1 ] == '>'), 
"InputStream::readToElementEnd - missing closing bracket of XML element" );
     return aText;
 }
 
diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index 10bc407..c2fbd65 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -258,11 +258,11 @@ void ShapeBase::finalizeFragmentImport()
 
 OUString ShapeBase::getShapeName() const
 {
-    if( maTypeModel.maShapeName.getLength() > 0 )
+    if( !maTypeModel.maShapeName.isEmpty() )
         return maTypeModel.maShapeName;
 
     OUString aBaseName = mrDrawing.getShapeBaseName( *this );
-    if( aBaseName.getLength() > 0 )
+    if( !aBaseName.isEmpty() )
     {
         sal_Int32 nShapeIdx = mrDrawing.getLocalShapeIndex( getShapeId() );
         if( nShapeIdx > 0 )
@@ -470,7 +470,7 @@ Reference< XShape > ComplexShape::implConvertAndInsert( const Reference< XShapes
             if( xShape.is() )
             {
                 // set the replacement graphic
-                if( aGraphicPath.getLength() > 0 )
+                if( !aGraphicPath.isEmpty() )
                 {
                     Reference< XGraphic > xGraphic = 
rFilter.getGraphicHelper().importEmbeddedGraphic( aGraphicPath );
                     if( xGraphic.is() )
@@ -487,11 +487,11 @@ Reference< XShape > ComplexShape::implConvertAndInsert( const Reference< 
XShapes
 
     // try to find registered form control info
     const ControlInfo* pControlInfo = mrDrawing.getControlInfo( maTypeModel.maShapeId );
-    if( pControlInfo && (pControlInfo->maFragmentPath.getLength() > 0) )
+    if( pControlInfo && !pControlInfo->maFragmentPath.isEmpty() )
     {
         OSL_ENSURE( nShapeType == VML_SHAPETYPE_HOSTCONTROL, "ComplexShape::implConvertAndInsert - 
unexpected shape type" );
         OUString aShapeName = getShapeName();
-        if( aShapeName.getLength() > 0 )
+        if( !aShapeName.isEmpty() )
         {
             OSL_ENSURE( aShapeName == pControlInfo->maName, "ComplexShape::implConvertAndInsert - 
control name mismatch" );
             // load the control properties from fragment
@@ -518,13 +518,13 @@ Reference< XShape > ComplexShape::implConvertAndInsert( const Reference< 
XShapes
     }
 
     // try to create a picture object
-    if( aGraphicPath.getLength() > 0 )
+    if( !aGraphicPath.isEmpty() )
     {
         Reference< XShape > xShape = mrDrawing.createAndInsertXShape( CREATE_OUSTRING( 
"com.sun.star.drawing.GraphicObjectShape" ), rxShapes, rShapeRect );
         if( xShape.is() )
         {
             OUString aGraphicUrl = rFilter.getGraphicHelper().importEmbeddedGraphicObject( 
aGraphicPath );
-            if( aGraphicUrl.getLength() > 0 )
+            if( !aGraphicUrl.isEmpty() )
             {
                 PropertySet aPropSet( xShape );
                 aPropSet.setProperty( PROP_GraphicURL, aGraphicUrl );
diff --git a/oox/source/vml/vmlshapecontainer.cxx b/oox/source/vml/vmlshapecontainer.cxx
index e8a8d02..0309839 100644
--- a/oox/source/vml/vmlshapecontainer.cxx
+++ b/oox/source/vml/vmlshapecontainer.cxx
@@ -52,8 +52,8 @@ void lclMapShapesById( RefMap< OUString, ShapeType >& orMap, const RefVector< Sh
     for( typename RefVector< ShapeType >::const_iterator aIt = rVector.begin(), aEnd = 
rVector.end(); aIt != aEnd; ++aIt )
     {
         const OUString& rShapeId = (*aIt)->getShapeId();
-        OSL_ENSURE( rShapeId.getLength() > 0, "lclMapShapesById - missing shape identifier" );
-        if( rShapeId.getLength() > 0 )
+        OSL_ENSURE( !rShapeId.isEmpty(), "lclMapShapesById - missing shape identifier" );
+        if( !rShapeId.isEmpty() )
         {
             OSL_ENSURE( orMap.find( rShapeId ) == orMap.end(), "lclMapShapesById - shape 
identifier already used" );
             orMap[ rShapeId ] = *aIt;
diff --git a/oox/source/vml/vmlshapecontext.cxx b/oox/source/vml/vmlshapecontext.cxx
index 6980ce3..e0e38ce 100644
--- a/oox/source/vml/vmlshapecontext.cxx
+++ b/oox/source/vml/vmlshapecontext.cxx
@@ -106,7 +106,7 @@ OptValue< DoublePair > lclDecodePercentPair( const AttributeList& rAttribs, sal_
  */
 bool lclDecodeVmlxBool( const OUString& rValue, bool bDefaultForEmpty )
 {
-    if( rValue.getLength() == 0 ) return bDefaultForEmpty;
+    if( rValue.isEmpty() ) return bDefaultForEmpty;
     sal_Int32 nToken = AttributeConversion::decodeToken( rValue );
     // anything else than 't' or 'True' is considered to be false, as specified
     return (nToken == XML_t) || (nToken == XML_True);
@@ -134,7 +134,7 @@ ContextHandlerRef ShapeLayoutContext::onCreateContext( sal_Int32 nElement, const
             while( nIndex >= 0 )
             {
                 OUString aToken = aBlockIds.getToken( 0, ' ', nIndex ).trim();
-                if( aToken.getLength() > 0 )
+                if( !aToken.isEmpty() )
                     mrDrawing.registerBlockId( aToken.toInt32() );
             }
         }
@@ -255,7 +255,7 @@ ShapeTypeContext::ShapeTypeContext( ContextHandler2Helper& rParent, ShapeType& r
     // shape identifier and shape name
     bool bHasOspid = rAttribs.hasAttribute( O_TOKEN( spid ) );
     mrTypeModel.maShapeId = rAttribs.getXString( bHasOspid ? O_TOKEN( spid ) : XML_id, OUString() 
);
-    OSL_ENSURE( mrTypeModel.maShapeId.getLength() > 0, "ShapeTypeContext::ShapeTypeContext - 
missing shape identifier" );
+    OSL_ENSURE( !mrTypeModel.maShapeId.isEmpty(), "ShapeTypeContext::ShapeTypeContext - missing 
shape identifier" );
     // if the o:spid attribute exists, the id attribute contains the user-defined shape name
     if( bHasOspid )
         mrTypeModel.maShapeName = rAttribs.getXString( XML_id, OUString() );
diff --git a/oox/source/xls/addressconverter.cxx b/oox/source/xls/addressconverter.cxx
index fd7217a..8a5a670 100644
--- a/oox/source/xls/addressconverter.cxx
+++ b/oox/source/xls/addressconverter.cxx
@@ -729,7 +729,7 @@ void AddressConverter::convertToCellRangeList( ApiCellRangeList& orRanges,
     while( (0 <= nPos) && (nPos < nLen) )
     {
         OUString aToken = rString.getToken( 0, ' ', nPos );
-        if( (aToken.getLength() > 0) && convertToCellRange( aRange, aToken, nSheet, true, 
bTrackOverflow ) )
+        if( !aToken.isEmpty() && convertToCellRange( aRange, aToken, nSheet, true, bTrackOverflow 
) )
             orRanges.push_back( aRange );
     }
 }
diff --git a/oox/source/xls/autofilterbuffer.cxx b/oox/source/xls/autofilterbuffer.cxx
index 7a3e6fa..ec92a90 100644
--- a/oox/source/xls/autofilterbuffer.cxx
+++ b/oox/source/xls/autofilterbuffer.cxx
@@ -129,7 +129,7 @@ bool lclTrimTrailingAsterisks( OUString& rValue )
 bool lclConvertWildcardsToRegExp( OUString& rValue )
 {
     // check existence of the wildcard characters '*' and '?'
-    if( (rValue.getLength() > 0) && ((rValue.indexOf( '*' ) >= 0) || (rValue.indexOf( '?' ) >= 0)) 
)
+    if( !rValue.isEmpty() && ((rValue.indexOf( '*' ) >= 0) || (rValue.indexOf( '?' ) >= 0)) )
     {
         OUStringBuffer aBuffer;
         aBuffer.ensureCapacity( rValue.getLength() + 5 );
@@ -249,7 +249,7 @@ void DiscreteFilter::importAttribs( sal_Int32 nElement, const AttributeList& rAt
         case XLS_TOKEN( filter ):
         {
             OUString aValue = rAttribs.getXString( XML_val, OUString() );
-            if( aValue.getLength() > 0 )
+            if( !aValue.isEmpty() )
                 maValues.push_back( aValue );
         }
         break;
@@ -276,7 +276,7 @@ void DiscreteFilter::importRecord( sal_Int32 nRecId, SequenceInputStream& rStrm
         case BIFF12_ID_DISCRETEFILTER:
         {
             OUString aValue = BiffHelper::readString( rStrm );
-            if( aValue.getLength() > 0 )
+            if( !aValue.isEmpty() )
                 maValues.push_back( aValue );
         }
         break;
@@ -384,7 +384,7 @@ void FilterCriterionModel::readBiffData( SequenceInputStream& rStrm )
         {
             rStrm.skip( 8 );
             OUString aValue = BiffHelper::readString( rStrm ).trim();
-            if( aValue.getLength() > 0 )
+            if( !aValue.isEmpty() )
                 maValue <<= aValue;
         }
         break;
@@ -468,7 +468,7 @@ void FilterCriterionModel::readString( BiffInputStream& rStrm, BiffType eBiff, r
             rStrm.readUniStringBody( mnStrLen, true ) :
             rStrm.readCharArrayUC( mnStrLen, eTextEnc, true );
         aValue = aValue.trim();
-        if( aValue.getLength() > 0 )
+        if( !aValue.isEmpty() )
             maValue <<= aValue;
     }
 }
@@ -494,7 +494,7 @@ void CustomFilter::importAttribs( sal_Int32 nElement, const AttributeList& rAttr
             FilterCriterionModel aCriterion;
             aCriterion.mnOperator = rAttribs.getToken( XML_operator, XML_equal );
             OUString aValue = rAttribs.getXString( XML_val, OUString() ).trim();
-            if( (aCriterion.mnOperator == XML_equal) || (aCriterion.mnOperator == XML_notEqual) || 
(aValue.getLength() > 0) )
+            if( (aCriterion.mnOperator == XML_equal) || (aCriterion.mnOperator == XML_notEqual) || 
(!aValue.isEmpty()) )
                 aCriterion.maValue <<= aValue;
             appendCriterion( aCriterion );
         }
@@ -554,7 +554,7 @@ ApiFilterSettings CustomFilter::finalizeImport( sal_Int32 /*nMaxCount*/ )
                 bool bNotEqual = nOperator == FilterOperator2::NOT_EQUAL;
                 if( bEqual || bNotEqual )
                 {
-                    if( aValue.getLength() == 0 )
+                    if( aValue.isEmpty() )
                     {
                         // empty comparison string: create empty/not empty filters
                         nOperator = bNotEqual ? FilterOperator2::NOT_EMPTY : 
FilterOperator2::EMPTY;
@@ -565,7 +565,7 @@ ApiFilterSettings CustomFilter::finalizeImport( sal_Int32 /*nMaxCount*/ )
                         bool bHasLeadingAsterisk = lclTrimLeadingAsterisks( aValue );
                         bool bHasTrailingAsterisk = lclTrimTrailingAsterisks( aValue );
                         // just '***' matches everything, do not create a filter field
-                        bValidOperator = aValue.getLength() > 0;
+                        bValidOperator = !aValue.isEmpty();
                         if( bValidOperator )
                         {
                             if( bHasLeadingAsterisk && bHasTrailingAsterisk )
diff --git a/oox/source/xls/condformatbuffer.cxx b/oox/source/xls/condformatbuffer.cxx
index 26355dc..c2341dc 100644
--- a/oox/source/xls/condformatbuffer.cxx
+++ b/oox/source/xls/condformatbuffer.cxx
@@ -542,7 +542,7 @@ void CondFormatRule::finalizeImport( const Reference< XSheetConditionalEntries >
         break;
     }
 
-    if( aReplaceFormula.getLength() > 0 )
+    if( !aReplaceFormula.isEmpty() )
     {
         OUString aAddress, aRanges, aText, aComp;
         sal_Int32 nStrPos = aReplaceFormula.getLength();
@@ -551,17 +551,17 @@ void CondFormatRule::finalizeImport( const Reference< 
XSheetConditionalEntries >
             switch( aReplaceFormula[ nStrPos + 1 ] )
             {
                 case 'B':       // current base address
-                    if( aAddress.getLength() == 0 )
+                    if( aAddress.isEmpty() )
                         aAddress = FormulaProcessorBase::generateAddress2dString( 
mrCondFormat.getRanges().getBaseAddress(), false );
                     aReplaceFormula = aReplaceFormula.replaceAt( nStrPos, 2, aAddress );
                 break;
                 case 'R':       // range list of conditional formatting
-                    if( aRanges.getLength() == 0 )
+                    if( aRanges.isEmpty() )
                         aRanges = FormulaProcessorBase::generateRangeList2dString( 
mrCondFormat.getRanges(), true, ',', true );
                     aReplaceFormula = aReplaceFormula.replaceAt( nStrPos, 2, aRanges );
                 break;
                 case 'T':       // comparison text
-                    if( aText.getLength() == 0 )
+                    if( aText.isEmpty() )
                         // quote the comparison text, and handle embedded quote characters
                         aText = FormulaProcessorBase::generateApiString( maModel.maText );
                     aReplaceFormula = aReplaceFormula.replaceAt( nStrPos, 2, aText );
@@ -579,7 +579,7 @@ void CondFormatRule::finalizeImport( const Reference< XSheetConditionalEntries >
                         OUString::valueOf( static_cast< sal_Int32 >( maModel.mbBottom ? 1 : 0 ) ) 
);
                 break;
                 case 'C':       // average comparison operator
-                    if( aComp.getLength() == 0 )
+                    if( aComp.isEmpty() )
                         aComp = maModel.mbAboveAverage ?
                             (maModel.mbEqualAverage ? CREATE_OUSTRING( ">=" ) : CREATE_OUSTRING( 
">" )) :
                             (maModel.mbEqualAverage ? CREATE_OUSTRING( "<=" ) : CREATE_OUSTRING( 
"<" ));
@@ -608,7 +608,7 @@ void CondFormatRule::finalizeImport( const Reference< XSheetConditionalEntries >
 
         // style name for the formatting attributes
         OUString aStyleName = getStyles().createDxfStyle( maModel.mnDxfId );
-        if( aStyleName.getLength() > 0 )
+        if( !aStyleName.isEmpty() )
             lclAppendProperty( aProps, CREATE_OUSTRING( "StyleName" ), aStyleName );
 
         // append the new rule
diff --git a/oox/source/xls/connectionsbuffer.cxx b/oox/source/xls/connectionsbuffer.cxx
index e066613..51b7970 100644
--- a/oox/source/xls/connectionsbuffer.cxx
+++ b/oox/source/xls/connectionsbuffer.cxx
@@ -124,7 +124,7 @@ void lclParseTables( WebPrModel::TablesVector& rTables, const OUString& rTableNa
 {
     rTables.clear();
     OUString aTableNames = rTableNames.trim();
-    while( aTableNames.getLength() > 0 )
+    while( !aTableNames.isEmpty() )
     {
         sal_Int32 nSep = -1;
         // table names are enclosed in double quotes
@@ -139,7 +139,7 @@ void lclParseTables( WebPrModel::TablesVector& rTables, const OUString& rTableNa
                 nSep = aTableNames.indexOf( ',', nEndQuote + 1 );
             // extract text between quote characters
             OUString aTableName = aTableNames.copy( 1, nEndQuote - 1 ).trim();
-            if( aTableName.getLength() > 0 )
+            if( !aTableName.isEmpty() )
                 rTables.push_back( Any( aTableName ) );
             else
                 rTables.push_back( Any() );
@@ -150,7 +150,7 @@ void lclParseTables( WebPrModel::TablesVector& rTables, const OUString& rTableNa
             if( nSep < 0 )
                 nSep = aTableNames.getLength();
             OUString aTableIndex = aTableNames.copy( 0, nSep ).trim();
-            if( (aTableIndex.getLength() > 0) && (aTableIndex[ 0 ] >= '1') && (aTableIndex[ 0 ] <= 
'9') )
+            if( !aTableIndex.isEmpty() && (aTableIndex[ 0 ] >= '1') && (aTableIndex[ 0 ] <= '9') )
                 rTables.push_back( Any( aTableIndex.toInt32() ) );
             else
                 rTables.push_back( Any() );
diff --git a/oox/source/xls/defnamesbuffer.cxx b/oox/source/xls/defnamesbuffer.cxx
index e2319ca..17418d6 100644
--- a/oox/source/xls/defnamesbuffer.cxx
+++ b/oox/source/xls/defnamesbuffer.cxx
@@ -244,7 +244,7 @@ DefinedNameBase::DefinedNameBase( const WorkbookHelper& rHelper ) :
 
 const OUString& DefinedNameBase::getUpcaseModelName() const
 {
-    if( maUpModelName.getLength() == 0 )
+    if( maUpModelName.isEmpty() )
         maUpModelName = lclGetUpcaseModelName( maModel.maName );
     return maUpModelName;
 }
@@ -279,7 +279,7 @@ Any DefinedNameBase::getReference( const CellAddress& rBaseAddr ) const
 
 ApiTokenSequence DefinedNameBase::importOoxFormula( sal_Int16 nBaseSheet )
 {
-    return (maModel.maFormula.getLength() > 0) ?
+    return (!maModel.maFormula.isEmpty()) ?
         getFormulaParser().importFormula( CellAddress( nBaseSheet, 0, 0 ), maModel.maFormula ) :
         getFormulaParser().convertErrorToFormula( BIFF_ERR_NAME );
 }
diff --git a/oox/source/xls/drawingfragment.cxx b/oox/source/xls/drawingfragment.cxx
index 98f5e28..75cf0c8 100644
--- a/oox/source/xls/drawingfragment.cxx
+++ b/oox/source/xls/drawingfragment.cxx
@@ -104,13 +104,13 @@ Shape::Shape( const WorksheetHelper& rHelper, const AttributeList& rAttribs, 
con
     WorksheetHelper( rHelper )
 {
     OUString aMacro = rAttribs.getXString( XML_macro, OUString() );
-    if( aMacro.getLength() > 0 )
+    if( !aMacro.isEmpty() )
         maMacroName = getFormulaParser().importMacroName( aMacro );
 }
 
 void Shape::finalizeXShape( XmlFilterBase& rFilter, const Reference< XShapes >& rxShapes )
 {
-    if( (maMacroName.getLength() > 0) && mxShape.is() )
+    if( !maMacroName.isEmpty() && mxShape.is() )
     {
         VbaMacroAttacherRef xAttacher( new ShapeMacroAttacher( maMacroName, mxShape ) );
         getBaseFilter().getVbaProject().registerMacroAttacher( xAttacher );
@@ -436,7 +436,7 @@ OUString VmlDrawing::getShapeBaseName( const ::oox::vml::ShapeBase& rShape ) con
 
 bool VmlDrawing::convertClientAnchor( Rectangle& orShapeRect, const OUString& rShapeAnchor ) const
 {
-    if( rShapeAnchor.getLength() == 0 )
+    if( rShapeAnchor.isEmpty() )
         return false;
     ShapeAnchor aAnchor( *this );
     aAnchor.importVmlAnchor( rShapeAnchor );
@@ -450,7 +450,7 @@ Reference< XShape > VmlDrawing::createAndInsertClientXShape( const ::oox::vml::S
     // simulate the legacy drawing controls with OLE form controls
     OUString aShapeName = rShape.getShapeName();
     const ::oox::vml::ClientData* pClientData = rShape.getClientData();
-    if( (aShapeName.getLength() > 0) && pClientData )
+    if( !aShapeName.isEmpty() && pClientData )
     {
         Rectangle aShapeRect = rShapeRect;
         const ::oox::vml::TextBox* pTextBox = rShape.getTextBox();
@@ -603,10 +603,10 @@ Reference< XShape > VmlDrawing::createAndInsertClientXShape( const 
::oox::vml::S
             Reference< XShape > xShape = createAndInsertXControlShape( aControl, rxShapes, 
aShapeRect, nCtrlIndex );
 
             // control shape macro
-            if( xShape.is() && (nCtrlIndex >= 0) && (pClientData->maFmlaMacro.getLength() > 0) )
+            if( xShape.is() && (nCtrlIndex >= 0) && !pClientData->maFmlaMacro.isEmpty() )
             {
                 OUString aMacroName = getFormulaParser().importMacroName( pClientData->maFmlaMacro 
);
-                if( aMacroName.getLength() > 0 )
+                if( !aMacroName.isEmpty() )
                 {
                     Reference< XIndexContainer > xFormIC = getControlForm().getXForm();
                     VbaMacroAttacherRef xAttacher( new VmlControlMacroAttacher( aMacroName, 
xFormIC, nCtrlIndex, pClientData->mnObjType, pClientData->mnDropStyle ) );
@@ -642,7 +642,7 @@ void VmlDrawing::notifyXShapeInserted( const Reference< XShape >& rxShape,
             aPropSet.setProperty( PROP_Printable, pClientData->mbPrintObject );
 
             // control source links
-            if( (pClientData->maFmlaLink.getLength() > 0) || (pClientData->maFmlaRange.getLength() 
0) )
+            if( !pClientData->maFmlaLink.isEmpty() || !pClientData->maFmlaRange.isEmpty() )
                 maControlConv.bindToSources( xCtrlModel, pClientData->maFmlaLink, 
pClientData->maFmlaRange, getSheetIndex() );
         }
         catch( Exception& )
@@ -656,7 +656,7 @@ void VmlDrawing::notifyXShapeInserted( const Reference< XShape >& rxShape,
 sal_uInt32 VmlDrawing::convertControlTextColor( const OUString& rTextColor ) const
 {
     // color attribute not present or 'auto' - use passed default color
-    if( (rTextColor.getLength() == 0) || rTextColor.equalsIgnoreAsciiCaseAsciiL( 
RTL_CONSTASCII_STRINGPARAM( "auto" ) ) )
+    if( rTextColor.isEmpty() || rTextColor.equalsIgnoreAsciiCaseAsciiL( 
RTL_CONSTASCII_STRINGPARAM( "auto" ) ) )
         return AX_SYSCOLOR_WINDOWTEXT;
 
     if( rTextColor[ 0 ] == '#' )
diff --git a/oox/source/xls/drawingmanager.cxx b/oox/source/xls/drawingmanager.cxx
index 93fc190..d8cf3b1 100644
--- a/oox/source/xls/drawingmanager.cxx
+++ b/oox/source/xls/drawingmanager.cxx
@@ -1317,10 +1317,10 @@ void BiffDrawingBase::finalizeImport()
 Reference< XShape > BiffDrawingBase::createAndInsertXShape( const OUString& rService,
         const Reference< XShapes >& rxShapes, const Rectangle& rShapeRect ) const
 {
-    OSL_ENSURE( rService.getLength() > 0, "BiffDrawingBase::createAndInsertXShape - missing UNO 
shape service name" );
+    OSL_ENSURE( !rService.isEmpty(), "BiffDrawingBase::createAndInsertXShape - missing UNO shape 
service name" );
     OSL_ENSURE( rxShapes.is(), "BiffDrawingBase::createAndInsertXShape - missing XShapes 
container" );
     Reference< XShape > xShape;
-    if( (rService.getLength() > 0) && rxShapes.is() ) try
+    if( !rService.isEmpty() && rxShapes.is() ) try
     {
         xShape.set( getBaseFilter().getModelFactory()->createInstance( rService ), UNO_QUERY_THROW 
);
         // insert shape into passed shape collection (maybe drawpage or group shape)
diff --git a/oox/source/xls/excelchartconverter.cxx b/oox/source/xls/excelchartconverter.cxx
index 5c63ee5..da938ea 100644
--- a/oox/source/xls/excelchartconverter.cxx
+++ b/oox/source/xls/excelchartconverter.cxx
@@ -82,7 +82,7 @@ Reference< XDataSequence > ExcelChartConverter::createDataSequence(
     if( rxDataProvider.is() )
     {
         OUString aRangeRep;
-        if( rDataSeq.maFormula.getLength() > 0 )
+        if( !rDataSeq.maFormula.isEmpty() )
         {
             // parse the formula string, create a token sequence
             FormulaParser& rParser = getFormulaParser();
@@ -105,7 +105,7 @@ Reference< XDataSequence > ExcelChartConverter::createDataSequence(
             aRangeRep = FormulaProcessorBase::generateApiArray( aMatrix );
         }
 
-        if( aRangeRep.getLength() > 0 ) try
+        if( !aRangeRep.isEmpty() ) try
         {
             // create the data sequence
             xDataSeq = rxDataProvider->createDataSequenceByRangeRepresentation( aRangeRep );
diff --git a/oox/source/xls/excelfilter.cxx b/oox/source/xls/excelfilter.cxx
index bbf6ce6..9ac3cf3 100644
--- a/oox/source/xls/excelfilter.cxx
+++ b/oox/source/xls/excelfilter.cxx
@@ -123,7 +123,7 @@ bool ExcelFilter::importDocument() throw()
     OOX_DUMP_FILE( ::oox::dump::xlsb::Dumper );
 
     OUString aWorkbookPath = getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( 
"officeDocument" ) );
-    if( aWorkbookPath.getLength() == 0 )
+    if( aWorkbookPath.isEmpty() )
         return false;
 
     /*  Construct the WorkbookGlobals object referred to by every instance of
diff --git a/oox/source/xls/excelhandlers.cxx b/oox/source/xls/excelhandlers.cxx
index a2019ed..271e9be 100644
--- a/oox/source/xls/excelhandlers.cxx
+++ b/oox/source/xls/excelhandlers.cxx
@@ -98,7 +98,7 @@ const sal_uInt16 BIFF_BOF_WORKSPACE         = 0x0100;   /// BIFF3-BIFF8 workspac
 BiffFragmentHandler::BiffFragmentHandler( const FilterBase& rFilter, const OUString& rStrmName )
 {
     // do not automatically close the root stream (indicated by empty stream name)
-    bool bRootStrm = rStrmName.getLength() == 0;
+    bool bRootStrm = rStrmName.isEmpty();
     mxXInStrm.reset( new BinaryXInputStream( rFilter.openInputStream( rStrmName ), !bRootStrm ) );
     mxBiffStrm.reset( new BiffInputStream( *mxXInStrm ) );
 }
diff --git a/oox/source/xls/excelvbaproject.cxx b/oox/source/xls/excelvbaproject.cxx
index c1306a8..12ba7cb 100644
--- a/oox/source/xls/excelvbaproject.cxx
+++ b/oox/source/xls/excelvbaproject.cxx
@@ -103,7 +103,7 @@ void ExcelVbaProject::prepareImport()
             PropertySet aSheetProp( xSheetsEnum->nextElement() );
             OUString aCodeName;
             aSheetProp.getProperty( aCodeName, PROP_CodeName );
-            if( aCodeName.getLength() > 0 )
+            if( !aCodeName.isEmpty() )
             {
                 aUsedCodeNames.insert( aCodeName );
             }
diff --git a/oox/source/xls/externallinkbuffer.cxx b/oox/source/xls/externallinkbuffer.cxx
index df5eccf..93e4fb3 100644
--- a/oox/source/xls/externallinkbuffer.cxx
+++ b/oox/source/xls/externallinkbuffer.cxx
@@ -112,7 +112,7 @@ ExternalName::ExternalName( const ExternalLink& rParentLink ) :
 void ExternalName::importDefinedName( const AttributeList& rAttribs )
 {
     maModel.maName = rAttribs.getXString( XML_name, OUString() );
-    OSL_ENSURE( maModel.maName.getLength() > 0, "ExternalName::importDefinedName - empty name" );
+    OSL_ENSURE( !maModel.maName.isEmpty(), "ExternalName::importDefinedName - empty name" );
     // zero-based index into sheet list of externalBook
     maModel.mnSheet = rAttribs.getInteger( XML_sheetId, -1 );
 }
@@ -120,7 +120,7 @@ void ExternalName::importDefinedName( const AttributeList& rAttribs )
 void ExternalName::importDdeItem( const AttributeList& rAttribs )
 {
     maModel.maName = rAttribs.getXString( XML_name, OUString() );
-    OSL_ENSURE( maModel.maName.getLength() > 0, "ExternalName::importDdeItem - empty name" );
+    OSL_ENSURE( !maModel.maName.isEmpty(), "ExternalName::importDdeItem - empty name" );
     maExtNameModel.mbOleObj     = false;
     maExtNameModel.mbStdDocName = rAttribs.getBool( XML_ole, false );
     maExtNameModel.mbNotify     = rAttribs.getBool( XML_advise, false );
@@ -135,7 +135,7 @@ void ExternalName::importValues( const AttributeList& rAttribs )
 void ExternalName::importOleItem( const AttributeList& rAttribs )
 {
     maModel.maName = rAttribs.getXString( XML_name, OUString() );
-    OSL_ENSURE( maModel.maName.getLength() > 0, "ExternalName::importOleItem - empty name" );
+    OSL_ENSURE( !maModel.maName.isEmpty(), "ExternalName::importOleItem - empty name" );
     maExtNameModel.mbOleObj    = true;
     maExtNameModel.mbNotify    = rAttribs.getBool( XML_advise, false );
     maExtNameModel.mbPreferPic = rAttribs.getBool( XML_preferPic, false );
@@ -145,7 +145,7 @@ void ExternalName::importOleItem( const AttributeList& rAttribs )
 void ExternalName::importExternalName( SequenceInputStream& rStrm )
 {
     rStrm >> maModel.maName;
-    OSL_ENSURE( maModel.maName.getLength() > 0, "ExternalName::importExternalName - empty name" );
+    OSL_ENSURE( !maModel.maName.isEmpty(), "ExternalName::importExternalName - empty name" );
 }
 
 void ExternalName::importExternalNameFlags( SequenceInputStream& rStrm )
@@ -233,7 +233,7 @@ void ExternalName::importExternalName( BiffInputStream& rStrm )
     maModel.maName = (getBiff() == BIFF8) ?
         rStrm.readUniStringBody( rStrm.readuInt8() ) :
         rStrm.readByteStringUC( false, getTextEncoding() );
-    OSL_ENSURE( maModel.maName.getLength() > 0, "ExternalName::importExternalName - empty name" );
+    OSL_ENSURE( !maModel.maName.isEmpty(), "ExternalName::importExternalName - empty name" );
 
     // load cell references that are stored in hidden external names (seen in BIFF3-BIFF4)
     bool bHiddenRef = (getBiff() <= BIFF4) && (maModel.maName.getLength() > 1) && (maModel.maName[ 
0 ] == '\x01') && (rStrm.getRemaining() > 2);
@@ -347,7 +347,7 @@ sal_Int32 ExternalName::getSheetCacheIndex() const
 
 bool ExternalName::getDdeItemInfo( DDEItemInfo& orItemInfo ) const
 {
-    if( (mrParentLink.getLinkType() == LINKTYPE_DDE) && (maModel.maName.getLength() > 0) )
+    if( (mrParentLink.getLinkType() == LINKTYPE_DDE) && !maModel.maName.isEmpty() )
     {
         orItemInfo.Item = maModel.maName;
         orItemInfo.Results = ContainerHelper::matrixToSequenceSequence( maResults );
@@ -358,7 +358,7 @@ bool ExternalName::getDdeItemInfo( DDEItemInfo& orItemInfo ) const
 
 bool ExternalName::getDdeLinkData( OUString& orDdeServer, OUString& orDdeTopic, OUString& 
orDdeItem )
 {
-    if( (mrParentLink.getLinkType() == LINKTYPE_DDE) && (maModel.maName.getLength() > 0) )
+    if( (mrParentLink.getLinkType() == LINKTYPE_DDE) && !maModel.maName.isEmpty() )
     {
         // try to create a DDE link and to set the imported link results
         if( !mbDdeLinkCreated ) try
@@ -620,7 +620,7 @@ void ExternalLink::importExternSheet( BiffInputStream& rStrm )
             maCalcSheets.push_back( getWorksheets().getCalcSheetIndex( aSheetName ) );
         break;
         case LINKTYPE_EXTERNAL:
-            insertExternalSheet( (aSheetName.getLength() > 0) ? aSheetName : 
WorksheetBuffer::getBaseFileName( maTargetUrl ) );
+            insertExternalSheet( aSheetName.isEmpty() ? WorksheetBuffer::getBaseFileName( 
maTargetUrl ) : aSheetName);
         break;
         default:;
     }
@@ -648,7 +648,7 @@ void ExternalLink::importExternalBook( BiffInputStream& rStrm )
 
     // parse the encoded URL
     OUString aDummySheetName = parseBiffTargetUrl( aTarget );
-    OSL_ENSURE( aDummySheetName.getLength() == 0, "ExternalLink::importExternalBook - sheet name 
in encoded URL" );
+    OSL_ENSURE( aDummySheetName.isEmpty(), "ExternalLink::importExternalBook - sheet name in 
encoded URL" );
     (void)aDummySheetName;  // prevent compiler warning
 
     // load external sheet names and create the sheet caches in the Calc document
@@ -819,7 +819,7 @@ void ExternalLink::setExternalTargetUrl( const OUString& rTargetUrl, const OUStr
     if( rTargetType == OOX_TARGETTYPE_EXTLINK )
     {
         maTargetUrl = getBaseFilter().getAbsoluteUrl( rTargetUrl );
-        if( maTargetUrl.getLength() > 0 )
+        if( !maTargetUrl.isEmpty() )
             meLinkType = LINKTYPE_EXTERNAL;
     }
     else if( rTargetType == OOX_TARGETTYPE_LIBRARY )
@@ -845,7 +845,7 @@ void ExternalLink::setDdeOleTargetUrl( const OUString& rClassName, const OUStrin
 {
     maClassName = rClassName;
     maTargetUrl = rTargetUrl;
-    meLinkType = ((maClassName.getLength() > 0) && (maTargetUrl.getLength() > 0)) ? eLinkType : 
LINKTYPE_UNKNOWN;
+    meLinkType = (maClassName.isEmpty() || maTargetUrl.isEmpty()) ?  LINKTYPE_UNKNOWN : eLinkType;
     OSL_ENSURE( meLinkType == eLinkType, "ExternalLink::setDdeOleTargetUrl - missing classname or 
target" );
 }
 
@@ -863,9 +863,9 @@ OUString ExternalLink::parseBiffTargetUrl( const OUString& rBiffTargetUrl )
     switch( getAddressConverter().parseBiffTargetUrl( aClassName, aTargetUrl, aSheetName, 
rBiffTargetUrl ) )
     {
         case BIFF_TARGETTYPE_URL:
-            if( aTargetUrl.getLength() == 0 )
+            if( aTargetUrl.isEmpty() )
             {
-                meLinkType = (aSheetName.getLength() > 0) ? LINKTYPE_INTERNAL : LINKTYPE_SELF;
+                meLinkType = aSheetName.isEmpty() ? LINKTYPE_SELF : LINKTYPE_INTERNAL;
             }
             else if( (aTargetUrl.getLength() == 1) && (aTargetUrl[ 0 ] == ':') )
             {
@@ -879,12 +879,12 @@ OUString ExternalLink::parseBiffTargetUrl( const OUString& rBiffTargetUrl )
         break;
 
         case BIFF_TARGETTYPE_SAMESHEET:
-            OSL_ENSURE( (aTargetUrl.getLength() == 0) && (aSheetName.getLength() == 0), 
"ExternalLink::parseBiffTargetUrl - unexpected target or sheet name" );
+            OSL_ENSURE( aTargetUrl.isEmpty() && aSheetName.isEmpty(), 
"ExternalLink::parseBiffTargetUrl - unexpected target or sheet name" );
             meLinkType = LINKTYPE_SAME;
         break;
 
         case BIFF_TARGETTYPE_LIBRARY:
-            OSL_ENSURE( aSheetName.getLength() == 0, "ExternalLink::parseBiffTargetUrl - 
unexpected sheet name" );
+            OSL_ENSURE( aSheetName.isEmpty(), "ExternalLink::parseBiffTargetUrl - unexpected sheet 
name" );
             setExternalTargetUrl( aTargetUrl, OOX_TARGETTYPE_LIBRARY );
         break;
 
@@ -900,7 +900,7 @@ OUString ExternalLink::parseBiffTargetUrl( const OUString& rBiffTargetUrl )
 
 void ExternalLink::insertExternalSheet( const OUString& rSheetName )
 {
-    OSL_ENSURE( rSheetName.getLength() > 0, "ExternalLink::insertExternalSheet - empty sheet name" 
);
+    OSL_ENSURE( !rSheetName.isEmpty(), "ExternalLink::insertExternalSheet - empty sheet name" );
     if( mxDocLink.is() )
     {
         Reference< XExternalSheetCache > xSheetCache = mxDocLink->addSheetCache( rSheetName, false 
);
diff --git a/oox/source/xls/formulabase.cxx b/oox/source/xls/formulabase.cxx
index abdfbf3..477a8f9 100644
--- a/oox/source/xls/formulabase.cxx
+++ b/oox/source/xls/formulabase.cxx
@@ -907,13 +907,13 @@ void FunctionProviderImpl::initFunc( const FunctionData& rFuncData, sal_uInt8 
nM
 
     if( getFlag( rFuncData.mnFlags, FUNCFLAG_MACROCALL ) )
     {
-        OSL_ENSURE( xFuncInfo->maOoxFuncName.getLength() > 0, "FunctionProviderImpl::initFunc - 
missing OOXML function name" );
+        OSL_ENSURE( !xFuncInfo->maOoxFuncName.isEmpty(), "FunctionProviderImpl::initFunc - missing 
OOXML function name" );
         OSL_ENSURE( !getFlag( rFuncData.mnFlags, FUNCFLAG_MACROCALLODF ), 
"FunctionProviderImpl::initFunc - unexpected flag FUNCFLAG_MACROCALLODF" );
         xFuncInfo->maBiffMacroName = CREATE_OUSTRING( "_xlfn." ) + xFuncInfo->maOoxFuncName;
     }
     else if( getFlag( rFuncData.mnFlags, FUNCFLAG_MACROCALLODF ) )
     {
-        OSL_ENSURE( xFuncInfo->maOdfFuncName.getLength() > 0, "FunctionProviderImpl::initFunc - 
missing ODF function name" );
+        OSL_ENSURE( !xFuncInfo->maOdfFuncName.isEmpty(), "FunctionProviderImpl::initFunc - missing 
ODF function name" );
         xFuncInfo->maBiffMacroName = CREATE_OUSTRING( "_xlfnodf." ) + xFuncInfo->maOdfFuncName;
     }
 
@@ -937,15 +937,15 @@ void FunctionProviderImpl::initFunc( const FunctionData& rFuncData, sal_uInt8 
nM
 
     // insert the function info into the member maps
     maFuncs.push_back( xFuncInfo );
-    if( xFuncInfo->maOdfFuncName.getLength() > 0 )
+    if( !xFuncInfo->maOdfFuncName.isEmpty() )
         maOdfFuncs[ xFuncInfo->maOdfFuncName ] = xFuncInfo;
-    if( xFuncInfo->maOoxFuncName.getLength() > 0 )
+    if( !xFuncInfo->maOoxFuncName.isEmpty() )
         maOoxFuncs[ xFuncInfo->maOoxFuncName ] = xFuncInfo;
     if( xFuncInfo->mnBiff12FuncId != NOID )
         maBiff12Funcs[ xFuncInfo->mnBiff12FuncId ] = xFuncInfo;
     if( xFuncInfo->mnBiffFuncId != NOID )
         maBiffFuncs[ xFuncInfo->mnBiffFuncId ] = xFuncInfo;
-    if( xFuncInfo->maBiffMacroName.getLength() > 0 )
+    if( !xFuncInfo->maBiffMacroName.isEmpty() )
         maMacroFuncs[ xFuncInfo->maBiffMacroName ] = xFuncInfo;
 }
 
@@ -1185,7 +1185,7 @@ bool OpCodeProviderImpl::initOpCode( sal_Int32& ornOpCode, const ApiTokenMap& 
rT
     if( aIt != rTokenMap.end() )
     {
         ornOpCode = aIt->second.OpCode;
-        if( rOoxName.getLength() > 0 )
+        if( !rOoxName.isEmpty() )
         {
             FormulaOpCodeMapEntry aEntry;
             aEntry.Name = rOoxName;
@@ -1217,7 +1217,7 @@ bool OpCodeProviderImpl::initOpCode( sal_Int32& ornOpCode, const ApiTokenMap& 
rT
 bool OpCodeProviderImpl::initFuncOpCode( FunctionInfo& orFuncInfo, const ApiTokenMap& 
rFuncTokenMap )
 {
     bool bIsValid = false;
-    if( orFuncInfo.maOdfFuncName.getLength() > 0 )
+    if( !orFuncInfo.maOdfFuncName.isEmpty() )
     {
         ApiTokenMap::const_iterator aIt = rFuncTokenMap.find( orFuncInfo.maOdfFuncName );
         if( aIt != rFuncTokenMap.end() )
@@ -1234,7 +1234,7 @@ bool OpCodeProviderImpl::initFuncOpCode( FunctionInfo& orFuncInfo, const 
ApiToke
 
             if( bIsValid && (orFuncInfo.mnApiOpCode == OPCODE_EXTERNAL) )
             {
-                bIsValid = (aIt->second.Data >>= orFuncInfo.maExtProgName) && 
(orFuncInfo.maExtProgName.getLength() > 0);
+                bIsValid = (aIt->second.Data >>= orFuncInfo.maExtProgName) && 
!orFuncInfo.maExtProgName.isEmpty();
                 OSL_ENSURE( bIsValid,
                     OStringBuffer( "OpCodeProviderImpl::initFuncOpCode - no programmatical name 
for external function \"" ).
                     append( OUStringToOString( orFuncInfo.maOdfFuncName, RTL_TEXTENCODING_ASCII_US 
) ).
@@ -1242,7 +1242,7 @@ bool OpCodeProviderImpl::initFuncOpCode( FunctionInfo& orFuncInfo, const 
ApiToke
             }
 
             // add to parser map, if OOXML function name exists
-            if( bIsValid && (orFuncInfo.maOoxFuncName.getLength() > 0) )
+            if( bIsValid && !orFuncInfo.maOoxFuncName.isEmpty() )
             {
                 // create the parser map entry
                 FormulaOpCodeMapEntry aEntry;
@@ -1254,7 +1254,7 @@ bool OpCodeProviderImpl::initFuncOpCode( FunctionInfo& orFuncInfo, const 
ApiToke
         else
         {
             // ignore entries for functions unknown by Calc *and* by Excel
-            bIsValid = orFuncInfo.maOoxFuncName.getLength() == 0;
+            bIsValid = orFuncInfo.maOoxFuncName.isEmpty();
         }
     }
     else if( orFuncInfo.mnBiffFuncId == BIFF_FUNC_EXTERNCALL )
@@ -1262,7 +1262,7 @@ bool OpCodeProviderImpl::initFuncOpCode( FunctionInfo& orFuncInfo, const 
ApiToke
         orFuncInfo.mnApiOpCode = OPCODE_EXTERNAL;
         bIsValid = true;
     }
-    else if( orFuncInfo.maOoxFuncName.getLength() > 0 )
+    else if( !orFuncInfo.maOoxFuncName.isEmpty() )
     {
         orFuncInfo.mnApiOpCode = OPCODE_BAD;
         bIsValid = true;
@@ -1284,7 +1284,7 @@ bool OpCodeProviderImpl::initFuncOpCodes( const ApiTokenMap& 
rIntFuncTokenMap, c
         // insert the function info into the maps
         if( xFuncInfo->mnApiOpCode != OPCODE_NONAME )
         {
-            if( (xFuncInfo->mnApiOpCode == OPCODE_EXTERNAL) && 
(xFuncInfo->maExtProgName.getLength() > 0) )
+            if( (xFuncInfo->mnApiOpCode == OPCODE_EXTERNAL) && !xFuncInfo->maExtProgName.isEmpty() 
)
                 maExtProgFuncs[ xFuncInfo->maExtProgName ] = xFuncInfo;
             else
                 maOpCodeFuncs[ xFuncInfo->mnApiOpCode ] = xFuncInfo;
@@ -1499,7 +1499,7 @@ OUString FormulaProcessorBase::generateApiAddressString( const CellAddress& 
rAdd
     OUString aCellName;
     PropertySet aCellProp( getCellFromDoc( rAddress ) );
     aCellProp.getProperty( aCellName, PROP_AbsoluteName );
-    OSL_ENSURE( aCellName.getLength() > 0, "FormulaProcessorBase::generateApiAddressString - 
cannot create cell address string" );
+    OSL_ENSURE( !aCellName.isEmpty(), "FormulaProcessorBase::generateApiAddressString - cannot 
create cell address string" );
     return aCellName;
 }
 
@@ -1508,7 +1508,7 @@ OUString FormulaProcessorBase::generateApiRangeString( const 
CellRangeAddress& r
     OUString aRangeName;
     PropertySet aRangeProp( getCellRangeFromDoc( rRange ) );
     aRangeProp.getProperty( aRangeName, PROP_AbsoluteName );
-    OSL_ENSURE( aRangeName.getLength() > 0, "FormulaProcessorBase::generateApiRangeString - cannot 
create cell range string" );
+    OSL_ENSURE( !aRangeName.isEmpty(), "FormulaProcessorBase::generateApiRangeString - cannot 
create cell range string" );
     return aRangeName;
 }
 
@@ -1518,7 +1518,7 @@ OUString FormulaProcessorBase::generateApiRangeListString( const 
ApiCellRangeLis
     for( ApiCellRangeList::const_iterator aIt = rRanges.begin(), aEnd = rRanges.end(); aIt != 
aEnd; ++aIt )
     {
         OUString aRangeName = generateApiRangeString( *aIt );
-        if( aRangeName.getLength() > 0 )
+        if( !aRangeName.isEmpty() )
         {
             if( aBuffer.getLength() > 0 )
                 aBuffer.append( API_TOKEN_SEP );
@@ -1671,7 +1671,7 @@ void FormulaProcessorBase::convertStringToStringList(
         ApiTokenSequence& orTokens, sal_Unicode cStringSep, bool bTrimLeadingSpaces ) const
 {
     OUString aString;
-    if( extractString( aString, orTokens ) && (aString.getLength() > 0) )
+    if( extractString( aString, orTokens ) && !aString.isEmpty() )
     {
         ::std::vector< ApiToken > aNewTokens;
         sal_Int32 nPos = 0;
diff --git a/oox/source/xls/formulaparser.cxx b/oox/source/xls/formulaparser.cxx
index fb4ca7b..b7f7ddc 100644
--- a/oox/source/xls/formulaparser.cxx
+++ b/oox/source/xls/formulaparser.cxx
@@ -111,7 +111,7 @@ const FunctionInfo* FormulaFinalizer::getFunctionInfo( ApiToken& orFuncToken )
             // write function op-code to the OPCODE_BAD token
             orFuncToken.OpCode = pLibFuncInfo->mnApiOpCode;
             // if it is an external function, insert programmatic function name
-            if( (orFuncToken.OpCode == OPCODE_EXTERNAL) && 
(pLibFuncInfo->maExtProgName.getLength() > 0) )
+            if( (orFuncToken.OpCode == OPCODE_EXTERNAL) && 
(!pLibFuncInfo->maExtProgName.isEmpty()) )
                 orFuncToken.Data <<= pLibFuncInfo->maExtProgName;
             else
                 orFuncToken.Data.clear();   // clear string from OPCODE_BAD
@@ -130,10 +130,10 @@ const FunctionInfo* FormulaFinalizer::getExternCallInfo( ApiToken& 
orFuncToken,
     {
         orFuncToken.OpCode = pFuncInfo->mnApiOpCode;
         // programmatic add-in function name
-        if( (pFuncInfo->mnApiOpCode == OPCODE_EXTERNAL) && (pFuncInfo->maExtProgName.getLength() > 
0) )
+        if( (pFuncInfo->mnApiOpCode == OPCODE_EXTERNAL) && !pFuncInfo->maExtProgName.isEmpty() )
             orFuncToken.Data <<= pFuncInfo->maExtProgName;
         // name of unsupported function, convert to OPCODE_BAD to preserve the name
-        else if( (pFuncInfo->mnApiOpCode == OPCODE_BAD) && (pFuncInfo->maOoxFuncName.getLength() > 
0) )
+        else if( (pFuncInfo->mnApiOpCode == OPCODE_BAD) && !pFuncInfo->maOoxFuncName.isEmpty() )
             orFuncToken.Data <<= pFuncInfo->maOoxFuncName;
         return pFuncInfo;
     }
@@ -146,7 +146,7 @@ const FunctionInfo* FormulaFinalizer::getExternCallInfo( ApiToken& orFuncToken,
     if( (rECToken.OpCode == OPCODE_NAME) && rECToken.Data.has< sal_Int32 >() )
     {
         OUString aDefName = resolveDefinedName( rECToken.Data.get< sal_Int32 >() );
-        if( aDefName.getLength() > 0 )
+        if( !aDefName.isEmpty() )
         {
             orFuncToken.OpCode = OPCODE_BAD;
             orFuncToken.Data <<= aDefName;
@@ -907,10 +907,10 @@ bool FormulaParserImpl::pushFunctionOperatorToken( const FunctionInfo& 
rFuncInfo
     if( bOk )
     {
        // create an external add-in call for the passed built-in function
-        if( (rFuncInfo.mnApiOpCode == OPCODE_EXTERNAL) && (rFuncInfo.maExtProgName.getLength() > 
0) )
+        if( (rFuncInfo.mnApiOpCode == OPCODE_EXTERNAL) && !rFuncInfo.maExtProgName.isEmpty() )
             getOperandToken( 1, 0, 0 ).Data <<= rFuncInfo.maExtProgName;
         // create a bad token with unsupported function name
-        else if( (rFuncInfo.mnApiOpCode == OPCODE_BAD) && (rFuncInfo.maOoxFuncName.getLength() > 
0) )
+        else if( (rFuncInfo.mnApiOpCode == OPCODE_BAD) && !rFuncInfo.maOoxFuncName.isEmpty() )
             getOperandToken( 1, 0, 0 ).Data <<= rFuncInfo.maOoxFuncName;
     }
     return bOk;
@@ -1029,14 +1029,14 @@ bool FormulaParserImpl::pushEmbeddedRefOperand( const DefinedNameBase& 
rName, bo
     Any aRefAny = rName.getReference( maBaseAddr );
     if( aRefAny.hasValue() )
         return pushAnyOperand( aRefAny, OPCODE_PUSH );
-    if( bPushBadToken && (rName.getModelName().getLength() > 0) && (rName.getModelName()[ 0 ] >= ' 
') )
+    if( bPushBadToken && !rName.getModelName().isEmpty() && (rName.getModelName()[ 0 ] >= ' ') )
         return pushValueOperand( rName.getModelName(), OPCODE_BAD );
     return pushBiffErrorOperand( BIFF_ERR_NAME );
 }
 
 bool FormulaParserImpl::pushDefinedNameOperand( const DefinedNameRef& rxDefName )
 {
-    if( !rxDefName || (rxDefName->getModelName().getLength() == 0) )
+    if( !rxDefName || rxDefName->getModelName().isEmpty() )
         return pushBiffErrorOperand( BIFF_ERR_NAME );
     if( rxDefName->isMacroFunction() )
         return pushValueOperand( rxDefName->getModelName(), OPCODE_MACRO );
@@ -2790,7 +2790,7 @@ bool lclExtractRefId( sal_Int32& rnRefId, OUString& rRemainder, const 
OUString&
         {
             rnRefId = rFormulaString.copy( 1, nBracketClose - 1 ).toInt32();
             rRemainder = rFormulaString.copy( nBracketClose + 1 );
-            return rRemainder.getLength() > 0;
+            return !rRemainder.isEmpty();
         }
     }
     return false;
@@ -2870,7 +2870,7 @@ ApiTokenSequence FormulaParser::convertNameToFormula( sal_Int32 nTokenIndex ) 
co
 
 ApiTokenSequence FormulaParser::convertNumberToHyperlink( const OUString& rUrl, double fValue ) 
const
 {
-    OSL_ENSURE( rUrl.getLength() > 0, "FormulaParser::convertNumberToHyperlink - missing URL" );
+    OSL_ENSURE( !rUrl.isEmpty(), "FormulaParser::convertNumberToHyperlink - missing URL" );
     if( const FunctionInfo* pFuncInfo = getFuncInfoFromBiffFuncId( BIFF_FUNC_HYPERLINK ) )
     {
         ApiTokenSequence aTokens( 6 );
diff --git a/oox/source/xls/numberformatsbuffer.cxx b/oox/source/xls/numberformatsbuffer.cxx
index cde0039..785fc46 100644
--- a/oox/source/xls/numberformatsbuffer.cxx
+++ b/oox/source/xls/numberformatsbuffer.cxx
@@ -1928,7 +1928,7 @@ void NumberFormat::setPredefinedId( const Locale& rLocale, sal_Int16 
nPredefId )
 
 sal_Int32 NumberFormat::finalizeImport( const Reference< XNumberFormats >& rxNumFmts, const 
Locale& rFromLocale )
 {
-    if( rxNumFmts.is() && (maModel.maFmtCode.getLength() > 0) )
+    if( rxNumFmts.is() && !maModel.maFmtCode.isEmpty() )
         maApiData.mnIndex = lclCreateFormat( rxNumFmts, maModel.maFmtCode, maModel.maLocale, 
rFromLocale );
     else
         maApiData.mnIndex = lclCreatePredefinedFormat( rxNumFmts, maModel.mnPredefId, 
maModel.maLocale );
@@ -1960,7 +1960,7 @@ NumberFormatsBuffer::NumberFormatsBuffer( const WorkbookHelper& rHelper ) :
         xConfigNA->getByName( CREATE_OUSTRING( "ooSetupSystemLocale" ) ) >>= maLocaleStr;
 
         // if set to "use system", get locale from system
-        if( maLocaleStr.getLength() == 0 )
+        if( maLocaleStr.isEmpty() )
         {
             aArgs[ 0 ] <<= CREATE_OUSTRING( "org.openoffice.System/L10N/" );
             xConfigNA.set( xConfigProv->createInstanceWithArguments(
diff --git a/oox/source/xls/pagesettings.cxx b/oox/source/xls/pagesettings.cxx
index c3cd279..0625bb3 100644
--- a/oox/source/xls/pagesettings.cxx
+++ b/oox/source/xls/pagesettings.cxx
@@ -454,7 +454,7 @@ void PageSettings::finalizeImport()
 void PageSettings::importPictureData( const Relations& rRelations, const OUString& rRelId )
 {
     OUString aPicturePath = rRelations.getFragmentPathFromRelId( rRelId );
-    if( aPicturePath.getLength() > 0 )
+    if( !aPicturePath.isEmpty() )
         maModel.maGraphicUrl = getBaseFilter().getGraphicHelper().importEmbeddedGraphicObject( 
aPicturePath );
 }
 
@@ -613,7 +613,7 @@ HeaderFooterParser::HeaderFooterParser( const WorkbookHelper& rHelper ) :
 
 double HeaderFooterParser::parse( const Reference< XHeaderFooterContent >& rxContext, const 
OUString& rData )
 {
-    if( !rxContext.is() || (rData.getLength() == 0) ||
+    if( !rxContext.is() || rData.isEmpty() ||
             !maPortions[ HF_LEFT ].initialize( rxContext->getLeftText() ) ||
             !maPortions[ HF_CENTER ].initialize( rxContext->getCenterText() ) ||
             !maPortions[ HF_RIGHT ].initialize( rxContext->getRightText() ) )
@@ -949,7 +949,7 @@ void HeaderFooterParser::appendField( const Reference< XTextContent >& rxContent
 
 void HeaderFooterParser::convertFontName( const OUString& rName )
 {
-    if( rName.getLength() > 0 )
+    if( !rName.isEmpty() )
     {
         // single dash is document default font
         if( (rName.getLength() == 1) && (rName[ 0 ] == '-') )
@@ -967,7 +967,7 @@ void HeaderFooterParser::convertFontStyle( const OUString& rStyle )
     while( (0 <= nPos) && (nPos < nLen) )
     {
         OString aToken = OUStringToOString( rStyle.getToken( 0, ' ', nPos ), RTL_TEXTENCODING_UTF8 
).toAsciiLowerCase();
-        if( aToken.getLength() > 0 )
+        if( !aToken.isEmpty() )
         {
             if( maBoldNames.count( aToken ) > 0 )
                 maFontModel.mbBold = true;
@@ -1211,7 +1211,7 @@ void PageSettingsConverter::writePageSettingsProperties(
     aPropMap[ PROP_FooterHeight ]          <<= maFooterData.mnHeight;
     aPropMap[ PROP_FooterBodyDistance ]    <<= maFooterData.mnBodyDist;
     // background image
-    if( rModel.maGraphicUrl.getLength() > 0 )
+    if( !rModel.maGraphicUrl.isEmpty() )
     {
         aPropMap[ PROP_BackGraphicURL ] <<= rModel.maGraphicUrl;
         aPropMap[ PROP_BackGraphicLocation ] <<= ::com::sun::star::style::GraphicLocation_TILED;
@@ -1225,8 +1225,8 @@ void PageSettingsConverter::convertHeaderFooterData(
         const OUString rOddContent, const OUString rEvenContent, bool bUseEvenContent,
         double fPageMargin, double fContentMargin )
 {
-    bool bHasOddContent  = rOddContent.getLength() > 0;
-    bool bHasEvenContent = bUseEvenContent && (rEvenContent.getLength() > 0);
+    bool bHasOddContent  = !rOddContent.isEmpty();
+    bool bHasEvenContent = bUseEvenContent && !rEvenContent.isEmpty();
 
     sal_Int32 nOddHeight  = bHasOddContent  ? writeHeaderFooter( rPropSet, orHFData.mnRightPropId, 
rOddContent  ) : 0;
     sal_Int32 nEvenHeight = bHasEvenContent ? writeHeaderFooter( rPropSet, orHFData.mnLeftPropId,  
rEvenContent ) : 0;
@@ -1262,9 +1262,9 @@ void PageSettingsConverter::convertHeaderFooterData(
 sal_Int32 PageSettingsConverter::writeHeaderFooter(
         PropertySet& rPropSet, sal_Int32 nPropId, const OUString& rContent )
 {
-    OSL_ENSURE( rContent.getLength() > 0, "PageSettingsConverter::writeHeaderFooter - empty h/f 
string found" );
+    OSL_ENSURE( !rContent.isEmpty(), "PageSettingsConverter::writeHeaderFooter - empty h/f string 
found" );
     sal_Int32 nHeight = 0;
-    if( rContent.getLength() > 0 )
+    if( !rContent.isEmpty() )
     {
         Reference< XHeaderFooterContent > xHFContent( rPropSet.getAnyProperty( nPropId ), 
UNO_QUERY );
         if( xHFContent.is() )
diff --git a/oox/source/xls/pivotcachebuffer.cxx b/oox/source/xls/pivotcachebuffer.cxx
index 49652da..e4ed2a3 100644
--- a/oox/source/xls/pivotcachebuffer.cxx
+++ b/oox/source/xls/pivotcachebuffer.cxx
@@ -889,7 +889,7 @@ OUString PivotCacheField::createParentGroupField( const Reference< XDataPilotFie
                 Reference< XNamed > xGroupName;
                 OUString aAutoName;
                 Reference< XIndexAccess > xGroupsIA( aGroupInfo.Groups, UNO_QUERY_THROW );
-                for( sal_Int32 nIdx = 0, nCount = xGroupsIA->getCount(); (nIdx < nCount) && 
(aAutoName.getLength() == 0); ++nIdx ) try
+                for( sal_Int32 nIdx = 0, nCount = xGroupsIA->getCount(); (nIdx < nCount) && 
(aAutoName.isEmpty()); ++nIdx ) try
                 {
                     Reference< XNameAccess > xItemsNA( xGroupsIA->getByIndex( nIdx ), 
UNO_QUERY_THROW );
                     if( xItemsNA->hasByName( aFirstItem ) )
@@ -901,17 +901,17 @@ OUString PivotCacheField::createParentGroupField( const Reference< 
XDataPilotFie
                 catch( Exception& )
                 {
                 }
-                OSL_ENSURE( aAutoName.getLength() > 0, "PivotCacheField::createParentGroupField - 
cannot find auto-generated group name" );
+                OSL_ENSURE( !aAutoName.isEmpty(), "PivotCacheField::createParentGroupField - 
cannot find auto-generated group name" );
 
                 // get the real group name from the list of group items
                 OUString aGroupName;
                 if( const PivotCacheItem* pGroupItem = maGroupItems.getCacheItem( static_cast< 
sal_Int32 >( aIt - aBeg ) ) )
                     aGroupName = pGroupItem->getName();
-                OSL_ENSURE( aGroupName.getLength() > 0, "PivotCacheField::createParentGroupField - 
cannot find group name" );
-                if( aGroupName.getLength() == 0 )
+                OSL_ENSURE( !aGroupName.isEmpty(), "PivotCacheField::createParentGroupField - 
cannot find group name" );
+                if( aGroupName.isEmpty() )
                     aGroupName = aAutoName;
 
-                if( xGroupName.is() && (aGroupName.getLength() > 0) )
+                if( xGroupName.is() && !aGroupName.isEmpty() )
                 {
                     // replace the auto-generated group name with the real name
                     if( aAutoName != aGroupName )
@@ -1238,8 +1238,8 @@ void PivotCache::finalizeImport()
         case XML_worksheet:
         {
             // decide whether an external document is used
-            bool bInternal = (maTargetUrl.getLength() == 0) && 
(maSheetSrcModel.maRelId.getLength() == 0);
-            bool bExternal = maTargetUrl.getLength() > 0;   // relation ID may be empty, e.g. BIFF 
import
+            bool bInternal = maTargetUrl.isEmpty() && maSheetSrcModel.maRelId.isEmpty();
+            bool bExternal = !maTargetUrl.isEmpty();   // relation ID may be empty, e.g. BIFF 
import
             OSL_ENSURE( bInternal || bExternal, "PivotCache::finalizeImport - invalid external 
document URL" );
             if( bInternal )
                 finalizeInternalSheetSource();
@@ -1329,13 +1329,13 @@ void PivotCache::importDConRef( BiffInputStream& rStrm )
 
     // the URL with (required) sheet name and optional URL of an external document
     importDConUrl( rStrm );
-    OSL_ENSURE( maSheetSrcModel.maSheet.getLength() > 0, "PivotCache::importDConRef - missing 
sheet name" );
+    OSL_ENSURE( !maSheetSrcModel.maSheet.isEmpty(), "PivotCache::importDConRef - missing sheet 
name" );
 }
 
 void PivotCache::importDConName( BiffInputStream& rStrm )
 {
     maSheetSrcModel.maDefName = (getBiff() == BIFF8) ? rStrm.readUniString() : 
rStrm.readByteStringUC( false, getTextEncoding() );
-    OSL_ENSURE( maSheetSrcModel.maDefName.getLength() > 0, "PivotCache::importDConName - missing 
defined name" );
+    OSL_ENSURE( !maSheetSrcModel.maDefName.isEmpty(), "PivotCache::importDConName - missing 
defined name" );
     importDConUrl( rStrm );
 }
 
@@ -1363,7 +1363,7 @@ void PivotCache::importDConUrl( BiffInputStream& rStrm )
         aEncodedUrl = rStrm.readByteStringUC( false, getTextEncoding() );
     }
 
-    if( aEncodedUrl.getLength() > 0 )
+    if( !aEncodedUrl.isEmpty() )
     {
         OUString aClassName;
         getAddressConverter().parseBiffTargetUrl( aClassName, maTargetUrl, 
maSheetSrcModel.maSheet, aEncodedUrl, true );
@@ -1376,7 +1376,7 @@ void PivotCache::finalizeInternalSheetSource()
     sal_Int16 nSheet = getWorksheets().getCalcSheetIndex( maSheetSrcModel.maSheet );
 
     // if cache is based on a defined name or table, try to resolve to cell range
-    if( maSheetSrcModel.maDefName.getLength() > 0 )
+    if( !maSheetSrcModel.maDefName.isEmpty() )
     {
         // local or global defined name
         if( const DefinedName* pDefName = getDefinedNames().getByModelName( 
maSheetSrcModel.maDefName, nSheet ).get() )
@@ -1401,7 +1401,7 @@ void PivotCache::finalizeInternalSheetSource()
         mbValidSource = true;
     }
     // else sheet has been deleted, generate the source data from cache
-    else if( maSheetSrcModel.maSheet.getLength() > 0 )
+    else if( !maSheetSrcModel.maSheet.isEmpty() )
     {
         prepareSourceDataSheet();
         // return here to skip the source range check below
@@ -1419,8 +1419,8 @@ void PivotCache::finalizeExternalSheetSource()
     /*  If pivot cache is based on external sheet data, try to restore sheet
         data from cache records. No support for external defined names or tables,
         sheet name and path to cache records fragment (OOXML only) are required. */
-    bool bHasRelation = (getFilterType() == FILTER_BIFF) || (maDefModel.maRelId.getLength() > 0);
-    if( bHasRelation && (maSheetSrcModel.maDefName.getLength() == 0) && 
(maSheetSrcModel.maSheet.getLength() > 0) )
+    bool bHasRelation = (getFilterType() == FILTER_BIFF) || !maDefModel.maRelId.isEmpty();
+    if( bHasRelation && maSheetSrcModel.maDefName.isEmpty() && !maSheetSrcModel.maSheet.isEmpty() )
         prepareSourceDataSheet();
 }
 
@@ -1462,7 +1462,7 @@ void PivotCacheBuffer::registerPivotCacheFragment( sal_Int32 nCacheId, const 
OUS
 {
     OSL_ENSURE( nCacheId >= 0, "PivotCacheBuffer::registerPivotCacheFragment - invalid pivot cache 
identifier" );
     OSL_ENSURE( maFragmentPaths.count( nCacheId ) == 0, 
"PivotCacheBuffer::registerPivotCacheFragment - fragment path exists already" );
-    if( (nCacheId >= 0) && (rFragmentPath.getLength() > 0) )
+    if( (nCacheId >= 0) && !rFragmentPath.isEmpty() )
         maFragmentPaths[ nCacheId ] = rFragmentPath;
 }
 
diff --git a/oox/source/xls/pivotcachefragment.cxx b/oox/source/xls/pivotcachefragment.cxx
index d86a158..4884976 100644
--- a/oox/source/xls/pivotcachefragment.cxx
+++ b/oox/source/xls/pivotcachefragment.cxx
@@ -208,7 +208,7 @@ void PivotCacheDefinitionFragment::finalizeImport()
     if( mrPivotCache.isValidDataSource() && mrPivotCache.isBasedOnDummySheet() )
     {
         OUString aRecFragmentPath = getRelations().getFragmentPathFromRelId( 
mrPivotCache.getRecordsRelId() );
-        if( aRecFragmentPath.getLength() > 0 )
+        if( !aRecFragmentPath.isEmpty() )
         {
             sal_Int16 nSheet = mrPivotCache.getSourceRange().Sheet;
             WorksheetGlobalsRef xSheetGlob = WorksheetHelper::constructGlobals( *this, 
ISegmentProgressBarRef(), SHEETTYPE_WORKSHEET, nSheet );
diff --git a/oox/source/xls/pivottablebuffer.cxx b/oox/source/xls/pivottablebuffer.cxx
index 3a8ea64..fefced8 100644
--- a/oox/source/xls/pivottablebuffer.cxx
+++ b/oox/source/xls/pivottablebuffer.cxx
@@ -529,7 +529,7 @@ void PivotTableField::finalizeImport( const Reference< XDataPilotDescriptor >& r
         xDPField.set( xDPFieldsIA->getByIndex( nDatabaseIdx ), UNO_QUERY_THROW );
         Reference< XNamed > xDPFieldName( xDPField, UNO_QUERY_THROW );
         maDPFieldName = xDPFieldName->getName();
-        OSL_ENSURE( maDPFieldName.getLength() > 0, "PivotTableField::finalizeImport - no field 
name in source data found" );
+        OSL_ENSURE( !maDPFieldName.isEmpty(), "PivotTableField::finalizeImport - no field name in 
source data found" );
 
         // try to convert grouping settings
         if( const PivotCacheField* pCacheField = mrPivotTable.getCacheField( mnFieldIndex ) )
@@ -566,14 +566,14 @@ void PivotTableField::finalizeImport( const Reference< XDataPilotDescriptor 
& r
 
 void PivotTableField::finalizeDateGroupingImport( const Reference< XDataPilotField >& 
rxBaseDPField, sal_Int32 nBaseFieldIdx )
 {
-    if( maDPFieldName.getLength() == 0 )    // prevent endless loops if file format is broken
+    if( maDPFieldName.isEmpty() )    // prevent endless loops if file format is broken
     {
         if( const PivotCacheField* pCacheField = mrPivotTable.getCacheField( mnFieldIndex ) )
         {
             if( !pCacheField->isDatabaseField() && pCacheField->hasDateGrouping() && 
(pCacheField->getGroupBaseField() == nBaseFieldIdx) )
             {
                 maDPFieldName = pCacheField->createDateGroupField( rxBaseDPField );
-                OSL_ENSURE( maDPFieldName.getLength() > 0, 
"PivotTableField::finalizeDateGroupingImport - cannot create date group field" );
+                OSL_ENSURE( !maDPFieldName.isEmpty(), "PivotTableField::finalizeDateGroupingImport 
- cannot create date group field" );
             }
         }
     }
@@ -581,7 +581,7 @@ void PivotTableField::finalizeDateGroupingImport( const Reference< XDataPilotFie
 
 void PivotTableField::finalizeParentGroupingImport( const Reference< XDataPilotField >& 
rxBaseDPField, PivotCacheGroupItemVector& orItemNames )
 {
-    if( maDPFieldName.getLength() == 0 )    // prevent endless loops if file format is broken
+    if( maDPFieldName.isEmpty() )    // prevent endless loops if file format is broken
     {
         if( const PivotCacheField* pCacheField = mrPivotTable.getCacheField( mnFieldIndex ) )
         {
@@ -651,7 +651,7 @@ void PivotTableField::convertPageField( const PTPageFieldModel& rPageField )
                 if( const PivotCacheItem* pSharedItem = pCacheField->getCacheItem( nCacheItem ) )
                 {
                     OUString aSelectedPage = pSharedItem->getName();
-                    if( aSelectedPage.getLength() > 0 )
+                    if( !aSelectedPage.isEmpty() )
                         aPropSet.setProperty( PROP_SelectedPage, aSelectedPage );
                 }
             }
@@ -1361,7 +1361,7 @@ void PivotTable::finalizeImport()
     if( getAddressConverter().validateCellRange( maLocationModel.maRange, true, true ) )
     {
         mpPivotCache = getPivotCaches().importPivotCacheFragment( maDefModel.mnCacheId );
-        if( mpPivotCache && mpPivotCache->isValidDataSource() && (maDefModel.maName.getLength() > 
0) )
+        if( mpPivotCache && mpPivotCache->isValidDataSource() && !maDefModel.maName.isEmpty() )
         {
             // clear destination area of the original pivot table
             try
@@ -1461,7 +1461,7 @@ void PivotTable::finalizeParentGroupingImport( const Reference< 
XDataPilotField
 Reference< XDataPilotField > PivotTable::getDataPilotField( const OUString& rFieldName ) const
 {
     Reference< XDataPilotField > xDPField;
-    if( (rFieldName.getLength() > 0) && mxDPDescriptor.is() ) try
+    if( !rFieldName.isEmpty() && mxDPDescriptor.is() ) try
     {
         Reference< XNameAccess > xDPFieldsNA( mxDPDescriptor->getDataPilotFields(), 
UNO_QUERY_THROW );
         xDPField.set( xDPFieldsNA->getByName( rFieldName ), UNO_QUERY );
diff --git a/oox/source/xls/querytablebuffer.cxx b/oox/source/xls/querytablebuffer.cxx
index 97b10fa..d904368 100644
--- a/oox/source/xls/querytablebuffer.cxx
+++ b/oox/source/xls/querytablebuffer.cxx
@@ -101,7 +101,7 @@ const sal_uInt32 BIFF_QTREFRESH_ADJUSTCOLWIDTH      = 0x00000002;
 
 void lclAppendWebQueryTableName( OUStringBuffer& rTables, const OUString& rTableName )
 {
-    if( rTableName.getLength() > 0 )
+    if( !rTableName.isEmpty() )
     {
         if( rTables.getLength() > 0 )
             rTables.append( sal_Unicode( ';' ) );
@@ -325,7 +325,7 @@ void QueryTable::finalizeImport()
         if( pWebPr && !pWebPr->mbXml )
         {
             OUString aFileUrl = getBaseFilter().getAbsoluteUrl( pWebPr->maUrl );
-            if( aFileUrl.getLength() > 0 )
+            if( !aFileUrl.isEmpty() )
             {
                 // resolve destination cell range (stored as defined name containing the range)
                 OUString aDefName = maModel.maDefName.replace( ' ', '_' ).replace( '-', '_' );
@@ -341,7 +341,7 @@ void QueryTable::finalizeImport()
                         CellAddress aDestPos( aDestRange.Sheet, aDestRange.StartColumn, 
aDestRange.StartRow );
                         // find tables mode: entire document, all tables, or specific tables
                         OUString aTables = pWebPr->mbHtmlTables ? lclBuildWebQueryTables( 
pWebPr->maTables ) : CREATE_OUSTRING( "HTML_all" );
-                        if( aTables.getLength() > 0 ) try
+                        if( !aTables.isEmpty() ) try
                         {
                             PropertySet aDocProps( getDocument() );
                             Reference< XAreaLinks > xAreaLinks( aDocProps.getAnyProperty( 
PROP_AreaLinks ), UNO_QUERY_THROW );
diff --git a/oox/source/xls/richstring.cxx b/oox/source/xls/richstring.cxx
index 7b025e5..48d5db8 100644
--- a/oox/source/xls/richstring.cxx
+++ b/oox/source/xls/richstring.cxx
@@ -579,9 +579,9 @@ RichStringPhoneticRef RichString::createPhonetic()
 void RichString::createTextPortions( const OString& rText, rtl_TextEncoding eTextEnc, 
FontPortionModelList& rPortions )
 {
     maTextPortions.clear();
-    sal_Int32 nStrLen = rText.getLength();
-    if( nStrLen > 0 )
+    if( !rText.isEmpty())
     {
+        sal_Int32 nStrLen = rText.getLength();
         // add leading and trailing string position to ease the following loop
         if( rPortions.empty() || (rPortions.front().mnPos > 0) )
             rPortions.insert( rPortions.begin(), FontPortionModel( 0, -1 ) );
@@ -610,9 +610,9 @@ void RichString::createTextPortions( const OString& rText, rtl_TextEncoding eTex
 void RichString::createTextPortions( const OUString& rText, FontPortionModelList& rPortions )
 {
     maTextPortions.clear();
-    sal_Int32 nStrLen = rText.getLength();
-    if( nStrLen > 0 )
+    if( !rText.isEmpty() )
     {
+         sal_Int32 nStrLen = rText.getLength();
         // add leading and trailing string position to ease the following loop
         if( rPortions.empty() || (rPortions.front().mnPos > 0) )
             rPortions.insert( rPortions.begin(), FontPortionModel( 0, -1 ) );
@@ -636,9 +636,9 @@ void RichString::createTextPortions( const OUString& rText, FontPortionModelList
 void RichString::createPhoneticPortions( const ::rtl::OUString& rText, PhoneticPortionModelList& 
rPortions, sal_Int32 nBaseLen )
 {
     maPhonPortions.clear();
-    sal_Int32 nStrLen = rText.getLength();
-    if( nStrLen > 0 )
+    if( !rText.isEmpty())
     {
+        sal_Int32 nStrLen = rText.getLength();
         // no portions - assign phonetic text to entire base text
         if( rPortions.empty() )
             rPortions.push_back( PhoneticPortionModel( 0, 0, nBaseLen ) );
diff --git a/oox/source/xls/scenariobuffer.cxx b/oox/source/xls/scenariobuffer.cxx
index 23636dd..b0d24c9 100644
--- a/oox/source/xls/scenariobuffer.cxx
+++ b/oox/source/xls/scenariobuffer.cxx
@@ -168,7 +168,7 @@ void Scenario::finalizeImport()
         if( !aIt->mbDeleted && rAddrConv.checkCellAddress( aIt->maPos, true ) )
             aRanges.push_back( CellRangeAddress( aIt->maPos.Sheet, aIt->maPos.Column, 
aIt->maPos.Row, aIt->maPos.Column, aIt->maPos.Row ) );
 
-    if( !aRanges.empty() && (maModel.maName.getLength() > 0) ) try
+    if( !aRanges.empty() && !maModel.maName.isEmpty() ) try
     {
         /*  Find an unused name for the scenario (Calc stores scenario data in
             hidden sheets named after the scenario following the base sheet). */
diff --git a/oox/source/xls/sheetdatabuffer.cxx b/oox/source/xls/sheetdatabuffer.cxx
index 0839b9d..1514eb8 100644
--- a/oox/source/xls/sheetdatabuffer.cxx
+++ b/oox/source/xls/sheetdatabuffer.cxx
@@ -724,7 +724,7 @@ void SheetDataBuffer::finalizeTableOperation( const CellRangeAddress& rRange, co
 {
     sal_Int16 nSheet = getSheetIndex();
     bool bOk = false;
-    if( !rModel.mbRef1Deleted && (rModel.maRef1.getLength() > 0) && (rRange.StartColumn > 0) && 
(rRange.StartRow > 0) )
+    if( !rModel.mbRef1Deleted && !rModel.maRef1.isEmpty() && (rRange.StartColumn > 0) && 
(rRange.StartRow > 0) )
     {
         CellRangeAddress aOpRange = rRange;
         CellAddress aRef1;
diff --git a/oox/source/xls/sheetdatacontext.cxx b/oox/source/xls/sheetdatacontext.cxx
index dfb9db0..b6415f3 100644
--- a/oox/source/xls/sheetdatacontext.cxx
+++ b/oox/source/xls/sheetdatacontext.cxx
@@ -217,7 +217,7 @@ void SheetDataContext::onEndElement()
         if( !mbHasFormula )
         {
             // no formula created: try to set the cell value
-            if( maCellValue.getLength() > 0 ) switch( maCellData.mnCellType )
+            if( !maCellValue.isEmpty() ) switch( maCellData.mnCellType )
             {
                 case XML_n:
                     mrSheetData.setValueCell( maCellData, maCellValue.toDouble() );
@@ -247,7 +247,7 @@ void SheetDataContext::onEndElement()
                 mrSheetData.setBlankCell( maCellData );
             }
         }
-        else if( maCellValue.getLength() > 0 ) switch( maCellData.mnCellType )
+        else if( !maCellValue.isEmpty() ) switch( maCellData.mnCellType )
         {
             case XML_n:
                 /* Set the pre-loaded value */
diff --git a/oox/source/xls/stylesbuffer.cxx b/oox/source/xls/stylesbuffer.cxx
index 28f7273..a09c4eb 100644
--- a/oox/source/xls/stylesbuffer.cxx
+++ b/oox/source/xls/stylesbuffer.cxx
@@ -946,7 +946,7 @@ void Font::importCfRule( BiffInputStream& rStrm )
     OSL_ENSURE( rStrm.getRemaining() >= 118, "Font::importCfRule - missing record data" );
     sal_Int64 nRecPos = rStrm.tell();
     maModel.maName = rStrm.readUniStringBody( rStrm.readuInt8() );
-    maUsedFlags.mbNameUsed = maModel.maName.getLength() > 0;
+    maUsedFlags.mbNameUsed = !maModel.maName.isEmpty();
     OSL_ENSURE( !rStrm.isEof() && (rStrm.tell() <= nRecPos + 64), "Font::importCfRule - font name 
too long" );
     rStrm.seek( nRecPos + 64 );
     rStrm >> nHeight >> nStyle >> nWeight >> nEscapement >> nUnderline;
@@ -1110,19 +1110,19 @@ void Font::writeToPropertyMap( PropertyMap& rPropMap, FontPropertyType 
ePropType
     // font name properties
     if( maUsedFlags.mbNameUsed )
     {
-        if( maApiData.maLatinFont.maName.getLength() > 0 )
+        if( !maApiData.maLatinFont.maName.isEmpty() )
         {
             rPropMap[ PROP_CharFontName ]    <<= maApiData.maLatinFont.maName;
             rPropMap[ PROP_CharFontFamily ]  <<= maApiData.maLatinFont.mnFamily;
             rPropMap[ PROP_CharFontCharSet ] <<= maApiData.maLatinFont.mnTextEnc;
         }
-        if( maApiData.maAsianFont.maName.getLength() > 0 )
+        if( !maApiData.maAsianFont.maName.isEmpty() )
         {
             rPropMap[ PROP_CharFontNameAsian ]    <<= maApiData.maAsianFont.maName;
             rPropMap[ PROP_CharFontFamilyAsian ]  <<= maApiData.maAsianFont.mnFamily;
             rPropMap[ PROP_CharFontCharSetAsian ] <<= maApiData.maAsianFont.mnTextEnc;
         }
-        if( maApiData.maCmplxFont.maName.getLength() > 0 )
+        if( !maApiData.maCmplxFont.maName.isEmpty() )
         {
             rPropMap[ PROP_CharFontNameComplex ]    <<= maApiData.maCmplxFont.maName;
             rPropMap[ PROP_CharFontFamilyComplex ]  <<= maApiData.maCmplxFont.mnFamily;
@@ -2809,7 +2809,7 @@ OUString lclGetBuiltinStyleName( sal_Int32 nBuiltinId, const OUString& rName, 
sa
     aStyleName.appendAscii( spcStyleNamePrefix );
     if( (0 <= nBuiltinId) && (nBuiltinId < snStyleNamesCount) && (sppcStyleNames[ nBuiltinId ][ 0 
] != 0) )
         aStyleName.appendAscii( sppcStyleNames[ nBuiltinId ] );
-    else if( rName.getLength() > 0 )
+    else if( !rName.isEmpty() )
         aStyleName.append( rName );
     else
         aStyleName.append( nBuiltinId );
@@ -2910,7 +2910,7 @@ void CellStyle::createCellStyle()
 {
     // #i1624# #i1768# ignore unnamed user styles
     if( !mbCreated )
-        mbCreated = maFinalName.getLength() == 0;
+        mbCreated = maFinalName.isEmpty();
 
     /*  #i103281# do not create another style of the same name, if it exists
         already. This is needed to prevent that styles pasted from clipboard
@@ -3036,7 +3036,7 @@ void CellStyleBuffer::finalizeImport()
     {
         const CellStyleModel& rModel = (*aIt)->getModel();
         // #i1624# #i1768# ignore unnamed user styles
-        if( rModel.maName.getLength() > 0 )
+        if( !rModel.maName.isEmpty() )
         {
             if( aCellStyles.count( rModel.maName ) > 0 )
                 aConflictNameStyles.push_back( *aIt );
@@ -3114,7 +3114,7 @@ OUString CellStyleBuffer::createCellStyle( const CellStyleRef& rxCellStyle ) 
con
     {
         rxCellStyle->createCellStyle();
         const OUString& rStyleName = rxCellStyle->getFinalStyleName();
-        if( rStyleName.getLength() > 0 )
+        if( !rStyleName.isEmpty() )
             return rStyleName;
     }
     // on error: fallback to default style
@@ -3423,7 +3423,7 @@ OUString StylesBuffer::createCellStyle( sal_Int32 nXfId ) const
 OUString StylesBuffer::createDxfStyle( sal_Int32 nDxfId ) const
 {
     OUString& rStyleName = maDxfStyles[ nDxfId ];
-    if( rStyleName.getLength() == 0 )
+    if( rStyleName.isEmpty() )
     {
         if( Dxf* pDxf = maDxfs.get( nDxfId ).get() )
         {
@@ -3435,7 +3435,7 @@ OUString StylesBuffer::createDxfStyle( sal_Int32 nDxfId ) const
             pDxf->writeToPropertySet( aPropSet );
         }
         // on error: fallback to default style
-        if( rStyleName.getLength() == 0 )
+        if( rStyleName.isEmpty() )
             rStyleName = maCellStyles.getDefaultStyleName();
     }
     return rStyleName;
diff --git a/oox/source/xls/tablebuffer.cxx b/oox/source/xls/tablebuffer.cxx
index cc6647a..baefc3d 100644
--- a/oox/source/xls/tablebuffer.cxx
+++ b/oox/source/xls/tablebuffer.cxx
@@ -94,7 +94,7 @@ void Table::finalizeImport()
     // ranges (or tables in their terminology) as Table1, Table2 etc.  We need
     // to import them as named db ranges because they may be referenced by
     // name in formula expressions.
-    if( (maModel.mnId > 0) && (maModel.maDisplayName.getLength() > 0) ) try
+    if( (maModel.mnId > 0) && !maModel.maDisplayName.isEmpty() ) try
     {
         maDBRangeName = maModel.maDisplayName;
         Reference< XDatabaseRange > xDatabaseRange(
@@ -154,7 +154,7 @@ void TableBuffer::insertTableToMaps( const TableRef& rxTable )
 {
     sal_Int32 nTableId = rxTable->getTableId();
     const OUString& rDispName = rxTable->getDisplayName();
-    if( (nTableId > 0) && (rDispName.getLength() > 0) )
+    if( (nTableId > 0) && !rDispName.isEmpty() )
     {
         OSL_ENSURE( !maIdTables.has( nTableId ), "TableBuffer::insertTableToMaps - multiple table 
identifier" );
         maIdTables[ nTableId ] = rxTable;
diff --git a/oox/source/xls/workbookfragment.cxx b/oox/source/xls/workbookfragment.cxx
index 9d22f5f..d4e61fb 100644
--- a/oox/source/xls/workbookfragment.cxx
+++ b/oox/source/xls/workbookfragment.cxx
@@ -200,25 +200,25 @@ void WorkbookFragment::finalizeImport()
 
     // read the theme substream
     OUString aThemeFragmentPath = getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( 
"theme" ) );
-    if( aThemeFragmentPath.getLength() > 0 )
+    if( !aThemeFragmentPath.isEmpty() )
         importOoxFragment( new ThemeFragmentHandler( getFilter(), aThemeFragmentPath, getTheme() ) 
);
     xGlobalSegment->setPosition( 0.25 );
 
     // read the styles substream (requires finalized theme buffer)
     OUString aStylesFragmentPath = getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( 
"styles" ) );
-    if( aStylesFragmentPath.getLength() > 0 )
+    if( !aStylesFragmentPath.isEmpty() )
         importOoxFragment( new StylesFragment( *this, aStylesFragmentPath ) );
     xGlobalSegment->setPosition( 0.5 );
 
     // read the shared string table substream (requires finalized styles buffer)
     OUString aSstFragmentPath = getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( 
"sharedStrings" ) );
-    if( aSstFragmentPath.getLength() > 0 )
+    if( !aSstFragmentPath.isEmpty() )
         importOoxFragment( new SharedStringsFragment( *this, aSstFragmentPath ) );
     xGlobalSegment->setPosition( 0.75 );
 
     // read the connections substream
     OUString aConnFragmentPath = getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( 
"connections" ) );
-    if( aConnFragmentPath.getLength() > 0 )
+    if( !aConnFragmentPath.isEmpty() )
         importOoxFragment( new ConnectionsFragment( *this, aConnFragmentPath ) );
     xGlobalSegment->setPosition( 1.0 );
 
@@ -240,8 +240,8 @@ void WorkbookFragment::finalizeImport()
         {
             // get fragment path of the sheet
             OUString aFragmentPath = getFragmentPathFromRelation( *pRelation );
-            OSL_ENSURE( aFragmentPath.getLength() > 0, "WorkbookFragment::finalizeImport - cannot 
access sheet fragment" );
-            if( aFragmentPath.getLength() > 0 )
+            OSL_ENSURE( !aFragmentPath.isEmpty(), "WorkbookFragment::finalizeImport - cannot 
access sheet fragment" );
+            if( !aFragmentPath.isEmpty() )
             {
                 double fSegmentLength = getProgressBar().getFreeLength() / (nWorksheetCount - 
nWorksheet);
                 ISegmentProgressBarRef xSheetSegment = getProgressBar().createSegment( 
fSegmentLength );
@@ -306,7 +306,7 @@ void WorkbookFragment::finalizeImport()
 
     // open the VBA project storage
     OUString aVbaFragmentPath = getFragmentPathFromFirstType( CREATE_MSOFFICE_RELATION_TYPE( 
"vbaProject" ) );
-    if( aVbaFragmentPath.getLength() > 0 )
+    if( !aVbaFragmentPath.isEmpty() )
     {
         Reference< XInputStream > xInStrm = getBaseFilter().openInputStream( aVbaFragmentPath );
         if( xInStrm.is() )
@@ -358,7 +358,7 @@ void WorkbookFragment::importPivotCache( SequenceInputStream& rStrm )
 void WorkbookFragment::importExternalLinkFragment( ExternalLink& rExtLink )
 {
     OUString aFragmentPath = getFragmentPathFromRelId( rExtLink.getRelId() );
-    if( aFragmentPath.getLength() > 0 )
+    if( !aFragmentPath.isEmpty() )
         importOoxFragment( new ExternalLinkFragment( *this, aFragmentPath, rExtLink ) );
 }
 
diff --git a/oox/source/xls/workbookhelper.cxx b/oox/source/xls/workbookhelper.cxx
index 3cc56cd..d1a61c9 100644
--- a/oox/source/xls/workbookhelper.cxx
+++ b/oox/source/xls/workbookhelper.cxx
@@ -365,7 +365,7 @@ Reference< XNamedRange > WorkbookGlobals::createNamedRangeObject( OUString& orNa
 {
     // create the name and insert it into the Calc document
     Reference< XNamedRange > xNamedRange;
-    if( orName.getLength() > 0 ) try
+    if( !orName.isEmpty() ) try
     {
         // find an unused name
         PropertySet aDocProps( mxDoc );
@@ -387,7 +387,7 @@ Reference< XNamedRange > WorkbookGlobals::createLocalNamedRangeObject( OUString&
 {
     // create the name and insert it into the Calc document
     Reference< XNamedRange > xNamedRange;
-    if( orName.getLength() > 0 ) try
+    if( !orName.isEmpty() ) try
     {
         // find an unused name
         Reference< XIndexAccess > xSheets(mxDoc->getSheets(), UNO_QUERY_THROW);
@@ -418,7 +418,7 @@ Reference< XDatabaseRange > WorkbookGlobals::createDatabaseRangeObject( OUString
 
     // create database range and insert it into the Calc document
     Reference< XDatabaseRange > xDatabaseRange;
-    if( bValidRange && (orName.getLength() > 0) ) try
+    if( bValidRange && !orName.isEmpty() ) try
     {
         // find an unused name
         PropertySet aDocProps( mxDoc );
diff --git a/oox/source/xls/worksheetbuffer.cxx b/oox/source/xls/worksheetbuffer.cxx
index 84ffb4c..be741ab 100644
--- a/oox/source/xls/worksheetbuffer.cxx
+++ b/oox/source/xls/worksheetbuffer.cxx
@@ -207,7 +207,7 @@ WorksheetBuffer::IndexNamePair WorksheetBuffer::createSheet( const OUString& rPr
         Reference< XIndexAccess > xSheetsIA( xSheets, UNO_QUERY_THROW );
         Reference< XNameAccess > xSheetsNA( xSheets, UNO_QUERY_THROW );
         sal_Int16 nCalcSheet = -1;
-        OUString aSheetName = (rPreferredName.getLength() == 0) ? CREATE_OUSTRING( "Sheet" ) : 
rPreferredName;
+        OUString aSheetName = rPreferredName.isEmpty() ? CREATE_OUSTRING( "Sheet" ) : 
rPreferredName;
         PropertySet aPropSet;
         if( nSheetPos < xSheetsIA->getCount() )
         {
diff --git a/oox/source/xls/worksheetfragment.cxx b/oox/source/xls/worksheetfragment.cxx
index 3832a1b..a045c46 100644
--- a/oox/source/xls/worksheetfragment.cxx
+++ b/oox/source/xls/worksheetfragment.cxx
@@ -224,7 +224,7 @@ WorksheetFragment::WorksheetFragment( const WorksheetHelper& rHelper, const OUSt
 
     // import comments related to this worksheet
     OUString aCommentsFragmentPath = getFragmentPathFromFirstType( CREATE_OFFICEDOC_RELATION_TYPE( 
"comments" ) );
-    if( aCommentsFragmentPath.getLength() > 0 )
+    if( !aCommentsFragmentPath.isEmpty() )
         importOoxFragment( new CommentsFragment( *this, aCommentsFragmentPath ) );
 }
 
@@ -731,7 +731,7 @@ void WorksheetFragment::importControl( SequenceInputStream& rStrm )
 void WorksheetFragment::importEmbeddedOleData( StreamDataSequence& orEmbeddedData, const OUString& 
rRelId )
 {
     OUString aFragmentPath = getFragmentPathFromRelId( rRelId );
-    if( aFragmentPath.getLength() > 0 )
+    if( !aFragmentPath.isEmpty() )
         getBaseFilter().importBinaryData( orEmbeddedData, aFragmentPath );
 }
 
diff --git a/oox/source/xls/worksheethelper.cxx b/oox/source/xls/worksheethelper.cxx
index e8a2972..14e7d24 100644
--- a/oox/source/xls/worksheethelper.cxx
+++ b/oox/source/xls/worksheethelper.cxx
@@ -979,7 +979,7 @@ void WorksheetGlobals::finalizeHyperlinkRanges() const
     {
         OUString aUrl = getHyperlinkUrl( *aIt );
         // try to insert URL into each cell of the range
-        if( aUrl.getLength() > 0 )
+        if( !aUrl.isEmpty() )
             for( CellAddress aAddress( getSheetIndex(), aIt->maRange.StartColumn, 
aIt->maRange.StartRow ); aAddress.Row <= aIt->maRange.EndRow; ++aAddress.Row )
                 for( aAddress.Column = aIt->maRange.StartColumn; aAddress.Column <= 
aIt->maRange.EndColumn; ++aAddress.Column )
                     insertHyperlink( aAddress, aUrl );
@@ -989,14 +989,14 @@ void WorksheetGlobals::finalizeHyperlinkRanges() const
 OUString WorksheetGlobals::getHyperlinkUrl( const HyperlinkModel& rHyperlink ) const
 {
     OUStringBuffer aUrlBuffer;
-    if( rHyperlink.maTarget.getLength() > 0 )
+    if( !rHyperlink.maTarget.isEmpty() )
         aUrlBuffer.append( getBaseFilter().getAbsoluteUrl( rHyperlink.maTarget ) );
-    if( rHyperlink.maLocation.getLength() > 0 )
+    if( !rHyperlink.maLocation.isEmpty() )
         aUrlBuffer.append( sal_Unicode( '#' ) ).append( rHyperlink.maLocation );
     OUString aUrl = aUrlBuffer.makeStringAndClear();
 
     // convert '#SheetName!A1' to '#SheetName.A1'
-    if( (aUrl.getLength() > 0) && (aUrl[ 0 ] == '#') )
+    if( !aUrl.isEmpty() && (aUrl[ 0 ] == '#') )
     {
         sal_Int32 nSepPos = aUrl.lastIndexOf( '!' );
         if( nSepPos > 0 )
@@ -1006,7 +1006,7 @@ OUString WorksheetGlobals::getHyperlinkUrl( const HyperlinkModel& rHyperlink 
) c
             // #i66592# convert sheet names that have been renamed on import
             OUString aSheetName = aUrl.copy( 1, nSepPos - 1 );
             OUString aCalcName = getWorksheets().getCalcSheetName( aSheetName );
-            if( aCalcName.getLength() > 0 )
+            if( !aCalcName.isEmpty() )
                 aUrl = aUrl.replaceAt( 1, nSepPos - 1, aCalcName );
         }
     }
@@ -1316,9 +1316,9 @@ void WorksheetGlobals::finalizeDrawings()
     {
         case FILTER_OOXML:
             // import DML and VML
-            if( maDrawingPath.getLength() > 0 )
+            if( !maDrawingPath.isEmpty() )
                 importOoxFragment( new DrawingFragment( *this, maDrawingPath ) );
-            if( maVmlDrawingPath.getLength() > 0 )
+            if( !maVmlDrawingPath.isEmpty() )
                 importOoxFragment( new VmlDrawingFragment( *this, maVmlDrawingPath ) );
         break;
 
-- 
1.7.5.4


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.