mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fdo#43460: Use isEmpty() instead of getLength() in svgio

Browse Source 
Change-Id: I65beeb09285553177a160e260d606e6ba3f1c023
Reviewed-on: https://gerrit.libreoffice.org/4641
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
This commit is contained in:
Chr. Rossmanith
2013-06-19 22:25:24 +02:00
committed by Markus Mohrhard
parent 07010b8ea2
commit 82a1d75ee5
2 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
svgio/source/svgreader/svgstyleattributes.cxx
View File

@@ -1199,7 +1199,7 @@ namespace svgio
if(pNode)
{
if(SVGTokenLinearGradient == pNode->getType() || SVGTokenRadialGradient == pNode->getType())
if(SVGTokenLinearGradient == pNode->getType() || SVGTokenRadialGradient == pNode->getType())
{
setSvgGradientNodeFill(static_cast< const SvgGradientNode* >(pNode));
}
@@ -1790,7 +1790,7 @@ namespace svgio
// style="display:none"
// taken from a svg example; this needs to be parsed and set at the owning node. Do not call
// mrOwner.parseAttribute(...) here, this would lead to a recursion
if(aContent.getLength())
if(!aContent.isEmpty())
{
mrOwner.setDisplay(getDisplayFromContent(aContent));
}
@@ -1798,7 +1798,7 @@ namespace svgio
}
case SVGTokenBaselineShift:
{
if(aContent.getLength())
if(!aContent.isEmpty())
{
static rtl::OUString aStrSub(rtl::OUString::createFromAscii("sub"));
static rtl::OUString aStrSuper(rtl::OUString::createFromAscii("super"));

4
svgio/source/svgreader/svgtools.cxx
View File

@@ -1169,9 +1169,7 @@ namespace svgio
bool readSvgPaint(const OUString& rCandidate, SvgPaint& rSvgPaint, OUString& rURL)
{
const sal_Int32 nLen(rCandidate.getLength());
if(nLen)
if( !rCandidate.isEmpty() )
{
basegfx::BColor aColor;