Hi Mehdi,
On 24/01/2012 10:02, [email protected] wrote:
Author: mehdi
Date: Tue Jan 24 10:02:36 2012
New Revision: 1235191
URL: http://svn.apache.org/viewvc?rev=1235191&view=rev
Log:
Corrected typo in error message
Well spotted. I think it's great that we now get an error instead of a
corrupt PDF in the case of license restricted Fonts, but the message
could be clearer, i.e. it does not say why the font is not embedable.
IMHO, the following would be clearer still:
String msg = "The license of font " + this.fontFileURI + " does not allow it to be
embedded in the output file.";
Thanks,
Chris
Modified:
xmlgraphics/fop/trunk/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java
Modified:
xmlgraphics/fop/trunk/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java?rev=1235191&r1=1235190&r2=1235191&view=diff
==============================================================================
---
xmlgraphics/fop/trunk/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java
(original)
+++
xmlgraphics/fop/trunk/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java
Tue Jan 24 10:02:36 2012
@@ -173,7 +173,7 @@ public class TTFFontLoader extends FontL
if (ttf.isEmbeddable()) {
returnFont.setEmbedFileName(this.fontFileURI);
} else {
- String msg = "The font " + this.fontFileURI + " is not
embedabble.";
+ String msg = "The font " + this.fontFileURI + " is not
embeddable.";
throw new RuntimeException(msg);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
