From 19e74fe5dacd03e0cb5582f840e15262e39fe24f Mon Sep 17 00:00:00 2001
From: Jiri Vanek <jvanek@redhat.com>
Date: Wed, 25 Sep 2013 18:50:18 +0200
Subject: Introduced logging bottleneck

---
 .../jnlp/services/XSingleInstanceService.java         | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

(limited to 'netx/net/sourceforge/jnlp/services/XSingleInstanceService.java')

diff --git a/netx/net/sourceforge/jnlp/services/XSingleInstanceService.java b/netx/net/sourceforge/jnlp/services/XSingleInstanceService.java
index 35da018..f82889e 100644
--- a/netx/net/sourceforge/jnlp/services/XSingleInstanceService.java
+++ b/netx/net/sourceforge/jnlp/services/XSingleInstanceService.java
@@ -33,6 +33,7 @@ import javax.management.InstanceAlreadyExistsException;
 import net.sourceforge.jnlp.JNLPFile;
 import net.sourceforge.jnlp.PluginBridge;
 import net.sourceforge.jnlp.runtime.JNLPRuntime;
+import net.sourceforge.jnlp.util.logging.OutputController;
 
 /**
  * This class implements SingleInstanceService
@@ -63,9 +64,7 @@ public class XSingleInstanceService implements ExtendedSingleInstanceService {
                 listeningSocket = new ServerSocket(0);
                 lockFile.createWithPort(listeningSocket.getLocalPort());
 
-                if (JNLPRuntime.isDebug()) {
-                    System.out.println("Starting SingleInstanceServer on port" + listeningSocket);
-                }
+                OutputController.getLogger().log("Starting SingleInstanceServer on port" + listeningSocket);
 
                 while (true) {
                     try {
@@ -76,19 +75,19 @@ public class XSingleInstanceService implements ExtendedSingleInstanceService {
                         notifySingleInstanceListeners(arguments);
                     } catch (Exception exception) {
                         // not much to do here...
-                        exception.printStackTrace();
+                        OutputController.getLogger().log(OutputController.Level.ERROR_ALL, exception);
                     }
 
                 }
             } catch (IOException e) {
-                e.printStackTrace();
+                OutputController.getLogger().log(OutputController.Level.ERROR_ALL, e);
             } finally {
                 if (listeningSocket != null) {
                     try {
                         listeningSocket.close();
                     } catch (IOException e) {
                         // Give up.
-                        e.printStackTrace();
+                        OutputController.getLogger().log(OutputController.Level.ERROR_ALL, e);
                     }
                 }
             }
@@ -136,9 +135,7 @@ public class XSingleInstanceService implements ExtendedSingleInstanceService {
         SingleInstanceLock lockFile = new SingleInstanceLock(jnlpFile);
         if (lockFile.isValid()) {
             int port = lockFile.getPort();
-            if (JNLPRuntime.isDebug()) {
-                System.out.println("Lock file is valid (port=" + port + "). Exiting.");
-            }
+            OutputController.getLogger().log("Lock file is valid (port=" + port + "). Exiting.");
 
             String[] args = null;
             if (jnlpFile.isApplet()) {
@@ -204,9 +201,7 @@ public class XSingleInstanceService implements ExtendedSingleInstanceService {
             serverCommunicationSocket.close();
 
         } catch (UnknownHostException unknownHost) {
-            if (JNLPRuntime.isDebug()) {
-                System.out.println("Unable to find localhost");
-            }
+            OutputController.getLogger().log("Unable to find localhost");
             throw new RuntimeException(unknownHost);
         }
     }
-- 
cgit v1.2.3