Apply Palantir formatter

Author: vy

log4j-1.2-api/src/main/java/org/apache/log4j/Appender.java

@@ -69,15 +69,13 @@ public interface Appender {
      */
     void doAppend(LoggingEvent event);
 
-
     /**
      * Get the name of this appender.
      *
      * @return name, may be null.
      */
     String getName();
 
-
     /**
      * Set the {@link ErrorHandler} for this appender.
      * @param errorHandler The error handler.
@@ -110,7 +108,6 @@ public interface Appender {
      */
     Layout getLayout();
 
-
     /**
      * Set the name of this appender. The name is used by other
      * components to identify this appender.

log4j-1.2-api/src/main/java/org/apache/log4j/AppenderSkeleton.java

@@ -43,19 +43,16 @@ public abstract class AppenderSkeleton implements Appender, OptionHandler {
     /**
      * Create new instance.
      */
-    public AppenderSkeleton() {
-    }
+    public AppenderSkeleton() {}
 
-    protected AppenderSkeleton(final boolean isActive) {
-    }
+    protected AppenderSkeleton(final boolean isActive) {}
 
     @Override
-    public void activateOptions() {
-    }
+    public void activateOptions() {}
 
     @Override
     public void addFilter(final Filter newFilter) {
-        if(headFilter == null) {
+        if (headFilter == null) {
             headFilter = tailFilter = newFilter;
         } else {
             tailFilter.setNext(newFilter);
@@ -71,8 +68,7 @@ public void clearFilters() {
     }
 
     @Override
-    public void finalize() {
-    }
+    public void finalize() {}
 
     @Override
     public ErrorHandler getErrorHandler() {
@@ -140,33 +136,21 @@ public void setThreshold(final Priority threshold) {
 
     public static class NoOpErrorHandler implements ErrorHandler {
         @Override
-        public void setLogger(final Logger logger) {
-
-        }
+        public void setLogger(final Logger logger) {}
 
         @Override
-        public void error(final String message, final Exception e, final int errorCode) {
-
-        }
+        public void error(final String message, final Exception e, final int errorCode) {}
 
         @Override
-        public void error(final String message) {
-
-        }
+        public void error(final String message) {}
 
         @Override
-        public void error(final String message, final Exception e, final int errorCode, final LoggingEvent event) {
-
-        }
+        public void error(final String message, final Exception e, final int errorCode, final LoggingEvent event) {}
 
         @Override
-        public void setAppender(final Appender appender) {
-
-        }
+        public void setAppender(final Appender appender) {}
 
         @Override
-        public void setBackupAppender(final Appender appender) {
-
-        }
+        public void setBackupAppender(final Appender appender) {}
     }
 }

log4j-1.2-api/src/main/java/org/apache/log4j/BasicConfigurator.java

@@ -60,6 +60,5 @@ public static void resetConfiguration() {
     /**
      * Constructs a new instance.
      */
-    protected BasicConfigurator() {
-    }
+    protected BasicConfigurator() {}
 }

log4j-1.2-api/src/main/java/org/apache/log4j/Category.java

@@ -24,7 +24,6 @@
 import java.util.Vector;
 import java.util.concurrent.ConcurrentMap;
 import java.util.stream.Collectors;
-
 import org.apache.log4j.bridge.AppenderAdapter;
 import org.apache.log4j.bridge.AppenderWrapper;
 import org.apache.log4j.bridge.LogEventWrapper;
@@ -141,12 +140,12 @@ public static void shutdown() {
      * The assigned level of this category. The <code>level</code> variable need not be assigned a value in which case it is
      * inherited form the hierarchy.
      */
-    volatile protected Level level;
+    protected volatile Level level;
 
     /**
      * The parent of this category. All categories have at least one ancestor which is the root category.
      */
-    volatile protected Category parent;
+    protected volatile Category parent;
 
     /**
      * Resource bundle for localized messages.
@@ -355,8 +354,8 @@ public boolean getAdditivity() {
     @SuppressWarnings("unchecked")
     public Enumeration getAllAppenders() {
         if (LogManager.isLog4jCorePresent()) {
-            final Collection<org.apache.logging.log4j.core.Appender> appenders = CategoryUtil.getAppenders(logger)
-                    .values();
+            final Collection<org.apache.logging.log4j.core.Appender> appenders =
+                    CategoryUtil.getAppenders(logger).values();
             return Collections.enumeration(appenders.stream()
                     // omit native Log4j 2.x appenders
                     .filter(AppenderAdapter.Adapter.class::isInstance)
@@ -386,23 +385,23 @@ public Priority getChainedPriority() {
 
     public Level getEffectiveLevel() {
         switch (logger.getLevel().getStandardLevel()) {
-        case ALL:
-            return Level.ALL;
-        case TRACE:
-            return Level.TRACE;
-        case DEBUG:
-            return Level.DEBUG;
-        case INFO:
-            return Level.INFO;
-        case WARN:
-            return Level.WARN;
-        case ERROR:
-            return Level.ERROR;
-        case FATAL:
-            return Level.FATAL;
-        default:
-            // TODO Should this be an IllegalStateException?
-            return Level.OFF;
+            case ALL:
+                return Level.ALL;
+            case TRACE:
+                return Level.TRACE;
+            case DEBUG:
+                return Level.DEBUG;
+            case INFO:
+                return Level.INFO;
+            case WARN:
+                return Level.WARN;
+            case ERROR:
+                return Level.ERROR;
+            case FATAL:
+                return Level.FATAL;
+            default:
+                // TODO Should this be an IllegalStateException?
+                return Level.OFF;
         }
     }
 
@@ -566,7 +565,11 @@ public void log(final String fqcn, final Priority priority, final Object message
         }
     }
 
-    void maybeLog(final String fqcn, final org.apache.logging.log4j.Level level, final Object message, final Throwable throwable) {
+    void maybeLog(
+            final String fqcn,
+            final org.apache.logging.log4j.Level level,
+            final Object message,
+            final Throwable throwable) {
         if (logger.isEnabled(level)) {
             final Message msg = createMessage(message);
             if (logger instanceof ExtendedLogger) {
@@ -587,7 +590,7 @@ void maybeLog(final String fqcn, final org.apache.logging.log4j.Level level, fin
     public void removeAllAppenders() {
         if (aai != null) {
             final Vector appenders = new Vector();
-            for (final Enumeration iter = aai.getAllAppenders(); iter != null && iter.hasMoreElements();) {
+            for (final Enumeration iter = aai.getAllAppenders(); iter != null && iter.hasMoreElements(); ) {
                 appenders.add(iter.nextElement());
             }
             aai.removeAllAppenders();
@@ -672,5 +675,4 @@ public void warn(final Object message) {
     public void warn(final Object message, final Throwable t) {
         maybeLog(FQCN, org.apache.logging.log4j.Level.WARN, message, t);
     }
-
 }

log4j-1.2-api/src/main/java/org/apache/log4j/CategoryKey.java

@@ -30,12 +30,12 @@ class CategoryKey {
     }
 
     @Override
-    final public int hashCode() {
+    public final int hashCode() {
         return hashCache;
     }
 
     @Override
-    final public boolean equals(final Object rArg) {
+    public final boolean equals(final Object rArg) {
         if (this == rArg) {
             return true;
         }

log4j-1.2-api/src/main/java/org/apache/log4j/ConsoleAppender.java

@@ -37,8 +37,7 @@ public class ConsoleAppender extends WriterAppender {
     /**
      * Constructs a non-configured appender.
      */
-    public ConsoleAppender() {
-    }
+    public ConsoleAppender() {}
 
     /**
      * Constructs a configured appender.
@@ -136,5 +135,4 @@ void targetWarn(final String val) {
         StatusLogger.getLogger().warn("[" + val + "] should be System.out or System.err.");
         StatusLogger.getLogger().warn("Using previously set target, System.out by default.");
     }
-
 }

log4j-1.2-api/src/main/java/org/apache/log4j/DefaultCategoryFactory.java

@@ -20,8 +20,7 @@
 
 class DefaultCategoryFactory implements LoggerFactory {
 
-    DefaultCategoryFactory() {
-    }
+    DefaultCategoryFactory() {}
 
     @Override
     public Logger makeNewLoggerInstance(final String name) {

log4j-1.2-api/src/main/java/org/apache/log4j/DefaultThrowableRenderer.java

@@ -16,15 +16,14 @@
  */
 package org.apache.log4j;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import java.io.IOException;
 import java.io.InterruptedIOException;
 import java.io.LineNumberReader;
 import java.io.PrintWriter;
 import java.io.StringReader;
 import java.io.StringWriter;
 import java.util.ArrayList;
-
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.apache.log4j.spi.ThrowableRenderer;
 
 /**
@@ -42,8 +41,7 @@ public final class DefaultThrowableRenderer implements ThrowableRenderer {
      */
     @SuppressFBWarnings(
             value = "INFORMATION_EXPOSURE_THROUGH_AN_ERROR_MESSAGE",
-            justification = "The throwable is formatted into a log file, which should be private."
-    )
+            justification = "The throwable is formatted into a log file, which should be private.")
     public static String[] render(final Throwable throwable) {
         final StringWriter sw = new StringWriter();
         final PrintWriter pw = new PrintWriter(sw);

log4j-1.2-api/src/main/java/org/apache/log4j/FileAppender.java

@@ -16,15 +16,14 @@
  */
 package org.apache.log4j;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import java.io.BufferedWriter;
 import java.io.File;
 import java.io.FileNotFoundException;
 import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.InterruptedIOException;
 import java.io.Writer;
-
-import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import org.apache.log4j.helpers.LogLog;
 import org.apache.log4j.helpers.QuietWriter;
 import org.apache.log4j.spi.ErrorCode;
@@ -65,8 +64,7 @@ public class FileAppender extends WriterAppender {
     /**
      * The default constructor does not do anything.
      */
-    public FileAppender() {
-    }
+    public FileAppender() {}
 
     /**
      * Constructs a FileAppender and open the file designated by <code>filename</code>. The opened filename will become the
@@ -104,7 +102,13 @@ public FileAppender(final Layout layout, final String filename, final boolean ap
      * file.
      * </p>
      */
-    public FileAppender(final Layout layout, final String filename, final boolean append, final boolean bufferedIO, final int bufferSize) throws IOException {
+    public FileAppender(
+            final Layout layout,
+            final String filename,
+            final boolean append,
+            final boolean bufferedIO,
+            final int bufferSize)
+            throws IOException {
         this.layout = layout;
         this.setFile(filename, append, bufferedIO, bufferSize);
     }
@@ -120,7 +124,8 @@ public void activateOptions() {
             try {
                 setFile(fileName, fileAppend, bufferedIO, bufferSize);
             } catch (java.io.IOException e) {
-                errorHandler.error("setFile(" + fileName + "," + fileAppend + ") call failed.", e, ErrorCode.FILE_OPEN_FAILURE);
+                errorHandler.error(
+                        "setFile(" + fileName + "," + fileAppend + ") call failed.", e, ErrorCode.FILE_OPEN_FAILURE);
             }
         } else {
             // LogLog.error("File option not set for appender ["+name+"].");
@@ -251,9 +256,9 @@ public void setFile(final String file) {
      */
     @SuppressFBWarnings(
             value = {"PATH_TRAVERSAL_IN", "PATH_TRAVERSAL_OUT"},
-            justification = "The file name comes from a configuration file."
-    )
-    public synchronized void setFile(String fileName, boolean append, boolean bufferedIO, int bufferSize) throws IOException {
+            justification = "The file name comes from a configuration file.")
+    public synchronized void setFile(String fileName, boolean append, boolean bufferedIO, int bufferSize)
+            throws IOException {
         LogLog.debug("setFile called: " + fileName + ", " + append);
 
         // It does not make sense to have immediate flush and bufferedIO.