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

THausherr · THausherr · commit d49b300e1242 · 2026-01-23T09:57:26.000Z
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1931483 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForTamil.java b/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForTamil.java
@@ -190,7 +190,7 @@ private List<Integer> getBeforeHalfGlyphIds()
 
     private List<Integer> getRephGlyphIds()
     {
-        List<Integer> result = new ArrayList<>();
+        List<Integer> result = new ArrayList<>(REPH_CHARS.length);
         for (char character : REPH_CHARS)
         {
             result.add(getGlyphId(character));
@@ -200,7 +200,7 @@ private List<Integer> getRephGlyphIds()
 
     private List<Integer> getbeforeRephGlyphIds()
     {
-        List<Integer> glyphIds = new ArrayList<>();
+        List<Integer> glyphIds = new ArrayList<>(REPH_CHARS.length);
         for (char character : BEFORE_REPH_CHARS)
         {
             glyphIds.add(getGlyphId(character));

Original file line number	Diff line number	Diff line change
`@@ -190,7 +190,7 @@ private List<Integer> getBeforeHalfGlyphIds()`
`190`	`190`
`191`	`191`	`private List<Integer> getRephGlyphIds()`
`192`	`192`	`{`
`193`		`- List<Integer> result = new ArrayList<>();`
	`193`	`+ List<Integer> result = new ArrayList<>(REPH_CHARS.length);`
`194`	`194`	`for (char character : REPH_CHARS)`
`195`	`195`	`{`
`196`	`196`	`result.add(getGlyphId(character));`
`@@ -200,7 +200,7 @@ private List<Integer> getRephGlyphIds()`
`200`	`200`
`201`	`201`	`private List<Integer> getbeforeRephGlyphIds()`
`202`	`202`	`{`
`203`		`- List<Integer> glyphIds = new ArrayList<>();`
	`203`	`+ List<Integer> glyphIds = new ArrayList<>(REPH_CHARS.length);`
`204`	`204`	`for (char character : BEFORE_REPH_CHARS)`
`205`	`205`	`{`
`206`	`206`	`glyphIds.add(getGlyphId(character));`