summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOlivier Hallot <olivier.hallot@alta.org.br>2011-12-23 18:23:14 -0200
committerIvan Timofeev <timofeev.i.s@gmail.com>2011-12-26 14:05:06 +0400
commite7bddf732798508e347221590110486e97bb45e6 (patch)
tree5b37f87e2683d758ae9429215fa9cdec26a40b4a
parentecf24119aeb5e25cf0f862b8af2070488cc57908 (diff)
Fix for fdo43460 Part XVII getLength() to isEmpty()
Part XVII Module fileaccess and filter
-rw-r--r--fileaccess/source/FileAccess.cxx2
-rw-r--r--filter/source/config/cache/basecontainer.cxx6
-rw-r--r--filter/source/config/cache/cacheitem.cxx4
-rw-r--r--filter/source/config/cache/cacheupdatelistener.cxx2
-rw-r--r--filter/source/config/cache/filtercache.cxx28
-rw-r--r--filter/source/config/cache/filterfactory.cxx6
-rw-r--r--filter/source/config/cache/querytokenizer.cxx2
-rw-r--r--filter/source/config/cache/typedetection.cxx42
-rw-r--r--filter/source/filtertracer/filtertracer.cxx2
-rw-r--r--filter/source/msfilter/escherex.cxx18
-rw-r--r--filter/source/msfilter/eschesdo.cxx2
-rw-r--r--filter/source/msfilter/msdffimp.cxx8
-rw-r--r--filter/source/msfilter/msfiltertracer.cxx14
-rw-r--r--filter/source/msfilter/mstoolbar.cxx6
-rw-r--r--filter/source/msfilter/msvbahelper.cxx16
-rw-r--r--filter/source/pdf/impdialog.cxx10
-rw-r--r--filter/source/pdf/pdfexport.cxx16
-rw-r--r--filter/source/placeware/exporter.cxx8
-rw-r--r--filter/source/placeware/tempfile.cxx2
-rw-r--r--filter/source/placeware/zip.cxx4
-rw-r--r--filter/source/svg/svgexport.cxx8
-rw-r--r--filter/source/svg/svgfontexport.cxx4
-rw-r--r--filter/source/svg/svgreader.cxx4
-rw-r--r--filter/source/svg/svgwriter.cxx6
-rw-r--r--filter/source/xmlfilteradaptor/XmlFilterAdaptor.cxx2
-rw-r--r--filter/source/xmlfilterdetect/filterdetect.cxx8
-rw-r--r--filter/source/xsltdialog/typedetectionexport.cxx6
-rw-r--r--filter/source/xsltdialog/typedetectionimport.cxx8
-rw-r--r--filter/source/xsltdialog/xmlfileview.cxx4
-rw-r--r--filter/source/xsltdialog/xmlfilterjar.cxx12
-rw-r--r--filter/source/xsltdialog/xmlfiltersettingsdialog.cxx14
-rw-r--r--filter/source/xsltdialog/xmlfiltertabdialog.cxx10
-rw-r--r--filter/source/xsltdialog/xmlfiltertabpagebasic.cxx4
-rw-r--r--filter/source/xsltdialog/xmlfiltertabpagexslt.cxx2
-rw-r--r--filter/source/xsltdialog/xmlfiltertestdialog.cxx16
-rw-r--r--filter/source/xsltfilter/XSLTFilter.cxx4
36 files changed, 155 insertions, 155 deletions
diff --git a/fileaccess/source/FileAccess.cxx b/fileaccess/source/FileAccess.cxx
index 15867f8e24fa..58521835dbcf 100644
--- a/fileaccess/source/FileAccess.cxx
+++ b/fileaccess/source/FileAccess.cxx
@@ -425,7 +425,7 @@ void OFileAccess::createFolder( const rtl::OUString& NewFolderURL )
throw(CommandAbortedException, Exception, RuntimeException)
{
// Does the folder already exist?
- if( !NewFolderURL.getLength() || isFolder( NewFolderURL ) )
+ if( NewFolderURL.isEmpty() || isFolder( NewFolderURL ) )
return;
// SfxContentHelper::MakeFolder
diff --git a/filter/source/config/cache/basecontainer.cxx b/filter/source/config/cache/basecontainer.cxx
index ea47575cc80c..7fa2faac3dcf 100644
--- a/filter/source/config/cache/basecontainer.cxx
+++ b/filter/source/config/cache/basecontainer.cxx
@@ -219,7 +219,7 @@ void SAL_CALL BaseContainer::insertByName(const ::rtl::OUString& sItem ,
css::lang::WrappedTargetException ,
css::uno::RuntimeException )
{
- if (!sItem.getLength())
+ if (sItem.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "empty value not allowed as item name." )),
static_cast< css::container::XNameContainer* >(this),
@@ -283,7 +283,7 @@ void SAL_CALL BaseContainer::replaceByName(const ::rtl::OUString& sItem ,
css::lang::WrappedTargetException ,
css::uno::RuntimeException )
{
- if (!sItem.getLength())
+ if (sItem.isEmpty())
throw css::lang::IllegalArgumentException(
::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "empty value not allowed as item name." )),
static_cast< css::container::XNameContainer* >(this),
@@ -323,7 +323,7 @@ css::uno::Any SAL_CALL BaseContainer::getByName(const ::rtl::OUString& sItem)
css::lang::WrappedTargetException ,
css::uno::RuntimeException )
{
- if (!sItem.getLength())
+ if (sItem.isEmpty())
throw css::container::NoSuchElementException(
::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "An empty item cant be part of this cache!" )),
css::uno::Reference< css::uno::XInterface >(static_cast< css::container::XNameAccess* >(this), css::uno::UNO_QUERY));
diff --git a/filter/source/config/cache/cacheitem.cxx b/filter/source/config/cache/cacheitem.cxx
index 36211bee0a94..88153a092ada 100644
--- a/filter/source/config/cache/cacheitem.cxx
+++ b/filter/source/config/cache/cacheitem.cxx
@@ -75,7 +75,7 @@ void CacheItem::update(const CacheItem& rUpdateItem)
void CacheItem::validateUINames(const ::rtl::OUString& sActLocale)
{
- if (!sActLocale.getLength())
+ if (sActLocale.isEmpty())
return;
// 1) check UINames first
@@ -90,7 +90,7 @@ void CacheItem::validateUINames(const ::rtl::OUString& sActLocale)
if (pUIName != end())
pUIName->second >>= sUIName;
- if (sUIName.getLength())
+ if (!sUIName.isEmpty())
{
// 1a) set UIName inside list of UINames for current locale
lUINames[sActLocale] <<= sUIName;
diff --git a/filter/source/config/cache/cacheupdatelistener.cxx b/filter/source/config/cache/cacheupdatelistener.cxx
index bf5f964e2f6a..09997222e1fe 100644
--- a/filter/source/config/cache/cacheupdatelistener.cxx
+++ b/filter/source/config/cache/cacheupdatelistener.cxx
@@ -179,7 +179,7 @@ void SAL_CALL CacheUpdateListener::changesOccurred(const css::util::ChangesEven
}
}
- if ( ! sNode.getLength() )
+ if ( sNode.isEmpty() )
continue;
OUStringList::const_iterator pIt = ::std::find(lChangedItems.begin(), lChangedItems.end(), sNode);
diff --git a/filter/source/config/cache/filtercache.cxx b/filter/source/config/cache/filtercache.cxx
index 31a61b120b1f..b623e136f7d2 100644
--- a/filter/source/config/cache/filtercache.cxx
+++ b/filter/source/config/cache/filtercache.cxx
@@ -232,24 +232,24 @@ void FilterCache::load(EFillState eRequired,
if (m_eFillState == E_CONTAINS_NOTHING)
{
impl_getDirectCFGValue(CFGDIRECTKEY_OFFICELOCALE) >>= m_sActLocale;
- if (!m_sActLocale.getLength())
+ if (m_sActLocale.isEmpty())
{
_FILTER_CONFIG_LOG_1_("FilterCache::ctor() ... could not specify office locale => use default \"%s\"\n", _FILTER_CONFIG_TO_ASCII_(DEFAULT_OFFICELOCALE));
m_sActLocale = DEFAULT_OFFICELOCALE;
}
impl_getDirectCFGValue(CFGDIRECTKEY_FORMATNAME) >>= m_sFormatName;
- if (!m_sFormatName.getLength())
+ if (m_sFormatName.isEmpty())
impl_getDirectCFGValue(CFGDIRECTKEY_PRODUCTNAME) >>= m_sFormatName;
- if (!m_sFormatName.getLength())
+ if (m_sFormatName.isEmpty())
{
_FILTER_CONFIG_LOG_1_("FilterCache::ctor() ... could not specify format name => use default \"%s\"\n", _FILTER_CONFIG_TO_ASCII_(DEFAULT_FORMATNAME));
m_sFormatName = DEFAULT_FORMATNAME;
}
impl_getDirectCFGValue(CFGDIRECTKEY_FORMATVERSION) >>= m_sFormatVersion;
- if (!m_sFormatVersion.getLength())
+ if (m_sFormatVersion.isEmpty())
{
_FILTER_CONFIG_LOG_1_("FilterCache::ctor() ... could not specify format version => use default \"%s\"\n", _FILTER_CONFIG_TO_ASCII_(DEFAULT_FORMATVERSION));
m_sFormatVersion = DEFAULT_FORMATVERSION;
@@ -557,7 +557,7 @@ void FilterCache::addStatePropsToItem( EItemType eType,
::rtl::OUString sDefaultFrameLoader;
if (
(aDirectValue >>= sDefaultFrameLoader) &&
- (sDefaultFrameLoader.getLength() ) &&
+ (!sDefaultFrameLoader.isEmpty() ) &&
(sItem.equals(sDefaultFrameLoader) )
)
{
@@ -911,8 +911,8 @@ css::uno::Any FilterCache::impl_getDirectCFGValue(const ::rtl::OUString& sDirect
if (
(!::utl::splitLastFromConfigurationPath(sDirectKey, sRoot, sKey)) ||
- (!sRoot.getLength() ) ||
- (!sKey.getLength() )
+ (sRoot.isEmpty() ) ||
+ (sKey.isEmpty() )
)
return css::uno::Any();
@@ -1066,7 +1066,7 @@ void FilterCache::impl_validateAndOptimize()
// Because these informations are available as type properties!
::rtl::OUString sDetectService;
aType[PROPNAME_DETECTSERVICE ] >>= sDetectService;
- if (sDetectService.getLength())
+ if (!sDetectService.isEmpty())
impl_resolveItem4TypeRegistration(&m_lDetectServices, sDetectService, sType);
// get its registration for file Extensions AND(!) URLPattern ...
@@ -1157,7 +1157,7 @@ void FilterCache::impl_validateAndOptimize()
::rtl::OUString sPrefFilter;
aType[PROPNAME_PREFERREDFILTER] >>= sPrefFilter;
- if (!sPrefFilter.getLength())
+ if (sPrefFilter.isEmpty())
{
// OK - there is no filter for this type. But thats not an error.
// May be it can be handled by a ContentHandler ...
@@ -1166,10 +1166,10 @@ void FilterCache::impl_validateAndOptimize()
sal_Bool bReferencedByLoader = sal_True;
sal_Bool bReferencedByHandler = sal_True;
if (bAllLoadersShouldExist)
- bReferencedByLoader = (impl_searchFrameLoaderForType(sType).getLength()!=0);
+ bReferencedByLoader = !impl_searchFrameLoaderForType(sType).isEmpty();
if (bAllHandlersShouldExist)
- bReferencedByHandler = (impl_searchContentHandlerForType(sType).getLength()!=0);
+ bReferencedByHandler = !impl_searchContentHandlerForType(sType).isEmpty();
if (
(!bReferencedByLoader ) &&
@@ -1184,7 +1184,7 @@ void FilterCache::impl_validateAndOptimize()
}
}
- if (sPrefFilter.getLength())
+ if (!sPrefFilter.isEmpty())
{
CacheItemList::const_iterator pIt2 = m_lFilters.find(sPrefFilter);
if (pIt2 == m_lFilters.end())
@@ -1260,7 +1260,7 @@ void FilterCache::impl_validateAndOptimize()
if (
(!(aDirectValue >>= sDefaultFrameLoader)) ||
- (!sDefaultFrameLoader.getLength() )
+ (sDefaultFrameLoader.isEmpty() )
)
{
sLog.appendAscii("error\t:\t" );
@@ -2371,7 +2371,7 @@ CacheItem FilterCache::impl_readOldItem(const css::uno::Reference< css::containe
xItem->getByName(::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "Data" ))) >>= sData;
lData = impl_tokenizeString(sData, (sal_Unicode)',');
if (
- (!sData.getLength()) ||
+ (sData.isEmpty()) ||
(lData.size()<1 )
)
{
diff --git a/filter/source/config/cache/filterfactory.cxx b/filter/source/config/cache/filterfactory.cxx
index f2b138e929ee..4620b3e4b4f3 100644
--- a/filter/source/config/cache/filterfactory.cxx
+++ b/filter/source/config/cache/filterfactory.cxx
@@ -173,7 +173,7 @@ css::uno::Reference< css::uno::XInterface > SAL_CALL FilterFactory::createInstan
// create service instance
css::uno::Reference< css::uno::XInterface > xFilter;
- if (sFilterService.getLength())
+ if (!sFilterService.isEmpty())
xFilter = m_xSMGR->createInstance(sFilterService);
// initialize filter
@@ -377,7 +377,7 @@ OUStringList FilterFactory::impl_queryMatchByDocumentService(const QueryTokenize
// "matchByDocumentService=com.sun.star..." => only filter matching this document service will be adressed
::rtl::OUString sCheckValue = aFilter.getUnpackedValueOrDefault(PROPNAME_DOCUMENTSERVICE, ::rtl::OUString());
if (
- ( sDocumentService.getLength() ) &&
+ (!sDocumentService.isEmpty() ) &&
(!sDocumentService.equals(QUERY_CONSTVALUE_ALL)) &&
(!sCheckValue.equals(sDocumentService) )
)
@@ -491,7 +491,7 @@ OUStringList FilterFactory::impl_getSortedFilterList(const QueryTokenizer& lToke
// simple search for filters of one specific module.
OUStringList lFilterList;
- if (sModule.getLength())
+ if (!sModule.isEmpty())
lFilterList = impl_getSortedFilterListForModule(sModule, nIFlags, nEFlags);
else
{
diff --git a/filter/source/config/cache/querytokenizer.cxx b/filter/source/config/cache/querytokenizer.cxx
index 9e2157e5771f..273b4d425652 100644
--- a/filter/source/config/cache/querytokenizer.cxx
+++ b/filter/source/config/cache/querytokenizer.cxx
@@ -50,7 +50,7 @@ QueryTokenizer::QueryTokenizer(const ::rtl::OUString& sQuery)
while(token != -1)
{
::rtl::OUString sToken = sQuery.getToken(0, ':', token);
- if (sToken.getLength())
+ if (!sToken.isEmpty())
{
sal_Int32 equal = sToken.indexOf('=');
diff --git a/filter/source/config/cache/typedetection.cxx b/filter/source/config/cache/typedetection.cxx
index cc91746c2b37..f1a835dbd266 100644
--- a/filter/source/config/cache/typedetection.cxx
+++ b/filter/source/config/cache/typedetection.cxx
@@ -180,7 +180,7 @@ TypeDetection::~TypeDetection()
// the might needed input stream, start detection
// which uses all registered deep detection services.
if (
- (!sType.getLength()) &&
+ (sType.isEmpty()) &&
(bAllowDeep )
)
{
@@ -192,13 +192,13 @@ TypeDetection::~TypeDetection()
// pure deep detection failed
// => ask might existing InteractionHandler
// means: ask user for it's decision
- if (!sType.getLength())
+ if (sType.isEmpty())
sType = impl_askUserForTypeAndFilterIfAllowed(stlDescriptor);
//*******************************************
// no real detected type - but a might valid one.
// update descriptor and set last chance for return.
- if (!sType.getLength() && sLastChance.getLength())
+ if (sType.isEmpty() && !sLastChance.isEmpty())
{
OSL_FAIL("set first flat detected type without a registered deep detection service as \"last chance\" ... nevertheless some other deep detections said \"NO\". I TRY IT!");
sType = sLastChance;
@@ -229,7 +229,7 @@ void TypeDetection::impl_checkResultsAndAddBestFilter(::comphelper::MediaDescrip
::rtl::OUString sFilter = rDescriptor.getUnpackedValueOrDefault(
::comphelper::MediaDescriptor::PROP_FILTERNAME(),
::rtl::OUString());
- if (sFilter.getLength())
+ if (!sFilter.isEmpty())
return;
// b)
@@ -238,7 +238,7 @@ void TypeDetection::impl_checkResultsAndAddBestFilter(::comphelper::MediaDescrip
::rtl::OUString sDocumentService = rDescriptor.getUnpackedValueOrDefault(
::comphelper::MediaDescriptor::PROP_DOCUMENTSERVICE(),
::rtl::OUString());
- if (sDocumentService.getLength())
+ if (!sDocumentService.isEmpty())
{
try
{
@@ -308,7 +308,7 @@ void TypeDetection::impl_checkResultsAndAddBestFilter(::comphelper::MediaDescrip
// <- SAFE
}
- if (sFilter.getLength() > 0)
+ if (!sFilter.isEmpty())
{
rDescriptor[::comphelper::MediaDescriptor::PROP_TYPENAME() ] <<= sRealType;
rDescriptor[::comphelper::MediaDescriptor::PROP_FILTERNAME()] <<= sFilter;
@@ -394,7 +394,7 @@ void TypeDetection::impl_checkResultsAndAddBestFilter(::comphelper::MediaDescrip
sFilter = ::rtl::OUString();
}
- if (sFilter.getLength())
+ if (!sFilter.isEmpty())
{
rDescriptor[::comphelper::MediaDescriptor::PROP_TYPENAME() ] <<= sType ;
rDescriptor[::comphelper::MediaDescriptor::PROP_FILTERNAME()] <<= sFilter;
@@ -505,7 +505,7 @@ sal_Bool TypeDetection::impl_getPreselectionForType(const ::rtl::OUString& sPreS
}
// if its a valid type - set it on all return values!
- if (sType.getLength())
+ if (!sType.isEmpty())
{
FlatDetectionInfo aInfo;
aInfo.sType = sType;
@@ -578,7 +578,7 @@ sal_Bool TypeDetection::impl_getPreselectionForFilter(const ::rtl::OUString& sPr
rInfo.bPreselectedByFilter = sal_True;
}
- if (sFilter.getLength())
+ if (!sFilter.isEmpty())
return sal_True;
else
return sal_False;
@@ -687,15 +687,15 @@ void TypeDetection::impl_getPreselection(const css::util::URL& aP
*/
::rtl::OUString sSelectedType = rDescriptor.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_TYPENAME(), ::rtl::OUString());
- if (sSelectedType.getLength())
+ if (!sSelectedType.isEmpty())
impl_getPreselectionForType(sSelectedType, aParsedURL, rFlatTypes);
::rtl::OUString sSelectedFilter = rDescriptor.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_FILTERNAME(), ::rtl::OUString());
- if (sSelectedFilter.getLength())
+ if (!sSelectedFilter.isEmpty())
impl_getPreselectionForFilter(sSelectedFilter, aParsedURL, rFlatTypes);
::rtl::OUString sSelectedDoc = rDescriptor.getUnpackedValueOrDefault(::comphelper::MediaDescriptor::PROP_DOCUMENTSERVICE(), ::rtl::OUString());
- if (sSelectedDoc.getLength())
+ if (!sSelectedDoc.isEmpty())
impl_getPreselectionForDocumentService(sSelectedDoc, aParsedURL, rFlatTypes);
}
@@ -757,7 +757,7 @@ void TypeDetection::impl_getPreselection(const css::util::URL& aP
aType[PROPNAME_DETECTSERVICE] >>= sDetectService;
// c)
- if (!sDetectService.getLength())
+ if (sDetectService.isEmpty())
{
// accept or not accept flat types without deep detection: that's the question :-)
// May be there exists some states, where we have to use our LastChance feature instead
@@ -778,7 +778,7 @@ void TypeDetection::impl_getPreselection(const css::util::URL& aP
// preselected by the user can be used as LAST CHANCE in case no other type could
// be detected. Of course only the first type without deep detector can be used.
// Further ones has to be ignored.
- if (rLastChance.getLength() < 1)
+ if (rLastChance.isEmpty())
rLastChance = sFlatType;
continue;
@@ -791,7 +791,7 @@ void TypeDetection::impl_getPreselection(const css::util::URL& aP
::rtl::OUString sDeepType = impl_askDetectService(sDetectService, rDescriptor);
// d)
- if (sDeepType.getLength())
+ if (!sDeepType.isEmpty())
return sDeepType;
}
catch(const css::container::NoSuchElementException&)
@@ -843,7 +843,7 @@ namespace
{
const ::rtl::OUString& sDetectService = *pIt;
::rtl::OUString sDeepType = impl_askDetectService(sDetectService, rDescriptor);
- if (sDeepType.getLength())
+ if (!sDeepType.isEmpty())
return sDeepType;
lInsideUsedDetectors.push_back(sDetectService);
}
@@ -863,7 +863,7 @@ namespace
::rtl::OUString sPreselDocumentService = rDescriptor.getUnpackedValueOrDefault(
::comphelper::MediaDescriptor::PROP_DOCUMENTSERVICE(),
::rtl::OUString());
- if (sPreselDocumentService.getLength())
+ if (!sPreselDocumentService.isEmpty())
{
for ( pIt = lDetectors.begin();
pIt != lDetectors.end() ;
@@ -921,7 +921,7 @@ namespace
if (bMatchDetectorToDocumentService)
{
::rtl::OUString sDeepType = impl_askDetectService(sDetectService, rDescriptor);
- if (sDeepType.getLength())
+ if (!sDeepType.isEmpty())
return sDeepType;
lInsideUsedDetectors.push_back(sDetectService);
}
@@ -942,7 +942,7 @@ namespace
continue;
::rtl::OUString sDeepType = impl_askDetectService(sDetectService, rDescriptor);
- if (sDeepType.getLength())
+ if (!sDeepType.isEmpty())
return sDeepType;
}
@@ -1090,7 +1090,7 @@ void TypeDetection::impl_seekStreamToZero(comphelper::MediaDescriptor& rDescript
// and not for "missing files". Especialy if detection is done by a stream only
// we cant check if the stream points to an "existing content"!
if (
- (!sURL.getLength() ) || // "non existing file" ?
+ (sURL.isEmpty() ) || // "non existing file" ?
(!xStream.is() ) || // non existing file !
(sURL.equalsIgnoreAsciiCaseAsciiL("private:stream", 14)) // not a good idea .-)
)
@@ -1134,7 +1134,7 @@ void TypeDetection::impl_openStream(::comphelper::MediaDescriptor& rDescriptor)
sal_Bool bSuccess = sal_False;
::rtl::OUString sURL = rDescriptor.getUnpackedValueOrDefault( ::comphelper::MediaDescriptor::PROP_URL(), ::rtl::OUString() );
sal_Bool bRequestedReadOnly = rDescriptor.getUnpackedValueOrDefault( ::comphelper::MediaDescriptor::PROP_READONLY(), sal_False );
- if ( sURL.getLength() && ::utl::LocalFileHelper::IsLocalFile( INetURLObject( sURL ).GetMainURL( INetURLObject::NO_DECODE ) ) )
+ if ( !sURL.isEmpty() && ::utl::LocalFileHelper::IsLocalFile( INetURLObject( sURL ).GetMainURL( INetURLObject::NO_DECODE ) ) )
{
// OOo uses own file locking mechanics in case of local file
bSuccess = rDescriptor.addInputStreamOwnLock();
diff --git a/filter/source/filtertracer/filtertracer.cxx b/filter/source/filtertracer/filtertracer.cxx
index 00ca6fc8b3c7..5cfa62950805 100644
--- a/filter/source/filtertracer/filtertracer.cxx
+++ b/filter/source/filtertracer/filtertracer.cxx
@@ -133,7 +133,7 @@ void SAL_CALL FilterTracer::initialize( const SEQ( NMSP_UNO::Any )& aArguments )
}
// check if we have to create the XOutputStream
- if ( !mxOutputStream.is() && msURL.getLength() )
+ if ( !(mxOutputStream.is() || msURL.isEmpty()) )
{
mpStream = ::utl::UcbStreamHelper::CreateStream( msURL, STREAM_WRITE | STREAM_TRUNC | STREAM_SHARE_DENYNONE );
if ( mpStream )
diff --git a/filter/source/msfilter/escherex.cxx b/filter/source/msfilter/escherex.cxx
index 10f17706357d..b8d64f794097 100644
--- a/filter/source/msfilter/escherex.cxx
+++ b/filter/source/msfilter/escherex.cxx
@@ -746,7 +746,7 @@ sal_Bool EscherPropertyContainer::GetLineArrow( const sal_Bool bLineStart,
sal_Int16 nWhich = bLineStart ? XATTR_LINESTART : XATTR_LINEEND;
SvxUnogetApiNameForItem( nWhich, aArrowStartName, aApiName );
- if ( aApiName.getLength() )
+ if ( !aApiName.isEmpty() )
{
/* todo:
@@ -1131,7 +1131,7 @@ sal_Bool EscherPropertyContainer::CreateOLEGraphicProperties(
{
GraphicObject aGraphicObject( *pGraphic );
rtl::OString aUniqueId( aGraphicObject.GetUniqueID() );
- if ( aUniqueId.getLength() )
+ if ( !aUniqueId.isEmpty() )
{
AddOpt( ESCHER_Prop_fillType, ESCHER_FillPicture );
uno::Reference< beans::XPropertySet > aXPropSet( rXShape, uno::UNO_QUERY );
@@ -1165,7 +1165,7 @@ sal_Bool EscherPropertyContainer::CreateOLEGraphicProperties(
sal_Bool EscherPropertyContainer::ImplCreateEmbeddedBmp( const rtl::OString& rUniqueId )
{
- if( rUniqueId.getLength() > 0 )
+ if( !rUniqueId.isEmpty() )
{
EscherGraphicProvider aProvider;
SvMemoryStream aMemStrm;
@@ -1409,7 +1409,7 @@ sal_Bool EscherPropertyContainer::CreateGraphicProperties(
delete pIn;
}
}
- if ( !aUniqueId.getLength() )
+ if ( aUniqueId.isEmpty() )
{
if ( pGraphicProvider )
{
@@ -1419,7 +1419,7 @@ sal_Bool EscherPropertyContainer::CreateGraphicProperties(
{
rtl::OUString aRelUrl( INetURLObject::GetRelURL( rBaseURI, aGraphicUrl,
INetURLObject::WAS_ENCODED, INetURLObject::DECODE_TO_IURI, RTL_TEXTENCODING_UTF8, INetURLObject::FSYS_DETECT ) );
- if ( aRelUrl.getLength() )
+ if ( !aRelUrl.isEmpty() )
aGraphicUrl = aRelUrl;
}
}
@@ -1427,7 +1427,7 @@ sal_Bool EscherPropertyContainer::CreateGraphicProperties(
}
}
- if ( aGraphicUrl.Len() || aUniqueId.getLength() )
+ if ( aGraphicUrl.Len() || !aUniqueId.isEmpty() )
{
if ( bMirrored || nAngle )
{
@@ -1454,7 +1454,7 @@ sal_Bool EscherPropertyContainer::CreateGraphicProperties(
else
AddOpt( ESCHER_Prop_fillType, ESCHER_FillPicture );
- if ( aUniqueId.getLength() )
+ if ( !aUniqueId.isEmpty() )
{
// write out embedded graphic
if ( pGraphicProvider && pPicOutStrm && pShapeBoundRect )
@@ -2960,7 +2960,7 @@ void EscherPropertyContainer::CreateCustomShapeProperties( const MSO_SPT eShapeT
uno::Reference< text::XSimpleText > xText( rXShape, uno::UNO_QUERY );
if ( xText.is() )
aText = xText->getString();
- if ( !aText.getLength() )
+ if ( aText.isEmpty() )
aText = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "your text" )); // todo: moving into a resource
AddOpt( DFF_Prop_gtextUNICODE, aText );
@@ -2969,7 +2969,7 @@ void EscherPropertyContainer::CreateCustomShapeProperties( const MSO_SPT eShapeT
const rtl::OUString sCharFontName ( RTL_CONSTASCII_USTRINGPARAM( "CharFontName" ) );
uno::Any aAny = aXPropSet->getPropertyValue( sCharFontName );
aAny >>= aFontName;
- if ( !aFontName.getLength() )
+ if ( aFontName.isEmpty() )
aFontName = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "Arial Black" ));
AddOpt( DFF_Prop_gtextFont, aFontName );
diff --git a/filter/source/msfilter/eschesdo.cxx b/filter/source/msfilter/eschesdo.cxx
index 96e81d9dd037..256b06a1249d 100644
--- a/filter/source/msfilter/eschesdo.cxx
+++ b/filter/source/msfilter/eschesdo.cxx
@@ -1119,7 +1119,7 @@ sal_uInt32 ImplEESdrObject::ImplGetText()
sal_Bool ImplEESdrObject::ImplHasText() const
{
Reference< XText > xXText( mXShape, UNO_QUERY );
- return xXText.is() && xXText->getString().getLength();
+ return xXText.is() && !xXText->getString().isEmpty();
}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/filter/source/msfilter/msdffimp.cxx b/filter/source/msfilter/msdffimp.cxx
index 4b3decedb737..038dcbe0b12f 100644
--- a/filter/source/msfilter/msdffimp.cxx
+++ b/filter/source/msfilter/msdffimp.cxx
@@ -5125,7 +5125,7 @@ SdrObject* SvxMSDffManager::ImportShape( const DffRecordHeader& rHd, SvStream& r
if( pRet )
{
::rtl::OUString aObjName = GetPropertyString( DFF_Prop_wzName, rSt );
- if( aObjName.getLength() > 0 )
+ if( !aObjName.isEmpty() )
pRet->SetName( aObjName );
}
@@ -7337,14 +7337,14 @@ com::sun::star::uno::Reference < com::sun::star::embed::XEmbeddedObject > SvxMS
else
aFilterName = GetFilterNameFromClassID_Impl( aStgNm );
- uno::Sequence < beans::PropertyValue > aMedium( aFilterName.getLength() ? 3 : 2);
+ uno::Sequence < beans::PropertyValue > aMedium( aFilterName.isEmpty() ? 2 : 3);
aMedium[0].Name = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "InputStream" ) );
uno::Reference < io::XInputStream > xStream = new ::utl::OSeekableInputStreamWrapper( *pStream );
aMedium[0].Value <<= xStream;
aMedium[1].Name = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "URL" ) );
aMedium[1].Value <<= ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "private:stream" ) );
- if ( aFilterName.getLength() )
+ if ( !aFilterName.isEmpty() )
{
aMedium[2].Name = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "FilterName" ) );
aMedium[2].Value <<= aFilterName;
@@ -7356,7 +7356,7 @@ com::sun::star::uno::Reference < com::sun::star::embed::XEmbeddedObject > SvxMS
if ( !xObj.is() )
{
- if( aFilterName.getLength() )
+ if( !aFilterName.isEmpty() )
{
// throw the filter parameter away as workaround
aMedium.realloc( 2 );
diff --git a/filter/source/msfilter/msfiltertracer.cxx b/filter/source/msfilter/msfiltertracer.cxx
index f4840bc074de..0bc2ccbbf2c5 100644
--- a/filter/source/msfilter/msfiltertracer.cxx
+++ b/filter/source/msfilter/msfiltertracer.cxx
@@ -71,20 +71,20 @@ MSFilterTracer::MSFilterTracer( const ::rtl::OUString& rConfigPath, uno::Sequenc
rtl::OUString aName( mpCfgItem->ReadString( rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "Name" )), aEmptyString ) );
rtl::OUString aDocumentURL( mpCfgItem->ReadString( rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "DocumentURL" )), aEmptyString ) );
INetURLObject aLogFile( aDocumentURL );
- if ( aLogFile.GetMainURL( INetURLObject::NO_DECODE ).getLength() )
+ if ( !aLogFile.GetMainURL( INetURLObject::NO_DECODE ).isEmpty() )
{
- if ( aPath.getLength() )
+ if ( !aPath.isEmpty() )
{
String aOldName( aLogFile.getName( INetURLObject::LAST_SEGMENT, true, INetURLObject::NO_DECODE ) );
aLogFile = INetURLObject( aPath );
aLogFile.insertName( aOldName );
}
- if ( aName.getLength() )
+ if ( !aName.isEmpty() )
aLogFile.setName( aName );
}
else
{
- if ( aPath.getLength() )
+ if ( !aPath.isEmpty() )
aLogFile = INetURLObject( aPath );
else
{
@@ -96,7 +96,7 @@ MSFilterTracer::MSFilterTracer( const ::rtl::OUString& rConfigPath, uno::Sequenc
aLogFile .removeFinalSlash();
}
}
- if ( !aName.getLength() )
+ if ( aName.isEmpty() )
aName = rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "tracer" ));
aLogFile.insertName( aName );
}
@@ -191,7 +191,7 @@ void MSFilterTracer::Trace( const rtl::OUString& rElement, const rtl::OUString&
if ( mbEnabled && mxLogger.is() )
{
sal_Bool bFilter = sal_False;
- if ( rMessage.getLength() && mxTextSearch.is() )
+ if ( !rMessage.isEmpty() && mxTextSearch.is() )
{
maSearchOptions.searchString = rMessage;
mxTextSearch->setOptions( maSearchOptions );
@@ -203,7 +203,7 @@ void MSFilterTracer::Trace( const rtl::OUString& rElement, const rtl::OUString&
uno::Reference < xml::sax::XAttributeList > xAttrList( new SvXMLAttributeList( *mpAttributeList ) );
if ( mxHandler.is() )
mxHandler->startElement( rElement, xAttrList );
- if ( rMessage.getLength() )
+ if ( !rMessage.isEmpty() )
{
rtl::OUString aEmpty;
mxLogger->logp( 0, aEmpty, aEmpty, rMessage );
diff --git a/filter/source/msfilter/mstoolbar.cxx b/filter/source/msfilter/mstoolbar.cxx
index ed1c986dd2c7..044c6910b946 100644
--- a/filter/source/msfilter/mstoolbar.cxx
+++ b/filter/source/msfilter/mstoolbar.cxx
@@ -328,7 +328,7 @@ bool TBCData::ImportToolBarControl( CustomToolBarImportHelper& helper, std::vect
if ( TBCBitMap* pIcon = pSpecificInfo->getIcon() )
{
// Without a command openoffice won't display the icon
- if ( sCommand.getLength() )
+ if ( !sCommand.isEmpty() )
{
BitmapEx aBitEx( pIcon->getBitMap() );
if ( pSpecificInfo->getIconMask() )
@@ -345,7 +345,7 @@ bool TBCData::ImportToolBarControl( CustomToolBarImportHelper& helper, std::vect
{
rtl::OUString sBuiltInCmd = helper.MSOTCIDToOOCommand( *pSpecificInfo->getBtnFace() );
- if ( sBuiltInCmd.getLength() )
+ if ( !sBuiltInCmd.isEmpty() )
{
uno::Sequence< rtl::OUString> sCmds(1);
sCmds[ 0 ] = sBuiltInCmd;
@@ -505,7 +505,7 @@ TBCGeneralInfo::ImportToolBarControlData( CustomToolBarImportHelper& helper, std
beans::PropertyValue aProp;
// probably access to the header would be a better test than seeing if there is an action, e.g.
// if ( rHeader.getTct() == 0x01 && rHeader.getTcID() == 0x01 ) // not defined, probably this is a command
- if ( extraInfo.getOnAction().getLength() )
+ if ( !extraInfo.getOnAction().isEmpty() )
{
aProp.Name = rtl::OUString( RTL_CONSTASCII_USTRINGPARAM("CommandURL") );
ooo::vba::MacroResolvedInfo aMacroInf = ooo::vba::resolveVBAMacro( &helper.GetDocShell(), extraInfo.getOnAction(), true );
diff --git a/filter/source/msfilter/msvbahelper.cxx b/filter/source/msfilter/msvbahelper.cxx
index ca7f34d22c76..07a40c05092b 100644
--- a/filter/source/msfilter/msvbahelper.cxx
+++ b/filter/source/msfilter/msvbahelper.cxx
@@ -106,7 +106,7 @@ SfxObjectShell* findShellForUrl( const rtl::OUString& sMacroURLOrPath )
, rtl::OUStringToOString( aURL, RTL_TEXTENCODING_UTF8 ).getStr()
);
::rtl::OUString aName = xModel->getURL() ;
- if (0 == aName.getLength())
+ if (aName.isEmpty())
{
const static rtl::OUString sTitle( RTL_CONSTASCII_USTRINGPARAM("Title" ) );
@@ -143,7 +143,7 @@ SfxObjectShell* findShellForUrl( const rtl::OUString& sMacroURLOrPath )
// sometimes just the name of the document ( without the path
// is used
bool bDocNameNoPathMatch = false;
- if ( aURL.getLength() && aURL.indexOf( '/' ) == -1 )
+ if ( !aURL.isEmpty() && aURL.indexOf( '/' ) == -1 )
{
sal_Int32 lastSlashIndex = xModel->getURL().lastIndexOf( '/' );
if ( lastSlashIndex > -1 )
@@ -231,7 +231,7 @@ bool hasMacro( SfxObjectShell* pShell, const String& sLibrary, String& sMod, con
if( BasicManager* pBasicMgr = pShell ? pShell->GetBasicManager() : 0 )
{
aPrjName = pBasicMgr->GetName();
- if( aPrjName.getLength() == 0 )
+ if( aPrjName.isEmpty() )
aPrjName = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "Standard" ) );
}
return aPrjName;
@@ -262,7 +262,7 @@ void parseMacro( const rtl::OUString& sMacro, String& sContainer, String& sModul
{
if( pShell )
{
- ::rtl::OUString aLibName = (rLibName.getLength() > 0) ? rLibName : getDefaultProjectName( pShell );
+ ::rtl::OUString aLibName = rLibName.isEmpty() ? getDefaultProjectName( pShell ) : rLibName ;
String aModuleName = rModuleName;
if( hasMacro( pShell, aLibName, aModuleName, rMacroName ) )
return ::rtl::OUStringBuffer( aLibName ).append( sal_Unicode( '.' ) ).append( aModuleName ).append( sal_Unicode( '.' ) ).append( rMacroName ).makeStringAndClear();
@@ -372,7 +372,7 @@ MacroResolvedInfo resolveVBAMacro( SfxObjectShell* pShell, const rtl::OUString&
uno::Reference< document::XDocumentProperties > xDocProps( xDocPropSupp->getDocumentProperties(), uno::UNO_QUERY_THROW );
rtl::OUString sCreatedFrom = xDocProps->getTemplateURL();
- if ( sCreatedFrom.getLength() )
+ if ( !sCreatedFrom.isEmpty() )
{
INetURLObject aObj;
aObj.SetURL( sCreatedFrom );
@@ -393,7 +393,7 @@ MacroResolvedInfo resolveVBAMacro( SfxObjectShell* pShell, const rtl::OUString&
sCreatedFrom = sCreatedFrom.copy( 0, nIndex );
rtl::OUString sPrj;
- if ( sCreatedFrom.getLength() && xPrjNameCache->hasByName( sCreatedFrom ) )
+ if ( !sCreatedFrom.isEmpty() && xPrjNameCache->hasByName( sCreatedFrom ) )
{
xPrjNameCache->getByName( sCreatedFrom ) >>= sPrj;
// Make sure we don't double up with this project
@@ -538,7 +538,7 @@ void SAL_CALL VBAMacroResolver::initialize( const uno::Sequence< uno::Any >& rAr
throw uno::RuntimeException();
// second argument: VBA project name
- if( !(rArgs[ 1 ] >>= maProjectName) || (maProjectName.getLength() == 0) )
+ if( !(rArgs[ 1 ] >>= maProjectName) || (maProjectName.isEmpty()) )
throw uno::RuntimeException();
}
@@ -551,7 +551,7 @@ void SAL_CALL VBAMacroResolver::initialize( const uno::Sequence< uno::Any >& rAr
// the name may be enclosed in apostrophs
::rtl::OUString aMacroName = trimMacroName( rVBAMacroName );
- if( aMacroName.getLength() == 0 )
+ if( aMacroName.isEmpty() )
throw lang::IllegalArgumentException();
// external references not supported here (syntax is "url!macroname" or "[url]!macroname" or "[url]macroname")
diff --git a/filter/source/pdf/impdialog.cxx b/filter/source/pdf/impdialog.cxx
index 650f07b1efc2..2f0682c78640 100644
--- a/filter/source/pdf/impdialog.cxx
+++ b/filter/source/pdf/impdialog.cxx
@@ -156,7 +156,7 @@ ImpPDFTabDialog::ImpPDFTabDialog( Window* pParent,
else if ( nLen == 1 )
{
Reference< text::XTextRange > xTextRange( xIndexAccess->getByIndex( 0 ), UNO_QUERY );
- if ( xTextRange.is() && ( xTextRange->getString().getLength() == 0 ) )
+ if ( xTextRange.is() && ( xTextRange->getString().isEmpty() ) )
mbSelectionPresent = sal_False;
}
}
@@ -252,8 +252,8 @@ ImpPDFTabDialog::ImpPDFTabDialog( Window* pParent,
//change text on the Ok button: get the relevant string from resources, update it on the button
//according to the exported pdf file destination: send as e-mail or write to file?
- GetOKButton().SetText( ( sOkButtonText.getLength() > 0 ) ?
- sOkButtonText : OUString( String( PDFFilterResId( STR_PDF_EXPORT ) ) ));
+ GetOKButton().SetText( ( sOkButtonText.isEmpty() ) ?
+ OUString( String( PDFFilterResId( STR_PDF_EXPORT ) ) ) : sOkButtonText );
GetCancelButton().SetClickHdl(LINK(this, ImpPDFTabDialog, CancelHdl));
@@ -1217,8 +1217,8 @@ IMPL_LINK( ImpPDFTabSecurityPage, ClickmaPbSetPwdHdl, void*, EMPTYARG )
rtl::OUString aUserPW( aPwdDialog.GetPassword() );
rtl::OUString aOwnerPW( aPwdDialog.GetPassword2() );
- mbHaveUserPassword = (aUserPW.getLength() != 0);
- mbHaveOwnerPassword = (aOwnerPW.getLength() != 0);
+ mbHaveUserPassword = !aUserPW.isEmpty();
+ mbHaveOwnerPassword = !aOwnerPW.isEmpty();
mxPreparedPasswords = vcl::PDFWriter::InitEncryption( aOwnerPW, aUserPW, true );
diff --git a/filter/source/pdf/pdfexport.cxx b/filter/source/pdf/pdfexport.cxx
index f023f04e463b..a027f7479e68 100644
--- a/filter/source/pdf/pdfexport.cxx
+++ b/filter/source/pdf/pdfexport.cxx
@@ -325,7 +325,7 @@ static OUString getMimetypeForDocument( const Reference< XMultiServiceFactory >&
if( xModuleManager.is() && xStore.is() )
{
OUString aDocServiceName = xModuleManager->identify( Reference< XInterface >( xStore, uno::UNO_QUERY ) );
- if ( aDocServiceName.getLength() )
+ if ( !aDocServiceName.isEmpty() )
{
// get the actual filter name
OUString aFilterName;
@@ -349,7 +349,7 @@ static OUString getMimetypeForDocument( const Reference< XMultiServiceFactory >&
if ( xApplConfig.is() )
{
xApplConfig->getByName( OUString( RTL_CONSTASCII_USTRINGPARAM( "ooSetupFactoryActualFilter" ) ) ) >>= aFilterName;
- if( aFilterName.getLength() )
+ if( !aFilterName.isEmpty() )
{
// find the related type name
OUString aTypeName;
@@ -363,7 +363,7 @@ static OUString getMimetypeForDocument( const Reference< XMultiServiceFactory >&
if ( aFilterData[nInd].Name.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Type" ) ) )
aFilterData[nInd].Value >>= aTypeName;
- if ( aTypeName.getLength() )
+ if ( !aTypeName.isEmpty() )
{
// find the mediatype
Reference< container::XNameAccess > xTypeDetection(
@@ -658,7 +658,7 @@ sal_Bool PDFExport::Export( const OUString& rFile, const Sequence< PropertyValue
//set check for permission change password
// if not enabled and no permission password, force permissions to default as if PDF where without encryption
- if( mbRestrictPermissions && (xEnc.is() || aPermissionPassword.getLength() > 0) )
+ if( mbRestrictPermissions && (xEnc.is() || !aPermissionPassword.isEmpty()) )
{
mbEncrypt = sal_True;
//permission set as desired, done after
@@ -710,7 +710,7 @@ sal_Bool PDFExport::Export( const OUString& rFile, const Sequence< PropertyValue
aContext.Encryption.CanExtractForAccessibility = mbCanExtractForAccessibility;
if( mbEncrypt && ! xEnc.is() )
xEnc = PDFWriter::InitEncryption( aPermissionPassword, aOpenPassword, aContext.Encryption.Security128bit );
- if( mbEncrypt && aPermissionPassword.getLength() && ! aPreparedPermissionPassword.getLength() )
+ if( mbEncrypt && !aPermissionPassword.isEmpty() && ! aPreparedPermissionPassword.getLength() )
aPreparedPermissionPassword = comphelper::OStorageHelper::CreatePackageEncryptionData( aPermissionPassword );
}
// after this point we don't need the legacy clear passwords anymore
@@ -828,7 +828,7 @@ sal_Bool PDFExport::Export( const OUString& rFile, const Sequence< PropertyValue
aRenderOptions[ 5 ].Name = OUString( RTL_CONSTASCII_USTRINGPARAM( "PageRange" ) );
aRenderOptions[ 5 ].Value <<= aPageRange;
- if( aPageRange.getLength() || !aSelection.hasValue() )
+ if( !aPageRange.isEmpty() || !aSelection.hasValue() )
{
aSelection = Any();
aSelection <<= mxSrcDoc;
@@ -866,7 +866,7 @@ sal_Bool PDFExport::Export( const OUString& rFile, const Sequence< PropertyValue
bSecondPassForImpressNotes = sal_True; // the export notes checkbox needs to be disabled
}
- if( !aPageRange.getLength() )
+ if( aPageRange.isEmpty() )
{
aPageRange = OUStringBuffer()
.append( static_cast< sal_Int32 >( 1 ) )
@@ -903,7 +903,7 @@ sal_Bool PDFExport::Export( const OUString& rFile, const Sequence< PropertyValue
// if during the export the doc locale was set copy it to PDF writer
const com::sun::star::lang::Locale& rLoc( pPDFExtOutDevData->GetDocumentLocale() );
- if( rLoc.Language.getLength() )
+ if( !rLoc.Language.isEmpty() )
pPDFWriter->SetDocumentLocale( rLoc );
if( bRet )
diff --git a/filter/source/placeware/exporter.cxx b/filter/source/placeware/exporter.cxx
index b925b655de5f..8473e3885713 100644
--- a/filter/source/placeware/exporter.cxx
+++ b/filter/source/placeware/exporter.cxx
@@ -202,7 +202,7 @@ static void createSlideFile( Reference< XComponent > xDoc, ZipFile& rZipFile, co
Reference< XDocumentProperties > xDocProps( xDPS->getDocumentProperties() );
aTemp = xDocProps->getTitle();
- if( 0 == aTemp.getLength() )
+ if( aTemp.isEmpty() )
{
sal_Int32 nPos1 = rURL.lastIndexOf( (sal_Unicode)'/' );
if( -1 != nPos1 )
@@ -229,7 +229,7 @@ static void createSlideFile( Reference< XComponent > xDoc, ZipFile& rZipFile, co
aTemp = xDocProps->getAuthor();
- if( aTemp.getLength() )
+ if( !aTemp.isEmpty() )
{
aInfo += OString( "PresenterName: " );
aInfo += convertString( aTemp );
@@ -243,7 +243,7 @@ static void createSlideFile( Reference< XComponent > xDoc, ZipFile& rZipFile, co
PageEntry* pEntry = (*aIter++);
aInfo += OString( "slide: " );
- if( pEntry->getTitle().getLength() )
+ if( !pEntry->getTitle().isEmpty() )
{
aInfo += convertString( pEntry->getTitle() );
}
@@ -261,7 +261,7 @@ static void createSlideFile( Reference< XComponent > xDoc, ZipFile& rZipFile, co
aInfo += aNewLine;
- if( pEntry->getNotes().getLength() )
+ if( !pEntry->getNotes().isEmpty() )
{
aInfo += OString( "notes: " );
aInfo += convertString( pEntry->getNotes() );
diff --git a/filter/source/placeware/tempfile.cxx b/filter/source/placeware/tempfile.cxx
index ad41465c8791..eab253fc9f2d 100644
--- a/filter/source/placeware/tempfile.cxx
+++ b/filter/source/placeware/tempfile.cxx
@@ -139,7 +139,7 @@ TempFile::~TempFile()
{
close();
- if( maURL.getLength() )
+ if( !maURL.isEmpty() )
osl::File::remove( maURL );
}
diff --git a/filter/source/placeware/zip.cxx b/filter/source/placeware/zip.cxx
index 32d819a62c4c..549bfcaafb8c 100644
--- a/filter/source/placeware/zip.cxx
+++ b/filter/source/placeware/zip.cxx
@@ -254,9 +254,9 @@ bool ZipFile::addFile( osl::File& rFile, const OString& rName )
if( !mbOpen )
return false;
- OSL_ASSERT( 0 != rName.getLength() );
+ OSL_ASSERT( !rName.isEmpty() );
- if(0 == rName.getLength())
+ if(rName.isEmpty())
return false;
mnRC = rFile.open( osl_File_OpenFlag_Read );
diff --git a/filter/source/svg/svgexport.cxx b/filter/source/svg/svgexport.cxx
index d22c700b8e3b..6149ddf2960d 100644
--- a/filter/source/svg/svgexport.cxx
+++ b/filter/source/svg/svgexport.cxx
@@ -1325,7 +1325,7 @@ sal_Bool SVGFilter::implExportPages( const SVGFilter::XDrawPageSequence & rxPage
mpSVGExport->AddAttribute( XML_NAMESPACE_NONE, "id", sPageId );
OUString sPageName = implGetInterfaceName( rxPages[i] );
- if( sPageName.getLength() && !mbSinglePage )
+ if( !(sPageName.isEmpty() || mbSinglePage ))
mpSVGExport->AddAttribute( XML_NAMESPACE_NONE, aOOOAttrName, sPageName );
{
@@ -1565,7 +1565,7 @@ sal_Bool SVGFilter::implExportShape( const Reference< XShape >& rxShape )
OUString aTitle;
xShapePropSet->getPropertyValue( B2UCONST( "Title" ) ) >>= aTitle;
- if( aTitle.getLength() )
+ if( !aTitle.isEmpty() )
{
SvXMLElementExport aExp2( *mpSVGExport, XML_NAMESPACE_NONE, "title", sal_True, sal_True );
xExtDocHandler->characters( aTitle );
@@ -1573,7 +1573,7 @@ sal_Bool SVGFilter::implExportShape( const Reference< XShape >& rxShape )
OUString aDescription;
xShapePropSet->getPropertyValue( B2UCONST( "Description" ) ) >>= aDescription;
- if( aDescription.getLength() )
+ if( !aDescription.isEmpty() )
{
SvXMLElementExport aExp2( *mpSVGExport, XML_NAMESPACE_NONE, "desc", sal_True, sal_True );
xExtDocHandler->characters( aDescription );
@@ -1582,7 +1582,7 @@ sal_Bool SVGFilter::implExportShape( const Reference< XShape >& rxShape )
Reference< XInterface > xRef( rxShape, UNO_QUERY );
const OUString& rShapeId = implGetValidIDFromInterface( xRef );
- if( rShapeId.getLength() )
+ if( !rShapeId.isEmpty() )
{
mpSVGExport->AddAttribute( XML_NAMESPACE_NONE, "id", rShapeId );
//mpSVGExport->AddAttributeIdLegacy( XML_NAMESPACE_DRAW, rShapeId );
diff --git a/filter/source/svg/svgfontexport.cxx b/filter/source/svg/svgfontexport.cxx
index 4b374c059afa..eef0e6f43832 100644
--- a/filter/source/svg/svgfontexport.cxx
+++ b/filter/source/svg/svgfontexport.cxx
@@ -140,7 +140,7 @@ void SVGFontExport::implCollectGlyphs()
break;
}
- if( aText.getLength() )
+ if( !aText.isEmpty() )
{
GlyphSet& rGlyphSet = implGetGlyphSet( aVDev.GetFont() );
::com::sun::star::uno::Reference< ::com::sun::star::i18n::XBreakIterator > xBI(
@@ -286,7 +286,7 @@ void SVGFontExport::implEmbedGlyph( OutputDevice& rOut, const ::rtl::OUString& r
mrExport.AddAttribute( XML_NAMESPACE_NONE, "horiz-adv-x", ::rtl::OUString::valueOf( aBoundRect.GetWidth() ) );
const ::rtl::OUString aPathString( SVGActionWriter::GetPathString( aPolyPoly, sal_False ) );
- if( aPathString.getLength() )
+ if( !aPathString.isEmpty() )
{
mrExport.AddAttribute( XML_NAMESPACE_NONE, "d", aPathString );
}
diff --git a/filter/source/svg/svgreader.cxx b/filter/source/svg/svgreader.cxx
index cfeaaf4c892b..c25bdbc16c7c 100644
--- a/filter/source/svg/svgreader.cxx
+++ b/filter/source/svg/svgreader.cxx
@@ -1060,7 +1060,7 @@ struct AnnotatingVisitor
{
aCurrToken=sValue.getToken(0,';',nIndex);
- if( aCurrToken.getLength() )
+ if( !aCurrToken.isEmpty() )
{
// split attrib & value
nDummyIndex=0;
@@ -1621,7 +1621,7 @@ struct ShapeWritingVisitor
if (rShearX != 0.0f)
sTransformValue += USTR("skewX(")+::rtl::OUString::valueOf(rShearX)+USTR(") ");
- if (!sTransformValue.getLength())
+ if (sTransformValue.isEmpty())
return;
xAttrs->AddAttribute( USTR("draw:transform"), sTransformValue);
}
diff --git a/filter/source/svg/svgwriter.cxx b/filter/source/svg/svgwriter.cxx
index 6a3c0b0e5fe7..9f70046dfe7f 100644
--- a/filter/source/svg/svgwriter.cxx
+++ b/filter/source/svg/svgwriter.cxx
@@ -161,7 +161,7 @@ void SVGAttributeWriter::AddColorAttr( const char* pColorAttrName,
mrExport.AddAttribute( XML_NAMESPACE_NONE, pColorAttrName, aColor );
- if( aColorOpacity.getLength() && mrExport.IsUseOpacity() )
+ if( !aColorOpacity.isEmpty() && mrExport.IsUseOpacity() )
mrExport.AddAttribute( XML_NAMESPACE_NONE, pColorOpacityAttrName, aColorOpacity );
}
@@ -177,7 +177,7 @@ void SVGAttributeWriter::AddPaintAttr( const Color& rLineColor, const Color& rFi
AddGradientDef( *pObjBoundRect, *pFillGradient, aGradientId );
- if( aGradientId.getLength() )
+ if( !aGradientId.isEmpty() )
{
::rtl::OUString aGradientURL( B2UCONST( "url(#" ) );
mrExport.AddAttribute( XML_NAMESPACE_NONE, aXMLAttrFill, ( aGradientURL += aGradientId ) += B2UCONST( ")" ) );
@@ -673,7 +673,7 @@ void SVGActionWriter::ImplWriteShape( const SVGShapeDescriptor& rShape, sal_Bool
mpContext->AddPaintAttr( rShape.maShapeLineColor, rShape.maShapeFillColor, &aBoundRect, rShape.mapShapeGradient.get() );
- if( rShape.maId.getLength() )
+ if( !rShape.maId.isEmpty() )
mrExport.AddAttribute( XML_NAMESPACE_NONE, aXMLAttrId, rShape.maId );
if( rShape.mnStrokeWidth )
diff --git a/filter/source/xmlfilteradaptor/XmlFilterAdaptor.cxx b/filter/source/xmlfilteradaptor/XmlFilterAdaptor.cxx
index 78fcd893dfd9..3e3abb02d09c 100644
--- a/filter/source/xmlfilteradaptor/XmlFilterAdaptor.cxx
+++ b/filter/source/xmlfilteradaptor/XmlFilterAdaptor.cxx
@@ -150,7 +150,7 @@ sal_Bool SAL_CALL XmlFilterAdaptor::importImpl( const Sequence< ::com::sun::star
//********************
//Template Loading if Required
//********************
- if (msTemplateName.getLength()){
+ if (!msTemplateName.isEmpty()){
Reference< XStyleFamiliesSupplier > xstylefamiliessupplier(mxDoc, UNO_QUERY);
Reference< XNameAccess >xName;
diff --git a/filter/source/xmlfilterdetect/filterdetect.cxx b/filter/source/xmlfilterdetect/filterdetect.cxx
index cb68f5747257..b70194bfa12c 100644
--- a/filter/source/xmlfilterdetect/filterdetect.cxx
+++ b/filter/source/xmlfilterdetect/filterdetect.cxx
@@ -198,17 +198,17 @@ bool isXMLStream(const ::rtl::OString& aHeaderStrm)
sal_Int32 new_nlength=0;
sal_Int32 i = 0 ;
- while ((i < nLength) && (sTypeName.getLength() == 0))
+ while ((i < nLength) && (sTypeName.isEmpty()))
{
Any elem = xTypeCont->getByName(myTypes[i]);
elem >>=lProps;
new_nlength = lProps.getLength();
sal_Int32 j =0;
- while (j < new_nlength && (sTypeName.getLength() == 0))
+ while (j < new_nlength && (sTypeName.isEmpty()))
{
::rtl::OUString tmpStr;
lProps[j].Value >>=tmpStr;
- if ((lProps[j].Name.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("ClipboardFormat"))) && tmpStr.getLength())
+ if ((lProps[j].Name.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("ClipboardFormat"))) && !tmpStr.isEmpty())
{
sTypeName = supportedByType(tmpStr,resultString, myTypes[i]);
}
@@ -222,7 +222,7 @@ bool isXMLStream(const ::rtl::OString& aHeaderStrm)
OSL_FAIL( "An Exception occurred while opening File stream" );
}
- if (sTypeName.getLength())
+ if (!sTypeName.isEmpty())
{
if (location == aArguments.getLength())
{
diff --git a/filter/source/xsltdialog/typedetectionexport.cxx b/filter/source/xsltdialog/typedetectionexport.cxx
index d373fe4898be..a01b462c440e 100644
--- a/filter/source/xsltdialog/typedetectionexport.cxx
+++ b/filter/source/xsltdialog/typedetectionexport.cxx
@@ -52,7 +52,7 @@ TypeDetectionExporter::TypeDetectionExporter( Reference< XMultiServiceFactory >&
static OUString createRelativeURL( const OUString& rFilterName, const OUString& rURL )
{
- if( rURL.getLength() &&
+ if( !rURL.isEmpty() &&
(rURL.compareToAscii( RTL_CONSTASCII_STRINGPARAM("http:") ) != 0) &&
(rURL.compareToAscii( RTL_CONSTASCII_STRINGPARAM("shttp:") ) != 0) &&
(rURL.compareToAscii( RTL_CONSTASCII_STRINGPARAM("jar:") ) != 0) &&
@@ -60,7 +60,7 @@ static OUString createRelativeURL( const OUString& rFilterName, const OUString&
{
INetURLObject aURL( rURL );
OUString aName( aURL.GetName() );
- if( aName.getLength() == 0 )
+ if( aName.isEmpty() )
{
sal_Int32 nPos = rURL.lastIndexOf( sal_Unicode( '/' ) );
if( nPos == -1 )
@@ -138,7 +138,7 @@ void TypeDetectionExporter::doExport( Reference< XOutputStream > xOS, const XML
OUString sValue( sal_Unicode('0') );
sValue += sComma;
sValue += sComma;
- if( pFilter->maDocType.getLength() )
+ if( !pFilter->maDocType.isEmpty() )
{
sValue += sDocTypePrefix;
sValue += pFilter->maDocType;
diff --git a/filter/source/xsltdialog/typedetectionimport.cxx b/filter/source/xsltdialog/typedetectionimport.cxx
index b370414dd1cd..218434840900 100644
--- a/filter/source/xsltdialog/typedetectionimport.cxx
+++ b/filter/source/xsltdialog/typedetectionimport.cxx
@@ -207,13 +207,13 @@ filter_info_impl* TypeDetectionImporter::createFilterForNode( Node * pNode )
if( pTypeNode == NULL )
bOk = false;
- if( pFilter->maFilterName.getLength() == 0 )
+ if( pFilter->maFilterName.isEmpty() )
bOk = false;
- if( pFilter->maInterfaceName.getLength() == 0 )
+ if( pFilter->maInterfaceName.isEmpty() )
bOk = false;
- if( pFilter->maType.getLength() == 0 )
+ if( pFilter->maType.isEmpty() )
bOk = false;
if( pFilter->maFlags == 0 )
@@ -225,7 +225,7 @@ filter_info_impl* TypeDetectionImporter::createFilterForNode( Node * pNode )
if( aAdapterService != sXSLTFilterService )
bOk = false;
- if( pFilter->maExtension.getLength() == 0 )
+ if( pFilter->maExtension.isEmpty() )
bOk = false;
if( !bOk )
diff --git a/filter/source/xsltdialog/xmlfileview.cxx b/filter/source/xsltdialog/xmlfileview.cxx
index 899a0a8b9c74..c38dfc932c18 100644
--- a/filter/source/xsltdialog/xmlfileview.cxx
+++ b/filter/source/xsltdialog/xmlfileview.cxx
@@ -519,7 +519,7 @@ XMLSourceFileDialog::XMLSourceFileDialog( Window* pParent, ResMgr& rResMgr, cons
XMLSourceFileDialog::~XMLSourceFileDialog()
{
- if( maFileURL.getLength() )
+ if( !maFileURL.isEmpty() )
osl::File::remove( maFileURL );
delete mpTextWindow;
@@ -528,7 +528,7 @@ XMLSourceFileDialog::~XMLSourceFileDialog()
void XMLSourceFileDialog::ShowWindow( const rtl::OUString& rFileName, const filter_info_impl* pFilterInfo )
{
EnterWait();
- if( maFileURL.getLength() )
+ if( !maFileURL.isEmpty() )
{
osl::File::remove( maFileURL );
delete mpTextWindow;
diff --git a/filter/source/xsltdialog/xmlfilterjar.cxx b/filter/source/xsltdialog/xmlfilterjar.cxx
index 8c4863aa032e..078b62cb774b 100644
--- a/filter/source/xsltdialog/xmlfilterjar.cxx
+++ b/filter/source/xsltdialog/xmlfilterjar.cxx
@@ -146,7 +146,7 @@ static void addFile( Reference< XInterface > xRootFolder, Reference< XSingleServ
void XMLFilterJarHelper::addFile( Reference< XInterface > xRootFolder, Reference< XSingleServiceFactory > xFactory, const OUString& rSourceFile ) throw( Exception )
{
- if( rSourceFile.getLength() &&
+ if( !rSourceFile.isEmpty() &&
(rSourceFile.compareToAscii( RTL_CONSTASCII_STRINGPARAM("http:") ) != 0) &&
(rSourceFile.compareToAscii( RTL_CONSTASCII_STRINGPARAM("shttp:") ) != 0) &&
(rSourceFile.compareToAscii( RTL_CONSTASCII_STRINGPARAM("jar:") ) != 0) &&
@@ -209,14 +209,14 @@ bool XMLFilterJarHelper::savePackage( const OUString& rPackageURL, const XMLFilt
if( xFilterRoot.is() )
{
- if( pFilter->maDTD.getLength() )
+ if( !pFilter->maDTD.isEmpty() )
addFile( xFilterRoot, xFactory, pFilter->maDTD );
- if( pFilter->maExportXSLT.getLength() )
+ if( !pFilter->maExportXSLT.isEmpty() )
addFile( xFilterRoot, xFactory, pFilter->maExportXSLT );
try
{
- if( pFilter->maImportXSLT.getLength() )
+ if( !pFilter->maImportXSLT.isEmpty() )
addFile( xFilterRoot, xFactory, pFilter->maImportXSLT );
}
catch(const com::sun::star::container::ElementExistException&)
@@ -226,7 +226,7 @@ bool XMLFilterJarHelper::savePackage( const OUString& rPackageURL, const XMLFilt
OSL_FAIL( "XMLFilterJarHelper::same named xslt filter exception!" );
}
- if( pFilter->maImportTemplate.getLength() )
+ if( !pFilter->maImportTemplate.isEmpty() )
addFile( xFilterRoot, xFactory, pFilter->maImportTemplate );
}
@@ -382,7 +382,7 @@ bool XMLFilterJarHelper::copyFile( Reference< XHierarchicalNameAccess > xIfc, OU
rURL = URIHelper::SmartRel2Abs( aBaseURL, szPackagePath, Link(), false );
- if( rURL.getLength() )
+ if( !rURL.isEmpty() )
{
// create output directory if needed
if( !createDirectory( rURL ) )
diff --git a/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx b/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx
index 306fe595dee8..4a04257e2efa 100644
--- a/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx
+++ b/filter/source/xsltdialog/xmlfiltersettingsdialog.cxx
@@ -503,12 +503,12 @@ bool XMLFilterSettingsDialog::insertOrEdit( filter_info_impl* pNewInfo, const fi
}
// check if we need to copy the template
- if( pFilterEntry->maImportTemplate.getLength() )
+ if( !pFilterEntry->maImportTemplate.isEmpty() )
{
if( !pFilterEntry->maImportTemplate.matchIgnoreAsciiCase( sTemplatePath ) )
{
INetURLObject aSourceURL( pFilterEntry->maImportTemplate );
- if( aSourceURL.GetName().getLength() != 0 )
+ if( !aSourceURL.GetName().isEmpty() )
{
OUString aDestURL( sTemplatePath );
aDestURL += pFilterEntry->maFilterName;
@@ -531,13 +531,13 @@ bool XMLFilterSettingsDialog::insertOrEdit( filter_info_impl* pNewInfo, const fi
if( bOk )
{
- if( pFilterEntry->maType.getLength() == 0 )
+ if( pFilterEntry->maType.isEmpty() )
{
pFilterEntry->maType = createUniqueTypeName( pNewInfo->maFilterName );
}
// update import/export flags
- if( pFilterEntry->maImportXSLT.getLength() )
+ if( !pFilterEntry->maImportXSLT.isEmpty() )
{
pFilterEntry->maFlags |= 1;
}
@@ -546,7 +546,7 @@ bool XMLFilterSettingsDialog::insertOrEdit( filter_info_impl* pNewInfo, const fi
pFilterEntry->maFlags &= ~1;
}
- if( pFilterEntry->maExportXSLT.getLength() )
+ if( !pFilterEntry->maExportXSLT.isEmpty() )
{
pFilterEntry->maFlags |= 2;
}
@@ -1379,7 +1379,7 @@ OUString getApplicationUIName( const OUString& rServiceName )
else
{
OUString aRet = String( RESID( STR_UNKNOWN_APPLICATION ) );
- if( rServiceName.getLength() )
+ if( !rServiceName.isEmpty() )
{
aRet += OUString( RTL_CONSTASCII_USTRINGPARAM( " (" ));
aRet += rServiceName;
@@ -1581,7 +1581,7 @@ String XMLFilterListBox::getEntryString( const filter_info_impl* pInfo ) const
String aEntryStr( pInfo->maFilterName );
aEntryStr += '\t';
// aEntryStr += String( getApplicationUIName( pInfo->maDocumentService ) );
- if ( pInfo->maExportService.getLength() > 0 )
+ if ( !pInfo->maExportService.isEmpty() )
aEntryStr += String( getApplicationUIName( pInfo->maExportService ) );
else
aEntryStr += String( getApplicationUIName( pInfo->maImportService ) );
diff --git a/filter/source/xsltdialog/xmlfiltertabdialog.cxx b/filter/source/xsltdialog/xmlfiltertabdialog.cxx
index 8b046f93706d..242ef898ced6 100644
--- a/filter/source/xsltdialog/xmlfiltertabdialog.cxx
+++ b/filter/source/xsltdialog/xmlfiltertabdialog.cxx
@@ -134,10 +134,10 @@ bool XMLFilterTabDialog::onOk()
String aReplace2;
// 1. see if the filter name is ok
- if( (mpNewInfo->maFilterName.getLength() == 0) || (mpNewInfo->maFilterName != mpOldInfo->maFilterName) )
+ if( (mpNewInfo->maFilterName.isEmpty()) || (mpNewInfo->maFilterName != mpOldInfo->maFilterName) )
{
// if the user deleted the filter name, we reset the original filter name
- if( mpNewInfo->maFilterName.getLength() == 0 )
+ if( mpNewInfo->maFilterName.isEmpty() )
{
mpNewInfo->maFilterName = mpOldInfo->maFilterName;
}
@@ -166,10 +166,10 @@ bool XMLFilterTabDialog::onOk()
}
// 2. see if the interface name is ok
- if( (mpNewInfo->maInterfaceName.getLength() == 0) || (mpNewInfo->maInterfaceName != mpOldInfo->maInterfaceName) )
+ if( (mpNewInfo->maInterfaceName.isEmpty()) || (mpNewInfo->maInterfaceName != mpOldInfo->maInterfaceName) )
{
// if the user deleted the interface name, we reset the original filter name
- if( mpNewInfo->maInterfaceName.getLength() == 0 )
+ if( mpNewInfo->maInterfaceName.isEmpty() )
{
mpNewInfo->maInterfaceName = mpOldInfo->maInterfaceName;
}
@@ -272,7 +272,7 @@ bool XMLFilterTabDialog::onOk()
}
// see if we have at least an import or an export dtd
- if((mpNewInfo->maImportXSLT.getLength() == 0) && (mpNewInfo->maExportXSLT.getLength() == 0) )
+ if((mpNewInfo->maImportXSLT.isEmpty()) && (mpNewInfo->maExportXSLT.isEmpty()) )
{
nErrorId = STR_ERROR_EXPORT_XSLT_NOT_FOUND;
nErrorPage = RID_XML_FILTER_TABPAGE_XSLT;
diff --git a/filter/source/xsltdialog/xmlfiltertabpagebasic.cxx b/filter/source/xsltdialog/xmlfiltertabpagebasic.cxx
index 847087e0c929..8042a2c7000f 100644
--- a/filter/source/xsltdialog/xmlfiltertabpagebasic.cxx
+++ b/filter/source/xsltdialog/xmlfiltertabpagebasic.cxx
@@ -110,7 +110,7 @@ bool XMLFilterTabPageBasic::FillInfo( filter_info_impl* pInfo )
pInfo->maComment = string_encode( maEDDescription.GetText() );
- if( pInfo->maDocumentService.getLength() )
+ if( !pInfo->maDocumentService.isEmpty() )
{
std::vector< application_info_impl* >& rInfos = getApplicationInfos();
std::vector< application_info_impl* >::iterator aIter( rInfos.begin() );
@@ -140,7 +140,7 @@ void XMLFilterTabPageBasic::SetInfo(const filter_info_impl* pInfo)
if( pInfo->maDocumentService.getLength() )
maCBApplication.SetText( getApplicationUIName( pInfo->maDocumentService ) );
*/
- if( pInfo->maExportService.getLength() )
+ if( !pInfo->maExportService.isEmpty() )
maCBApplication.SetText( getApplicationUIName( pInfo->maExportService ) );
else
maCBApplication.SetText( getApplicationUIName( pInfo->maImportService ) );
diff --git a/filter/source/xsltdialog/xmlfiltertabpagexslt.cxx b/filter/source/xsltdialog/xmlfiltertabpagexslt.cxx
index 30442a3bd141..9df711f327b6 100644
--- a/filter/source/xsltdialog/xmlfiltertabpagexslt.cxx
+++ b/filter/source/xsltdialog/xmlfiltertabpagexslt.cxx
@@ -161,7 +161,7 @@ void XMLFilterTabPageXSLT::SetURL( SvtURLBox& rURLBox, const OUString& rURL )
rURLBox.SetBaseURL( rURL );
rURLBox.SetText( rURL );
}
- else if( rURL.getLength() )
+ else if( !rURL.isEmpty() )
{
rtl::OUString aURL( rURL );
aURL = URIHelper::SmartRel2Abs( sInstPath, aURL, Link(), false );
diff --git a/filter/source/xsltdialog/xmlfiltertestdialog.cxx b/filter/source/xsltdialog/xmlfiltertestdialog.cxx
index aaf274fa8e17..0c341aadddbc 100644
--- a/filter/source/xsltdialog/xmlfiltertestdialog.cxx
+++ b/filter/source/xsltdialog/xmlfiltertestdialog.cxx
@@ -261,7 +261,7 @@ void XMLFilterTestDialog::updateCurrentDocumentButtonState( Reference< XComponen
}
}
- if( 0 == aTitle.getLength() )
+ if( aTitle.isEmpty() )
{
Reference< XStorable > xStorable( xCurrentDocument, UNO_QUERY );
if( xStorable.is() )
@@ -307,13 +307,13 @@ void XMLFilterTestDialog::initDialog()
maFLImport.Enable( bImport );
maFTImportXSLT.Enable( bImport );
maFTImportXSLTFile.Enable( bImport );
- maFTImportTemplate.Enable( bImport && mpFilterInfo->maImportTemplate.getLength() );
- maFTImportTemplateFile.Enable( bImport && mpFilterInfo->maImportTemplate.getLength() );
+ maFTImportTemplate.Enable( bImport && !mpFilterInfo->maImportTemplate.isEmpty() );
+ maFTImportTemplateFile.Enable( bImport && !mpFilterInfo->maImportTemplate.isEmpty() );
maFTTransformFile.Enable( bImport );
maCBXDisplaySource.Enable( bImport );
maPBImportBrowse.Enable( bImport );
- maPBRecentDocument.Enable( bImport && maImportRecentFile.getLength() );
- maFTNameOfRecentFile.Enable( bImport && maImportRecentFile.getLength() );
+ maPBRecentDocument.Enable( bImport && !maImportRecentFile.isEmpty() );
+ maFTNameOfRecentFile.Enable( bImport && !maImportRecentFile.isEmpty() );
maFTImportXSLTFile.SetText( getFileNameFromURL( mpFilterInfo->maImportXSLT ) );
maFTImportTemplateFile.SetText( getFileNameFromURL( mpFilterInfo->maImportTemplate ) );
@@ -376,7 +376,7 @@ void XMLFilterTestDialog::onExportBrowse()
}
- if( (nFound == 15) && (aType.getLength() && aService == mpFilterInfo->maDocumentService) )
+ if( (nFound == 15) && (!aType.isEmpty() && aService == mpFilterInfo->maDocumentService) )
{
// see if this filter is not supressed in dialog
if( (nFlags & 0x1000) == 0 )
@@ -484,8 +484,8 @@ void XMLFilterTestDialog::doExport( Reference< XComponent > xComp )
// create xslt exporter
Reference< XOutputStream > xIS( new comphelper::OSLOutputStreamWrapper( aOutputFile ) );
- int bUseDTD = (mpFilterInfo->maDTD.getLength() != 0) ? 1 : 0 ;
- int bUseDocType = (mpFilterInfo->maDocType.getLength() != 0 ) ? 1 : 0;
+ int bUseDTD = mpFilterInfo->maDTD.isEmpty() ? 0 : 1 ;
+ int bUseDocType = mpFilterInfo->maDocType.isEmpty() ? 0 : 1;
Sequence< PropertyValue > aSourceData( 2 + bUseDTD + bUseDocType );
int i = 0;
diff --git a/filter/source/xsltfilter/XSLTFilter.cxx b/filter/source/xsltfilter/XSLTFilter.cxx
index e272ecdab4bc..c61fe17f306c 100644
--- a/filter/source/xsltfilter/XSLTFilter.cxx
+++ b/filter/source/xsltfilter/XSLTFilter.cxx
@@ -322,7 +322,7 @@ m_rServiceFactory(r), m_bTerminated(sal_False), m_bError(sal_False)
args[2] <<= nv;
OUString serviceName(RTL_CONSTASCII_USTRINGPARAM("com.sun.star.comp.documentconversion.LibXSLTTransformer"));
- if (msUserData[1].getLength() != 0)
+ if (!msUserData[1].isEmpty())
serviceName = msUserData[1];
m_tcontrol = Reference<XActiveDataControl> (m_rServiceFactory->createInstanceWithArguments(serviceName, args), UNO_QUERY);
@@ -486,7 +486,7 @@ m_rServiceFactory(r), m_bTerminated(sal_False), m_bError(sal_False)
args[3] <<= nv;
OUString serviceName(RTL_CONSTASCII_USTRINGPARAM("com.sun.star.comp.documentconversion.LibXSLTTransformer"));
- if (msUserData[1].getLength() != 0)
+ if (!msUserData[1].isEmpty())
serviceName = msUserData[1];
m_tcontrol = Reference<XActiveDataControl> (m_rServiceFactory->createInstanceWithArguments(serviceName, args), UNO_QUERY);