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 2015/10/20 19:04:02 UTC

svn commit: r1709647 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java

Author: tilman
Date: Tue Oct 20 17:04:02 2015
New Revision: 1709647

URL: http://svn.apache.org/viewvc?rev=1709647&view=rev
Log:
PDFBOX-3038: return BBox from font descriptor if font BBox empty

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java?rev=1709647&r1=1709646&r2=1709647&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDType1Font.java Tue Oct 20 17:04:02 2015
@@ -36,6 +36,7 @@ import org.apache.pdfbox.cos.COSDictiona
 import org.apache.pdfbox.cos.COSName;
 import org.apache.pdfbox.cos.COSStream;
 import org.apache.pdfbox.pdmodel.PDDocument;
+import org.apache.pdfbox.pdmodel.common.PDRectangle;
 import org.apache.pdfbox.pdmodel.common.PDStream;
 import org.apache.pdfbox.pdmodel.font.encoding.Encoding;
 import org.apache.pdfbox.pdmodel.font.encoding.StandardEncoding;
@@ -435,7 +436,14 @@ public class PDType1Font extends PDSimpl
     @Override
     public BoundingBox getBoundingBox() throws IOException
     {
-        return genericFont.getFontBBox();
+        BoundingBox fontBBox = genericFont.getFontBBox();
+        if (fontBBox.getWidth() == 0 && fontBBox.getHeight() == 0)
+        {
+            PDRectangle fdBB = getFontDescriptor().getFontBoundingBox();
+            fontBBox = new BoundingBox(fdBB.getLowerLeftX(), fdBB.getLowerLeftY(),
+                    fdBB.getUpperRightX(), fdBB.getUpperRightY());
+        }
+        return fontBBox;
     }
 
     //@Override