Revert "Revert "Gjoranv/java9 prep 05""

2 files changed, 5 insertions, 7 deletions

diff --git a/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java b/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java
index 619643a32b6..db2ed6b0b83 100644
--- a/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java
+++ b/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java
@@ -5,6 +5,7 @@ import com.yahoo.log.LogLevel;
 import com.yahoo.plugin.Plugin;
 import com.yahoo.plugin.SystemPropertyConfig;
 
+import java.lang.reflect.InvocationTargetException;
 import java.util.logging.Logger;
 
 /**
@@ -20,8 +21,8 @@ public abstract class AbstractPluginLoader implements PluginLoader {
     protected void loadFromClass(Class<? extends Plugin> pluginClass) {
         Plugin plugin;
         try {
-            plugin = (Plugin) pluginClass.newInstance();
-        } catch (InstantiationException | IllegalAccessException e) {
+            plugin = pluginClass.getDeclaredConstructor().newInstance();
+        } catch (InstantiationException | IllegalAccessException | NoSuchMethodException | InvocationTargetException e) {
             log.log(LogLevel.ERROR, pluginClass.getName() + ": load failed: " + e);
             throw new RuntimeException(e);
         }
diff --git a/logserver/src/main/java/com/yahoo/logserver/Server.java b/logserver/src/main/java/com/yahoo/logserver/Server.java
index f87c66ecd92..36b1c3e31a0 100644
--- a/logserver/src/main/java/com/yahoo/logserver/Server.java
+++ b/logserver/src/main/java/com/yahoo/logserver/Server.java
@@ -50,9 +50,6 @@ public class Server implements Runnable {
 
     private final boolean isInitialized;
 
-    /**
-     * Server constructor
-     */
     private Server() {
         dispatch = new LogDispatcher();
         dispatch.setBatchedMode(true);
@@ -75,7 +72,7 @@ public class Server implements Runnable {
         return ht;
     }
 
-    public void registerPluginLoader(PluginLoader loader) {
+    private void registerPluginLoader(PluginLoader loader) {
         loader.loadPlugins();
     }
 
@@ -90,7 +87,7 @@ public class Server implements Runnable {
         unregisterLogHandler(lh, threadName);
     }
 
-    public void unregisterLogHandler(LogHandler lh, String threadName) {
+    private void unregisterLogHandler(LogHandler lh, String threadName) {
         HandlerThread ht = getHandlerThread(threadName);
         ht.unregisterHandler(lh);
         threadNameForHandler.remove(lh);