1 files changed, 9 insertions, 82 deletions

diff --git a/src/java/org/apache/fop/fonts/apps/PFMReader.java b/src/java/org/apache/fop/fonts/apps/PFMReader.java
index f5ca0e192..ebf7bd2ba 100644
--- a/src/java/org/apache/fop/fonts/apps/PFMReader.java
+++ b/src/java/org/apache/fop/fonts/apps/PFMReader.java
@@ -19,75 +19,34 @@
 package org.apache.fop.fonts.apps;
 
 import java.io.File;
-import java.io.InputStream;
 import java.io.IOException;
-import java.util.Map;
-import java.util.List;
+import java.io.InputStream;
 import java.util.Iterator;
+import java.util.Map;
+
 import javax.xml.parsers.DocumentBuilderFactory;
 import javax.xml.transform.Transformer;
 import javax.xml.transform.TransformerException;
 import javax.xml.transform.TransformerFactory;
 
-import org.w3c.dom.Document;
-import org.w3c.dom.Element;
-
-import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
-
-//FOP
 import org.apache.fop.Version;
 import org.apache.fop.fonts.type1.PFMFile;
 import org.apache.fop.util.CommandLineLogger;
+import org.w3c.dom.Document;
+import org.w3c.dom.Element;
 
 /**
  * A tool which reads PFM files from Adobe Type 1 fonts and creates
  * XML font metrics file for use in FOP.
  */
-public class PFMReader {
+public class PFMReader extends AbstractFontReader {
     
     /**
-     * logging instance
-     */
-    protected static Log log;
-
-    /**
      * Main constructor.
      */
     public PFMReader() {
-        // Create logger if necessary here to allow embedding of PFMReader in
-        // other applications. There is a possible but harmless synchronization
-        // issue.
-        if (log == null) {
-            log = LogFactory.getLog(PFMReader.class);
-        }
-    }
-    
-    /**
-     * Parse commandline arguments. put options in the HashMap and return
-     * arguments in the String array
-     * the arguments: -fn Perpetua,Bold -cn PerpetuaBold per.ttf Perpetua.xml
-     * returns a String[] with the per.ttf and Perpetua.xml. The hash
-     * will have the (key, value) pairs: (-fn, Perpetua) and (-cn, PerpetuaBold)
-     */
-    private static String[] parseArguments(Map options, String[] args) {
-        List arguments = new java.util.ArrayList();
-        for (int i = 0; i < args.length; i++) {
-            if (args[i].startsWith("-")) {
-                if ("-d".equals(args[i]) || "-q".equals(args[i])) {
-                    options.put(args[i], "");
-                } else if ((i + 1) < args.length && !args[i + 1].startsWith("-")) {
-                    options.put(args[i], args[i + 1]);
-                    i++;
-                } else {
-                    options.put(args[i], "");
-                }
-            } else {
-                arguments.add(args[i]);
-            }
-        }
-
-        return (String[])arguments.toArray(new String[arguments.size()]);
+        super();
     }
 
     private static void displayUsage() {
@@ -142,15 +101,8 @@ public class PFMReader {
                                             CommandLineLogger.class.getName());
         }
 
-        //Determine log level
-        if (options.get("-d") != null) {
-            setLogLevel("debug");
-        } else if (options.get("-q") != null) {
-            setLogLevel("error");
-        } else {
-            setLogLevel("info");
-        }
-        
+        determineLogLevel(options);
+
         PFMReader app = new PFMReader();
 
         log.info("PFM Reader for Apache FOP " + Version.getVersion() + "\n");
@@ -194,15 +146,6 @@ public class PFMReader {
         }
     }
 
-    private static void setLogLevel(String level) {
-        // Set the evel for future loggers.
-        LogFactory.getFactory().setAttribute("level", level);
-        if (log instanceof CommandLineLogger) {
-            // Set the level for the logger creates already.
-            ((CommandLineLogger) log).setLogLevel(level);
-        }
-    }
-
     /**
      * Read a PFM file and returns it as an object.
      *
@@ -408,22 +351,6 @@ public class PFMReader {
         }
         return doc;
     }
-
-
-    private String escapeString(String str) {
-        StringBuffer esc = new StringBuffer();
-
-        for (int i = 0; i < str.length(); i++) {
-            if (str.charAt(i) == '\\') {
-                esc.append("\\\\");
-            } else {
-                esc.append(str.charAt(i));
-            }
-        }
-
-        return esc.toString();
-    }
-
 }