svn commit: r1758804 - in /xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts: FontReader.java LazyFont.java XMLFontMetricsReader.java

Author: gadams
Date: Thu Sep  1 16:14:25 2016
New Revision: 1758804

URL: http://svn.apache.org/viewvc?rev=1758804&view=rev
Log:
Rename o.a.f.fonts.FontReader to XMLMetricsFontReader, then deprecate the latter.

Added:
    xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/XMLFontMetricsReader.java
      - copied, changed from r1758803, xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/FontReader.java
Removed:
    xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/FontReader.java
Modified:
    xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/LazyFont.java

Modified: xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/LazyFont.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/LazyFont.java?rev=1758804&r1=1758803&r2=1758804&view=diff
==============================================================================
--- xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/LazyFont.java (original)
+++ xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/LazyFont.java Thu Sep  1 16:14:25 2016
@@ -101,12 +101,13 @@ public class LazyFont extends Typeface i
         if (!isMetricsLoaded) {
             try {
                 if (fontUris.getMetrics() != null) {
-                    /**@todo Possible thread problem here */
-                    FontReader reader = null;
+                    // Use of XML based font metrics is DEPRECATED!
+                    // @todo Possible thread problem here
+                    XMLFontMetricsReader reader = null;
                     InputStream in = resourceResolver.getResource(fontUris.getMetrics());
                     InputSource src = new InputSource(in);
                     src.setSystemId(fontUris.getMetrics().toASCIIString());
-                    reader = new FontReader(src, resourceResolver);
+                    reader = new XMLFontMetricsReader(src, resourceResolver);
                     reader.setKerningEnabled(useKerning);
                     reader.setAdvancedEnabled(useAdvanced);
                     if (this.embedded) {

Copied: xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/XMLFontMetricsReader.java (from r1758803, xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/FontReader.java)
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/XMLFontMetricsReader.java?p2=xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/XMLFontMetricsReader.java&p1=xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/FontReader.java&r1=1758803&r2=1758804&rev=1758804&view=diff
==============================================================================
--- xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/FontReader.java (original)
+++ xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/fonts/XMLFontMetricsReader.java Thu Sep  1 16:14:25 2016
@@ -43,16 +43,20 @@ import org.apache.fop.apps.io.InternalRe
 import org.apache.fop.fonts.apps.TTFReader;
 
 /**
- * Class for reading a metric.xml file and creating a font object.
- * Typical usage:
+ * <p>Class for reading a metric.xml file and creating a font object.
+ * Typical usage:</p>
  * <pre>
- * FontReader reader = new FontReader(<path til metrics.xml>);
+ * XMLFontMetricsReader reader = new XMLFontMetricsReader(<path til metrics.xml>);
  * reader.setFontEmbedPath(<path to a .ttf or .pfb file or null to diable embedding>);
  * reader.useKerning(true);
  * Font f = reader.getFont();
  * </pre>
+ * <p><strong>N.B. This is deprecated functionality and is expected to be
+ * removed from a future version of FOP. New applications using FOP should
+ * not make direct or implied use of this mechanism.</strong></p>
  */
-public class FontReader extends DefaultHandler {
+@Deprecated
+public class XMLFontMetricsReader extends DefaultHandler {
 
     private boolean isCID;
     private CustomFont returnFont;
@@ -69,12 +73,12 @@ public class FontReader extends DefaultH
     private List<CMapSegment> bfranges;
 
     /**
-     * Construct a FontReader object from a path to a metric.xml file
+     * Construct a XMLFontMetricsReader object from a path to a metric.xml file
      * and read metric data
      * @param source Source of the font metric file
      * @throws FOPException if loading the font fails
      */
-    public FontReader(InputSource source, InternalResourceResolver resourceResolver) throws FOPException {
+    public XMLFontMetricsReader(InputSource source, InternalResourceResolver resourceResolver) throws FOPException {
         this.resourceResolver = resourceResolver;
         createFont(source);
     }