/hg/icedtea-web: minor refactoring + dead code removal

dlila at icedtea.classpath.org dlila at icedtea.classpath.org
Mon Feb 28 14:27:22 PST 2011


changeset 26b24fb414ec in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=26b24fb414ec
author: Denis Lila <dlila at redhat.com>
date: Mon Feb 28 17:30:40 2011 -0500

	minor refactoring + dead code removal


diffstat:

2 files changed, 7 insertions(+), 3 deletions(-)
ChangeLog                                        |    7 +++++++
plugin/icedteanp/java/sun/applet/PluginMain.java |    3 ---

diffs (117 lines):

diff -r 54077eaf4670 -r 26b24fb414ec ChangeLog
--- a/ChangeLog	Mon Feb 28 14:30:39 2011 -0500
+++ b/ChangeLog	Mon Feb 28 17:30:40 2011 -0500
@@ -1,3 +1,14 @@ 2011-02-28  Omair Majid  <omajid at redhat.
+2011-02-28  Denis Lila  <dlila at redhat.com>
+
+	* plugin/icedteanp/java/sun/applet/PluginMain.java
+	(redirectStreams, streamHandler, securityContext) make them local.
+	(theVersion): make it private.
+	(PluginMain): make it private. Empty the body.
+	(main): Do all the work that used to be in PluginMain.
+	(connect): make it static, and now it returns a PluginStreamHandler
+	instead of setting a static variable.
+	(messageAvailable, getMessage): Remove.
+
 2011-02-28  Omair Majid  <omajid at redhat.com>
 
 	* netx/net/sourceforge/jnlp/resources/Messages.properties: Add Password,
diff -r 54077eaf4670 -r 26b24fb414ec plugin/icedteanp/java/sun/applet/PluginMain.java
--- a/plugin/icedteanp/java/sun/applet/PluginMain.java	Mon Feb 28 14:30:39 2011 -0500
+++ b/plugin/icedteanp/java/sun/applet/PluginMain.java	Mon Feb 28 17:30:40 2011 -0500
@@ -85,14 +85,9 @@ public class PluginMain {
     public static final String PLUGIN_STDERR_FILE = "java.stderr";
     public static final String PLUGIN_STDOUT_FILE = "java.stdout";
 
-    final boolean redirectStreams = System.getenv().containsKey("ICEDTEAPLUGIN_DEBUG");
-    static PluginStreamHandler streamHandler;
-
     // This is used in init().  Getting rid of this is desirable but depends
     // on whether the property that uses it is necessary/standard.
-    public static final String theVersion = System.getProperty("java.version");
-
-    private PluginAppletSecurityContext securityContext;
+    private static final String theVersion = System.getProperty("java.version");
 
     /**
      * The main entry point into AppletViewer.
@@ -105,7 +100,24 @@ public class PluginMain {
         }
 
         try {
-            PluginMain pm = new PluginMain(args[0], args[1]);
+            PluginStreamHandler streamHandler = connect(args[0], args[1]);
+            boolean redirectStreams = System.getenv().containsKey("ICEDTEAPLUGIN_DEBUG");
+
+            // must be called before JNLPRuntime.initialize()
+            JNLPRuntime.setRedirectStreams(redirectStreams);
+
+            PluginAppletSecurityContext sc = new PluginAppletSecurityContext(0);
+            sc.prePopulateLCClasses();
+            PluginAppletSecurityContext.setStreamhandler(streamHandler);
+            AppletSecurityContextManager.addContext(0, sc);
+
+            PluginAppletViewer.setStreamhandler(streamHandler);
+            PluginAppletViewer.setPluginCallRequestFactory(new PluginCallRequestFactory());
+
+            init();
+
+            // Streams set. Start processing.
+            streamHandler.startProcessing();
         } catch (Exception e) {
             e.printStackTrace();
             System.err.println("Something very bad happened. I don't know what to do, so I am going to exit :(");
@@ -113,34 +125,19 @@ public class PluginMain {
         }
     }
 
-    public PluginMain(String inPipe, String outPipe) {
-
-        connect(inPipe, outPipe);
-
-        // must be called before JNLPRuntime.initialize()
-        JNLPRuntime.setRedirectStreams(redirectStreams);
-
-        securityContext = new PluginAppletSecurityContext(0);
-        securityContext.prePopulateLCClasses();
-        securityContext.setStreamhandler(streamHandler);
-        AppletSecurityContextManager.addContext(0, securityContext);
-
-        PluginAppletViewer.setStreamhandler(streamHandler);
-        PluginAppletViewer.setPluginCallRequestFactory(new PluginCallRequestFactory());
-
-        init();
-
-        // Streams set. Start processing.
-        streamHandler.startProcessing();
+    private PluginMain() {
+        // The PluginMain constructor should never, EVER, be called
     }
 
-    public void connect(String inPipe, String outPipe) {
+    private static PluginStreamHandler connect(String inPipe, String outPipe) {
+        PluginStreamHandler streamHandler = null;
         try {
             streamHandler = new PluginStreamHandler(new FileInputStream(inPipe), new FileOutputStream(outPipe));
             PluginDebug.debug("Streams initialized");
         } catch (IOException ioe) {
             ioe.printStackTrace();
         }
+        return streamHandler;
     }
 
     private static void init() {
@@ -206,13 +203,4 @@ public class PluginMain {
         CookieManager ckManager = new PluginCookieManager();
         CookieHandler.setDefault(ckManager);
     }
-
-    static boolean messageAvailable() {
-        return streamHandler.messageAvailable();
-    }
-
-    static String getMessage() {
-        return streamHandler.getMessage();
-    }
-
 }



More information about the distro-pkg-dev mailing list