Merge branch 'develop'

java-io-base/pom.xml

@@ -5,7 +5,7 @@
     <parent>
         <groupId>com.github.nbbrd.java-io-util</groupId>
         <artifactId>java-io-parent</artifactId>
-        <version>0.0.14</version>
+        <version>0.0.15</version>
     </parent>
 
     <artifactId>java-io-base</artifactId>

java-io-base/src/main/java/internal/io/IOIterators.java

@@ -44,12 +44,12 @@ public enum Empty implements IOIterator<Object> {
         INSTANCE;
 
         @Override
-        public boolean hasNextWithIO() throws IOException {
+        public boolean hasNextWithIO() {
             return false;
         }
 
         @Override
-        public Object nextWithIO() throws IOException, NoSuchElementException {
+        public Object nextWithIO() throws NoSuchElementException {
             throw new NoSuchElementException();
         }
 
@@ -73,12 +73,12 @@ public static final class Singleton<E> implements IOIterator<E> {
         private boolean first = true;
 
         @Override
-        public boolean hasNextWithIO() throws IOException {
+        public boolean hasNextWithIO() {
             return first;
         }
 
         @Override
-        public E nextWithIO() throws IOException, NoSuchElementException {
+        public E nextWithIO() throws NoSuchElementException {
             if (!hasNextWithIO()) {
                 throw new NoSuchElementException();
             }

java-io-base/src/main/java/internal/io/text/InternalFormatter.java

@@ -42,6 +42,7 @@ public CharSequence formatTemporalAccessor(DateTimeFormatter formatter, Temporal
             try {
                 return formatter.format(value);
             } catch (DateTimeException ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -68,6 +69,7 @@ public CharSequence formatStringList(Function<Stream<CharSequence>, String> join
             try {
                 return joiner.apply(value.stream().map(CharSequence.class::cast));
             } catch (Exception ex) {
+                doNothing(ex);
             }
         }
         return null;

java-io-base/src/main/java/internal/io/text/InternalParser.java

@@ -41,6 +41,7 @@
 @lombok.experimental.UtilityClass
 public class InternalParser {
 
+    @SuppressWarnings("unchecked")
     public <T> T parseTemporalAccessor(DateTimeFormatter formatter, TemporalQuery<T>[] queries, CharSequence input) {
         if (input != null) {
             try {
@@ -53,6 +54,7 @@ public <T> T parseTemporalAccessor(DateTimeFormatter formatter, TemporalQuery<T>
                         return (T) formatter.parseBest(input, queries);
                 }
             } catch (DateTimeParseException ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -94,6 +96,7 @@ public double[] parseDoubleArray(CharSequence input) {
                 }
                 return result;
             } catch (Exception ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -115,6 +118,7 @@ public String[] parseStringArray(CharSequence input) {
                 }
                 return result;
             } catch (Exception ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -125,6 +129,7 @@ public Integer parseInteger(CharSequence input) {
             try {
                 return Integer.valueOf(input.toString());
             } catch (NumberFormatException ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -135,6 +140,7 @@ public Long parseLong(CharSequence input) {
             try {
                 return Long.valueOf(input.toString());
             } catch (NumberFormatException ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -145,6 +151,7 @@ public Double parseDouble(CharSequence input) {
             try {
                 return Double.valueOf(input.toString());
             } catch (NumberFormatException ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -155,6 +162,7 @@ public Charset parseCharset(CharSequence input) {
             try {
                 return Charset.forName(input.toString());
             } catch (IllegalArgumentException ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -183,6 +191,7 @@ public <T extends Enum<T>> T parseEnum(Class<T> enumClass, CharSequence input) {
             try {
                 return Enum.valueOf(enumClass, input.toString());
             } catch (IllegalArgumentException ex) {
+                doNothing(ex);
             }
         }
         return null;
@@ -291,6 +300,7 @@ public URL parseURL(CharSequence input) {
             try {
                 return new URL(input.toString());
             } catch (MalformedURLException ex) {
+                doNothing(ex);
             }
         }
         return null;

java-io-base/src/main/java/module-info.java

@@ -27,5 +27,5 @@
     exports nbbrd.io.text;
     exports nbbrd.io.zip;
 
-    exports internal.io.text to nbbrd.io.xml;
+    exports internal.io.text to nbbrd.io.xml, nbbrd.io.xml.bind;
 }

java-io-base/src/main/java/nbbrd/io/AbstractIOIterator.java

@@ -34,7 +34,7 @@ public abstract class AbstractIOIterator<E> implements IOIterator<E> {
 
     private enum State {
         COMPUTED, NOT_COMPUTED, DONE
-    };
+    }
 
     private State state = State.NOT_COMPUTED;
 

java-io-base/src/main/java/nbbrd/io/FileFormatter.java

@@ -33,7 +33,7 @@ default void formatPath(@NonNull T value, @NonNull Path target) throws IOExcepti
         }
     }
 
-    default void formatStream(@NonNull T value, IOSupplier<? extends OutputStream> target) throws IOException {
+    default void formatStream(@NonNull T value, @NonNull IOSupplier<? extends OutputStream> target) throws IOException {
         Objects.requireNonNull(value, "value");
         Objects.requireNonNull(target, "target");
         try (OutputStream resource = LegacyFiles.checkResource(target.getWithIO(), "Missing OutputStream")) {

java-io-base/src/main/java/nbbrd/io/IOIterator.java

@@ -67,6 +67,7 @@ default void forEachRemainingWithIO(@NonNull IOConsumer<? super E> action) throw
         return new IOIterators.Unchecked<>(this);
     }
 
+    @SuppressWarnings("unchecked")
     @StaticFactoryMethod
     static <E> @NonNull IOIterator<E> empty() {
         return (IOIterator<E>) IOIterators.Empty.INSTANCE;

java-io-base/src/main/java/nbbrd/io/WrappedIOException.java

@@ -35,7 +35,7 @@ public static IOException wrap(@NonNull Throwable ex) {
     @NonNull
     public static Throwable unwrap(@NonNull IOException ex) {
         Objects.requireNonNull(ex);
-        return ex instanceof WrappedIOException ? ((WrappedIOException) ex).getCause() : ex;
+        return ex instanceof WrappedIOException ? ex.getCause() : ex;
     }
 
     private WrappedIOException(Throwable ex) {

java-io-base/src/main/java/nbbrd/io/function/IOConsumer.java

@@ -28,7 +28,7 @@
 /**
  * Represents an operation that accepts a single input argument and returns no
  * result. Unlike most other functional interfaces, {@code Consumer} is expected
- * to operate via side-effects.
+ * to operate via side effects.
  *
  * @param <T> the type of the input to the operation
  */

java-io-base/src/main/java/nbbrd/io/function/IOPredicate.java

@@ -139,7 +139,7 @@ public interface IOPredicate<T> {
     @JdkWithIO
     @StaticFactoryMethod
     static <T> @NonNull IOPredicate<T> isEqual(Object targetRef) {
-        return (null == targetRef) ? Objects::isNull : (object) -> targetRef.equals(object);
+        return (null == targetRef) ? Objects::isNull : targetRef::equals;
     }
 
     @StaticFactoryMethod

java-io-base/src/main/java/nbbrd/io/sys/ProcessReader.java

@@ -32,7 +32,7 @@ public class ProcessReader {
         return newReader(new ProcessBuilder(args).start());
     }
 
-    public static @NonNull BufferedReader newReader(@NonNull Process process) throws IOException {
+    public static @NonNull BufferedReader newReader(@NonNull Process process) {
         return new BufferedReader(new InputStreamReader(new ProcessInputStream(process), Charset.defaultCharset()));
     }
 
@@ -69,6 +69,7 @@ public void close() throws IOException {
         }
 
         // we need the process to end, else we'll get an illegal Thread State Exception
+        @SuppressWarnings("StatementWithEmptyBody")
         private void readUntilEnd() throws IOException {
             while (delegate.read() != -1) {
             }

java-io-base/src/main/java/nbbrd/io/sys/SystemProperties.java

@@ -68,7 +68,7 @@ public final class SystemProperties {
 
     /**
      * Path used to find directories and JAR archives containing class files.
-     * Elements of the class path are separated by a platform-specific character specified in the path.separator property.
+     * Elements of the class path are separated by a platform-specific character specified in the path separator property.
      *
      * @return
      */

java-io-base/src/main/java/nbbrd/io/text/Parser.java

@@ -87,6 +87,7 @@ public interface Parser<T> {
         return o -> after.apply(parse(o));
     }
 
+    @SuppressWarnings("unchecked")
     @StaticFactoryMethod
     static <T> @NonNull Parser<T> onDateTimeFormatter(@NonNull DateTimeFormatter formatter, TemporalQuery<T>... queries) {
         Objects.requireNonNull(formatter);
@@ -170,7 +171,7 @@ public interface Parser<T> {
     }
 
     @StaticFactoryMethod
-    static <T extends Enum<T>> @NonNull Parser<T> onEnum(Class<T> type, ToIntFunction<T> function) {
+    static <T extends Enum<T>> @NonNull Parser<T> onEnum(@NonNull Class<T> type, @NonNull ToIntFunction<T> function) {
         final T[] values = type.getEnumConstants();
         Objects.requireNonNull(function);
         return onInteger().andThen(code -> InternalParser.parse(values, function, code));
@@ -192,7 +193,7 @@ public interface Parser<T> {
     }
 
     @StaticFactoryMethod
-    static @NonNull Parser<List<String>> onStringList(@NonNull Function<CharSequence, Stream<String>> splitter) {
+    static @NonNull Parser<List<String>> onStringList(@NonNull Function<CharSequence, @NonNull Stream<String>> splitter) {
         Objects.requireNonNull(splitter);
         return o -> InternalParser.parseStringList(splitter, o);
     }

java-io-base/src/main/java/nbbrd/io/text/TextParser.java

@@ -48,15 +48,15 @@ default T parseResource(@NonNull Class<?> type, @NonNull String name, @NonNull C
     }
 
     @NonNull
-    default T parseReader(IOSupplier<? extends Reader> source) throws IOException {
+    default T parseReader(@NonNull IOSupplier<? extends Reader> source) throws IOException {
         Objects.requireNonNull(source, "source");
         try (Reader resource = LegacyFiles.checkResource(source.getWithIO(), "Missing Reader")) {
             return parseReader(resource);
         }
     }
 
     @NonNull
-    default T parseStream(IOSupplier<? extends InputStream> source, @NonNull Charset encoding) throws IOException {
+    default T parseStream(@NonNull IOSupplier<? extends InputStream> source, @NonNull Charset encoding) throws IOException {
         Objects.requireNonNull(source, "source");
         Objects.requireNonNull(encoding, "encoding");
         try (InputStream resource = LegacyFiles.checkResource(source.getWithIO(), "Missing InputStream")) {

java-io-base/src/test/java/_test/IOIteratorAssertions.java

@@ -42,6 +42,7 @@ private static <E> void assertIteratorBehavior(Supplier<IOIterator<E>> iterable)
                 .isThrownBy(iterator::nextWithIO);
     }
 
+    @SafeVarargs
     public static <E> void assertContent(Supplier<IOIterator<E>> iterable, E... content) throws IOException {
         assertThat(iterable.get().asUnchecked())
                 .toIterable()