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/06/24 11:24:31 UTC
svn commit: r1799759 -
/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/NamingTable.java
Author: tilman
Date: Sat Jun 24 11:24:31 2017
New Revision: 1799759
URL: http://svn.apache.org/viewvc?rev=1799759&view=rev
Log:
PDFBOX-2852: use switch
Modified:
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/NamingTable.java
Modified: pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/NamingTable.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/NamingTable.java?rev=1799759&r1=1799758&r2=1799759&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/NamingTable.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/NamingTable.java Sat Jun 24 11:24:31 2017
@@ -94,18 +94,20 @@ public class NamingTable extends TTFTabl
}
else if (platform == NameRecord.PLATFORM_ISO)
{
- if (encoding == 0)
+ switch (encoding)
{
- charset = Charsets.US_ASCII;
- }
- else if (encoding == 1)
- {
- //not sure is this is correct??
- charset = Charsets.ISO_10646;
- }
- else if (encoding == 2)
- {
- charset = Charsets.ISO_8859_1;
+ case 0:
+ charset = Charsets.US_ASCII;
+ break;
+ case 1:
+ //not sure is this is correct??
+ charset = Charsets.ISO_10646;
+ break;
+ case 2:
+ charset = Charsets.ISO_8859_1;
+ break;
+ default:
+ break;
}
}
String string = data.readString(nr.getStringLength(), charset);