You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ja...@apache.org on 2014/08/28 02:27:28 UTC
svn commit: r1621003 -
/pdfbox/branches/no-awt/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/ExternalFonts.java
Author: jahewson
Date: Thu Aug 28 00:27:28 2014
New Revision: 1621003
URL: http://svn.apache.org/r1621003
Log:
PDFBOX-2262: Changed default font to Times-Roman
Modified:
pdfbox/branches/no-awt/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/ExternalFonts.java
Modified: pdfbox/branches/no-awt/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/ExternalFonts.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/no-awt/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/ExternalFonts.java?rev=1621003&r1=1621002&r2=1621003&view=diff
==============================================================================
--- pdfbox/branches/no-awt/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/ExternalFonts.java (original)
+++ pdfbox/branches/no-awt/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/ExternalFonts.java Thu Aug 28 00:27:28 2014
@@ -161,8 +161,7 @@ public final class ExternalFonts
*/
public static TrueTypeFont getFallbackFont(PDFontDescriptor fontDescriptor)
{
- String fontName = "Times";
-
+ String fontName;
if (fontDescriptor != null)
{
// heuristic detection of bold
@@ -227,11 +226,16 @@ public final class ExternalFonts
}
}
}
+ else
+ {
+ // if there is no FontDescriptor then we just fall back to Times Roman
+ fontName = "Times-Roman";
+ }
TrueTypeFont ttf = getTrueTypeFont(fontName);
if (ttf == null)
{
- throw new IllegalStateException("No fonts available on the system");
+ throw new IllegalStateException("No fonts available on the system for " + fontName);
}
return ttf;
}