ppkarwasz commented on code in PR #2249: URL: https://github.com/apache/logging-log4j2/pull/2249#discussion_r1482842721
########## log4j-api/src/main/java/org/apache/logging/log4j/status/StatusConsoleListener.java: ########## @@ -56,50 +54,59 @@ public StatusConsoleListener(final Level level) { * @throws NullPointerException on null {@code level} or {@code stream} */ public StatusConsoleListener(final Level level, final PrintStream stream) { - this(level, stream, SimpleLoggerFactory.getInstance()); - } - - StatusConsoleListener(final Level level, final PrintStream stream, final SimpleLoggerFactory loggerFactory) { - this.level = Objects.requireNonNull(level, "level"); - this.stream = Objects.requireNonNull(stream, "stream"); - this.logger = Objects.requireNonNull(loggerFactory, "loggerFactory") - .createSimpleLogger( - "StatusConsoleListener", level, ParameterizedNoReferenceMessageFactory.INSTANCE, stream); + this.level = requireNonNull(level, "level"); + this.stream = requireNonNull(stream, "stream"); } /** * Sets the level to a new value. - * @param level The new Level. + * + * @param level the new level + * @throws NullPointerException on null {@code level} */ public void setLevel(final Level level) { - this.level = level; + this.level = requireNonNull(level, "level"); } /** - * Return the Log Level for which the Listener should receive events. - * @return the Log Level. + * Sets the output stream to a new value. + * + * @param stream the new output stream + * @throws NullPointerException on null {@code stream} + * @since 2.23.0 Review Comment: ```suggestion * @since 2.23 ``` Nit: we don't need to specify the patch version where this was introduced, since it **always** must be `0`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org