add back the public access thing

tildejustin · Dec 17, 2023 · 25e87e7 · 25e87e7
1 parent 509b4e1
commit 25e87e7
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/src/main/java/me/modmuss50/optifabric/mod/OptifineInjector.java b/src/main/java/me/modmuss50/optifabric/mod/OptifineInjector.java
@@ -2,6 +2,7 @@
 
 import com.chocohead.mm.api.ClassTinkerers;
 import me.modmuss50.optifabric.patcher.*;
+import net.fabricmc.loader.api.FabricLoader;
 import org.objectweb.asm.Opcodes;
 import org.objectweb.asm.tree.*;
 
@@ -12,7 +13,6 @@ public class OptifineInjector {
     private static final List<String> patched = new ArrayList<>();
     ClassCache classCache;
     // i have no idea why and how this works, if you know better please let me know
-    @SuppressWarnings("CommentedOutCode")
     public final Consumer<ClassNode> transformer = target -> {
         if (patched.contains(target.name)) {
             System.out.println("Already patched" + target.name);
@@ -40,12 +40,11 @@ public class OptifineInjector {
         }
 
         // let's make every class we touch public
-        // is this necessary? I don't think so
-        // if (!FabricLoader.getInstance().isDevelopmentEnvironment()) {
-        //     target.access = modAccess(target.access);
-        //     target.methods.forEach(methodNode -> methodNode.access = modAccess(methodNode.access));
-        //     target.fields.forEach(fieldNode -> fieldNode.access = modAccess(fieldNode.access));
-        // }
+        if (!FabricLoader.getInstance().isDevelopmentEnvironment()) {
+            target.access = modAccess(target.access);
+            target.methods.forEach(methodNode -> methodNode.access = modAccess(methodNode.access));
+            target.fields.forEach(fieldNode -> fieldNode.access = modAccess(fieldNode.access));
+        }
     };
 
     public OptifineInjector(ClassCache classCache) {