Checkstyle fixes

jknack · jknack · commit 5ba38c1288e7 · 2020-01-14T01:36:55.000-03:00
diff --git a/jooby/src/main/java/io/jooby/AccessLogHandler.java b/jooby/src/main/java/io/jooby/AccessLogHandler.java
@@ -1,3 +1,8 @@
+/**
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
 package io.jooby;
 
 import org.slf4j.Logger;
@@ -167,9 +172,12 @@ public class AccessLogHandler implements Route.Decorator {
   private static final char BR = ']';
   private static final char Q = '\"';
 
-  private static Function<Context, String> USER_OR_DASH = ctx -> Optional.ofNullable(ctx.getUser())
-      .map(Object::toString)
-      .orElse(DASH);
+  private static final Function<Context, String> USER_OR_DASH = ctx ->
+      Optional.ofNullable(ctx.getUser())
+          .map(Object::toString)
+          .orElse(DASH);
+  /** Default buffer size. */
+  private static final int MESSAGE_SIZE = 256;
 
   /** The logging system. */
   private final Logger log = LoggerFactory.getLogger(getClass());
@@ -207,7 +215,7 @@ public AccessLogHandler() {
     long timestamp = System.currentTimeMillis();
     return ctx -> {
       ctx.onComplete(context -> {
-        StringBuilder sb = new StringBuilder(256);
+        StringBuilder sb = new StringBuilder(MESSAGE_SIZE);
         sb.append(ctx.getRemoteAddress());
         sb.append(SP).append(DASH).append(SP);
         sb.append(userId.apply(ctx));
diff --git a/jooby/src/main/java/io/jooby/CompletionListeners.java b/jooby/src/main/java/io/jooby/CompletionListeners.java
@@ -1,20 +1,41 @@
+/**
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
 package io.jooby;
 
 import javax.annotation.Nonnull;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.stream.Stream;
 
+/**
+ * Utility class that group one or more completion listeners and execute them in reverse order.
+ *
+ * @author edgar
+ * @since 2.5.2
+ */
 public class CompletionListeners {
   private List<Route.Complete> listeners;
 
+  /**
+   * Add a listener.
+   *
+   * @param listener Listener.
+   */
   public void addListener(@Nonnull Route.Complete listener) {
     if (listeners == null) {
       listeners = new ArrayList<>();
     }
     listeners.add(listener);
   }
 
+  /**
+   * Execute all listeners.
+   *
+   * @param ctx Listeners.
+   */
   public void run(@Nonnull Context ctx) {
     if (listeners != null) {
       Throwable cause = null;
diff --git a/jooby/src/main/java/io/jooby/Route.java b/jooby/src/main/java/io/jooby/Route.java
@@ -9,7 +9,6 @@
 import io.jooby.exception.NotAcceptableException;
 import io.jooby.exception.NotFoundException;
 import io.jooby.exception.UnsupportedMediaType;
-import sun.security.krb5.Config;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -462,7 +461,7 @@ public Route(@Nonnull String method, @Nonnull String pattern, @Nonnull Handler h
   /**
    * Recreate a path pattern using the given variables.
    *
-   * <code>reserve(/{k1}/{k2}, {"k1": ""foo", "k2": "bar"}) => /foo/bar</code>
+   * <code>reserve(/{k1}/{k2}, {"k1": ""foo", "k2": "bar"}) =&gt; /foo/bar</code>
    *
    * @param keys Path keys.
    * @return Path.
@@ -474,7 +473,7 @@ public Route(@Nonnull String method, @Nonnull String pattern, @Nonnull Handler h
   /**
    * Recreate a path pattern using the given variables.
    *
-   * <code>reserve(/{k1}/{k2}, "foo", "bar") => /foo/bar</code>
+   * <code>reserve(/{k1}/{k2}, "foo", "bar") =&gt; /foo/bar</code>
    *
    * @param values Values.
    * @return Path.
diff --git a/modules/jooby-jetty/src/main/java/io/jooby/internal/jetty/JettyOutputStream.java b/modules/jooby-jetty/src/main/java/io/jooby/internal/jetty/JettyOutputStream.java
@@ -1,3 +1,8 @@
+/**
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
 package io.jooby.internal.jetty;
 
 import java.io.IOException;
diff --git a/modules/jooby-utow/src/main/java/io/jooby/internal/utow/UtowCompletionListener.java b/modules/jooby-utow/src/main/java/io/jooby/internal/utow/UtowCompletionListener.java
@@ -1,3 +1,8 @@
+/**
+ * Jooby https://jooby.io
+ * Apache License Version 2.0 https://jooby.io/LICENSE.txt
+ * Copyright 2014 Edgar Espina
+ */
 package io.jooby.internal.utow;
 
 import io.jooby.CompletionListeners;

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,6 @@`
`9`	`9`	`import io.jooby.exception.NotAcceptableException;`
`10`	`10`	`import io.jooby.exception.NotFoundException;`
`11`	`11`	`import io.jooby.exception.UnsupportedMediaType;`
`12`		`-import sun.security.krb5.Config;`
`13`	`12`
`14`	`13`	`import javax.annotation.Nonnull;`
`15`	`14`	`import javax.annotation.Nullable;`
`@@ -462,7 +461,7 @@ public Route(@Nonnull String method, @Nonnull String pattern, @Nonnull Handler h`
`462`	`461`	`/**`
`463`	`462`	`* Recreate a path pattern using the given variables.`
`464`	`463`	`*`
`465`		`- * <code>reserve(/{k1}/{k2}, {"k1": ""foo", "k2": "bar"}) => /foo/bar</code>`
	`464`	`+ * <code>reserve(/{k1}/{k2}, {"k1": ""foo", "k2": "bar"}) => /foo/bar</code>`
`466`	`465`	`*`
`467`	`466`	`* @param keys Path keys.`
`468`	`467`	`* @return Path.`
`@@ -474,7 +473,7 @@ public Route(@Nonnull String method, @Nonnull String pattern, @Nonnull Handler h`
`474`	`473`	`/**`
`475`	`474`	`* Recreate a path pattern using the given variables.`
`476`	`475`	`*`
`477`		`- * <code>reserve(/{k1}/{k2}, "foo", "bar") => /foo/bar</code>`
	`476`	`+ * <code>reserve(/{k1}/{k2}, "foo", "bar") => /foo/bar</code>`
`478`	`477`	`*`
`479`	`478`	`* @param values Values.`
`480`	`479`	`* @return Path.`