null logger

config/src/main/java/module-info.java

@@ -1,4 +1,5 @@
 import org.xbib.config.ConfigLogger;
+import org.xbib.config.NullConfigLogger;
 import org.xbib.config.SystemConfigLogger;
 import org.xbib.settings.SettingsLoader;
 
@@ -6,7 +7,7 @@ module org.xbib.config {
     exports org.xbib.config;
     uses ConfigLogger;
     uses SettingsLoader;
-    provides ConfigLogger with SystemConfigLogger;
+    provides ConfigLogger with NullConfigLogger, SystemConfigLogger;
     requires org.xbib.settings.api;
     requires transitive org.xbib.settings.datastructures;
 }

config/src/main/java/org/xbib/config/ConfigLoader.java

@@ -40,7 +40,7 @@ public class ConfigLoader {
         private static ConfigLogger createConfigLogger() {
             ServiceLoader<ConfigLogger> serviceLoader = ServiceLoader.load(ConfigLogger.class);
             Optional<ConfigLogger> optionalConfigLogger = serviceLoader.findFirst();
-            return optionalConfigLogger.orElse(new SystemConfigLogger());
+            return optionalConfigLogger.orElse(new NullConfigLogger());
         }
 
         private static final ConfigLoader configLoader = new ConfigLoader().withLogger(createConfigLogger());

config/src/main/java/org/xbib/config/NullConfigLogger.java

@@ -0,0 +1,16 @@
+package org.xbib.config;
+
+public class NullConfigLogger implements ConfigLogger {
+
+    @Override
+    public void info(String string) {
+    }
+
+    @Override
+    public void warn(String message) {
+    }
+
+    @Override
+    public void error(String message) {
+    }
+}

config/src/test/resources/META-INF/services/org.xbib.config.ConfigLogger

@@ -1 +1,2 @@
+org.xbib.config.NullConfigLogger
 org.xbib.config.SystemConfigLogger

settings-datastructures/src/main/java/org/xbib/settings/datastructures/DatastructureSettings.java

@@ -17,7 +17,7 @@ public class DatastructureSettings implements Settings {
 
     private final TinyMap<String, String> map;
 
-    protected DatastructureSettings(TinyMap<String, String> map) {
+    DatastructureSettings(TinyMap<String, String> map) {
         this.map = map;
     }