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 2021/10/02 13:23:53 UTC
svn commit: r1893821 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java
Author: tilman
Date: Sat Oct 2 13:23:53 2021
New Revision: 1893821
URL: http://svn.apache.org/viewvc?rev=1893821&view=rev
Log:
PDFBOX-4892: optimize + remove unneeded code, as suggested by valerybokov
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java?rev=1893821&r1=1893820&r2=1893821&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java Sat Oct 2 13:23:53 2021
@@ -241,7 +241,7 @@ public class PDType1Font extends PDSimpl
length1 = repairLength1(bytes, length1);
length2 = repairLength2(bytes, length1, length2);
- if (bytes.length > 0 && (bytes[0] & 0xff) == PFB_START_MARKER)
+ if ((bytes[0] & 0xff) == PFB_START_MARKER)
{
// some bad files embed the entire PFB, see PDFBOX-2607
t1 = Type1Font.createWithPFB(bytes);
@@ -394,7 +394,6 @@ public class PDType1Font extends PDSimpl
@Override
public float getHeight(int code) throws IOException
{
- String name = codeToName(code);
if (getStandard14AFM() != null)
{
String afmName = getEncoding().getName(code);
@@ -402,6 +401,8 @@ public class PDType1Font extends PDSimpl
}
else
{
+ String name = codeToName(code);
+
// todo: should be scaled by font matrix
return (float) genericFont.getPath(name).getBounds().getHeight();
}