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/03 22:15:58 UTC

svn commit: r1785407 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java

Author: tilman
Date: Fri Mar  3 22:15:58 2017
New Revision: 1785407

URL: http://svn.apache.org/viewvc?rev=1785407&view=rev
Log:
PDFBOX-2852: use switch

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java?rev=1785407&r1=1785406&r2=1785407&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java Fri Mar  3 22:15:58 2017
@@ -677,18 +677,18 @@ public class CFFParser
         DictData.Entry encodingEntry = topDict.getEntry("Encoding");
         CFFEncoding encoding;
         int encodingId = encodingEntry != null ? encodingEntry.getNumber(0).intValue() : 0;
-        if (encodingId == 0)
+        switch (encodingId)
         {
-            encoding = CFFStandardEncoding.getInstance();
-        }
-        else if (encodingId == 1)
-        {
-            encoding = CFFExpertEncoding.getInstance();
-        }
-        else
-        {
-            input.setPosition(encodingId);
-            encoding = readEncoding(input, charset);
+            case 0:
+                encoding = CFFStandardEncoding.getInstance();
+                break;
+            case 1:
+                encoding = CFFExpertEncoding.getInstance();
+                break;
+            default:
+                input.setPosition(encodingId);
+                encoding = readEncoding(input, charset);
+                break;
         }
         font.setEncoding(encoding);