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 2015/02/19 20:28:10 UTC

svn commit: r1660981 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/function/type4/InstructionSequenceBuilder.java

Author: tilman
Date: Thu Feb 19 19:28:10 2015
New Revision: 1660981

URL: http://svn.apache.org/r1660981
Log:
PDFBOX-2576: make class final; make fields final

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/function/type4/InstructionSequenceBuilder.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/function/type4/InstructionSequenceBuilder.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/function/type4/InstructionSequenceBuilder.java?rev=1660981&r1=1660980&r2=1660981&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/function/type4/InstructionSequenceBuilder.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/function/type4/InstructionSequenceBuilder.java Thu Feb 19 19:28:10 2015
@@ -25,11 +25,11 @@ import java.util.regex.Pattern;
  *
  * @version $Revision$
  */
-public class InstructionSequenceBuilder extends Parser.AbstractSyntaxHandler
+public final class InstructionSequenceBuilder extends Parser.AbstractSyntaxHandler
 {
 
-    private InstructionSequence mainSequence = new InstructionSequence();
-    private Stack<InstructionSequence> seqStack = new Stack<InstructionSequence>();
+    private final InstructionSequence mainSequence = new InstructionSequence();
+    private final Stack<InstructionSequence> seqStack = new Stack<InstructionSequence>();
 
     private InstructionSequenceBuilder()
     {
@@ -67,6 +67,7 @@ public class InstructionSequenceBuilder
     private static final Pattern REAL_PATTERN = Pattern.compile("[\\-]?\\d*\\.\\d*([Ee]\\-?\\d+)?");
 
     /** {@inheritDoc} */
+    @Override
     public void token(CharSequence text)
     {
         String token = text.toString();