From 074d8c686ea2d4133e7c6bd27f52b04b8b17254d Mon Sep 17 00:00:00 2001
From: Basil Crow <me@basilcrow.com>
Date: Wed, 26 Oct 2022 08:43:46 -0700
Subject: [PATCH] Clean up `JnlpSlaveRestarterInstaller` (#7302)

---
 .../JnlpSlaveRestarterInstaller.java          | 26 +++++++------------
 1 file changed, 10 insertions(+), 16 deletions(-)

diff --git a/core/src/main/java/jenkins/slaves/restarter/JnlpSlaveRestarterInstaller.java b/core/src/main/java/jenkins/slaves/restarter/JnlpSlaveRestarterInstaller.java
index 3d8297504365..17978efe72ec 100644
--- a/core/src/main/java/jenkins/slaves/restarter/JnlpSlaveRestarterInstaller.java
+++ b/core/src/main/java/jenkins/slaves/restarter/JnlpSlaveRestarterInstaller.java
@@ -52,20 +52,20 @@ public Void call() throws Exception {
             install(c, listener);
             return null;
         }
-    }
 
-    private static void install(Computer c, TaskListener listener) {
-        try {
-            final List<SlaveRestarter> restarters = new ArrayList<>(SlaveRestarter.all());
+        private static void install(Computer c, TaskListener listener) {
+            try {
+                final List<SlaveRestarter> restarters = new ArrayList<>(SlaveRestarter.all());
 
-            VirtualChannel ch = c.getChannel();
-            if (ch == null) return;  // defensive check
+                VirtualChannel ch = c.getChannel();
+                if (ch == null) return;  // defensive check
 
-            List<SlaveRestarter> effective = ch.call(new FindEffectiveRestarters(restarters));
+                List<SlaveRestarter> effective = ch.call(new FindEffectiveRestarters(restarters));
 
-            LOGGER.log(FINE, "Effective SlaveRestarter on {0}: {1}", new Object[] {c.getName(), effective});
-        } catch (Throwable e) {
-            Functions.printStackTrace(e, listener.error("Failed to install restarter"));
+                LOGGER.log(FINE, "Effective SlaveRestarter on {0}: {1}", new Object[] {c.getName(), effective});
+            } catch (Throwable e) {
+                Functions.printStackTrace(e, listener.error("Failed to install restarter"));
+            }
         }
     }
 
@@ -81,12 +81,6 @@ public List<SlaveRestarter> call() throws IOException {
             Engine e = Engine.current();
             if (e == null) return null;    // not running under Engine
 
-            try {
-                Engine.class.getMethod("addListener", EngineListener.class);
-            } catch (NoSuchMethodException ignored) {
-                return null;    // running with older version of remoting that doesn't support adding listener
-            }
-
             // filter out ones that doesn't apply
             restarters.removeIf(r -> !r.canWork());