From a39beed8dc1c831f6bfb1f8ed741d679802433bb Mon Sep 17 00:00:00 2001 From: jimtng <2554958+jimtng@users.noreply.github.com> Date: Sat, 8 Apr 2023 03:47:01 +1000 Subject: [PATCH] EventLogger minor refactor (#3523) * EventLogger minor refactor Signed-off-by: Jimmy Tanagra * Fix null check Signed-off-by: Jimmy Tanagra --------- Signed-off-by: Jimmy Tanagra --- .../openhab/core/io/monitor/internal/EventLogger.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/bundles/org.openhab.core.io.monitor/src/main/java/org/openhab/core/io/monitor/internal/EventLogger.java b/bundles/org.openhab.core.io.monitor/src/main/java/org/openhab/core/io/monitor/internal/EventLogger.java index a60a2b07b..2f857b7eb 100644 --- a/bundles/org.openhab.core.io.monitor/src/main/java/org/openhab/core/io/monitor/internal/EventLogger.java +++ b/bundles/org.openhab.core.io.monitor/src/main/java/org/openhab/core/io/monitor/internal/EventLogger.java @@ -14,6 +14,7 @@ package org.openhab.core.io.monitor.internal; import java.util.HashMap; import java.util.Map; +import java.util.Objects; import java.util.Set; import org.eclipse.jdt.annotation.NonNullByDefault; @@ -73,13 +74,8 @@ public class EventLogger implements EventSubscriber, ReadyTracker { } private Logger getLogger(String eventType) { - String loggerName = "openhab.event." + eventType; - Logger logger = eventLoggers.get(loggerName); - if (logger == null) { - logger = LoggerFactory.getLogger(loggerName); - eventLoggers.put(loggerName, logger); - } - return logger; + return Objects.requireNonNull( + eventLoggers.computeIfAbsent(eventType, type -> LoggerFactory.getLogger("openhab.event." + eventType))); } @Override