You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by cl...@apache.org on 2022/07/22 17:52:23 UTC

[activemq-artemis] 29/42: fix checkstyle on processor

This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a commit to branch new-logging
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git

commit a633cd2e7a84b06de9e1ba94392aad910398e5fc
Author: Robbie Gemmell <ro...@apache.org>
AuthorDate: Thu Jul 21 16:26:05 2022 +0100

    fix checkstyle on processor
---
 .../java/org/apache/activemq/artemis/logprocessor/LogProcessor.java  | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/artemis-log-processor/src/main/java/org/apache/activemq/artemis/logprocessor/LogProcessor.java b/artemis-log-processor/src/main/java/org/apache/activemq/artemis/logprocessor/LogProcessor.java
index 5f077ab0cc..f7e2ed5279 100644
--- a/artemis-log-processor/src/main/java/org/apache/activemq/artemis/logprocessor/LogProcessor.java
+++ b/artemis-log-processor/src/main/java/org/apache/activemq/artemis/logprocessor/LogProcessor.java
@@ -42,7 +42,6 @@ import org.apache.activemq.artemis.logprocessor.annotation.LogBundle;
 import org.apache.activemq.artemis.logprocessor.annotation.GetLogger;
 import org.apache.activemq.artemis.logprocessor.annotation.LogMessage;
 import org.apache.activemq.artemis.logprocessor.annotation.Message;
-import org.slf4j.helpers.MessageFormatter;
 
 @SupportedAnnotationTypes({"org.apache.activemq.artemis.logprocessor.annotation.LogBundle"})
 @SupportedSourceVersion(SourceVersion.RELEASE_11)
@@ -380,7 +379,7 @@ public class LogProcessor extends AbstractProcessor {
          }
 
          if (DEBUG) {
-            debug("Parameter " + parameter + (isException? "is" : "is not") + " an exception");
+            debug("Parameter " + parameter + (isException ? "is" : "is not") + " an exception");
          }
 
          if (bundleAnnotation.enforceExceptionParameterAsLast() && isException && parameters.hasNext()) {
@@ -429,7 +428,7 @@ public class LogProcessor extends AbstractProcessor {
       String formattingString = encodeSpecialChars(bundleAnnotation.projectCode() + messageAnnotation.id() + ": " + messageAnnotation.value());
       if (exceptionParameter != null) {
          writerOutput.println("     FormattingTuple output = org.slf4j.helpers.MessageFormatter.arrayFormat(\"" + formattingString + "\",new Object[] {" + callList + "});");
-         writerOutput.println("     logger." + methodName + "(output.getMessage(), " + exceptionParameter.getSimpleName() +");");
+         writerOutput.println("     logger." + methodName + "(output.getMessage(), " + exceptionParameter.getSimpleName() + ");");
       } else {
          if (!hasParameters) {
             writerOutput.println("      logger." + methodName + "(\"" + formattingString + "\");");