slawekjaranowski commented on code in PR #11969:
URL: https://github.com/apache/maven/pull/11969#discussion_r3113171736


##########
maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleLogger.java:
##########
@@ -29,24 +29,38 @@
  */
 public class MavenSimpleLogger extends SimpleLogger {
 
+    private String traceRenderedLevel;
+    private String debugRenderedLevel;
+    private String infoRenderedLevel;
+    private String warnRenderedLevel;
+    private String errorRenderedLevel;
+
     MavenSimpleLogger(String name) {
         super(name);
     }
 
     @Override
     protected String renderLevel(int level) {
+        if (traceRenderedLevel == null) {

Review Comment:
   we will create it twice .... next time last object will be used, the same is 
in master
   I think synchronization is not needed here



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to