Checkstyle updates

Author: jeffgbutler

src/main/java/org/mybatis/dynamic/sql/AbstractListValueCondition.java

@@ -28,6 +28,8 @@ public abstract class AbstractListValueCondition<T> implements VisitableConditio
     protected final Collection<T> values;
 
     /**
+     * Callback to execute when the list is empty.
+     *
      * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
@@ -38,9 +40,11 @@ protected AbstractListValueCondition(Collection<T> values) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Construct a new condition with a callback.
+     *
      * @param values values
      * @param emptyCallback empty callback
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     protected AbstractListValueCondition(Collection<T> values, Callback emptyCallback) {
@@ -110,9 +114,9 @@ protected <R, S extends AbstractListValueCondition<R>> S mapSupport(Function<? s
     /**
      * Specifies a callback function to be called if the value list is empty when rendered.
      *
-     * @deprecated in favor of the statement configuration functions
      * @param callback a callback function - typically throws an exception to block the statement from executing
      * @return this condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     public abstract AbstractListValueCondition<T> withListEmptyCallback(Callback callback);

src/main/java/org/mybatis/dynamic/sql/Callback.java

@@ -18,6 +18,8 @@
 import java.util.function.Function;
 
 /**
+ * Provides a callback function for empty "in" conditions.
+ *
  * @deprecated in favor of the new statement configuration functionality
  */
 @Deprecated

src/main/java/org/mybatis/dynamic/sql/where/condition/IsIn.java

@@ -39,8 +39,10 @@ public static <T> IsIn<T> empty() {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build an empty condition.
+     *
      * @return a new empty condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     private <S> IsIn<S> emptyWithCallBack() {
@@ -52,9 +54,11 @@ protected IsIn(Collection<T> values) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build a new condition with a callback.
+     *
      * @param values values
      * @param emptyCallback empty callback
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     protected IsIn(Collection<T> values, Callback emptyCallback) {
@@ -68,9 +72,11 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build a new condition with a callback.
+     *
      * @param callback a callback function - typically throws an exception to block the statement from executing
      * @return this condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     @Override

src/main/java/org/mybatis/dynamic/sql/where/condition/IsInCaseInsensitive.java

@@ -35,8 +35,10 @@ public static IsInCaseInsensitive empty() {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build an empty condition.
+     *
      * @return a new empty condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     private IsInCaseInsensitive emptyWithCallback() {
@@ -48,9 +50,11 @@ protected  IsInCaseInsensitive(Collection<String> values) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build a new instance with a callback.
+     *
      * @param values values
      * @param emptyCallback empty callback
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     protected  IsInCaseInsensitive(Collection<String> values, Callback emptyCallback) {
@@ -65,9 +69,11 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build a new instance with a callback.
+     *
      * @param callback a callback function - typically throws an exception to block the statement from executing
      * @return this condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     @Override

src/main/java/org/mybatis/dynamic/sql/where/condition/IsNotIn.java

@@ -39,8 +39,10 @@ public static <T> IsNotIn<T> empty() {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build an empty condition.
+     *
      * @return a new empty condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     private <S> IsNotIn<S> emptyWithCallback() {
@@ -52,9 +54,11 @@ protected IsNotIn(Collection<T> values) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build a new condition with a callback.
+     *
      * @param values values
      * @param emptyCallback empty callback
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     protected IsNotIn(Collection<T> values, Callback emptyCallback) {
@@ -69,10 +73,11 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
     }
 
     /**
+     * Build a new instance with a callback.
      *
-     * @deprecated in favor of the statement configuration functions
      * @param callback a callback function - typically throws an exception to block the statement from executing
      * @return this condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     @Override

src/main/java/org/mybatis/dynamic/sql/where/condition/IsNotInCaseInsensitive.java

@@ -35,8 +35,10 @@ public static IsNotInCaseInsensitive empty() {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build an empty condition.
+     *
      * @return a new empty condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     private IsNotInCaseInsensitive emptyWithCallback() {
@@ -48,9 +50,11 @@ protected IsNotInCaseInsensitive(Collection<String> values) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build a new instance with a callback.
+     *
      * @param values values
      * @param emptyCallback empty callback
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     protected IsNotInCaseInsensitive(Collection<String> values, Callback emptyCallback) {
@@ -65,9 +69,11 @@ public String renderCondition(String columnName, Stream<String> placeholders) {
     }
 
     /**
-     * @deprecated in favor of the statement configuration functions
+     * Build a new instance with a callback.
+     *
      * @param callback a callback function - typically throws an exception to block the statement from executing
      * @return this condition
+     * @deprecated in favor of the statement configuration functions
      */
     @Deprecated
     @Override