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/10/16 17:13:27 UTC

svn commit: r1904620 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Font.java

Author: tilman
Date: Sun Oct 16 17:13:27 2022
New Revision: 1904620

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

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Font.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Font.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Font.java?rev=1904620&r1=1904619&r2=1904620&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Font.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Font.java Sun Oct 16 17:13:27 2022
@@ -90,8 +90,8 @@ public final class Type1Font implements
     Encoding encoding = null;
     int paintType;
     int fontType;
-    List<Number> fontMatrix = new ArrayList<>();
-    List<Number> fontBBox = new ArrayList<>();
+    List<Number> fontMatrix = Collections.emptyList();
+    List<Number> fontBBox = Collections.emptyList();
     int uniqueID;
     float strokeWidth;
     String fontID = "";
@@ -108,17 +108,17 @@ public final class Type1Font implements
     float underlineThickness;
 
     // Private dictionary
-    List<Number> blueValues = new ArrayList<>();
-    List<Number> otherBlues = new ArrayList<>();
-    List<Number> familyBlues = new ArrayList<>();
-    List<Number> familyOtherBlues = new ArrayList<>();
+    List<Number> blueValues = Collections.emptyList();
+    List<Number> otherBlues = Collections.emptyList();
+    List<Number> familyBlues = Collections.emptyList();
+    List<Number> familyOtherBlues = Collections.emptyList();
     float blueScale;
     int blueShift;
     int blueFuzz;
-    List<Number> stdHW = new ArrayList<>();
-    List<Number> stdVW = new ArrayList<>();
-    List<Number> stemSnapH = new ArrayList<>();
-    List<Number> stemSnapV = new ArrayList<>();
+    List<Number> stdHW = Collections.emptyList();
+    List<Number> stdVW = Collections.emptyList();
+    List<Number> stemSnapH = Collections.emptyList();
+    List<Number> stemSnapV = Collections.emptyList();
     boolean forceBold;
     int languageGroup;
 
@@ -263,6 +263,7 @@ public final class Type1Font implements
      * 
      * @return the font matrix
      */
+    @Override
     public List<Number> getFontMatrix()
     {
         return Collections.unmodifiableList(fontMatrix);