From c6c409470fda56efed184c75ac4034607333dd30 Mon Sep 17 00:00:00 2001
From: Glen Mazza <gmazza@apache.org>
Date: Mon, 4 Aug 2003 18:53:17 +0000
Subject: [PATCH] 1) Command-line option -l now sets default locale for entire
 instance, not just that of AWTRenderer. 2) (minor)
 CommandLineOptions.getInputHandler() returns error if input mode not properly
 set.

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@196774 13f79535-47bb-0310-9956-ffa450edef68
---
 src/java/org/apache/fop/apps/AWTStarter.java      |  8 +-------
 .../org/apache/fop/apps/CommandLineOptions.java   | 15 +++------------
 2 files changed, 4 insertions(+), 19 deletions(-)

diff --git a/src/java/org/apache/fop/apps/AWTStarter.java b/src/java/org/apache/fop/apps/AWTStarter.java
index caf6bd39b..e7dd49240 100644
--- a/src/java/org/apache/fop/apps/AWTStarter.java
+++ b/src/java/org/apache/fop/apps/AWTStarter.java
@@ -87,13 +87,7 @@ public class AWTStarter extends CommandLineStarter {
     }
 
     private void init() throws FOPException {
-        //Creates Translator according to the language
-        String language = commandLineOptions.getLanguage();
-        if (language == null) {
-            translator = new Translator(Locale.getDefault());
-        } else {
-            translator = new Translator(new Locale(language, ""));
-        }
+        translator = new Translator();
         AWTRenderer renderer = new AWTRenderer(translator);
         frame = createPreviewDialog(renderer, translator);
         renderer.setComponent(frame);
diff --git a/src/java/org/apache/fop/apps/CommandLineOptions.java b/src/java/org/apache/fop/apps/CommandLineOptions.java
index 713f96066..264b6dd38 100644
--- a/src/java/org/apache/fop/apps/CommandLineOptions.java
+++ b/src/java/org/apache/fop/apps/CommandLineOptions.java
@@ -53,6 +53,7 @@ package org.apache.fop.apps;
 // java
 import java.io.File;
 import java.io.FileNotFoundException;
+import java.util.Locale;
 
 // Avalon
 import org.apache.avalon.framework.logger.ConsoleLogger;
@@ -110,8 +111,6 @@ public class CommandLineOptions {
     private int inputmode = NOT_SET;
     /* output mode */
     private int outputmode = NOT_SET;
-    /* language for user information */
-    private String language = null;
 
     private java.util.HashMap rendererOptions;
 
@@ -233,7 +232,7 @@ public class CommandLineOptions {
                 || (args[i + 1].charAt(0) == '-')) {
             throw new FOPException("if you use '-l', you must specify a language");
         } else {
-            language = args[i + 1];
+            Locale.setDefault(new Locale(args[i + 1], ""));
             return 1;
         }
     }
@@ -496,7 +495,7 @@ public class CommandLineOptions {
         case XSLT_INPUT:
             return new XSLTInputHandler(xmlfile, xsltfile);
         default:
-            return new FOInputHandler(fofile);
+            throw new FOPException("Invalid inputmode setting!");
         }
     }
 
@@ -564,14 +563,6 @@ public class CommandLineOptions {
         return userConfigFile;
     }
 
-    /**
-     * Returns the default language
-     * @return the default language
-     */
-    public String getLanguage() {
-        return language;
-    }
-
     /**
      * Indicates if FOP should be silent.
      * @return true if should be silent
-- 
2.39.5