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

[activemq-artemis] 05/06: format some generated bits more consistent with regular code style

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

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

commit d704500af32ecd49e854bd516bc1467a7435c7b6
Author: Robbie Gemmell <ro...@apache.org>
AuthorDate: Fri Sep 23 17:14:54 2022 +0100

    format some generated bits more consistent with regular code style
---
 .../annotation/processor/LogAnnotationProcessor.java     | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/artemis-log-annotation-processor/src/main/java/org/apache/activemq/artemis/logs/annotation/processor/LogAnnotationProcessor.java b/artemis-log-annotation-processor/src/main/java/org/apache/activemq/artemis/logs/annotation/processor/LogAnnotationProcessor.java
index 2d9e0bd666..67f83d3650 100644
--- a/artemis-log-annotation-processor/src/main/java/org/apache/activemq/artemis/logs/annotation/processor/LogAnnotationProcessor.java
+++ b/artemis-log-annotation-processor/src/main/java/org/apache/activemq/artemis/logs/annotation/processor/LogAnnotationProcessor.java
@@ -287,18 +287,17 @@ public class LogAnnotationProcessor extends AbstractProcessor {
       }
 
       // the real implementation
-      writerOutput.println(")");
-      writerOutput.println("   {");
+      writerOutput.println(") {");
 
       String formattingString = encodeSpecialChars(bundleAnnotation.projectCode() + messageAnnotation.id() + ": " + messageAnnotation.value());
       if (!hasParameters) {
-         writerOutput.println("     String returnString = \"" + formattingString + "\";");
+         writerOutput.println("      String returnString = \"" + formattingString + "\";");
       } else {
-         writerOutput.println("     String returnString = MessageFormatter.arrayFormat(\"" + formattingString + "\", new Object[]{" + callList + "}).getMessage();");
+         writerOutput.println("      String returnString = MessageFormatter.arrayFormat(\"" + formattingString + "\", new Object[]{" + callList + "}).getMessage();");
       }
 
       if (executableMember.getReturnType().toString().equals(String.class.getName())) {
-         writerOutput.println("     return returnString;");
+         writerOutput.println("      return returnString;");
       } else {
          writerOutput.println();
          writerOutput.println("      {");
@@ -378,7 +377,9 @@ public class LogAnnotationProcessor extends AbstractProcessor {
       // This is really a debug output
       writerOutput.println("   // " + loggerAnnotation.toString());
       writerOutput.println("   @Override");
-      writerOutput.println("   public Logger " + executableMember.getSimpleName() + "() { return logger; }");
+      writerOutput.println("   public Logger " + executableMember.getSimpleName() + "() {");
+      writerOutput.println("      return logger;");
+      writerOutput.println("   }");
       writerOutput.println();
    }
 
@@ -424,8 +425,7 @@ public class LogAnnotationProcessor extends AbstractProcessor {
          }
       }
 
-      writerOutput.println(")");
-      writerOutput.println("   {");
+      writerOutput.println(") {");
 
       // the one that will be used on the logger call
       StringBuffer callList = new StringBuffer();