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 2017/03/23 18:50:26 UTC

svn commit: r1788302 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Parser.java

Author: tilman
Date: Thu Mar 23 18:50:26 2017
New Revision: 1788302

URL: http://svn.apache.org/viewvc?rev=1788302&view=rev
Log:
PDFBOX-2852: move string literal on the left side of string comparison

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

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Parser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Parser.java?rev=1788302&r1=1788301&r2=1788302&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Parser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/type1/Type1Parser.java Thu Mar 23 18:50:26 2017
@@ -502,19 +502,19 @@ final class Type1Parser
             // key/value
             String key = read(Token.LITERAL).getText();
 
-            if (key.equals("Subrs"))
+            if ("Subrs".equals(key))
             {
                 readSubrs(lenIV);
             }
-            else if (key.equals("OtherSubrs"))
+            else if ("OtherSubrs".equals(key))
             {
                 readOtherSubrs();
             }
-            else if (key.equals("lenIV"))
+            else if ("lenIV".equals(key))
             {
                 lenIV = readDictValue().get(0).intValue();
             }
-            else if (key.equals("ND"))
+            else if ("ND".equals(key))
             {
                 read(Token.START_PROC);
                 read(Token.NAME, "noaccess");
@@ -523,7 +523,7 @@ final class Type1Parser
                 read(Token.NAME, "executeonly");
                 read(Token.NAME, "def");
             }
-            else if (key.equals("NP"))
+            else if ("NP".equals(key))
             {
                 read(Token.START_PROC);
                 read(Token.NAME, "noaccess");
@@ -532,7 +532,7 @@ final class Type1Parser
                 read(Token.NAME, "executeonly");
                 read(Token.NAME, "def");
             }
-            else if (key.equals("RD"))
+            else if ("RD".equals(key))
             {
                 // /RD {string currentfile exch readstring pop} bind executeonly def
                 read(Token.START_PROC);