summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJelle van der Waa <jelle@vdwaa.nl>2013-06-18 11:42:05 +0200
committerNoel Power <noel.power@suse.com>2013-06-18 17:15:41 +0000
commit733d88433c560af8f51f010fcaae9ce7a29b0325 (patch)
treeef2e3361a444d3e7cfc332b1cfd4e93f7067a072
parent307aeb6e97294e554ecdfec642c0ac6e926ecde7 (diff)
fdo#43460 tools: use isEmpty()
Change-Id: I832473cab555d8ac5c6b2a47d51124ac63e996e9 Reviewed-on: https://gerrit.libreoffice.org/4333 Reviewed-by: Noel Power <noel.power@suse.com> Tested-by: Noel Power <noel.power@suse.com>
-rw-r--r--tools/source/fsys/urlobj.cxx34
-rw-r--r--tools/source/memtools/multisel.cxx2
-rw-r--r--tools/source/stream/stream.cxx6
3 files changed, 21 insertions, 21 deletions
diff --git a/tools/source/fsys/urlobj.cxx b/tools/source/fsys/urlobj.cxx
index 071ca7a31803..77128d71b1e9 100644
--- a/tools/source/fsys/urlobj.cxx
+++ b/tools/source/fsys/urlobj.cxx
@@ -899,7 +899,7 @@ bool INetURLObject::setAbsURIRef(OUString const & rTheAbsURIRef,
eCharset,
false);
}
- if (aSynAuthority.getLength() == 0)
+ if (aSynAuthority.isEmpty())
{
setInvalid();
return false;
@@ -965,7 +965,7 @@ bool INetURLObject::setAbsURIRef(OUString const & rTheAbsURIRef,
false);
}
}
- if (aSynAuthority.getLength() == 0)
+ if (aSynAuthority.isEmpty())
{
setInvalid();
return false;
@@ -1987,7 +1987,7 @@ bool INetURLObject::convertAbsToRel(OUString const & rTheAbsURIRef,
// new relative URL would neither be empty nor start with <"/"> nor start
// with <1*rseg> (i.e., it could be mistaken for an absolute URL starting
// with a scheme part), then the new relative URL is prefixed with "./":
- if (aSynRelURIRef.getLength() == 0)
+ if (aSynRelURIRef.isEmpty())
{
if (pSubjectPathEnd - pSubjectPathBegin >= nMatch + 2
&& pSubjectPathBegin[nMatch] == '/'
@@ -2886,12 +2886,12 @@ bool INetURLObject::setHost(OUString const & rTheHost, bool bOctets,
}
break;
case INET_PROT_LDAP:
- if (aSynHost.getLength() == 0 && m_aPort.isPresent())
+ if (aSynHost.isEmpty() && m_aPort.isPresent())
return false;
break;
default:
- if (aSynHost.getLength() == 0)
+ if (aSynHost.isEmpty())
return false;
break;
}
@@ -2944,7 +2944,7 @@ bool INetURLObject::parsePath(INetProtocol eScheme,
appendUCS4(aTheSynPath, nUTF32, eEscapeType, bOctets,
PART_HTTP_PATH, '%', eCharset, true);
}
- if (aTheSynPath.getLength() == 0)
+ if (aTheSynPath.isEmpty())
aTheSynPath.append(sal_Unicode('/'));
break;
@@ -2965,7 +2965,7 @@ bool INetURLObject::parsePath(INetProtocol eScheme,
appendUCS4(aTheSynPath, nUTF32, eEscapeType, bOctets,
PART_HTTP_PATH, '%', eCharset, true);
}
- if (aTheSynPath.getLength() == 0)
+ if (aTheSynPath.isEmpty())
aTheSynPath.append(sal_Unicode('/'));
break;
@@ -3008,7 +3008,7 @@ bool INetURLObject::parsePath(INetProtocol eScheme,
appendUCS4(aTheSynPath, nUTF32, eEscapeType, bOctets,
PART_PCHAR, '%', eCharset, true);
}
- if (aTheSynPath.getLength() == 0)
+ if (aTheSynPath.isEmpty())
aTheSynPath.append(sal_Unicode('/'));
break;
}
@@ -3076,7 +3076,7 @@ bool INetURLObject::parsePath(INetProtocol eScheme,
appendUCS4(aTheSynPath, nUTF32, eEscapeType, bOctets,
PART_NEWS_ARTICLE_LOCALPART, '%', eCharset, true);
}
- if (aTheSynPath.getLength() == 0)
+ if (aTheSynPath.isEmpty())
return false;
++pPos;
aTheSynPath.append(sal_Unicode('@'));
@@ -3197,7 +3197,7 @@ bool INetURLObject::parsePath(INetProtocol eScheme,
appendUCS4(aTheSynPath, nUTF32, eEscapeType, bOctets,
PART_PCHAR, '%', eCharset, false);
}
- if (aTheSynPath.getLength() == 0)
+ if (aTheSynPath.isEmpty())
aTheSynPath.append(sal_Unicode('/'));
break;
@@ -3362,7 +3362,7 @@ bool INetURLObject::parsePath(INetProtocol eScheme,
appendUCS4(aTheSynPath, nUTF32, eEscapeType, bOctets,
PART_URIC, '%', eCharset, true);
}
- if (aTheSynPath.getLength() == 0)
+ if (aTheSynPath.isEmpty())
return false;
break;
default:
@@ -3732,7 +3732,7 @@ INetURLObject::getAbbreviated(
}
else
{
- if (m_aAbsURIRef.getLength() != 0)
+ if (!m_aAbsURIRef.isEmpty())
{
sal_Unicode const * pSchemeBegin
= m_aAbsURIRef.getStr();
@@ -3874,7 +3874,7 @@ INetURLObject::getAbbreviated(
aBuffer.
append(decode(m_aFragment, cEscapePrefix, eMechanism, eCharset));
}
- if (aBuffer.getLength() != 0)
+ if (!aBuffer.isEmpty())
{
OUStringBuffer aResult(aBuffer);
if (rStringWidth->queryStringWidth(aResult.makeStringAndClear())
@@ -3884,7 +3884,7 @@ INetURLObject::getAbbreviated(
if (i == 0)
{
aBuffer.setLength(aBuffer.getLength() - 1);
- if (aBuffer.getLength() == 0)
+ if (aBuffer.isEmpty())
break;
}
else
@@ -4088,7 +4088,7 @@ bool INetURLObject::ConcatData(INetProtocol eTheScheme,
break;
case INET_PROT_LDAP:
- if (aSynHost.getLength() == 0 && nThePort != 0)
+ if (aSynHost.isEmpty() && nThePort != 0)
{
setInvalid();
return false;
@@ -4096,7 +4096,7 @@ bool INetURLObject::ConcatData(INetProtocol eTheScheme,
break;
default:
- if (aSynHost.getLength() == 0)
+ if (aSynHost.isEmpty())
{
setInvalid();
return false;
@@ -4289,7 +4289,7 @@ bool INetURLObject::removeSegment(sal_Int32 nIndex, bool bIgnoreFinalSlash)
else
aNewPath.append(m_aAbsURIRef.getStr() + aSegment.getEnd(),
m_aPath.getEnd() - aSegment.getEnd());
- if (aNewPath.getLength() == 0 && !aSegment.isEmpty() &&
+ if (aNewPath.isEmpty() && !aSegment.isEmpty() &&
m_aAbsURIRef[aSegment.getBegin()] == '/')
{
aNewPath.append(sal_Unicode('/'));
diff --git a/tools/source/memtools/multisel.cxx b/tools/source/memtools/multisel.cxx
index d91c49983426..25e4d706b9d6 100644
--- a/tools/source/memtools/multisel.cxx
+++ b/tools/source/memtools/multisel.cxx
@@ -714,7 +714,7 @@ bool StringRangeEnumerator::setRange( const OUString& i_rNewRange, bool i_bStric
{
while( *pInput >= sal_Unicode('0') && *pInput <= sal_Unicode('9') )
aNumberBuf.append( *pInput++ );
- if( aNumberBuf.getLength() )
+ if( !aNumberBuf.isEmpty() )
{
sal_Int32 nNumber = aNumberBuf.makeStringAndClear().toInt32() + mnOffset;
aNumbers.push_back( nNumber );
diff --git a/tools/source/stream/stream.cxx b/tools/source/stream/stream.cxx
index e80256652e40..cc7667dd17bd 100644
--- a/tools/source/stream/stream.cxx
+++ b/tools/source/stream/stream.cxx
@@ -585,7 +585,7 @@ sal_Bool SvStream::ReadLine( OString& rStr, sal_Int32 nMaxBytesToRead )
aBuf.append(buf, n);
}
- if ( !bEnd && !GetError() && aBuf.getLength() )
+ if ( !bEnd && !GetError() && !aBuf.isEmpty() )
bEnd = sal_True;
nOldFilePos += nTotalLen;
@@ -627,7 +627,7 @@ sal_Bool SvStream::ReadUniStringLine( OUString& rStr, sal_Int32 nMaxCodepointsTo
nLen /= sizeof(sal_Unicode);
if ( !nLen )
{
- if ( aBuf.getLength() == 0 )
+ if ( aBuf.isEmpty() )
{
// exit on first BlockRead error
bIsEof = sal_True;
@@ -671,7 +671,7 @@ sal_Bool SvStream::ReadUniStringLine( OUString& rStr, sal_Int32 nMaxCodepointsTo
aBuf.append( buf, n );
}
- if ( !bEnd && !GetError() && aBuf.getLength() )
+ if ( !bEnd && !GetError() && !aBuf.isEmpty() )
bEnd = sal_True;
nOldFilePos += nTotalLen * sizeof(sal_Unicode);