use isEmpty() instead of getLength() where appropriate

Change-Id: I87bb3473af631dc321c5a8c2b06b0b5ccdf8e73b
Reviewed-on: https://gerrit.libreoffice.org/11813Reviewed-by: Christina Roßmanith <ChrRossmanith@web.de>
Tested-by: Christina Roßmanith <ChrRossmanith@web.de>

svgio/source/svgreader/svgnode.cxx

@@ -135,7 +135,7 @@ namespace svgio
                 }
 
                 // check for class-dependent references to CssStyles
-                if(rClassStr.getLength())
+                if(!rClassStr.isEmpty())
                 {
                     OUString aNewConcatenated(aConcatenated);
 
@@ -213,7 +213,7 @@ namespace svgio
             {
                 // #i125293# rOriginal will be the last element in the linked list; use no CssStyleParent
                 // there (reset it) to ensure that the parent hierarchy will be used when it's base
-                // is referenced. This new chaning inserts the CssStyles before the original style,
+                // is referenced. This new chaining inserts the CssStyles before the original style,
                 // this makes the whole process much safer since the original style when used will
                 // be not different to the situation without CssStyles; thus loops which may be caused
                 // by trying to use the parent hierarchy of the owner of the style will be avoided

svgio/source/svgreader/svgstylenode.cxx

@@ -85,7 +85,7 @@ namespace svgio
         {
             // aSelectors: CssStyle selectors, any combination, no comma separations, no spaces at start/end
             // rNewStyle: the already preapared style to register on that name
-            if(aSelectors.getLength())
+            if(!aSelectors.isEmpty())
             {
                 std::vector< OUString > aSelectorParts;
                 const sal_Int32 nLen(aSelectors.getLength());
@@ -100,7 +100,7 @@ namespace svgio
                     skip_char(aSelectors, sal_Unicode(' '), nPos, nLen);
                     const OUString aSelectorPart(aToken.makeStringAndClear().trim());
 
-                    if(aSelectorPart.getLength())
+                    if(!aSelectorPart.isEmpty())
                     {
                         aSelectorParts.push_back(aSelectorPart);
                     }
@@ -147,7 +147,7 @@ namespace svgio
         {
             // aSelectors: possible comma-separated list of CssStyle definitions, no spaces at start/end
             // aContent: the svg style definitions as string
-            if(aSelectors.getLength() && aContent.getLength())
+            if(!aSelectors.isEmpty() && !aContent.isEmpty())
             {
                 // create new style and add to local list (for ownership control)
                 SvgStyleAttributes* pNewStyle = new SvgStyleAttributes(*this);
@@ -203,7 +203,7 @@ namespace svgio
                     const OUString aSelectors(aToken.makeStringAndClear().trim());
                     OUString aContent;
 
-                    if(aSelectors.getLength() && nPos < nLen)
+                    if(!aSelectors.isEmpty() && nPos < nLen)
                     {
                         // isolate content as text, embraced by '{' and '}'
                         copyToLimiter(aSelectorsAndContent, sal_Unicode('}'), nPos, aToken, nLen);
@@ -212,7 +212,7 @@ namespace svgio
                         aContent = aToken.makeStringAndClear().trim();
                     }
 
-                    if(aSelectors.getLength() && aContent.getLength())
+                    if(!aSelectors.isEmpty() && !aContent.isEmpty())
                     {
                         addCssStyleSheet(aSelectors, aContent);
                     }