You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pdfbox.apache.org by le...@apache.org on 2023/01/01 21:19:38 UTC

svn commit: r1906329 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java

Author: lehmi
Date: Sun Jan  1 21:19:38 2023
New Revision: 1906329

URL: http://svn.apache.org/viewvc?rev=1906329&view=rev
Log:
PDFBOX-5178: sonar fix

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java?rev=1906329&r1=1906328&r2=1906329&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java Sun Jan  1 21:19:38 2023
@@ -160,11 +160,7 @@ public abstract class BaseParser
         Optional<COSObjectKey> foundKey = document.getXrefTable().keySet().stream()
                 .filter(k -> k.getNumber() == num && k.getGeneration() == gen) //
                 .findAny();
-        if (foundKey == null || !foundKey.isPresent())
-        {
-            return new COSObjectKey(num, gen);
-        }
-        return foundKey.get();
+        return foundKey.isPresent() ? foundKey.get() : new COSObjectKey(num, gen);
     }
 
     /**