Fixed #68

HMCL/src/main/java/org/jackhuang/hellominecraft/launcher/core/install/optifine/OptiFineInstaller.java

@@ -72,7 +72,8 @@ public class OptiFineInstaller extends Task implements PreviousResultRegistrar<F
                     mv.mainClass = "net.minecraft.launchwrapper.Launch";
                     mv.libraries.add(1, new MinecraftLibrary("net.minecraft:launchwrapper:1.7"));
                 }
-                mv.minecraftArguments += " --tweakClass optifine.OptiFineTweaker";
+                if (!mv.minecraftArguments.contains("FMLTweaker"))
+                    mv.minecraftArguments += " --tweakClass optifine.OptiFineTweaker";
             }
         }
         File loc = new File(service.baseDirectory(), "versions/" + mv.id);

HMCL/src/main/java/org/jackhuang/hellominecraft/launcher/core/launch/GameLauncher.java

@@ -120,7 +120,7 @@ public class GameLauncher {
      *
      * @throws IOException failed creating process
      */
-    public void launch(List str) throws IOException {
+    public void launch(List<String> str) throws IOException {
         if (!service.version().onLaunch(options.getLaunchVersion()))
             return;
         if (StrUtils.isNotBlank(options.getPrecalledCommand())) {
@@ -133,6 +133,7 @@ public class GameLauncher {
             }
         }
         HMCLog.log("Starting process");
+        HMCLog.log(str.toString());
         ProcessBuilder builder = new ProcessBuilder(str);
         if (options.getLaunchVersion() == null || service.baseDirectory() == null)
             throw new Error("Fucking bug!");

HMCL/src/main/java/org/jackhuang/hellominecraft/launcher/core/version/MinecraftVersion.java

@@ -121,7 +121,7 @@ public class MinecraftVersion implements Cloneable, Comparable<MinecraftVersion>
             this.assets != null ? this.assets : parent.assets,
             this.jar != null ? this.jar : parent.jar,
             null, this.runDir, parent.minimumLauncherVersion,
-            this.libraries != null ? ArrayUtils.merge(this.libraries, parent.libraries) : parent.libraries, this.hidden,
+            this.libraries != null ? ArrayUtils.merge(parent.libraries, this.libraries) : parent.libraries, this.hidden,
             this.downloads != null ? this.downloads : parent.downloads,
             this.assetIndex != null ? this.assetIndex : parent.assetIndex);
 

HMCLAPI/src/main/java/org/jackhuang/hellominecraft/util/StrUtils.java

@@ -184,7 +184,7 @@ public final class StrUtils {
         return sb.toString();
     }
 
-    public static boolean isEquals(String base, String to) {
+    public static boolean equals(String base, String to) {
         if (base == null)
             return (to == null);
         else
@@ -240,4 +240,14 @@ public final class StrUtils {
         t.printStackTrace(writer);
         return trace.toString();
     }
+
+    public static List<Integer> findAllPos(String t, String p) {
+        ArrayList<Integer> ret = new ArrayList<>();
+        int i = 0, index;
+        while ((index = t.indexOf(p, i)) != -1) {
+            ret.add(index);
+            i = index + p.length();
+        }
+        return ret;
+    }
 }

HMCLAPI/src/main/java/org/jackhuang/hellominecraft/util/system/ZipEngine.java

@@ -25,6 +25,7 @@ import java.io.FileInputStream;
 import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
+import java.util.HashSet;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipOutputStream;
 import org.jackhuang.hellominecraft.util.func.BiFunction;
@@ -100,7 +101,7 @@ public class ZipEngine {
                     pathName = pathNameCallback.apply(pathName, true);
                 if (pathName == null)
                     continue;
-                zos.putNextEntry(new ZipEntry(pathName));
+                put(new ZipEntry(pathName));
                 putDirectoryImpl(file, basePath, pathNameCallback);
             } else {
                 if (".DS_Store".equals(file.getName())) // For Mac computers.
@@ -122,7 +123,7 @@ public class ZipEngine {
     public void putStream(InputStream is, String pathName) throws IOException {
         int length;
         try (BufferedInputStream bis = new BufferedInputStream(is)) {
-            zos.putNextEntry(new ZipEntry(pathName));
+            put(new ZipEntry(pathName));
             while ((length = bis.read(buf)) > 0)
                 zos.write(buf, 0, length);
         }
@@ -136,4 +137,11 @@ public class ZipEngine {
         putStream(new ByteArrayInputStream(text.getBytes(encoding)), pathName);
     }
 
+    protected HashSet<String> names = new HashSet<>();
+
+    public void put(ZipEntry entry) throws IOException {
+        if (names.add(entry.getName()))
+            zos.putNextEntry(entry);
+    }
+
 }