Author: tilman
Date: Mon Apr  7 10:17:29 2025
New Revision: 1924896

URL: http://svn.apache.org/viewvc?rev=1924896&view=rev
Log:
PDFBOX-5660: Sonar fix

Modified:
    
pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/GlyphSubstitutionTableTest.java
    pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/BidiTest.java
    
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/TestTextStripper.java

Modified: 
pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/GlyphSubstitutionTableTest.java
URL: 
http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/GlyphSubstitutionTableTest.java?rev=1924896&r1=1924895&r2=1924896&view=diff
==============================================================================
--- 
pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/GlyphSubstitutionTableTest.java
 (original)
+++ 
pdfbox/trunk/fontbox/src/test/java/org/apache/fontbox/ttf/GlyphSubstitutionTableTest.java
 Mon Apr  7 10:17:29 2025
@@ -102,7 +102,7 @@ class GlyphSubstitutionTableTest
                     break;
                 }
 
-                if (line.trim().length() == 0)
+                if (line.isBlank())
                 {
                     continue;
                 }

Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/BidiTest.java
URL: 
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/BidiTest.java?rev=1924896&r1=1924895&r2=1924896&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/BidiTest.java 
(original)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/BidiTest.java Mon 
Apr  7 10:17:29 2025
@@ -153,12 +153,12 @@ class BidiTest
             while (true)
             {
                 String expectedLine = expectedReader.readLine();
-                while( expectedLine != null && expectedLine.trim().length() == 
0 )
+                while (expectedLine != null && expectedLine.isBlank())
                 {
                     expectedLine = expectedReader.readLine();
                 }
                 String actualLine = actualReader.readLine();
-                while( actualLine != null && actualLine.trim().length() == 0 )
+                while (actualLine != null && actualLine.isBlank())
                 {
                     actualLine = actualReader.readLine();
                 }

Modified: 
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/TestTextStripper.java
URL: 
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/TestTextStripper.java?rev=1924896&r1=1924895&r2=1924896&view=diff
==============================================================================
--- 
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/TestTextStripper.java 
(original)
+++ 
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/text/TestTextStripper.java 
Mon Apr  7 10:17:29 2025
@@ -317,12 +317,12 @@ class TestTextStripper
             while (true)
             {
                 String expectedLine = expectedReader.readLine();
-                while( expectedLine != null && expectedLine.trim().length() == 
0 )
+                while (expectedLine != null && expectedLine.isBlank())
                 {
                     expectedLine = expectedReader.readLine();
                 }
                 String actualLine = actualReader.readLine();
-                while( actualLine != null && actualLine.trim().length() == 0 )
+                while (actualLine != null && actualLine.isBlank())
                 {
                     actualLine = actualReader.readLine();
                 }
@@ -567,7 +567,7 @@ class TestTextStripper
         File inDirExt = new File("target/test-input-ext");
         File outDirExt = new File("target/test-output-ext");
 
-            if ((filename == null) || (filename.length() == 0)) 
+            if (filename == null || filename.isEmpty())
             {
                 doTestDir(inDir, outDir);
                 if (inDirExt.exists())


Reply via email to