PDFBOX-5660: optimize, as suggested by Valery Bokov; closes #223

THausherr · THausherr · commit 055513deb3fb · 2025-08-25T15:07:16.000Z
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1927998 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/pdfbox/src/main/java/org/apache/pdfbox/contentstream/operator/graphics/ClosePath.java b/pdfbox/src/main/java/org/apache/pdfbox/contentstream/operator/graphics/ClosePath.java
@@ -43,12 +43,13 @@ public ClosePath(PDFGraphicsStreamEngine context)
     @Override
     public void process(Operator operator, List<COSBase> operands) throws IOException
     {
-        if (getGraphicsContext().getCurrentPoint() == null)
+        PDFGraphicsStreamEngine context = getGraphicsContext();
+        if (context.getCurrentPoint() == null)
         {
             LOG.warn("ClosePath without initial MoveTo");
             return;
         }
-        getGraphicsContext().closePath();
+        context.closePath();
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -43,12 +43,13 @@ public ClosePath(PDFGraphicsStreamEngine context)`
`43`	`43`	`@Override`
`44`	`44`	`public void process(Operator operator, List<COSBase> operands) throws IOException`
`45`	`45`	`{`
`46`		`- if (getGraphicsContext().getCurrentPoint() == null)`
	`46`	`+ PDFGraphicsStreamEngine context = getGraphicsContext();`
	`47`	`+ if (context.getCurrentPoint() == null)`
`47`	`48`	`{`
`48`	`49`	`LOG.warn("ClosePath without initial MoveTo");`
`49`	`50`	`return;`
`50`	`51`	`}`
`51`		`- getGraphicsContext().closePath();`
	`52`	`+ context.closePath();`
`52`	`53`	`}`
`53`	`54`
`54`	`55`	`@Override`