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:42 UTC

svn commit: r1894099 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java

Author: tilman
Date: Sun Oct 10 10:04:42 2021
New Revision: 1894099

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

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

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java?rev=1894099&r1=1894098&r2=1894099&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java Sun Oct 10 10:04:42 2021
@@ -997,9 +997,6 @@ public class PageDrawer extends PDFGraph
                 case PathIterator.SEG_CUBICTO:
                     return false;
 
-                case PathIterator.SEG_CLOSE:
-                    break;
-
                 default:
                     break;
             }
@@ -2024,12 +2021,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<Boolean>();
         List<PDPropertyList> oCGs = ocmd.getOCGs();
         if (oCGs.isEmpty())
         {
             return false;
         }
+        List<Boolean> visibles = new ArrayList<Boolean>();
         for (PDPropertyList prop : oCGs)
         {
             visibles.add(!isHiddenOCG(prop));