You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2020/10/23 15:48:48 UTC
svn commit: r1882786 -
/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CharStringHandler.java
Author: lehmi
Date: Fri Oct 23 15:48:48 2020
New Revision: 1882786
URL: http://svn.apache.org/viewvc?rev=1882786&view=rev
Log:
PDFBOX-4892: replace Stack with ArrayList
Modified:
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CharStringHandler.java
Modified: pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CharStringHandler.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CharStringHandler.java?rev=1882786&r1=1882785&r2=1882786&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CharStringHandler.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/cff/CharStringHandler.java Fri Oct 23 15:48:48 2020
@@ -17,8 +17,8 @@
package org.apache.fontbox.cff;
+import java.util.ArrayList;
import java.util.List;
-import java.util.Stack;
/**
* A Handler for CharStringCommands.
@@ -37,24 +37,24 @@ public abstract class CharStringHandler
*/
public List<Number> handleSequence(List<Object> sequence)
{
- Stack<Number> stack = new Stack<Number>();
+ List<Number> numbers = new ArrayList<Number>();
for (Object obj : sequence)
{
if (obj instanceof CharStringCommand)
{
- List<Number> results = handleCommand(stack, (CharStringCommand)obj);
- stack.clear(); // this is basically returning the new stack
+ List<Number> results = handleCommand(numbers, (CharStringCommand)obj);
+ numbers.clear();
if (results != null)
{
- stack.addAll(results);
+ numbers.addAll(results);
}
}
else
{
- stack.push((Number)obj);
+ numbers.add((Number) obj);
}
}
- return stack;
+ return numbers;
}
/**