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 2019/07/14 07:09:25 UTC
svn commit: r1863037 -
/pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1CFont.java
Author: tilman
Date: Sun Jul 14 07:09:25 2019
New Revision: 1863037
URL: http://svn.apache.org/viewvc?rev=1863037&view=rev
Log:
PDFBOX-4595: fix bug that returned 0 instead of cache value, as suggested by Zhichao Yang
Modified:
pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1CFont.java
Modified: pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1CFont.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1CFont.java?rev=1863037&r1=1863036&r2=1863037&view=diff
==============================================================================
--- pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1CFont.java (original)
+++ pdfbox/branches/issue4569/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1CFont.java Sun Jul 14 07:09:25 2019
@@ -318,12 +318,16 @@ public class PDType1CFont extends PDSimp
public float getHeight(int code) throws IOException
{
String name = codeToName(code);
- float height = 0;
+ float height;
if (!glyphHeights.containsKey(name))
{
height = (float)cffFont.getType1CharString(name).getBounds().getHeight(); // todo: cffFont could be null
glyphHeights.put(name, height);
}
+ else
+ {
+ height = glyphHeights.get(name);
+ }
return height;
}