You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ms...@apache.org on 2016/12/10 13:00:51 UTC

svn commit: r1773516 - /pdfbox/trunk/examples/src/main/java/org/apache/pdfbox/examples/pdmodel/CreatePDFA.java

Author: msahyoun
Date: Sat Dec 10 13:00:51 2016
New Revision: 1773516

URL: http://svn.apache.org/viewvc?rev=1773516&view=rev
Log:
PDFBOX-3618: add comment that developer should check the license terms.

Modified:
    pdfbox/trunk/examples/src/main/java/org/apache/pdfbox/examples/pdmodel/CreatePDFA.java

Modified: pdfbox/trunk/examples/src/main/java/org/apache/pdfbox/examples/pdmodel/CreatePDFA.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/examples/src/main/java/org/apache/pdfbox/examples/pdmodel/CreatePDFA.java?rev=1773516&r1=1773515&r2=1773516&view=diff
==============================================================================
--- pdfbox/trunk/examples/src/main/java/org/apache/pdfbox/examples/pdmodel/CreatePDFA.java (original)
+++ pdfbox/trunk/examples/src/main/java/org/apache/pdfbox/examples/pdmodel/CreatePDFA.java Sat Dec 10 13:00:51 2016
@@ -66,9 +66,14 @@ public final class CreatePDFA
             PDFont font = PDType0Font.load(doc, new File(fontfile));
 
             // A PDF/A file needs to have the font embedded if the font is used for text rendering
-            // in rendering modes other than text rendering mode 3
+            // in rendering modes other than text rendering mode 3.
+            //
             // This requirement includes the PDF standard fonts, so don't use their static PDFType1Font classes such as
             // PDFType1Font.HELVETICA.
+            //
+            // As there are many different font licenses it is up to the developer to check if the license terms for the
+            // font loaded allows embedding in the PDF.
+            // 
             if (!font.isEmbedded())
             {
             	throw new IllegalStateException("PDF/A compliance requires that all fonts used for"