From 376cb3fd21b7881cad8df1caa5f38be2c140b0da Mon Sep 17 00:00:00 2001 From: Jim Hahn Date: Fri, 20 Aug 2021 17:02:13 -0400 Subject: Fix sonars and java warnings in common Issue-ID: POLICY-3284 Change-Id: I13fc28c6ae681871be8bb9cf3d222d7c65459a44 Signed-off-by: Jim Hahn --- .../policy/common/utils/logging/LoggerUtils.java | 31 +++------------------- 1 file changed, 4 insertions(+), 27 deletions(-) (limited to 'utils/src/main/java') diff --git a/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java b/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java index 9276f749..89575eb2 100644 --- a/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java +++ b/utils/src/main/java/org/onap/policy/common/utils/logging/LoggerUtils.java @@ -23,13 +23,15 @@ package org.onap.policy.common.utils.logging; import ch.qos.logback.classic.LoggerContext; +import lombok.AccessLevel; +import lombok.NoArgsConstructor; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.slf4j.Marker; import org.slf4j.MarkerFactory; - -public class LoggerUtils { +@NoArgsConstructor(access = AccessLevel.PRIVATE) +public final class LoggerUtils { private static final Logger LOGGER = LoggerFactory.getLogger(LoggerUtils.class); @@ -70,17 +72,6 @@ public class LoggerUtils { */ public static final Marker TRANSACTION_LOG_MARKER = MarkerFactory.getMarker(TRANSACTION_LOG_MARKER_NAME); - /** - * Logger delegate. - */ - private final Logger mlogger; - - /** - * Constructor. - */ - public LoggerUtils(final Logger loggerP) { - this.mlogger = checkNotNull(loggerP); - } /** * Set the log level of a logger. @@ -111,18 +102,4 @@ public class LoggerUtils { return logger.getLevel().toString(); } - - /** - * Dependency-free nullcheck. - * - * @param in to be checked - * @param argument (and return) type - * @return input arg - */ - private static T checkNotNull(final T in) { - if (in == null) { - throw new NullPointerException(); - } - return in; - } } -- cgit 1.2.3-korg