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 2022/02/12 11:19:45 UTC
svn commit: r1897995 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/shading/PDShadingType5.java
Author: tilman
Date: Sat Feb 12 11:19:45 2022
New Revision: 1897995
URL: http://svn.apache.org/viewvc?rev=1897995&view=rev
Log:
PDFBOX-4892: refactor long method
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/shading/PDShadingType5.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/shading/PDShadingType5.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/shading/PDShadingType5.java?rev=1897995&r1=1897994&r2=1897995&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/shading/PDShadingType5.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/shading/PDShadingType5.java Sat Feb 12 11:19:45 2022
@@ -132,13 +132,12 @@ public class PDShadingType5 extends PDTr
}
}
int rowNum = vlist.size() / numPerRow;
- Vertex[][] latticeArray = new Vertex[rowNum][numPerRow];
- List<ShadedTriangle> list = new ArrayList<>();
if (rowNum < 2)
{
// must have at least two rows; if not, return empty list
- return list;
+ return Collections.emptyList();
}
+ Vertex[][] latticeArray = new Vertex[rowNum][numPerRow];
for (int i = 0; i < rowNum; i++)
{
for (int j = 0; j < numPerRow; j++)
@@ -147,8 +146,14 @@ public class PDShadingType5 extends PDTr
}
}
+ return createShadedTriangleList(rowNum, numPerRow, latticeArray);
+ }
+
+ private List<ShadedTriangle> createShadedTriangleList(int rowNum, int numPerRow, Vertex[][] latticeArray)
+ {
Point2D[] ps = new Point2D[3]; // array will be shallow-cloned in ShadedTriangle constructor
float[][] cs = new float[3][];
+ List<ShadedTriangle> list = new ArrayList<>();
for (int i = 0; i < rowNum - 1; i++)
{
for (int j = 0; j < numPerRow - 1; j++)