Fix some warnings

Signed-off-by: Taeik Lim <[email protected]>

buildSrc/src/main/kotlin/spring.batch.plus.java-library-conventions.gradle.kts

@@ -29,6 +29,12 @@ tasks.compileJava {
     options.encoding = "UTF-8"
 }
 
+tasks.javadoc {
+    options {
+        (this as CoreJavadocOptions).addStringOption("Xdoclint:none", "-quiet")
+    }
+}
+
 sourceSets {
     create("integrationTest") {
         compileClasspath += sourceSets.main.get().output

spring-batch-plus-kotlin/src/main/kotlin/com/navercorp/spring/batch/plus/kotlin/item/adapter/ItemDelegates.kt

@@ -16,6 +16,8 @@
  * limitations under the License.
  */
 
+@file:Suppress("DEPRECATION")
+
 package com.navercorp.spring.batch.plus.kotlin.item.adapter
 
 import com.navercorp.spring.batch.plus.item.adapter.AdapterFactory

spring-batch-plus-kotlin/src/test/kotlin/com/navercorp/spring/batch/plus/kotlin/item/adapter/ItemDelegatesTest.kt

@@ -16,6 +16,8 @@
  * limitations under the License.
  */
 
+@file:Suppress("DEPRECATION")
+
 package com.navercorp.spring.batch.plus.kotlin.item.adapter
 
 import com.navercorp.spring.batch.plus.item.adapter.ItemProcessorAdapter

spring-batch-plus/src/main/java/com/navercorp/spring/batch/plus/item/adapter/AdapterFactory.java

@@ -42,9 +42,12 @@ public final class AdapterFactory {
 	 * Create an adapter which adapt {@link ItemStreamReaderDelegate} to {@link ItemStreamReader}
 	 * with {@link StepScope} bound proxy implementation. It creates new instance for every {@link StepScope}.
 	 *
-	 * @deprecated use {@link com.navercorp.spring.batch.plus.step.adapter.AdapterFactory#itemStreamReader(ItemStreamFluxReaderDelegate)} instead.
-	 * @return an adapted ItemStreamReader
+	 * @deprecated use {@link com.navercorp.spring.batch.plus.step.adapter.AdapterFactory#itemStreamReader(
+	 *ItemStreamFluxReaderDelegate)} instead.
+	 *
 	 * @param <T> a read item type
+	 * @param delegate a delegate
+	 * @return an adapted ItemStreamReader
 	 */
 	@Deprecated
 	public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamReaderDelegate<T> delegate) {
@@ -55,11 +58,14 @@ public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamReader
 	/**
 	 * Create an adapter which adapt {@link ItemProcessorDelegate} to {@link ItemProcessor}.
 	 *
-	 * @deprecated use {@link com.navercorp.spring.batch.plus.step.adapter.AdapterFactory#itemProcessor(com.navercorp.spring.batch.plus.step.adapter.ItemProcessorDelegate)} instead.
-	 * @param delegate a delegate
-	 * @return an adapted ItemProcessor
+	 * @deprecated use
+	 * {@link com.navercorp.spring.batch.plus.step.adapter.AdapterFactory#itemProcessor(
+	 *com.navercorp.spring.batch.plus.step.adapter.ItemProcessorDelegate)} instead.
+	 *
 	 * @param <I> an item type to process
 	 * @param <O> a processed item type
+	 * @param delegate a delegate
+	 * @return an adapted ItemProcessor
 	 */
 	@Deprecated
 	public static <I, O> ItemProcessor<I, O> itemProcessor(@NonNull ItemProcessorDelegate<I, O> delegate) {
@@ -69,10 +75,12 @@ public static <I, O> ItemProcessor<I, O> itemProcessor(@NonNull ItemProcessorDel
 	/**
 	 * Create an adapter which adapt {@link ItemStreamWriterDelegate} to {@link ItemStreamWriter}.
 	 *
-	 * @deprecated use {@link com.navercorp.spring.batch.plus.step.adapter.AdapterFactory#itemStreamWriter(com.navercorp.spring.batch.plus.step.adapter.ItemStreamWriterDelegate)} instead.
+	 * @deprecated use {@link com.navercorp.spring.batch.plus.step.adapter.AdapterFactory#itemStreamWriter(
+	 *com.navercorp.spring.batch.plus.step.adapter.ItemStreamWriterDelegate)} instead.
+	 *
+	 * @param <T> an item type to write
 	 * @param delegate a delegate
 	 * @return an adapted ItemStreamWriter
-	 * @param <T> an item type to write
 	 */
 	@Deprecated
 	public static <T> ItemStreamWriter<T> itemStreamWriter(@NonNull ItemStreamWriterDelegate<T> delegate) {

spring-batch-plus/src/main/java/com/navercorp/spring/batch/plus/step/adapter/AdapterFactory.java

@@ -43,9 +43,9 @@ public final class AdapterFactory {
 	 * Create an adapter which adapt {@link ItemStreamFluxReaderDelegate} to {@link ItemStreamReader}
 	 * with {@link StepScope} bound proxy implementation. It creates new instance for every {@link StepScope}.
 	 *
+	 * @param <T> a read item type
 	 * @param delegate a delegate
 	 * @return an adapted ItemStreamReader
-	 * @param <T> a read item type
 	 */
 	public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamFluxReaderDelegate<T> delegate) {
 		Objects.requireNonNull(delegate, "ItemStreamReader delegate is null");
@@ -56,9 +56,9 @@ public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamFluxRe
 	 * Create an adapter which adapt {@link ItemStreamIterableReaderDelegate} to {@link ItemStreamReader}
 	 * with {@link StepScope} bound proxy implementation. It creates new instance for every {@link StepScope}.
 	 *
+	 * @param <T> a read item type
 	 * @param delegate a delegate
 	 * @return an adapted ItemStreamReader
-	 * @param <T> a read item type
 	 */
 	public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamIterableReaderDelegate<T> delegate) {
 		Objects.requireNonNull(delegate, "ItemStreamReader delegate is null");
@@ -69,9 +69,9 @@ public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamIterab
 	 * Create an adapter which adapt {@link ItemStreamIteratorReaderDelegate} to {@link ItemStreamReader}
 	 * with {@link StepScope} bound proxy implementation. It creates new instance for every {@link StepScope}.
 	 *
+	 * @param <T> a read item type
 	 * @param delegate a delegate
 	 * @return an adapted ItemStreamReader
-	 * @param <T> a read item type
 	 */
 	public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamIteratorReaderDelegate<T> delegate) {
 		Objects.requireNonNull(delegate, "ItemStreamReader delegate is null");
@@ -82,9 +82,9 @@ public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamIterat
 	 * Create an adapter which adapt {@link ItemStreamSimpleReaderDelegate} to {@link ItemStreamReader}
 	 * with {@link StepScope} bound proxy implementation. It creates new instance for every {@link StepScope}.
 	 *
+	 * @param <T> a read item type
 	 * @param delegate a delegate
 	 * @return an adapted ItemStreamReader
-	 * @param <T> a read item type
 	 */
 	public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamSimpleReaderDelegate<T> delegate) {
 		Objects.requireNonNull(delegate, "ItemStreamReader delegate is null");
@@ -94,10 +94,10 @@ public static <T> ItemStreamReader<T> itemStreamReader(@NonNull ItemStreamSimple
 	/**
 	 * Create an adapter which adapt {@link ItemProcessorDelegate} to {@link ItemProcessor}.
 	 *
-	 * @param delegate a delegate
-	 * @return an adapted ItemProcessor
 	 * @param <I> an item type to process
 	 * @param <O> a processed item type
+	 * @param delegate a delegate
+	 * @return an adapted ItemProcessor
 	 */
 	public static <I, O> ItemProcessor<I, O> itemProcessor(@NonNull ItemProcessorDelegate<I, O> delegate) {
 		return ItemProcessorAdapter.of(delegate);
@@ -106,9 +106,9 @@ public static <I, O> ItemProcessor<I, O> itemProcessor(@NonNull ItemProcessorDel
 	/**
 	 * Create an adapter which adapt {@link ItemStreamWriterDelegate} to {@link ItemStreamWriter}.
 	 *
+	 * @param <T> an item type to write
 	 * @param delegate a delegate
 	 * @return an adapted ItemStreamWriter
-	 * @param <T> an item type to write
 	 */
 	public static <T> ItemStreamWriter<T> itemStreamWriter(@NonNull ItemStreamWriterDelegate<T> delegate) {
 		return ItemStreamWriterAdapter.of(delegate);