Issue #5828 - cleanups from review

Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
jetty · Feb 15, 2021 · 0e4ed5c · 0e4ed5c
1 parent 805d6ef
commit 0e4ed5c
Showing 1 changed file with 3 additions and 16 deletions.
diff --git a/.../java/org/eclipse/jetty/websocket/javax/client/JavaxWebSocketClientContainerProvider.java b/.../java/org/eclipse/jetty/websocket/javax/client/JavaxWebSocketClientContainerProvider.java
@@ -71,23 +71,10 @@ public WebSocketContainer getContainer(HttpClient httpClient)
     }
 
     // See: https://github.com/eclipse-ee4j/websocket-api/issues/212
-    private WebSocketContainer registerShutdown(JavaxWebSocketClientContainer container)
+    private void registerShutdown(JavaxWebSocketClientContainer container)
     {
-        // Register as JVM runtime shutdown hook?
+        // Register as JVM runtime shutdown hook.
         ShutdownThread.register(container);
-
-        if (!container.isStarted())
-        {
-            try
-            {
-                container.start();
-            }
-            catch (Exception e)
-            {
-                throw new RuntimeException("Unable to start Client Container", e);
-            }
-        }
-
-        return container;
+        LifeCycle.start(container);
     }
 }