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 2020/04/30 18:21:08 UTC
svn commit: r1877226 -
/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java
Author: tilman
Date: Thu Apr 30 18:21:08 2020
New Revision: 1877226
URL: http://svn.apache.org/viewvc?rev=1877226&view=rev
Log:
PDFBOX-4071: SonarQube fix: replace Stack with Deque
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java
Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java?rev=1877226&r1=1877225&r2=1877226&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1CharStringParser.java Thu Apr 30 18:21:08 2020
@@ -17,9 +17,10 @@
package org.apache.fontbox.cff;
import java.io.IOException;
+import java.util.ArrayDeque;
import java.util.ArrayList;
+import java.util.Deque;
import java.util.List;
-import java.util.Stack;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
@@ -123,7 +124,7 @@ public class Type1CharStringParser
Integer numArgs = (Integer)sequence.remove(sequence.size()-1);
// othersubrs 0-3 have their own semantics
- Stack<Integer> results = new Stack<>();
+ Deque<Integer> results = new ArrayDeque<>();
switch (othersubrNum)
{
case 0: