You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2021/10/10 10:04:46 UTC

svn commit: r1894100 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java

Author: tilman
Date: Sun Oct 10 10:04:45 2021
New Revision: 1894100

URL: http://svn.apache.org/viewvc?rev=1894100&view=rev
Log:
PDFBOX-4892: optimize, as suggested by valerybokov

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java?rev=1894100&r1=1894099&r2=1894100&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java Sun Oct 10 10:04:45 2021
@@ -921,9 +921,6 @@ public class PageDrawer extends PDFGraph
                 case PathIterator.SEG_CUBICTO:
                     return false;
 
-                case PathIterator.SEG_CLOSE:
-                    break;
-
                 default:
                     break;
             }
@@ -1948,12 +1945,12 @@ public class PageDrawer extends PDFGraph
             // support seems to be optional, and is approximated by /P and /OCGS
             LOG.info("/VE entry ignored in Optional Content Membership Dictionary");
         }
-        List<Boolean> visibles = new ArrayList<>();
         List<PDPropertyList> oCGs = ocmd.getOCGs();
         if (oCGs.isEmpty())
         {
             return false;
         }
+        List<Boolean> visibles = new ArrayList<>();
         oCGs.forEach(prop -> visibles.add(!isHiddenOCG(prop)));
         COSName visibilityPolicy = ocmd.getVisibilityPolicy();