Make code fully Java 8 compatible

Author: jjohannes

src/main/java/de/jjohannes/gradle/javamodules/ExtraModuleInfoPluginExtension.java

@@ -11,6 +11,7 @@
  * Here the class is used as extension that can be configured in the build script
  * and as input to the ExtraModuleInfoTransform that add the information to Jars.
  */
+@SuppressWarnings("unused")
 public abstract class ExtraModuleInfoPluginExtension {
 
     abstract public MapProperty<String, ModuleSpec> getModuleSpecs();

src/main/java/de/jjohannes/gradle/javamodules/ExtraModuleInfoTransform.java

@@ -21,8 +21,10 @@
 import javax.annotation.Nullable;
 import java.io.BufferedReader;
 import java.io.ByteArrayInputStream;
+import java.io.ByteArrayOutputStream;
 import java.io.File;
 import java.io.IOException;
+import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.OutputStream;
 import java.nio.charset.StandardCharsets;
@@ -234,7 +236,7 @@ private static Map<String, String[]> copyAndExtractProviders(JarInputStream inpu
         JarEntry jarEntry = inputStream.getNextJarEntry();
         Map<String, String[]> providers = new LinkedHashMap<>();
         while (jarEntry != null) {
-            byte[] content = inputStream.readAllBytes();
+            byte[] content = readAllBytes(inputStream);
             String entryName = jarEntry.getName();
             if (entryName.startsWith(SERVICES_PREFIX) && !entryName.equals(SERVICES_PREFIX)) {
                 providers.put(entryName.substring(SERVICES_PREFIX.length()), extractImplementations(content));
@@ -324,7 +326,7 @@ private static void copyEntries(JarInputStream inputStream, JarOutputStream outp
         while (jarEntry != null) {
             try {
                 outputStream.putNextEntry(jarEntry);
-                outputStream.write(inputStream.readAllBytes());
+                outputStream.write(readAllBytes(inputStream));
                 outputStream.closeEntry();
             } catch (ZipException e) {
                 if (!e.getMessage().startsWith("duplicate entry:")) {
@@ -334,4 +336,16 @@ private static void copyEntries(JarInputStream inputStream, JarOutputStream outp
             jarEntry = inputStream.getNextJarEntry();
         }
     }
+
+    public static byte[] readAllBytes(InputStream inputStream) throws IOException {
+        final int bufLen = 4 * 0x400;
+        byte[] buf = new byte[bufLen];
+        int readLen;
+        try (ByteArrayOutputStream outputStream = new ByteArrayOutputStream()) {
+            while ((readLen = inputStream.read(buf, 0, bufLen)) != -1) {
+                outputStream.write(buf, 0, readLen);
+            }
+            return outputStream.toByteArray();
+        }
+    }
 }

src/main/java/de/jjohannes/gradle/javamodules/ModuleInfo.java

@@ -6,6 +6,7 @@
 /**
  * Data class to hold the information that should be added as module-info.class to an existing Jar file.
  */
+@SuppressWarnings("unused")
 public class ModuleInfo extends ModuleSpec {
 
     private final String moduleVersion;

src/main/java/de/jjohannes/gradle/javamodules/ModuleSpec.java

@@ -7,6 +7,7 @@
 /**
  * Details that real Modules and Automatic-Module-Names share.
  */
+@SuppressWarnings("unused")
 abstract public class ModuleSpec implements Serializable {
 
     private final String identifier;