[Libreoffice-commits] core.git: tools/source
Jelle van der Waa
jelle at vdwaa.nl
Tue Jun 18 10:16:00 PDT 2013
tools/source/fsys/urlobj.cxx | 34 +++++++++++++++++-----------------
tools/source/memtools/multisel.cxx | 2 +-
tools/source/stream/stream.cxx | 6 +++---
3 files changed, 21 insertions(+), 21 deletions(-)
New commits:
commit 733d88433c560af8f51f010fcaae9ce7a29b0325
Author: Jelle van der Waa <jelle at vdwaa.nl>
Date: Tue Jun 18 11:42:05 2013 +0200
fdo#43460 tools: use isEmpty()
Change-Id: I832473cab555d8ac5c6b2a47d51124ac63e996e9
Reviewed-on: https://gerrit.libreoffice.org/4333
Reviewed-by: Noel Power <noel.power at suse.com>
Tested-by: Noel Power <noel.power at suse.com>
diff --git a/tools/source/fsys/urlobj.cxx b/tools/source/fsys/urlobj.cxx
index 071ca7a..77128d7 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 d91c499..25e4d70 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 e802566..cc7667d 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);
More information about the Libreoffice-commits
mailing list