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 2018/02/17 18:45:28 UTC
svn commit: r1824621 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java
Author: tilman
Date: Sat Feb 17 18:45:28 2018
New Revision: 1824621
URL: http://svn.apache.org/viewvc?rev=1824621&view=rev
Log:
PDFBOX-4106: Fix non-embedded vertical font rendering, by Aaron Madlon-Kay
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java?rev=1824621&r1=1824620&r2=1824621&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java Sat Feb 17 18:45:28 2018
@@ -421,8 +421,10 @@ public class PageDrawer extends PDFGraph
if (path != null)
{
- // stretch non-embedded glyph if it does not match the width contained in the PDF
- if (!font.isEmbedded())
+ // Stretch non-embedded glyph if it does not match the height/width contained in the PDF.
+ // Vertical fonts have zero X displacement, so the following code scales to 0 if we don't skip it.
+ // TODO: How should vertical fonts be handled?
+ if (!font.isEmbedded() && !font.isVertical())
{
float fontWidth = font.getWidthFromFont(code);
if (fontWidth > 0 && // ignore spaces