Merge branch 'master' into glparser-util

.gitignore

@@ -36,3 +36,5 @@ addon.local.gradle
 addon.local.gradle.kts
 addon.late.local.gradle
 addon.late.local.gradle.kts
+/*.csv
+!gl_functions.csv

README.MD

@@ -8,19 +8,19 @@
 * It's the angel that pulls you out of OptiFine's hell -Roadhog360
 
 ## **Beta Quality - Here there be Dragons** 
-* Shaders - Beta
+* Shaders sneak preview - Please don't report issues yet, but please do submit fixes
    * Complementary & BSL mostly function, but have some bugs
    * Sildurs, Seus, and any other shader probably doesn't work
 
 
 # Required Dependencies
-* [UniMixins](https://github.com/LegacyModdingMC/UniMixins/releases) >= 0.1.19
-* [GTNHLib](https://github.com/GTNewHorizons/GTNHLib/releases) >= 0.6.20
+* [UniMixins](https://github.com/LegacyModdingMC/UniMixins/releases) >= 0.1.14
+* [GTNHLib](https://github.com/GTNewHorizons/GTNHLib/releases) >= 0.4.2
 * NOTE: Some mods are not required, but a specific version is required if present - see: `Permanent Incompatibilities` below.
 
 # Known (temporary) Incompatibilities
 * Forge Relocation - Currently non Thread Safe; safe if not in use
-* ProjectRed Frames - Currently non Thread Safe; safe if not in use
+* ProjectRed Frames - Currently non Thread Safe; safe if not in use'
 
 # Potential Incompatibilities
 * EndlessIDs - Compat commits have been made, might work, might not - not fully tested
@@ -29,14 +29,13 @@
 * FalseTweaks - Not currently planned (Looking for volunteers) [Might work if you effectively disable most of the mod]
 * SmoothFont - Not currently planned (Looking for volunteers)
 * RPLE - Not currently planned (Looking for volunteers)
-* LiteLoader - Install [LiteloaderLoader](https://github.com/Midnight145/LiteloaderLoader)
 
 # Permanent Incompatibilities
 * OptiFine [Disabled, won't fix]
 * FastCraft [Disabled, won't fix]
 * BetterFPS [Disabled, won't fix]
 * OptimizationsAndTweaks [Disabled, won't fix]
-* [Amazing Trophies 1.2.1 and below](https://github.com/GTNewHorizons/Amazing-Trophies/releases) - Use 1.2.2+
+* [Amazing Trophies 1.2.1 and below](https://github.com/GTNewHorizons/Amazing-Trophies/releases) - use 1.2.2+
 * [LWJGL3ify 1.5.4 and below](https://github.com/GTNewHorizons/lwjgl3ify/releases/) - Use the latest version available, 1.5.7+
 * [ArchaicFix 0.6.2 and below](https://www.curseforge.com/minecraft/mc-mods/archaicfix/files/all?page=1&pageSize=20) - Use 0.7.0 or above
 * [Hodgepodge 2.4.3 and below](https://github.com/GTNewHorizons/Hodgepodge/releases) - Use 0.4.5 or above
@@ -65,7 +64,7 @@
   * Makamys for [Neodymium](https://github.com/makamys/Neodymium) - Various Tessellator -> Quad functionality taken under the LGPL 3
 * MCPatcherForge
   * Original [MCPatcher](https://bitbucket.org/prupe/mcpatcher/src/master/): Paul Rupe
-  * [Forge version](https://github.com/mist475/MCPatcherForge): Mist475 under LGPL 3.0 with default ctm textures for glass and bookshelves by DarkBum under CC-BY-SA
+  * [Forge version](https://bitbucket.org/prupe/mcpatcher/src/master/): Mist475 under LGPL 3.0 with default ctm textures for glass and bookshelves by DarkBum under CC-BY-SA
 * HUDCaching
   * Created by [Moulberry](https://github.com/Moulberry/MCHUDCaching), backported from [Patcher](https://github.com/Sk1erLLC/Patcher) mod by Alexdoru under the CC-BY-SA-NC 4.0 license
 * Angelica

dependencies.gradle

@@ -49,19 +49,19 @@ configurations {
 }
 
 dependencies {
-    compileOnly("com.github.GTNewHorizons:Hodgepodge:2.6.30")
+    compileOnly("com.github.GTNewHorizons:Hodgepodge:2.5.78")
 
-    compileOnly("com.gtnewhorizons.retrofuturabootstrap:RetroFuturaBootstrap:1.0.7") { transitive = false }
+    compileOnly("com.gtnewhorizons.retrofuturabootstrap:RetroFuturaBootstrap:0.4.0") { transitive = false }
 
-    compileOnly("org.projectlombok:lombok:1.18.34") { transitive = false }
-    annotationProcessor("org.projectlombok:lombok:1.18.34")
+    compileOnly("org.projectlombok:lombok:1.18.22") { transitive = false }
+    annotationProcessor("org.projectlombok:lombok:1.18.22")
 
     // Use modern models+textures to inject QuadProviders
-    devOnlyNonPublishable('com.github.GTNewHorizons:TX-Loader:1.8.2:dev')
+    devOnlyNonPublishable('com.github.GTNewHorizons:TX-Loader:1.7.2:dev')
 
     // Iris Shaders
-    compileOnly('org.jetbrains:annotations:26.0.2')
-    api("com.github.GTNewHorizons:GTNHLib:0.6.20:dev")
+    compileOnly('org.jetbrains:annotations:24.0.1')
+    api("com.github.GTNewHorizons:GTNHLib:0.5.20:dev")
     shadowImplementation("org.anarres:jcpp:1.4.14") // Apache 2.0
     shadowImplementation("org.taumc:glsl-transformation-lib:0.2.0-4.g6b42bca") {
         exclude module: "antlr4" // we only want to shadow the runtime module
@@ -70,20 +70,21 @@ dependencies {
     compileOnly "org.apache.ant:ant:1.8.2"
 
     // Because who doesn't want NEI
-    devOnlyNonPublishable('com.github.GTNewHorizons:NotEnoughItems:2.7.29-GTNH:dev')
-    devOnlyNonPublishable('com.github.GTNewHorizons:CodeChickenCore:1.4.1:dev')
+    devOnlyNonPublishable('com.github.GTNewHorizons:NotEnoughItems:2.6.45-GTNH:dev')
+    devOnlyNonPublishable('com.github.GTNewHorizons:CodeChickenCore:1.3.10:dev')
 
     // Notfine Deps
     compileOnly("thaumcraft:Thaumcraft:1.7.10-4.2.3.5:dev")
-    devOnlyNonPublishable("com.github.GTNewHorizons:Baubles-Expanded:2.1.7-GTNH:dev")
-    compileOnly("com.github.GTNewHorizons:twilightforest:2.7.5:dev") { transitive = false }
+    devOnlyNonPublishable("com.github.GTNewHorizons:Baubles:1.0.4:dev")
+    compileOnly("com.github.GTNewHorizons:twilightforest:2.6.35:dev") { transitive = false }
     compileOnly(rfg.deobf('curse.maven:witchery-69673:2234410'))
-    compileOnly("com.github.GTNewHorizons:TinkersConstruct:1.13.10-GTNH:dev") { transitive = false }
-    compileOnly("com.github.GTNewHorizons:Natura:2.8.2:dev")
+    compileOnly("com.github.GTNewHorizons:TinkersConstruct:1.12.14-GTNH:dev") { transitive = false }
+    compileOnly("com.github.GTNewHorizons:Natura:2.7.5:dev")
 
-    compileOnly("com.github.GTNewHorizons:ThaumicHorizons:1.7.1:dev")
+    compileOnly("com.github.GTNewHorizons:ThaumicHorizons:1.6.4:dev")
 
-    compileOnly("com.github.GTNewHorizons:Battlegear2:1.4.2:dev") { transitive = false }
+    compileOnly("com.github.GTNewHorizons:Battlegear2:1.4.1:dev") { transitive = false }
+    compileOnly(rfg.deobf('maven.modrinth:backhand:1.4.1')) { transitive = false }
 
     compileOnly("com.falsepattern:chunkapi-mc1.7.10:0.5.1:api") { transitive = false }
     compileOnly("com.falsepattern:endlessids-mc1.7.10:1.5-beta0003:dev") { transitive = false }
@@ -98,7 +99,8 @@ dependencies {
 
     compileOnly(rfg.deobf("curse.maven:campfirebackport-387444:4611675"))
     compileOnly(rfg.deobf("curse.maven:xaeros-minimap-263420:5060684"))
-    compileOnly(rfg.deobf("curse.maven:security-craft-64760:2818228"))
+
+    compileOnly("com.github.GTNewHorizons:HoloInventory:2.4.13-GTNH:dev")
 
     runtimeOnlyNonPublishable(rfg.deobf("CoreTweaks:CoreTweaks:0.3.3.2"))
 
@@ -109,10 +111,12 @@ dependencies {
     compileOnly("com.github.GTNewHorizons:BetterCrashes:1.4.0-GTNH:dev")
 
     // Tests
-    testImplementation(platform('org.junit:junit-bom:5.11.4'))
+    testImplementation(platform('org.junit:junit-bom:5.9.2'))
     testImplementation('org.junit.jupiter:junit-jupiter')
     testImplementation("org.mockito:mockito-core:3.+")
     testImplementation("org.lwjgl.lwjgl:lwjgl:2.9.4-nightly-20150209")
 
-    compileOnly("mega:fluidlogged-mc1.7.10:0.1.2")
+    devOnlyNonPublishable("com.github.javaparser:javaparser-symbol-solver-core:3.26.2") {
+        exclude module: "guava"
+    }
 }

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.13-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.9-bin.zip
 networkTimeout=10000
 validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME

gradlew

@@ -86,7 +86,8 @@ done
 # shellcheck disable=SC2034
 APP_BASE_NAME=${0##*/}
 # Discard cd standard output in case $CDPATH is set (https://github.com/gradle/gradle/issues/25036)
-APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s\n' "$PWD" ) || exit
+APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s
+' "$PWD" ) || exit
 
 # Use the maximum available, or set MAX_FD != -1 to use that value.
 MAX_FD=maximum
@@ -205,7 +206,7 @@ fi
 DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"'
 
 # Collect all arguments for the java command:
-#   * DEFAULT_JVM_OPTS, JAVA_OPTS, and optsEnvironmentVar are not allowed to contain shell fragments,
+#   * DEFAULT_JVM_OPTS, JAVA_OPTS, JAVA_OPTS, and optsEnvironmentVar are not allowed to contain shell fragments,
 #     and any embedded shellness will be escaped.
 #   * For example: A user cannot expect ${Hostname} to be expanded, as it is an environment variable and will be
 #     treated as '${Hostname}' itself on the command line.

repositories.gradle

@@ -8,13 +8,6 @@ repositories {
             includeGroup 'com.falsepattern'
         }
     }
-    maven {
-        name = 'mega'
-        url = 'https://mvn.falsepattern.com/gtmega_releases/'
-        content {
-            includeGroup 'mega'
-        }
-    }
     exclusiveContent {
         forRepository {
             ivy {

settings.gradle

@@ -16,5 +16,5 @@ pluginManagement {
 }
 
 plugins {
-    id 'com.gtnewhorizons.gtnhsettingsconvention' version '1.0.37'
+    id 'com.gtnewhorizons.gtnhsettingsconvention' version '1.0.29'
 }

src/main/java/GLFunctionParser.java

@@ -0,0 +1,120 @@
+import com.github.javaparser.StaticJavaParser;
+import com.github.javaparser.ast.CompilationUnit;
+import com.github.javaparser.ast.Node;
+import com.github.javaparser.ast.body.ClassOrInterfaceDeclaration;
+import com.github.javaparser.ast.body.MethodDeclaration;
+import com.github.javaparser.ast.body.Parameter;
+import com.github.javaparser.ast.visitor.VoidVisitorWithDefaults;
+
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.PrintWriter;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
+
+/**
+ * Used to generate gl_functions.csv. Which contains a mapping of OpenGL functions provided by LWJGL
+ * corresponding to the earliest GL version they appear in.
+ */
+public class GLFunctionParser {
+    final static String glPath = "util/gl"; // Path to LWJGL OpenGL bindings source code
+
+    public static void main(String[] args) throws FileNotFoundException {
+        HashMap<String, Set<String>> byFunction = new HashMap<>();
+        File dir = new File(glPath);
+        for (File file : dir.listFiles()) {
+            final String filename = file.getName();
+            if (filename.endsWith(".java") && filename.startsWith("GL") && Character.isDigit(filename.charAt(2))) {
+                Path p = Paths.get(filename);
+
+                String parsedFile = removeFileExtension(p.getFileName().toString());
+
+                CompilationUnit cu = StaticJavaParser.parse(file);
+                cu.accept(new VoidVisitorWithDefaults<>() {
+
+                    @Override
+                    public void defaultAction(Node n, Object arg) {
+                        for( Node child : n.getChildNodes()) {
+                            if(child instanceof ClassOrInterfaceDeclaration cd) {
+                                for (MethodDeclaration md : cd.getMethods()) {
+                                    String methodName = md.getNameAsString() + "(";
+                                    for (Parameter p : md.getParameters()) {
+                                        methodName = methodName + p.getTypeAsString();
+                                    }
+                                    methodName = methodName + ")";
+                                    if(methodName.startsWith("n")) continue;
+
+                                    Set<String> functionSet = byFunction.computeIfAbsent(methodName, k -> new HashSet<>());
+                                    functionSet.add(parsedFile);
+                                }
+                            }
+                        }
+                    }
+                }, null);
+
+            }
+        }
+
+        Map<String, String> functionVersionMap = new HashMap<>();
+
+        for(String function : byFunction.keySet()) {
+
+            Set<String> versionsForFunction = byFunction.get(function);
+            int lowestInt = 46;
+            String lowestString = "GL46";
+            for(String version : versionsForFunction) {
+                int i = Integer.parseInt(version.substring(2, 4));
+                if (i < lowestInt) {
+                    lowestInt = i;
+                    lowestString = version;
+                }
+            }
+            functionVersionMap.put(function, lowestString);
+        }
+
+        List<String[]> outputLines = new ArrayList<>();
+
+        for (Map.Entry<String, String> entry : functionVersionMap.entrySet()) {
+            outputLines.add(new String[] {entry.getKey(), entry.getValue()});
+        }
+
+        File output = new File("output.csv");
+        try (PrintWriter writer = new PrintWriter(output)) {
+            outputLines.stream().map(GLSMFunctionComparison::convertToCSV).forEach(writer::println);
+        }
+    }
+
+    public static String convertToCSV(String[] data) {
+        return Stream.of(data).map(GLSMFunctionComparison::escapeSpecialCharacters).collect(Collectors.joining(","));
+    }
+
+    public static String escapeSpecialCharacters(String data) {
+        if (data == null) {
+            throw new IllegalArgumentException("Input data cannot be null");
+        }
+        String escapedData = data.replaceAll("\\R", " ");
+        if (data.contains(",") || data.contains("\"") || data.contains("'")) {
+            data = data.replace("\"", "\"\"");
+            escapedData = "\"" + data + "\"";
+        }
+        return escapedData;
+    }
+
+    public static String removeFileExtension(String filename) {
+        if (filename == null || filename.isEmpty()) {
+            return filename;
+        }
+
+        String extPattern = "(?<!^)[.]" + "[^.]*$";
+        return filename.replaceAll(extPattern, "");
+    }
+
+}

src/main/java/GLSMFunctionComparison.java

@@ -0,0 +1,135 @@
+import com.github.javaparser.ParserConfiguration;
+import com.github.javaparser.StaticJavaParser;
+import com.github.javaparser.ast.CompilationUnit;
+import com.github.javaparser.ast.Node;
+import com.github.javaparser.ast.body.ClassOrInterfaceDeclaration;
+import com.github.javaparser.ast.body.MethodDeclaration;
+import com.github.javaparser.ast.body.Parameter;
+import com.github.javaparser.ast.visitor.VoidVisitorWithDefaults;
+
+import java.io.BufferedReader;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.FileReader;
+import java.io.IOException;
+import java.io.PrintWriter;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
+
+/**
+ * Used to compare OpenGL functions from gl_functions.csv to current GLStateManager
+ * source code to determine what functions have an implementation in GLSM, and what ones don't.
+ * Only considers functions which appear in GL 2.1 and lower
+ */
+public class GLSMFunctionComparison {
+
+    /**
+     * If set to true, then the outputted CSV will show functions which do have an implementation in GLSM
+     */
+    final static boolean INCLUDE_TRUE = false;
+
+    final static String glFunctionsCsvPath = "gl_functions.csv";
+    final static String glsmPath = "src/main/java/com/gtnewhorizons/angelica/glsm/GLStateManager.java";
+
+    public static Map<String, String> parseGLFunctions() {
+        Map<String, String> map = new HashMap<>();
+
+        try (BufferedReader br = new BufferedReader(new FileReader(glFunctionsCsvPath))) {
+            String line;
+            while ((line = br.readLine()) != null) {
+                String[] values = line.split(",");
+                if (values.length == 2) {
+                    String key = values[0].trim();
+                    String value = values[1].trim();
+                    int versionInt = Integer.parseInt(value.substring(2, 4));
+                    if (versionInt < 30) map.put(key, value);
+                }
+            }
+        } catch (IOException e) {
+            e.printStackTrace();
+        }
+        return map;
+    }
+
+    public static void main(String[] args) throws FileNotFoundException {
+        //GLSM will fail to parse without increasing language level
+        StaticJavaParser.getParserConfiguration().setLanguageLevel(ParserConfiguration.LanguageLevel.JAVA_17);
+
+        Map<String, String> functionMap = parseGLFunctions();
+        Set<String> glsmFunctions = new HashSet<>();
+
+        File glsmFile = new File(glsmPath);
+        CompilationUnit cu = StaticJavaParser.parse(glsmFile);
+        cu.accept(new VoidVisitorWithDefaults<>() {
+
+            @Override
+            public void defaultAction(Node n, Object arg) {
+                for (Node child : n.getChildNodes()) {
+                    if (child instanceof ClassOrInterfaceDeclaration cd) {
+                        for (MethodDeclaration md : cd.getMethods()) {
+                            String methodName = md.getNameAsString() + "(";
+                            for (Parameter p : md.getParameters()) {
+                                methodName = methodName + p.getTypeAsString();
+                            }
+                            methodName = methodName + ")";
+                            glsmFunctions.add(methodName);
+                        }
+                    }
+                }
+            }
+        }, null);
+
+        Map<String, String> functionGLSMMap = new HashMap<>();
+
+        for(String function : functionMap.keySet()) {
+            if (!glsmFunctions.contains(function)) {
+                functionGLSMMap.put(function, "false");
+            } else if (INCLUDE_TRUE) {
+                functionGLSMMap.put(function, "true");
+            }
+        }
+
+        List<String[]> outputLines = new ArrayList<>();
+
+        for (Map.Entry<String, String> entry : functionGLSMMap.entrySet()) {
+            outputLines.add(new String[] {entry.getKey(), functionMap.get(entry.getKey()), entry.getValue()});
+        }
+
+        File output = new File("glsm_function_comparison.csv");
+        try (PrintWriter writer = new PrintWriter(output)) {
+            outputLines.stream().map(GLSMFunctionComparison::convertToCSV).forEach(writer::println);
+        }
+    }
+
+    public static String convertToCSV(String[] data) {
+        return Stream.of(data).map(GLSMFunctionComparison::escapeSpecialCharacters).collect(Collectors.joining(","));
+    }
+
+    public static String escapeSpecialCharacters(String data) {
+        if (data == null) {
+            throw new IllegalArgumentException("Input data cannot be null");
+        }
+        String escapedData = data.replaceAll("\\R", " ");
+        if (data.contains(",") || data.contains("\"") || data.contains("'")) {
+            data = data.replace("\"", "\"\"");
+            escapedData = "\"" + data + "\"";
+        }
+        return escapedData;
+    }
+
+    public static String removeFileExtension(String filename) {
+        if (filename == null || filename.isEmpty()) {
+            return filename;
+        }
+
+        String extPattern = "(?<!^)[.]" + "[^.]*$";
+        return filename.replaceAll(extPattern, "");
+    }
+
+}

src/main/java/com/gtnewhorizons/angelica/AngelicaMod.java

@@ -15,13 +15,8 @@ import cpw.mods.fml.common.event.FMLPreInitializationEvent;
         modid = "angelica",
         name = "Angelica",
         version = Tags.VERSION,
-        dependencies = """
-            required-after:gtnhlib@[0.6.6,);\
-            before:lwjgl3ify@[1.5.3,);\
-            after:hodgepodge@[2.4.4,);\
-            after:CodeChickenCore@[1.2.0,);\
-            after:archaicfix@[0.7.0,);\
-            """,
+        dependencies = " before:lwjgl3ify@[1.5.3,);" + " after:hodgepodge@[2.4.4,);" + " after:CodeChickenCore@[1.2.0,);"
+                     + " after:archaicfix@[0.7.0,);" + " required-after:gtnhlib@[0.4.2,);",
         acceptedMinecraftVersions = "[1.7.10]",
         acceptableRemoteVersions = "*",
         guiFactory = "com.gtnewhorizons.angelica.config.AngelicaGuiConfigFactory")

src/main/java/com/gtnewhorizons/angelica/api/IBlockAccessExtended.java

@@ -1,8 +0,0 @@
-package com.gtnewhorizons.angelica.api;
-
-import net.minecraft.world.IBlockAccess;
-import net.minecraft.world.World;
-
-public interface IBlockAccessExtended extends IBlockAccess {
-    World getWorld();
-}

src/main/java/com/gtnewhorizons/angelica/api/TextureServices.java

@@ -1,27 +0,0 @@
-package com.gtnewhorizons.angelica.api;
-
-import com.gtnewhorizons.angelica.mixins.interfaces.IPatchedTextureAtlasSprite;
-import com.gtnewhorizons.angelica.mixins.interfaces.ITexturesCache;
-import net.minecraft.client.renderer.RenderBlocks;
-import net.minecraft.util.IIcon;
-
-/**
- * Exposes a method to the API that allow mods to add texture update compat
- */
-@SuppressWarnings("unused")
-public class TextureServices {
-
-    @SuppressWarnings("unused")
-    public static void updateBlockTextureAnimation(IIcon icon, RenderBlocks renderBlocks) {
-        if (renderBlocks instanceof ITexturesCache texturesCache) {
-            texturesCache.getRenderedTextures().add(icon);
-        }
-    }
-
-    @SuppressWarnings("unused")
-    public static void updateTextureAnimation(IIcon icon) {
-        if(icon instanceof IPatchedTextureAtlasSprite patchedSprite) {
-            patchedSprite.markNeedsAnimationUpdate();
-        }
-    }
-}

src/main/java/com/gtnewhorizons/angelica/client/font/BatchingFontRenderer.java

@@ -1,17 +1,16 @@
 package com.gtnewhorizons.angelica.client.font;
 
-import static com.gtnewhorizon.gtnhlib.bytebuf.MemoryUtilities.*;
-
+import com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil;
 import com.gtnewhorizons.angelica.glsm.GLStateManager;
 import com.gtnewhorizons.angelica.mixins.interfaces.FontRendererAccessor;
 import it.unimi.dsi.fastutil.chars.Char2ShortOpenHashMap;
 import it.unimi.dsi.fastutil.objects.ObjectArrayList;
 import jss.util.RandomXoshiro256StarStar;
-import net.minecraft.client.Minecraft;
 import net.minecraft.client.gui.FontRenderer;
-import net.minecraft.client.gui.ScaledResolution;
+import net.minecraft.client.renderer.texture.TextureManager;
 import net.minecraft.util.MathHelper;
 import net.minecraft.util.ResourceLocation;
+import org.lwjgl.BufferUtils;
 import org.lwjgl.opengl.GL11;
 
 import java.nio.ByteBuffer;
@@ -43,6 +42,8 @@ public class BatchingFontRenderer {
     private int[] colorCode;
     /** Location of the primary font atlas to bind. */
     protected final ResourceLocation locationFontTexture;
+    /** The RenderEngine used to load and setup glyph textures. */
+    private final TextureManager renderEngine;
     private final RandomXoshiro256StarStar fontRandom = new RandomXoshiro256StarStar();
 
     /** The full list of characters present in the default Minecraft font, excluding the Unicode font */
@@ -73,13 +74,14 @@ public class BatchingFontRenderer {
     }
 
     public BatchingFontRenderer(FontRenderer underlying, ResourceLocation[] unicodePageLocations, int[] charWidth,
-        byte[] glyphWidth, int[] colorCode, ResourceLocation locationFontTexture) {
+        byte[] glyphWidth, int[] colorCode, ResourceLocation locationFontTexture, TextureManager renderEngine) {
         this.underlying = underlying;
         this.unicodePageLocations = unicodePageLocations;
         this.charWidth = charWidth;
         this.glyphWidth = glyphWidth;
         this.colorCode = colorCode;
         this.locationFontTexture = locationFontTexture;
+        this.renderEngine = renderEngine;
 
         for (int i = 0; i < 64; i++) {
             batchCommandPool.add(new FontDrawCmd());
@@ -95,10 +97,10 @@ public class BatchingFontRenderer {
     private static final int INITIAL_BATCH_SIZE = 256;
     private static final ResourceLocation DUMMY_RESOURCE_LOCATION = new ResourceLocation("angelica$dummy",
         "this is invalid!");
-    private FloatBuffer batchVtxPositions = memAllocFloat(INITIAL_BATCH_SIZE * 2);
-    private ByteBuffer batchVtxColors = memAlloc(INITIAL_BATCH_SIZE * 4);
-    private FloatBuffer batchVtxTexCoords = memAllocFloat(INITIAL_BATCH_SIZE * 2);
-    private IntBuffer batchIndices = memAllocInt(INITIAL_BATCH_SIZE / 2 * 3);
+    private FloatBuffer batchVtxPositions = BufferUtils.createFloatBuffer(INITIAL_BATCH_SIZE * 2);
+    private ByteBuffer batchVtxColors = BufferUtils.createByteBuffer(INITIAL_BATCH_SIZE * 4);
+    private FloatBuffer batchVtxTexCoords = BufferUtils.createFloatBuffer(INITIAL_BATCH_SIZE * 2);
+    private IntBuffer batchIndices = BufferUtils.createIntBuffer(INITIAL_BATCH_SIZE / 2 * 3);
     private final ObjectArrayList<FontDrawCmd> batchCommands = ObjectArrayList.wrap(new FontDrawCmd[64], 0);
     private final ObjectArrayList<FontDrawCmd> batchCommandPool = ObjectArrayList.wrap(new FontDrawCmd[64], 0);
 
@@ -107,12 +109,12 @@ public class BatchingFontRenderer {
         final int oldCap = batchVtxPositions.capacity() / 2;
         if (vtxWriterIndex >= oldCap) {
             final int newCap = oldCap * 2;
-            batchVtxPositions = memRealloc(batchVtxPositions, newCap * 2);
-            batchVtxColors = memRealloc(batchVtxColors, newCap * 4);
-            batchVtxTexCoords = memRealloc(batchVtxTexCoords, newCap * 2);
+            batchVtxPositions = CompatMemoryUtil.memReallocDirect(batchVtxPositions, newCap * 2);
+            batchVtxColors = CompatMemoryUtil.memReallocDirect(batchVtxColors, newCap * 4);
+            batchVtxTexCoords = CompatMemoryUtil.memReallocDirect(batchVtxTexCoords, newCap * 2);
             final int oldIdxCap = batchIndices.capacity();
             final int newIdxCap = oldIdxCap * 2;
-            batchIndices = memRealloc(batchIndices, newIdxCap);
+            batchIndices = CompatMemoryUtil.memReallocDirect(batchIndices, newIdxCap);
         }
         final int idx = vtxWriterIndex;
         final int idx2 = idx * 2;
@@ -151,7 +153,7 @@ public class BatchingFontRenderer {
         return idx;
     }
 
-    private void pushDrawCmd(int startIdx, int idxCount, ResourceLocation texture, boolean isUnicode) {
+    private void pushDrawCmd(int startIdx, int idxCount, ResourceLocation texture) {
         if (!batchCommands.isEmpty()) {
             final FontDrawCmd lastCmd = batchCommands.get(batchCommands.size() - 1);
             final int prevEndVtx = lastCmd.startVtx + lastCmd.idxCount;
@@ -167,7 +169,7 @@ public class BatchingFontRenderer {
             }
         }
         final FontDrawCmd cmd = batchCommandPool.pop();
-        cmd.reset(startIdx, idxCount, texture, isUnicode);
+        cmd.reset(startIdx, idxCount, texture);
         batchCommands.add(cmd);
     }
 
@@ -175,14 +177,12 @@ public class BatchingFontRenderer {
 
         public int startVtx;
         public int idxCount;
-        public boolean isUnicode;
         public ResourceLocation texture;
 
-        public void reset(int startVtx, int vtxCount, ResourceLocation texture, boolean isUnicode) {
+        public void reset(int startVtx, int vtxCount, ResourceLocation texture) {
             this.startVtx = startVtx;
             this.idxCount = vtxCount;
             this.texture = texture;
-            this.isUnicode = isUnicode;
         }
 
         @Override
@@ -283,21 +283,7 @@ public class BatchingFontRenderer {
             }
             batchIndices.limit(cmd.startVtx + cmd.idxCount);
             batchIndices.position(cmd.startVtx);
-
-            Minecraft mc = Minecraft.getMinecraft();
-            int scaleFactor = new ScaledResolution(mc, mc.displayWidth, mc.displayHeight).getScaleFactor();
-            boolean shouldApplyFilter = cmd.isUnicode && scaleFactor % 2 != 0;
-
-            if (shouldApplyFilter) {
-                GLStateManager.glTexParameteri(GL11.GL_TEXTURE_2D, GL11.GL_TEXTURE_MIN_FILTER, GL11.GL_LINEAR);
-                GLStateManager.glTexParameteri(GL11.GL_TEXTURE_2D, GL11.GL_TEXTURE_MAG_FILTER, GL11.GL_LINEAR);
-            }
             GL11.glDrawElements(GL11.GL_TRIANGLES, batchIndices);
-
-            if (shouldApplyFilter) {
-                GLStateManager.glTexParameteri(GL11.GL_TEXTURE_2D, GL11.GL_TEXTURE_MIN_FILTER, GL11.GL_NEAREST);
-                GLStateManager.glTexParameteri(GL11.GL_TEXTURE_2D, GL11.GL_TEXTURE_MAG_FILTER, GL11.GL_NEAREST);
-            }
         }
 
         GL11.glDisableClientState(GL11.GL_TEXTURE_COORD_ARRAY);
@@ -384,7 +370,7 @@ public class BatchingFontRenderer {
                     if (curUnderline && underlineStartX != underlineEndX) {
                         final int ulIdx = idxWriterIndex;
                         pushUntexRect(underlineStartX, underlineY, underlineEndX - underlineStartX, 1.0f, curColor);
-                        pushDrawCmd(ulIdx, 6, null, false);
+                        pushDrawCmd(ulIdx, 6, null);
                         underlineStartX = underlineEndX;
                     }
                     if (curStrikethrough && strikethroughStartX != strikethroughEndX) {
@@ -395,7 +381,7 @@ public class BatchingFontRenderer {
                             strikethroughEndX - strikethroughStartX,
                             1.0f,
                             curColor);
-                        pushDrawCmd(ulIdx, 6, null, false);
+                        pushDrawCmd(ulIdx, 6, null);
                         strikethroughStartX = strikethroughEndX;
                     }
 
@@ -451,6 +437,8 @@ public class BatchingFontRenderer {
                     chr = MCFONT_CHARS.charAt(lutIndex);
                 }
 
+                final float shadowOffset = unicodeFlag ? 0.5F : 1.0F;
+
                 // Check ASCII space, NBSP, NNBSP
                 if (chr == ' ' || chr == '\u00A0' || chr == '\u202F') {
                     curX += 4;
@@ -464,7 +452,6 @@ public class BatchingFontRenderer {
                 final float uSz;
                 final float vSz;
                 final float itOff = curItalic ? 1.0F : 0.0F; // italic offset
-                final float shadowOffset;
                 final ResourceLocation texture;
 
                 if (lutIndex == -1 || unicodeFlag) {
@@ -472,24 +459,22 @@ public class BatchingFontRenderer {
                         continue;
                     }
                     // Draw unicode char
-                    shadowOffset = 0.5F;
                     final int uniPage = chr / 256;
                     texture = getUnicodePageLocation(uniPage);
                     final int startColumn = this.glyphWidth[chr] >>> 4;
                     final int endColumn = this.glyphWidth[chr] & 15;
                     final float startColumnF = (float) startColumn;
                     final float endColumnF = (float) (endColumn + 1);
-                    uStart = ((float) (chr % 16 * 16) + startColumnF + 0.21f) / 256.0f;
-                    vStart = ((float) ((chr & 255) / 16 * 16) + 0.21f) / 256.0f;
+                    uStart = ((float) (chr % 16 * 16) + startColumnF) / 256.0f;
+                    vStart = ((float) ((chr & 255) / 16 * 16)) / 256.0f;
                     final float chrWidth = endColumnF - startColumnF - 0.02F;
                     glyphW = chrWidth / 2.0f + 1.0f;
                     xAdvance = (endColumnF - startColumnF) / 2.0F + 1.0F;
-                    uSz = (chrWidth - 0.42f) / 256.0f;
-                    vSz = (16.0f - 0.42f) / 256.0f;
+                    uSz = chrWidth / 256.0f;
+                    vSz = 15.98f / 256.0f;
 
                 } else {
                     // Draw "ASCII" char
-                    shadowOffset = 1.0F;
                     uStart = ((lutIndex % 16) * 8) / 128.0F;
                     vStart = (float) ((lutIndex / 16) * 8) / 128.0F;
                     xAdvance = this.charWidth[lutIndex];
@@ -573,7 +558,7 @@ public class BatchingFontRenderer {
                     vtxCount += 4;
                 }
 
-                pushDrawCmd(idxId, vtxCount / 2 * 3, texture, chr > 255);
+                pushDrawCmd(idxId, vtxCount / 2 * 3, texture);
                 curX += xAdvance + (curBold ? shadowOffset : 0.0f);
                 underlineEndX = curX;
                 strikethroughEndX = curX;
@@ -582,7 +567,7 @@ public class BatchingFontRenderer {
             if (curUnderline && underlineStartX != underlineEndX) {
                 final int ulIdx = idxWriterIndex;
                 pushUntexRect(underlineStartX, underlineY, underlineEndX - underlineStartX, 1.0f, curColor);
-                pushDrawCmd(ulIdx, 6, null, false);
+                pushDrawCmd(ulIdx, 6, null);
             }
             if (curStrikethrough && strikethroughStartX != strikethroughEndX) {
                 final int ulIdx = idxWriterIndex;
@@ -592,7 +577,7 @@ public class BatchingFontRenderer {
                     strikethroughEndX - strikethroughStartX,
                     1.0f,
                     curColor);
-                pushDrawCmd(ulIdx, 6, null, false);
+                pushDrawCmd(ulIdx, 6, null);
             }
 
         } finally {

src/main/java/com/gtnewhorizons/angelica/compat/ExtendedBlockStorageExt.java

@@ -1,6 +1,8 @@
 package com.gtnewhorizons.angelica.compat;
 
 import com.falsepattern.chunk.api.DataRegistry;
+import com.gtnewhorizons.angelica.mixins.interfaces.ExtendedBlockStorageAccessor;
+import com.gtnewhorizons.angelica.mixins.interfaces.ExtendedNibbleArray;
 import com.gtnewhorizons.neid.mixins.interfaces.IExtendedBlockStorageMixin;
 
 import net.minecraft.world.chunk.Chunk;
@@ -17,7 +19,7 @@ public class ExtendedBlockStorageExt extends ExtendedBlockStorage {
     }
 
     public ExtendedBlockStorageExt(Chunk chunk, ExtendedBlockStorage storage) {
-        super(storage.yBase, storage.getSkylightArray() != null);
+        super(((ExtendedBlockStorageAccessor) storage).getYBase(), storage.getSkylightArray() != null);
 
         if (ModStatus.isChunkAPILoaded) {
             if (storage.getSkylightArray() != null) {
@@ -31,7 +33,7 @@ public class ExtendedBlockStorageExt extends ExtendedBlockStorage {
                 System.arraycopy(((IExtendedBlockStorageMixin)(Object)storage).getBlock16BArray(), 0, block16BArray, 0, block16BArray.length);
                 if(storage.getBlockMSBArray() != null) {
                     this.setBlockMSBArray(new NibbleArray(block16BArray.length, 4));
-                    copyNibbleArray(storage.getBlockMSBArray(), this.getBlockMSBArray());
+                    copyNibbleArray((ExtendedNibbleArray) storage.getBlockMSBArray(), (ExtendedNibbleArray) this.getBlockMSBArray());
                 }
                 arrayLen = block16BArray.length;
                 if (ModStatus.isNEIDMetadataExtended) {
@@ -50,31 +52,31 @@ public class ExtendedBlockStorageExt extends ExtendedBlockStorage {
                 System.arraycopy(storage.getBlockLSBArray(), 0, blockLSBArray, 0, blockLSBArray.length);
                 if(storage.getBlockMSBArray() != null) {
                     this.setBlockMSBArray(new NibbleArray(blockLSBArray.length, 4));
-                    copyNibbleArray(storage.getBlockMSBArray(), this.getBlockMSBArray());
+                    copyNibbleArray((ExtendedNibbleArray) storage.getBlockMSBArray(), (ExtendedNibbleArray) this.getBlockMSBArray());
                 }
                 arrayLen = blockLSBArray.length;
             }
 
 
-            if (!ModStatus.isNEIDMetadataExtended) copyNibbleArray(storage.getMetadataArray(), this.getMetadataArray());
-            copyNibbleArray(storage.getBlocklightArray(), this.getBlocklightArray());
+            if (!ModStatus.isNEIDMetadataExtended) copyNibbleArray((ExtendedNibbleArray) storage.getMetadataArray(), (ExtendedNibbleArray)this.getMetadataArray());
+            copyNibbleArray((ExtendedNibbleArray) storage.getBlocklightArray(), (ExtendedNibbleArray)this.getBlocklightArray());
             if(storage.getSkylightArray() != null) {
                 hasSky = true;
                 if(this.getSkylightArray() == null) {
                     this.setSkylightArray(new NibbleArray(arrayLen, 4));
                 }
-                copyNibbleArray(storage.getSkylightArray(), this.getSkylightArray());
+                copyNibbleArray((ExtendedNibbleArray) storage.getSkylightArray(), (ExtendedNibbleArray) this.getSkylightArray());
             }
         }
-        this.blockRefCount = storage.blockRefCount;
+        ((ExtendedBlockStorageAccessor) this).setBlockRefCount(((ExtendedBlockStorageAccessor) storage).getBlockRefCount());
     }
 
 
-    private static void copyNibbleArray(NibbleArray srcArray, NibbleArray dstArray) {
+    private static void copyNibbleArray(ExtendedNibbleArray srcArray, ExtendedNibbleArray dstArray) {
         if (srcArray == null || dstArray == null) {
             throw new RuntimeException("NibbleArray is null src: " + (srcArray == null) + " dst: " + (dstArray == null));
         }
-        final byte[] data = srcArray.data;
-        System.arraycopy(data, 0, dstArray.data, 0, data.length);
+        final byte[] data = srcArray.getData();
+        System.arraycopy(data, 0, dstArray.getData(), 0, data.length);
     }
 }

src/main/java/com/gtnewhorizons/angelica/compat/ModStatus.java

@@ -1,17 +1,14 @@
 package com.gtnewhorizons.angelica.compat;
 
-import com.gtnewhorizons.angelica.compat.backhand.BackhandReflectionCompat;
 import com.gtnewhorizons.angelica.helpers.LoadControllerHelper;
 import cpw.mods.fml.common.Loader;
+import cpw.mods.fml.common.versioning.ArtifactVersion;
 import cpw.mods.fml.common.versioning.DefaultArtifactVersion;
 import mods.battlegear2.Battlegear;
-import org.apache.logging.log4j.LogManager;
-import org.apache.logging.log4j.Logger;
+import net.dries007.holoInventory.HoloInventory;
+import xonin.backhand.Backhand;
 
 public class ModStatus {
-    public static final Logger LOGGER = LogManager.getLogger("ModCompat");
-
-    public static BackhandReflectionCompat backhandCompat;
     /**
      * Mixin Version
      */
@@ -20,6 +17,7 @@ public class ModStatus {
      * ASM Version
      */
     public static boolean isOldNEIDLoaded;
+
     public static boolean isBetterCrashesLoaded;
     public static boolean isNEIDMetadataExtended;
     public static boolean isLotrLoaded;
@@ -32,14 +30,8 @@ public class ModStatus {
     public static boolean isThaumcraftLoaded;
     public static boolean isThaumicHorizonsLoaded;
     public static boolean isBaublesLoaded;
-    public static boolean isFluidLoggedLoaded;
 
     public static void preInit(){
-        isBackhandLoaded = Loader.isModLoaded("backhand");
-        if(isBackhandLoaded) {
-            isBackhandLoaded = BackhandReflectionCompat.isBackhandLoaded();
-        }
-
         isBetterCrashesLoaded = Loader.isModLoaded("bettercrashes");
         isNEIDLoaded = Loader.isModLoaded("neid");
         isOldNEIDLoaded = Loader.isModLoaded("notenoughIDs");
@@ -49,12 +41,17 @@ public class ModStatus {
         isXaerosMinimapLoaded = Loader.isModLoaded("XaeroMinimap");
         isHoloInventoryLoaded = Loader.isModLoaded("holoinventory");
         isBattlegearLoaded = Loader.isModLoaded("battlegear2");
+        isBackhandLoaded = Loader.isModLoaded("backhand");
         isThaumcraftLoaded = Loader.isModLoaded("Thaumcraft");
         isThaumicHorizonsLoaded = Loader.isModLoaded("ThaumicHorizons");
         isBaublesLoaded = Loader.isModLoaded("Baubles");
-        isFluidLoggedLoaded = Loader.isModLoaded("fluidlogged");
 
-        isHoloInventoryLoaded = Loader.isModLoaded("holoinventory");
+        if (isHoloInventoryLoaded){
+            isHoloInventoryLoaded = new DefaultArtifactVersion("2.4.4-GTNH")
+                .compareTo(
+                    LoadControllerHelper.getOwningMod(HoloInventory.class).getProcessedVersion()
+                ) <= 0;
+        }
 
         // remove compat with original release of BG2
         if (isBattlegearLoaded){
@@ -64,9 +61,17 @@ public class ModStatus {
                 ) <= 0;
         }
 
+        if (isBackhandLoaded){
+            isBackhandLoaded = new DefaultArtifactVersion("1.4.1")
+                .compareTo(
+                    LoadControllerHelper.getOwningMod(Backhand.class).getProcessedVersion()
+                ) <= 0;
+        }
+
+
         isNEIDMetadataExtended = false;
         if (isNEIDLoaded) {
-            final int majorVersion = Integer.parseInt(Loader.instance().getIndexedModList().get("neid").getVersion().split("\\.")[0]);
+            int majorVersion = Integer.parseInt(Loader.instance().getIndexedModList().get("neid").getVersion().split("\\.")[0]);
             if (majorVersion >= 2) {
                 isNEIDMetadataExtended = true;
             }

src/main/java/com/gtnewhorizons/angelica/compat/backhand/BackhandReflectionCompat.java

@@ -1,76 +0,0 @@
-package com.gtnewhorizons.angelica.compat.backhand;
-
-import com.gtnewhorizons.angelica.helpers.LoadControllerHelper;
-import cpw.mods.fml.common.versioning.DefaultArtifactVersion;
-import cpw.mods.fml.relauncher.ReflectionHelper;
-import net.minecraft.entity.player.EntityPlayer;
-import net.minecraft.item.ItemStack;
-import java.lang.invoke.MethodHandle;
-import java.lang.invoke.MethodHandles;
-import java.lang.invoke.MethodType;
-
-import static com.gtnewhorizons.angelica.compat.ModStatus.LOGGER;
-
-public class BackhandReflectionCompat {
-    private static final String BACKHAND_CLASS = "xonin.backhand.Backhand";
-    private static final String ITEM_RENDERER_HOOKS_CLASS = "xonin.backhand.client.hooks.ItemRendererHooks";
-    private static final String BACKHAND_UTILS_CLASS = "xonin.backhand.api.core.BackhandUtils";
-
-    private static boolean isLoaded;
-    private static final MethodHandle renderOffhandReturn;
-    private static final MethodHandle getOffhandItem;
-
-    static {
-        MethodHandle renderOffhandReturnTemp = null;
-        MethodHandle getOffhandItemTemp = null;
-
-        try {
-            final Class<?> backhandClass = ReflectionHelper.getClass(BackhandReflectionCompat.class.getClassLoader(), BACKHAND_CLASS);
-            final boolean versionCheck = new DefaultArtifactVersion("1.6.9").compareTo(LoadControllerHelper.getOwningMod(backhandClass).getProcessedVersion()) <= 0;
-
-            if (versionCheck) {
-                final MethodHandles.Lookup lookup = MethodHandles.lookup();
-                final Class<?> itemRendererHooksClass = ReflectionHelper.getClass(BackhandReflectionCompat.class.getClassLoader(), ITEM_RENDERER_HOOKS_CLASS);
-                final Class<?> backhandUtilsClass = ReflectionHelper.getClass(BackhandReflectionCompat.class.getClassLoader(), BACKHAND_UTILS_CLASS);
-
-                renderOffhandReturnTemp = lookup.findStatic(itemRendererHooksClass, "renderOffhandReturn", MethodType.methodType(void.class, float.class));
-                getOffhandItemTemp = lookup.findStatic(backhandUtilsClass, "getOffhandItem", MethodType.methodType(ItemStack.class, EntityPlayer.class));
-                isLoaded = true;
-                LOGGER.info("Backhand compat loaded");
-            }
-        } catch (Exception e) {
-            LOGGER.error("Failed to load Backhand compat", e);
-            isLoaded = false;
-        }
-
-        renderOffhandReturn = renderOffhandReturnTemp;
-        getOffhandItem = getOffhandItemTemp;
-    }
-
-    public static boolean isBackhandLoaded() {
-        return isLoaded;
-    }
-
-    public static void renderOffhand(float partialTicks) {
-        if (isLoaded && renderOffhandReturn != null) {
-            try {
-                renderOffhandReturn.invokeExact(partialTicks);
-            } catch (Throwable e) {
-                LOGGER.error("Failed to invoke Backhand renderOffhandReturn", e);
-                isLoaded = false;
-            }
-        }
-    }
-
-    public static ItemStack getOffhandItem(EntityPlayer player) {
-        if (isLoaded && getOffhandItem != null) {
-            try {
-                return (ItemStack) getOffhandItem.invokeExact(player);
-            } catch (Throwable e) {
-                LOGGER.error("Failed to invoke Backhand getOffhandItem", e);
-                isLoaded = false;
-            }
-        }
-        return null;
-    }
-}

src/main/java/com/gtnewhorizons/angelica/compat/battlegear2/Battlegear2Compat.java

@@ -1,15 +0,0 @@
-package com.gtnewhorizons.angelica.compat.battlegear2;
-
-import mods.battlegear2.api.core.IInventoryPlayerBattle;
-import net.minecraft.entity.player.EntityPlayer;
-import net.minecraft.item.ItemStack;
-
-/**
- * This class exists to isolate classes that exist in Battlegear 2 but not
- * in Battlegear 2 for Backhand in order to avoid class loading exceptions.
- */
-public class Battlegear2Compat {
-    public static ItemStack getBattlegear2Offhand(EntityPlayer player) {
-        return ((IInventoryPlayerBattle) player.inventory).battlegear2$getCurrentOffhandWeapon();
-    }
-}

src/main/java/com/gtnewhorizons/angelica/compat/holoinventory/HoloInventoryReflectionCompat.java

@@ -1,93 +0,0 @@
-package com.gtnewhorizons.angelica.compat.holoinventory;
-
-import net.minecraftforge.client.event.RenderGameOverlayEvent;
-
-import java.lang.invoke.MethodHandle;
-import java.lang.invoke.MethodHandles;
-import java.lang.invoke.MethodType;
-import java.lang.reflect.Field;
-
-import static com.gtnewhorizons.angelica.compat.ModStatus.LOGGER;
-
-public class HoloInventoryReflectionCompat {
-    private static final String RENDERER_CLASS = "net.dries007.holoInventory.client.Renderer";
-
-    private static boolean isLoaded;
-    private static final MethodHandle renderEventPostMethod;
-    private static final MethodHandle renderEventMethod;
-    private static final MethodHandle angelicaOverrideSetter;
-
-    static {
-        MethodHandle renderEventPostMethodTemp = null;
-        MethodHandle renderEventMethodTemp = null;
-        MethodHandle angelicaOverrideSetterTemp = null;
-        final Object rendererInstance;
-        final Class<?> rendererClass;
-
-        try {
-            rendererClass = Class.forName(RENDERER_CLASS);
-
-            // Get and cache the Renderer.INSTANCE
-            final Field instanceField = rendererClass.getDeclaredField("INSTANCE");
-            instanceField.setAccessible(true);
-            rendererInstance = instanceField.get(null);
-
-            final MethodHandles.Lookup lookup = MethodHandles.lookup();
-            try {
-                renderEventPostMethodTemp = lookup.findVirtual(rendererClass, "renderEvent", MethodType.methodType(void.class, RenderGameOverlayEvent.Post.class));
-            } catch (NoSuchMethodException e) {
-                // Fallback for 2.5.0-GTNH and earlier
-                renderEventMethodTemp = lookup.findVirtual(rendererClass, "renderEvent", MethodType.methodType(void.class, RenderGameOverlayEvent.class));
-            }
-
-            final Field angelicaOverrideField = rendererClass.getDeclaredField("angelicaOverride");
-            angelicaOverrideField.setAccessible(true);
-            angelicaOverrideSetterTemp = lookup.unreflectSetter(angelicaOverrideField);
-
-            // Bind the method handles to the instance
-            if (rendererInstance != null) {
-                if (renderEventPostMethodTemp != null) {
-                    renderEventPostMethodTemp = renderEventPostMethodTemp.bindTo(rendererInstance);
-                }
-                if (renderEventMethodTemp != null) {
-                    renderEventMethodTemp = renderEventMethodTemp.bindTo(rendererInstance);
-                }
-                angelicaOverrideSetterTemp = angelicaOverrideSetterTemp.bindTo(rendererInstance);
-            }
-
-            isLoaded = true;
-            LOGGER.info("Successfully initialized HoloInventory compatibility layer");
-        } catch (Exception e) {
-            LOGGER.warn("Failed to initialize HoloInventory compatibility layer", e);
-            isLoaded = false;
-        }
-
-        renderEventPostMethod = renderEventPostMethodTemp;
-        renderEventMethod = renderEventMethodTemp;
-        angelicaOverrideSetter = angelicaOverrideSetterTemp;
-    }
-
-    public static void renderEvent(RenderGameOverlayEvent event) {
-        if (!isLoaded) return;
-        try {
-            if (event instanceof RenderGameOverlayEvent.Post && renderEventPostMethod != null) {
-                renderEventPostMethod.invokeExact((RenderGameOverlayEvent.Post) event);
-            } else {
-                renderEventMethod.invokeExact(event);
-            }
-        } catch (Throwable e) {
-            LOGGER.error("Failed to invoke HoloInventory renderEvent", e);
-            isLoaded = false;
-        }
-    }
-
-    public static void setAngelicaOverride(boolean value) {
-        if (!isLoaded) return;
-        try {
-            angelicaOverrideSetter.invokeExact(value);
-        } catch (Throwable e) {
-            LOGGER.error("Failed to set HoloInventory angelicaOverride", e);
-            isLoaded = false;
-        }
-    }
-}

src/main/java/com/gtnewhorizons/angelica/compat/lwjgl/CompatMemoryUtil.java

@@ -0,0 +1,327 @@
+package com.gtnewhorizons.angelica.compat.lwjgl;
+
+import static com.gtnewhorizons.angelica.compat.lwjgl.Pointer.BITS64;
+
+import it.unimi.dsi.fastutil.longs.LongPredicate;
+import java.lang.reflect.Field;
+import java.lang.reflect.Modifier;
+import java.nio.ByteBuffer;
+import java.nio.ByteOrder;
+import java.nio.FloatBuffer;
+import java.nio.IntBuffer;
+import org.lwjgl.BufferUtils;
+
+/**
+ * Backported from LWJGL3 under the BSD 3-Clause "New" or "Revised" License license
+ *
+ * <p>This class provides functionality for managing native memory.
+ *
+ * <p>All methods in this class will make use of {@link sun.misc.Unsafe} if it's available, for performance. If Unsafe is not available, the fallback
+ * implementations make use of reflection and, in the worst-case, JNI.</p>
+ *
+ * <p>Method names in this class are prefixed with {@code mem} to avoid ambiguities when used with static imports.</p>
+ */
+public class CompatMemoryUtil {
+    public static final long NULL = 0;
+
+    static final sun.misc.Unsafe UNSAFE;
+
+    private static final Class<? extends ByteBuffer> BUFFER_BYTE;
+    private static final Class<? extends IntBuffer> BUFFER_INT;
+    private static final Class<? extends FloatBuffer> BUFFER_FLOAT;
+
+    private static final long MARK;
+    private static final long POSITION;
+    private static final long LIMIT;
+    private static final long CAPACITY;
+
+    private static final long ADDRESS;
+
+    static {
+        ByteBuffer bb = ByteBuffer.allocateDirect(0).order(ByteOrder.nativeOrder());
+        BUFFER_BYTE = bb.getClass();
+        BUFFER_INT = bb.asIntBuffer().getClass();
+        BUFFER_FLOAT = bb.asFloatBuffer().getClass();
+
+        UNSAFE = getUnsafeInstance();
+
+        MARK = getMarkOffset();
+        POSITION = getPositionOffset();
+        LIMIT = getLimitOffset();
+        CAPACITY = getCapacityOffset();
+
+        ADDRESS = getAddressOffset();
+    }
+
+    public static ByteBuffer memReallocDirect(ByteBuffer old, int capacity) {
+        ByteBuffer newBuf = BufferUtils.createByteBuffer(capacity);
+        int oldPos = old.position();
+        old.rewind();
+        newBuf.put(old);
+        newBuf.position(Math.min(capacity, oldPos));
+        return newBuf;
+    }
+
+    public static IntBuffer memReallocDirect(IntBuffer old, int capacity) {
+        IntBuffer newBuf = BufferUtils.createIntBuffer(capacity);
+        int oldPos = old.position();
+        old.rewind();
+        newBuf.put(old);
+        newBuf.position(Math.min(capacity, oldPos));
+        return newBuf;
+    }
+
+    public static FloatBuffer memReallocDirect(FloatBuffer old, int capacity) {
+        FloatBuffer newBuf = BufferUtils.createFloatBuffer(capacity);
+        int oldPos = old.position();
+        old.rewind();
+        newBuf.put(old);
+        newBuf.position(Math.min(capacity, oldPos));
+        return newBuf;
+    }
+
+     /* -------------------------------------
+        -------------------------------------
+               UNSAFE MEMORY ACCESS API
+        -------------------------------------
+        ------------------------------------- */
+
+    private static final int  FILL_PATTERN_32 = Integer.divideUnsigned(-1, 255);
+    private static final long FILL_PATTERN_64 = Long.divideUnsigned(-1L, 255L);
+
+    /**
+     * Sets all bytes in a specified block of memory to a fixed value (usually zero).
+     *
+     * @param ptr   the starting memory address
+     * @param value the value to set (memSet will convert it to unsigned byte)
+     * @param bytes the number of bytes to set
+     */
+    public static void memSet(long ptr, int value, long bytes) {
+        if (/*DEBUG*/ false && (ptr == NULL || bytes < 0)) {
+            throw new IllegalArgumentException();
+        }
+
+        /*
+        - Unsafe.setMemory is very slow.
+        - A custom Java loop is fastest at small sizes, approximately up to 256 bytes.
+        - The native memset becomes fastest at bigger sizes, when the JNI overhead becomes negligible.
+        TODO: verify this
+         */
+
+        //UNSAFE.setMemory(ptr, bytes, (byte)(value & 0xFF));
+        //if (bytes < 256L) {
+        int p = (int)ptr;
+        if (BITS64) {
+            if ((p & 7) == 0) {
+                memSet64(ptr, value, (int)bytes & 0xFF);
+                return;
+            }
+        } else {
+            if ((p & 3) == 0) {
+                memSet32(p, value, (int)bytes & 0xFF);
+                return;
+            }
+        }
+        //}
+        //nmemset(ptr, value, bytes);
+    }
+    private static void memSet64(long ptr, int value, int bytes) {
+        int aligned = bytes & ~7;
+
+        // Aligned body
+        long valuel = (value & 0xFF) * FILL_PATTERN_64;
+        for (int i = 0; i < aligned; i += 8) {
+            UNSAFE.putLong(null, ptr + i, valuel);
+        }
+
+        // Unaligned tail
+        byte valueb = (byte)(value & 0xFF);
+        for (int i = aligned; i < bytes; i++) {
+            UNSAFE.putByte(null, ptr + i, valueb);
+        }
+    }
+    private static void memSet32(int ptr, int value, int bytes) {
+        int aligned = bytes & ~3;
+
+        // Aligned body
+        int vi = (value & 0xFF) * FILL_PATTERN_32;
+        for (int i = 0; i < aligned; i += 4) {
+            UNSAFE.putInt(null, (ptr + i) & 0xFFFF_FFFFL, vi);
+        }
+
+        // Unaligned tail
+        byte vb = (byte)(value & 0xFF);
+        for (int i = aligned; i < bytes; i++) {
+            UNSAFE.putByte(null, (ptr + i) & 0xFFFF_FFFFL, vb);
+        }
+    }
+
+    public static boolean memGetBoolean(long ptr) { return UNSAFE.getByte(null, ptr) != 0; }
+    public static byte memGetByte(long ptr)       { return UNSAFE.getByte(null, ptr); }
+    public static short memGetShort(long ptr)     { return UNSAFE.getShort(null, ptr); }
+    public static int memGetInt(long ptr)         { return UNSAFE.getInt(null, ptr); }
+    public static long memGetLong(long ptr)       { return UNSAFE.getLong(null, ptr); }
+    public static float memGetFloat(long ptr)     { return UNSAFE.getFloat(null, ptr); }
+    public static double memGetDouble(long ptr)   { return UNSAFE.getDouble(null, ptr); }
+
+    public static void memPutByte(long ptr, byte value)     { UNSAFE.putByte(null, ptr, value); }
+    public static void memPutShort(long ptr, short value)   { UNSAFE.putShort(null, ptr, value); }
+    public static void memPutInt(long ptr, int value)       { UNSAFE.putInt(null, ptr, value); }
+    public static void memPutLong(long ptr, long value)     { UNSAFE.putLong(null, ptr, value); }
+    public static void memPutFloat(long ptr, float value)   { UNSAFE.putFloat(null, ptr, value); }
+    public static void memPutDouble(long ptr, double value) { UNSAFE.putDouble(null, ptr, value); }
+
+    // -------------------------------------------------
+    // -------------------------------------------------
+    // -------------------------------------------------
+
+    private static sun.misc.Unsafe getUnsafeInstance() {
+        java.lang.reflect.Field[] fields = sun.misc.Unsafe.class.getDeclaredFields();
+
+        /*
+        Different runtimes use different names for the Unsafe singleton,
+        so we cannot use .getDeclaredField and we scan instead. For example:
+
+        Oracle: theUnsafe
+        PERC : m_unsafe_instance
+        Android: THE_ONE
+        */
+        for (java.lang.reflect.Field field : fields) {
+            if (!field.getType().equals(sun.misc.Unsafe.class)) {
+                continue;
+            }
+
+            int modifiers = field.getModifiers();
+            if (!(java.lang.reflect.Modifier.isStatic(modifiers) && java.lang.reflect.Modifier.isFinal(modifiers))) {
+                continue;
+            }
+
+            try {
+                field.setAccessible(true);
+                return (sun.misc.Unsafe)field.get(null);
+            } catch (Exception ignored) {
+            }
+            break;
+        }
+
+        throw new UnsupportedOperationException("LWJGL requires sun.misc.Unsafe to be available.");
+    }
+
+    /**
+     * Recursively searches an object and its superclasses for a field with the same name and type. If the field isn't
+     * public, it may not be stable across library/JVM versions, and these calls are likely not portable. You have been
+     * warned!
+     */
+    private static long getFieldOffset(Class<?> containerType, Class<?> fieldType, String name) {
+        Class<?> c = containerType;
+        while (c != Object.class) {
+            Field[] fields = c.getDeclaredFields();
+            for (Field field : fields) {
+                if (!field.getType().isAssignableFrom(fieldType) || Modifier.isStatic(field.getModifiers()) || field.isSynthetic()) {
+                    continue;
+                }
+
+                long offset = UNSAFE.objectFieldOffset(field);
+                if (field.getName().equals(name)) {
+                    return offset;
+                }
+            }
+            c = c.getSuperclass();
+        }
+        throw new UnsupportedOperationException("Failed to find field offset in class.");
+    }
+
+    private static long getFieldOffset(Class<?> containerType, Class<?> fieldType, LongPredicate predicate) {
+        Class<?> c = containerType;
+        while (c != Object.class) {
+            Field[] fields = c.getDeclaredFields();
+            for (Field field : fields) {
+                if (!field.getType().isAssignableFrom(fieldType) || Modifier.isStatic(field.getModifiers()) || field.isSynthetic()) {
+                    continue;
+                }
+
+                long offset = UNSAFE.objectFieldOffset(field);
+                if (predicate.test(offset)) {
+                    return offset;
+                }
+            }
+            c = c.getSuperclass();
+        }
+        throw new UnsupportedOperationException("Failed to find field offset in class.");
+    }
+
+    private static long getFieldOffsetInt(Object container, int value) {
+        return getFieldOffset(container.getClass(), int.class, offset -> UNSAFE.getInt(container, offset) == value);
+    }
+
+    private static long getAddressOffset() {
+        final ByteBuffer bb = ByteBuffer.allocateDirect(0);
+        return getFieldOffset(bb.getClass(), long.class, "address");
+
+        // TODO: do this portably
+        // long MAGIC_ADDRESS = 0xDEADBEEF8BADF00DL & (BITS32 ? 0xFFFF_FFFFL : 0xFFFF_FFFF_FFFF_FFFFL);
+        // ByteBuffer bb = Objects.requireNonNull(NewDirectByteBuffer(MAGIC_ADDRESS, 0));
+        //return getFieldOffset(bb.getClass(), long.class, offset -> UNSAFE.getLong(bb, offset) == MAGIC_ADDRESS);
+    }
+
+    private static final int MAGIC_CAPACITY = 0x0D15EA5E;
+    private static final int MAGIC_POSITION = 0x00FACADE;
+
+    private static long getMarkOffset() {
+        // TODO: is this actually reliable without NewDirectByteBuffer?
+        ByteBuffer bb = ByteBuffer.allocateDirect(0);
+        return getFieldOffsetInt(bb, -1);
+        // ByteBuffer bb = Objects.requireNonNull(NewDirectByteBuffer(1L, 0));
+        // return getFieldOffsetInt(bb, -1);
+    }
+
+    private static long getPositionOffset() {
+        ByteBuffer bb = ByteBuffer.allocateDirect(MAGIC_CAPACITY);
+        bb.position(MAGIC_POSITION);
+        return getFieldOffsetInt(bb, MAGIC_POSITION);
+    }
+
+    private static long getLimitOffset() {
+        ByteBuffer bb = ByteBuffer.allocateDirect(MAGIC_CAPACITY);
+        bb.limit(MAGIC_POSITION);
+        return getFieldOffsetInt(bb, MAGIC_POSITION);
+    }
+
+    private static long getCapacityOffset() {
+        ByteBuffer bb = ByteBuffer.allocateDirect(MAGIC_CAPACITY);
+        bb.limit(0);
+        return getFieldOffsetInt(bb, MAGIC_CAPACITY);
+    }
+
+    static IntBuffer wrapBufferInt(long address, int capacity) {
+        IntBuffer buffer;
+        try {
+            buffer = (IntBuffer)UNSAFE.allocateInstance(BUFFER_INT);
+        } catch (InstantiationException e) {
+            throw new UnsupportedOperationException(e);
+        }
+
+        UNSAFE.putLong(buffer, ADDRESS, address);
+        UNSAFE.putInt(buffer, MARK, -1);
+        UNSAFE.putInt(buffer, LIMIT, capacity);
+        UNSAFE.putInt(buffer, CAPACITY, capacity);
+
+        return buffer;
+    }
+
+    static FloatBuffer wrapBufferFloat(long address, int capacity) {
+        FloatBuffer buffer;
+        try {
+            buffer = (FloatBuffer)UNSAFE.allocateInstance(BUFFER_FLOAT);
+        } catch (InstantiationException e) {
+            throw new UnsupportedOperationException(e);
+        }
+
+        UNSAFE.putLong(buffer, ADDRESS, address);
+        UNSAFE.putInt(buffer, MARK, -1);
+        UNSAFE.putInt(buffer, LIMIT, capacity);
+        UNSAFE.putInt(buffer, CAPACITY, capacity);
+
+        return buffer;
+    }
+}

src/main/java/com/gtnewhorizons/angelica/compat/lwjgl/MemoryStack.java

@@ -0,0 +1,382 @@
+/*
+ * This class is backported from LWJGL3 under the BSD 3-clause "New" or "Revised" License
+ */
+
+package com.gtnewhorizons.angelica.compat.lwjgl;
+
+
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.memPutDouble;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.memPutFloat;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.memPutInt;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.memPutLong;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.memPutShort;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.memSet;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.wrapBufferFloat;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.wrapBufferInt;
+
+import com.gtnewhorizons.angelica.loading.AngelicaTweaker;
+import java.nio.ByteBuffer;
+import java.nio.FloatBuffer;
+import java.nio.IntBuffer;
+import java.util.Arrays;
+import org.jetbrains.annotations.Nullable;
+import org.lwjgl.BufferUtils;
+import org.lwjgl.MemoryUtil;
+
+/**
+ * An off-heap memory stack.
+ *
+ * <p>This class should be used in a thread-local manner for stack allocations.</p>
+ *
+ * @ see Configuration#STACK_SIZE
+ * @ see Configuration#DEBUG_STACK
+ */
+@SuppressWarnings("LombokGetterMayBeUsed")
+public class MemoryStack extends Pointer.Default implements AutoCloseable {
+
+    private static final int DEFAULT_STACK_SIZE   = /*Configuration.STACK_SIZE.get(64)*/ 64 * 1024;
+    private static final int DEFAULT_STACK_FRAMES = 8;
+
+    private static final ThreadLocal<MemoryStack> TLS = ThreadLocal.withInitial(MemoryStack::create);
+
+    static {
+        if (DEFAULT_STACK_SIZE < 0) {
+            throw new IllegalStateException("Invalid stack size.");
+        }
+    }
+
+    @SuppressWarnings({"FieldCanBeLocal", "unused"})
+    private final @Nullable ByteBuffer container;
+
+    private final int size;
+
+    private int pointer;
+
+    private   int[] frames;
+    protected int   frameIndex;
+
+    /**
+     * Creates a new {@code MemoryStack} backed by the specified memory region.
+     *
+     * <p>In the initial state, there is no active stack frame. The {@link #push} method must be used before any allocations.</p>
+     *
+     * @param container the backing memory buffer, may be null
+     * @param address   the backing memory address
+     * @param size      the backing memory size
+     */
+    protected MemoryStack(@Nullable ByteBuffer container, long address, int size) {
+        super(address);
+        this.container = container;
+
+        this.size = size;
+        this.pointer = size;
+
+        this.frames = new int[DEFAULT_STACK_FRAMES];
+    }
+
+    /**
+     * Creates a new {@code MemoryStack} with the default size.
+     *
+     * <p>In the initial state, there is no active stack frame. The {@link #push} method must be used before any allocations.</p>
+     */
+    public static MemoryStack create() {
+        return create(DEFAULT_STACK_SIZE);
+    }
+
+    /**
+     * Creates a new {@code MemoryStack} with the specified size.
+     *
+     * <p>In the initial state, there is no active stack frame. The {@link #push} method must be used before any allocations.</p>
+     *
+     * @param capacity the maximum number of bytes that may be allocated on the stack
+     */
+    public static MemoryStack create(int capacity) {
+        return create(BufferUtils.createByteBuffer(capacity));
+    }
+
+    /**
+     * Creates a new {@code MemoryStack} backed by the specified memory buffer.
+     *
+     * <p>In the initial state, there is no active stack frame. The {@link #push} method must be used before any allocations.</p>
+     *
+     * @param buffer the backing memory buffer
+     */
+    public static MemoryStack create(ByteBuffer buffer) {
+        long address = MemoryUtil.getAddress(buffer);
+        int  size    = buffer.remaining();
+        return new MemoryStack(buffer, address, size);
+    }
+
+    /**
+     * Creates a new {@code MemoryStack} backed by the specified memory region.
+     *
+     * <p>In the initial state, there is no active stack frame. The {@link #push} method must be used before any allocations.</p>
+     *
+     * @param address the backing memory address
+     * @param size    the backing memory size
+     */
+    public static MemoryStack ncreate(long address, int size) {
+        return new MemoryStack(null, address, size);
+    }
+
+    /**
+     * Stores the current stack pointer and pushes a new frame to the stack.
+     *
+     * <p>This method should be called when entering a method, before doing any stack allocations. When exiting a method, call the {@link #pop} method to
+     * restore the previous stack frame.</p>
+     *
+     * <p>Pairs of push/pop calls may be nested. Care must be taken to:</p>
+     * <ul>
+     * <li>match every push with a pop</li>
+     * <li>not call pop before push has been called at least once</li>
+     * <li>not nest push calls to more than the maximum supported depth</li>
+     * </ul>
+     *
+     * @return this stack
+     */
+    public MemoryStack push() {
+        if (frameIndex == frames.length) {
+            frameOverflow();
+        }
+
+        frames[frameIndex++] = pointer;
+        return this;
+    }
+
+    private void frameOverflow() {
+        if (/*DEBUG*/ false) {
+            AngelicaTweaker.LOGGER.warn("[WARNING] Out of frame stack space (" + frames.length + ") in thread: " + Thread.currentThread());
+        }
+        frames = Arrays.copyOf(frames, frames.length * 3 / 2);
+    }
+
+    /**
+     * Pops the current stack frame and moves the stack pointer to the end of the previous stack frame.
+     *
+     * @return this stack
+     */
+    public MemoryStack pop() {
+        pointer = frames[--frameIndex];
+        return this;
+    }
+
+    /**
+     * Calls {@link #pop} on this {@code MemoryStack}.
+     *
+     * <p>This method should not be used directly. It is called automatically when the {@code MemoryStack} is used as a resource in a try-with-resources
+     * statement.</p>
+     */
+    @Override
+    public void close() {
+        //noinspection resource
+        pop();
+    }
+
+    /**
+     * Returns the address of the backing off-heap memory.
+     *
+     * <p>The stack grows "downwards", so the bottom of the stack is at {@code address + size}, while the top is at {@code address}.</p>
+     */
+    public long getAddress() {
+        return address;
+    }
+
+    /**
+     * Returns the size of the backing off-heap memory.
+     *
+     * <p>This is the maximum number of bytes that may be allocated on the stack.</p>
+     */
+    public int getSize() {
+        return size;
+    }
+
+    /**
+     * Returns the current frame index.
+     *
+     * <p>This is the current number of nested {@link #push} calls.</p>
+     */
+    public int getFrameIndex() {
+        return frameIndex;
+    }
+
+    /** Returns the memory address at the current stack pointer. */
+    public long getPointerAddress() {
+        return address + (pointer & 0xFFFF_FFFFL);
+    }
+
+    /**
+     * Returns the current stack pointer.
+     *
+     * <p>The stack grows "downwards", so when the stack is empty {@code pointer} is equal to {@code size}. On every allocation {@code pointer} is reduced by
+     * the allocated size (after alignment) and {@code address + pointer} points to the first byte of the last allocation.</p>
+     *
+     * <p>Effectively, this methods returns how many more bytes may be allocated on the stack.</p>
+     */
+    public int getPointer() {
+        return pointer;
+    }
+
+    /**
+     * Sets the current stack pointer.
+     *
+     * <p>This method directly manipulates the stack pointer. Using it irresponsibly may break the internal state of the stack. It should only be used in rare
+     * cases or in auto-generated code.</p>
+     */
+    public void setPointer(int pointer) {
+        if (/*CHECKS*/ false) {
+            checkPointer(pointer);
+        }
+
+        this.pointer = pointer;
+    }
+
+    private void checkPointer(int pointer) {
+        if (pointer < 0 || size < pointer) {
+            throw new IndexOutOfBoundsException("Invalid stack pointer");
+        }
+    }
+
+    private static void checkAlignment(int alignment) {
+        if (Integer.bitCount(alignment) != 1) {
+            throw new IllegalArgumentException("Alignment must be a power-of-two value.");
+        }
+    }
+
+    /**
+     * Calls {@link #nmalloc(int, int)} with {@code alignment} equal to {@link Pointer#POINTER_SIZE POINTER_SIZE}.
+     *
+     * @param size the allocation size
+     *
+     * @return the memory address on the stack for the requested allocation
+     */
+    public long nmalloc(int size) {
+        return nmalloc(POINTER_SIZE, size);
+    }
+
+    /**
+     * Allocates a block of {@code size} bytes of memory on the stack. The content of the newly allocated block of memory is not initialized, remaining with
+     * indeterminate values.
+     *
+     * @param alignment the required alignment
+     * @param size      the allocation size
+     *
+     * @return the memory address on the stack for the requested allocation
+     */
+    public long nmalloc(int alignment, int size) {
+        // Align address to the specified alignment
+        long address = (this.address + pointer - size) & ~Integer.toUnsignedLong(alignment - 1);
+
+        pointer = (int)(address - this.address);
+        if (/*CHECKS*/ false && pointer < 0) {
+            throw new OutOfMemoryError("Out of stack space.");
+        }
+
+        return address;
+    }
+
+    /**
+     * Allocates a block of memory on the stack for an array of {@code num} elements, each of them {@code size} bytes long, and initializes all its bits to
+     * zero.
+     *
+     * @param alignment the required element alignment
+     * @param num       num  the number of elements to allocate
+     * @param size      the size of each element
+     *
+     * @return the memory address on the stack for the requested allocation
+     */
+    public long ncalloc(int alignment, int num, int size) {
+        int  bytes   = num * size;
+        long address = nmalloc(alignment, bytes);
+        memSet(address, 0, bytes);
+        return address;
+    }
+
+    // -------------------------------------------------
+
+    /** Unsafe version of {@link #shorts(short)}. */
+    public long nshort(short value) {
+        long a = nmalloc(2, 2);
+        memPutShort(a, value);
+        return a;
+    }
+
+    // -------------------------------------------------
+
+    /** Int version of {@link #malloc(int)}. */
+    public IntBuffer mallocInt(int size) { return wrapBufferInt(nmalloc(4, size << 2), size); }
+
+    /** Unsafe version of {@link #ints(int)}. */
+    public long nint(int value) {
+        long a = nmalloc(4, 4);
+        memPutInt(a, value);
+        return a;
+    }
+
+    // -------------------------------------------------
+
+    /** Unsafe version of {@link #longs(long)}. */
+    public long nlong(long value) {
+        long a = nmalloc(8, 8);
+        memPutLong(a, value);
+        return a;
+    }
+
+    // -------------------------------------------------
+
+    /** Float version of {@link #malloc(int)}. */
+    public FloatBuffer mallocFloat(int size) { return wrapBufferFloat(nmalloc(4, size << 2), size); }
+
+    /** Unsafe version of {@link #floats(float)}. */
+    public long nfloat(float value) {
+        long a = nmalloc(4, 4);
+        memPutFloat(a, value);
+        return a;
+    }
+
+    // -------------------------------------------------
+
+    /** Unsafe version of {@link #doubles(double)}. */
+    public long ndouble(double value) {
+        long a = nmalloc(8, 8);
+        memPutDouble(a, value);
+        return a;
+    }
+
+    // -------------------------------------------------
+    // -------------------------------------------------
+    // -------------------------------------------------
+
+
+    /** Returns the stack of the current thread. */
+    public static MemoryStack stackGet() {
+        return TLS.get();
+    }
+
+    /**
+     * Calls {@link #push} on the stack of the current thread.
+     *
+     * @return the stack of the current thread.
+     */
+    public static MemoryStack stackPush() {
+        return stackGet().push();
+    }
+
+    /**
+     * Calls {@link #pop} on the stack of the current thread.
+     *
+     * @return the stack of the current thread.
+     */
+    public static MemoryStack stackPop() {
+        return stackGet().pop();
+    }
+
+    /** Thread-local version of {@link #nmalloc(int)}. */
+    public static long nstackMalloc(int size) { return stackGet().nmalloc(size); }
+    /** Thread-local version of {@link #nmalloc(int, int)}. */
+    public static long nstackMalloc(int alignment, int size) { return stackGet().nmalloc(alignment, size); }
+    /** Thread-local version of {@link #ncalloc}. */
+    public static long nstackCalloc(int alignment, int num, int size) { return stackGet().ncalloc(alignment, num, size); }
+
+    // -------------------------------------------------
+
+}

src/main/java/com/gtnewhorizons/angelica/compat/lwjgl/Pointer.java

@@ -0,0 +1,93 @@
+/*
+ * This class is backported from LWJGL3 under the BSD 3-clause "New" or "Revised" License
+ */
+package com.gtnewhorizons.angelica.compat.lwjgl;
+
+
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.NULL;
+import static com.gtnewhorizons.angelica.compat.lwjgl.CompatMemoryUtil.UNSAFE;
+import static org.lwjgl.LWJGLUtil.CHECKS;
+
+import net.minecraft.util.Util;
+import org.jetbrains.annotations.Nullable;
+
+/**
+ * Pointer interface.
+ *
+ * <p>LWJGL can run on both 32bit and 64bit architectures. Since LWJGL applications deal with native memory directly, this interface provides necessary
+ * information about the underlying architecture of the running JVM process.</p>
+ *
+ * <p>When interacting with native functions, pointer values are mapped to Java {@code long}. LWJGL automatically converts long values to the correct pointer
+ * addresses when used in native code. Native functions sometimes require arrays of pointer values; the {@link PointerBuffer} class may be used for that
+ * purpose. It has an API similar to a {@link java.nio.LongBuffer} but handles pointer casts automatically.</p>
+ */
+public interface Pointer {
+
+    /** The pointer size in bytes. Will be 4 on a 32bit JVM and 8 on a 64bit one. */
+    int POINTER_SIZE = UNSAFE.addressSize();
+
+    /** The pointer size power-of-two. Will be 2 on a 32bit JVM and 3 on a 64bit one. */
+    int POINTER_SHIFT = POINTER_SIZE == 8 ? 3 : 2;
+
+    /** The value of {@code sizeof(long)} for the current platform. */
+    int CLONG_SIZE  = POINTER_SIZE == 8 && Util.getOSType() == Util.EnumOS.WINDOWS ? 4 : POINTER_SIZE;
+
+    /** The value of {@code sizeof(long)} as a power-of-two. */
+    int CLONG_SHIFT = CLONG_SIZE == 8 ? 3 : 2;
+
+    /** Will be true on a 32bit JVM. */
+    boolean BITS32 = POINTER_SIZE * 8 == 32;
+
+    /** Will be true on a 64bit JVM. */
+    boolean BITS64 = POINTER_SIZE * 8 == 64;
+
+    /**
+     * Returns the raw pointer address as a {@code long} value.
+     *
+     * @return the pointer address
+     */
+    long address();
+
+    /** Default {@link Pointer} implementation. */
+    abstract class Default implements Pointer {
+
+        // Removed final due to JDK-8139758. TODO: Restore if the fix is backported to JDK 8.
+        protected long address;
+
+        protected Default(long address) {
+            if (CHECKS && address == NULL) {
+                throw new NullPointerException();
+            }
+            this.address = address;
+        }
+
+        @Override
+        public long address() {
+            return address;
+        }
+
+        public boolean equals(@Nullable Object o) {
+            if (this == o) {
+                return true;
+            }
+            if (!(o instanceof Pointer)) {
+                return false;
+            }
+
+            Pointer that = (Pointer)o;
+
+            return address == that.address();
+        }
+
+        public int hashCode() {
+            return (int)(address ^ (address >>> 32));
+        }
+
+        @Override
+        public String toString() {
+            return String.format("%s pointer [0x%X]", getClass().getSimpleName(), address);
+        }
+
+    }
+
+}

src/main/java/com/gtnewhorizons/angelica/compat/mojang/ByteBufferBackedInputStream.java

@@ -15,7 +15,6 @@ public class ByteBufferBackedInputStream extends InputStream {
         this.buf = buf;
     }
 
-    @Override
     public int read() throws IOException {
         if (!buf.hasRemaining()) {
             return -1;

src/main/java/com/gtnewhorizons/angelica/compat/mojang/ChunkPos.java

@@ -40,7 +40,6 @@ public class ChunkPos {
         return (long)x & 4294967295L | ((long)z & 4294967295L) << 32;
     }
 
-    @Override
     public int hashCode()
     {
         final int i = 1664525 * this.x + 1013904223;
@@ -48,7 +47,6 @@ public class ChunkPos {
         return i ^ j;
     }
 
-    @Override
     public boolean equals(Object object) {
         if (this == object) {
             return true;

src/main/java/com/gtnewhorizons/angelica/compat/mojang/InteractionHand.java

@@ -1,23 +1,10 @@
 package com.gtnewhorizons.angelica.compat.mojang;
 
 
-import com.gtnewhorizons.angelica.compat.ModStatus;
-import net.minecraft.entity.player.EntityPlayer;
-import net.minecraft.item.ItemStack;
-
 public enum InteractionHand {
     MAIN_HAND,
     OFF_HAND;
 
     InteractionHand() {
     }
-
-    public ItemStack getItemInHand(EntityPlayer player){
-        if (ModStatus.isBackhandLoaded && this == InteractionHand.OFF_HAND){ // off hand (requires backhand)
-            return ModStatus.backhandCompat.getOffhandItem(player);
-        }
-        else { // main hand
-            return player.getHeldItem();
-        }
-    }
 }

src/main/java/com/gtnewhorizons/angelica/compat/mojang/NativeImage.java

@@ -1,5 +1,6 @@
 package com.gtnewhorizons.angelica.compat.mojang;
 
+import com.gtnewhorizons.angelica.compat.lwjgl.MemoryStack;
 import java.awt.image.BufferedImage;
 import java.io.File;
 import java.io.IOException;
@@ -7,8 +8,6 @@ import java.io.InputStream;
 import java.nio.ByteBuffer;
 import java.nio.IntBuffer;
 import javax.imageio.ImageIO;
-
-import com.gtnewhorizon.gtnhlib.bytebuf.MemoryUtilities;
 import lombok.Getter;
 import net.coderbot.iris.Iris;
 import org.lwjgl.opengl.GL11;
@@ -59,20 +58,15 @@ public class NativeImage extends BufferedImage {
 
 //        final int width = GL11.glGetTexLevelParameteri(GL11.GL_TEXTURE_2D, level, GL11.GL_TEXTURE_WIDTH);
 //        final int height = GL11.glGetTexLevelParameteri(GL11.GL_TEXTURE_2D, level, GL11.GL_TEXTURE_HEIGHT);
-
-        final IntBuffer buffer = MemoryUtilities.memAllocInt(size);
-
-        try {
+        try (final MemoryStack stack = MemoryStack.stackPush()) {
+            final IntBuffer buffer = stack.mallocInt(size);
             GL11.glGetTexImage(GL11.GL_TEXTURE_2D, level, format.glFormat, GL12.GL_UNSIGNED_INT_8_8_8_8_REV, buffer);
 
             int[] data = new int[size];
             buffer.get(data);
             setRGB(0, 0, width, height, data, 0, width);
-        } finally {
-            MemoryUtilities.memFree(buffer);
         }
     }
-
     public void writeToFile(File file) throws IOException{
         try {
             ImageIO.write(this, "png", file);
@@ -110,15 +104,18 @@ public class NativeImage extends BufferedImage {
 
 
     public enum Format {
-        RGBA(4, GL11.GL_RGBA),
-        RGB(3, GL11.GL_RGB);
+        RGBA(4, GL11.GL_RGBA, BufferedImage.TYPE_INT_ARGB),
+        RGB(3, GL11.GL_RGB, BufferedImage.TYPE_INT_RGB);
 
         private final int components;
         private final int glFormat;
+        private final int imageType;
 
-        Format(int components, int glFormat) {
+
+        Format(int components, int glFormat, int imageType) {
             this.components = components;
             this.glFormat = glFormat;
+            this.imageType = imageType;
         }
     }
 

src/main/java/com/gtnewhorizons/angelica/compat/toremove/RenderLayer.java

@@ -72,6 +72,7 @@ public abstract class RenderLayer extends RenderPhase { // Aka: RenderType (Iris
         private final MultiPhaseParameters phases;
         private final int hash;
         private final Optional<RenderLayer> affectedOutline;
+        private final boolean outline;
 
         private MultiPhase(String name, VertexFormat vertexFormat, int drawMode, int expectedBufferSize, boolean hasCrumbling, boolean translucent, MultiPhaseParameters phases) {
             super(name, vertexFormat, drawMode, expectedBufferSize, () -> {
@@ -83,6 +84,7 @@ public abstract class RenderLayer extends RenderPhase { // Aka: RenderType (Iris
             this.affectedOutline = phases.outlineMode == RenderLayer.OutlineMode.AFFECTS_OUTLINE ? phases.texture.getId().map((arg2) -> {
                 return getOutline(arg2, phases.cull);
             }) : Optional.empty();
+            this.outline = phases.outlineMode == RenderLayer.OutlineMode.IS_OUTLINE;
             this.hash = Objects.hash(new Object[]{super.hashCode(), phases});
         }
 
@@ -95,23 +97,20 @@ public abstract class RenderLayer extends RenderPhase { // Aka: RenderType (Iris
             return this.affectedOutline;
         }
 
-        @Override
         public boolean equals(@Nullable Object object) {
             return this == object;
         }
 
-        @Override
         public int hashCode() {
             return this.hash;
         }
 
-        @Override
         public String toString() {
             return "RenderType[" + this.phases + ']';
         }
 
         static {
-            CACHE = new ObjectOpenCustomHashSet<>(RenderLayer.MultiPhase.HashStrategy.INSTANCE);
+            CACHE = new ObjectOpenCustomHashSet(RenderLayer.MultiPhase.HashStrategy.INSTANCE);
         }
 
         static enum HashStrategy implements Hash.Strategy<MultiPhase> {
@@ -169,7 +168,6 @@ public abstract class RenderLayer extends RenderPhase { // Aka: RenderType (Iris
             this.phases = ImmutableList.of(this.texture, this.transparency, this.diffuseLighting, this.shadeModel, this.alpha, this.depthTest, this.cull, this.lightmap, this.fog, this.layering, this.target, this.texturing, this.writeMaskState);
         }
 
-        @Override
         public boolean equals(Object object) {
             if (this == object) {
                 return true;
@@ -181,12 +179,10 @@ public abstract class RenderLayer extends RenderPhase { // Aka: RenderType (Iris
             }
         }
 
-        @Override
         public int hashCode() {
             return Objects.hash(new Object[]{this.phases, this.outlineMode});
         }
 
-        @Override
         public String toString() {
             return "CompositeState[" + this.phases + ", outlineProperty=" + this.outlineMode + ']';
         }
@@ -297,7 +293,6 @@ public abstract class RenderLayer extends RenderPhase { // Aka: RenderType (Iris
             this.name = name;
         }
 
-        @Override
         public String toString() {
             return this.name;
         }

src/main/java/com/gtnewhorizons/angelica/compat/toremove/RenderPhase.java

@@ -11,6 +11,7 @@ import org.lwjgl.opengl.GL11;
 import javax.annotation.Nullable;
 import java.util.Objects;
 import java.util.Optional;
+import java.util.OptionalDouble;
 
 
 public abstract class RenderPhase {
@@ -97,7 +98,6 @@ public abstract class RenderPhase {
         this.endAction.run();
     }
 
-    @Override
     public boolean equals(@Nullable Object object) {
         if (this == object) {
             return true;
@@ -109,12 +109,10 @@ public abstract class RenderPhase {
         }
     }
 
-    @Override
     public int hashCode() {
         return this.name.hashCode();
     }
 
-    @Override
     public String toString() {
         return this.name;
     }

src/main/java/com/gtnewhorizons/angelica/config/AngelicaConfig.java

@@ -38,7 +38,7 @@ public class AngelicaConfig {
     public static boolean enableIris;
 
     @Config.Comment("Enable MCPatcherForge features, still in Alpha. Individual features are toggled in mcpatcher.json")
-    @Config.DefaultBoolean(true)
+    @Config.DefaultBoolean(false)
     @Config.RequiresMcRestart
     public static boolean enableMCPatcherForgeFeatures;
 
@@ -133,37 +133,19 @@ public class AngelicaConfig {
     @Config.RequiresMcRestart
     public static boolean enableDebugLogging;
 
-    @Config.Comment("Enables PBR atlas dumping")
-    @Config.DefaultBoolean(false)
-    @Config.Name("Enable PBR Debug")
-    public static boolean enablePBRDebug;
-
     @Config.Comment("Enable Zoom")
     @Config.DefaultBoolean(true)
     @Config.RequiresMcRestart
     public static boolean enableZoom;
 
-    @Config.Comment("Optimizes in-world item rendering [Experimental]")
-    @Config.DefaultBoolean(false)
+    @Config.Comment("Optimizes in-world item rendering")
+    @Config.DefaultBoolean(true)
     @Config.RequiresMcRestart
-    public static boolean optimizeInWorldItemRendering_WIP;
+    public static boolean optimizeInWorldItemRendering;
 
     @Config.Comment("Max amount of display lists to cache for optimized item rendering. Higher number will use more VRAM")
     @Config.DefaultInt(256)
     @Config.RangeInt(min = 64, max = 1024)
     public static int itemRendererDisplayListCacheSize;
 
-    @Config.Comment("Render distance for the spinning mob inside mod spawners")
-    @Config.DefaultDouble(16D)
-    @Config.RangeDouble(min = 16D, max = 64D)
-    public static double mobSpawnerRenderDistance;
-
-    @Config.Comment("Switches to an alternate FPS limiter that gives more stable frametimes, in exchange for slightly " +
-        "more latency. Will never introduce more than one frame of latency, and has a lower impact at higher framerates.")
-    @Config.DefaultBoolean(false)
-    public static boolean sleepBeforeSwap;
-
-    @Config.Comment("Allows unicode languages to use an odd gui scale")
-    @Config.DefaultBoolean(true)
-    public static boolean removeUnicodeEvenScaling;
 }

src/main/java/com/gtnewhorizons/angelica/config/CompatConfig.java

@@ -39,9 +39,4 @@ public class CompatConfig {
     @Config.DefaultBoolean(true)
     @Config.RequiresMcRestart
     public static boolean fixImmersiveEngineering;
-
-    @Config.Comment("Apply fixes to Security Craft")
-    @Config.DefaultBoolean(true)
-    @Config.RequiresMcRestart
-    public static boolean fixSecurityCraft;
 }

src/main/java/com/gtnewhorizons/angelica/debug/F3Graph.java

@@ -28,10 +28,9 @@ import static org.lwjgl.opengl.GL20.glUniform1;
 import static org.lwjgl.opengl.GL20.glUniform1f;
 import static org.lwjgl.opengl.GL20.glUniform1i;
 import static org.lwjgl.opengl.GL20.glVertexAttribPointer;
-import static com.gtnewhorizon.gtnhlib.bytebuf.MemoryStack.*;
 
-import com.gtnewhorizon.gtnhlib.bytebuf.MemoryStack;
 import com.gtnewhorizon.gtnhlib.client.renderer.shader.ShaderProgram;
+import com.gtnewhorizons.angelica.compat.lwjgl.MemoryStack;
 import java.nio.FloatBuffer;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.gui.FontRenderer;
@@ -132,7 +131,7 @@ public abstract class F3Graph {
         // Load vertex buffer
         vertBuf = glGenBuffers();
         glBindBuffer(GL_ARRAY_BUFFER, vertBuf);
-        try (final MemoryStack stack = stackPush()) {
+        try (final MemoryStack stack = MemoryStack.stackPush()) {
             final FloatBuffer vertices = stack.mallocFloat(VERT_COUNT * VERT_FLOATS);
             // Since we use a triangle strip, we only need 4 vertices. The quad extends to the top of the screen so spikes
             // don't get truncated. The max height is replaced in the vert shader, no need to be precise.

src/main/java/com/gtnewhorizons/angelica/dynamiclights/DynamicLights.java

@@ -6,18 +6,19 @@ import com.gtnewhorizon.gtnhlib.blockpos.IBlockPos;
 import com.gtnewhorizon.gtnhlib.util.CoordinatePacker;
 import com.gtnewhorizons.angelica.api.IDynamicLightProducer;
 import com.gtnewhorizons.angelica.compat.ModStatus;
-import com.gtnewhorizons.angelica.compat.battlegear2.Battlegear2Compat;
 import com.gtnewhorizons.angelica.config.AngelicaConfig;
 import it.unimi.dsi.fastutil.longs.LongOpenHashSet;
 import it.unimi.dsi.fastutil.objects.ObjectOpenHashSet;
 import me.jellysquid.mods.sodium.client.render.SodiumWorldRenderer;
 import mods.battlegear2.api.core.IBattlePlayer;
+import mods.battlegear2.api.core.IInventoryPlayerBattle;
 import net.irisshaders.iris.api.v0.IrisApi;
 import net.minecraft.block.Block;
 import net.minecraft.block.material.Material;
 import net.minecraft.entity.Entity;
 import net.minecraft.entity.EntityLivingBase;
 import net.minecraft.entity.item.EntityItem;
+import net.minecraft.entity.monster.EntityCreeper;
 import net.minecraft.entity.player.EntityPlayer;
 import net.minecraft.init.Blocks;
 import net.minecraft.init.Items;
@@ -27,6 +28,7 @@ import net.minecraft.item.ItemStack;
 import net.minecraft.util.MathHelper;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
+import xonin.backhand.api.core.BackhandUtils;
 
 import java.util.Set;
 import java.util.concurrent.locks.ReentrantReadWriteLock;
@@ -376,13 +378,13 @@ public class DynamicLights {
                 player instanceof IBattlePlayer battlePlayer &&
                 battlePlayer.battlegear2$isBattlemode()
             ) {
-                ItemStack offhand = Battlegear2Compat.getBattlegear2Offhand(player);
+                ItemStack offhand = ((IInventoryPlayerBattle) player.inventory).battlegear2$getCurrentOffhandWeapon();
                 if (offhand != null) {
                     luminance = Math.max(luminance, getLuminanceFromItemStack(offhand, inWater));
                 }
             }
             else if (ModStatus.isBackhandLoaded && living instanceof EntityPlayer player){
-                ItemStack offhand = ModStatus.backhandCompat.getOffhandItem(player);
+                ItemStack offhand = BackhandUtils.getOffhandItem(player);
                 if (offhand != null) {
                     luminance = Math.max(luminance, getLuminanceFromItemStack(offhand, inWater));
                 }

src/main/java/com/gtnewhorizons/angelica/glsm/Feature.java

@@ -7,11 +7,9 @@ import it.unimi.dsi.fastutil.ints.Int2ObjectOpenHashMap;
 import org.lwjgl.opengl.GL11;
 import org.lwjgl.opengl.GL13;
 
-import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.HashSet;
-import java.util.List;
 import java.util.Map;
 import java.util.Set;
 
@@ -21,25 +19,21 @@ public class Feature {
         GL13.GL_MULTISAMPLE_BIT, GL11.GL_PIXEL_MODE_BIT, GL11.GL_POINT_BIT, GL11.GL_POLYGON_BIT, GL11.GL_POLYGON_STIPPLE_BIT, GL11.GL_SCISSOR_BIT,
         GL11.GL_STENCIL_BUFFER_BIT, GL11.GL_TEXTURE_BIT, GL11.GL_TRANSFORM_BIT, GL11.GL_VIEWPORT_BIT };
 
-    static final Int2ObjectMap<List<IStateStack<?>>> maskToFeaturesMap = new Int2ObjectOpenHashMap<>();
+    static final Int2ObjectMap<Set<IStateStack<?>>> maskToFeaturesMap = new Int2ObjectOpenHashMap<>();
 
-    static List<IStateStack<?>> maskToFeatures(int mask) {
+    static Set<IStateStack<?>> maskToFeatures(int mask) {
         if(maskToFeaturesMap.containsKey(mask)) {
             return maskToFeaturesMap.get(mask);
         }
 
         final Set<IStateStack<?>> features = new HashSet<>();
-
         for(int attrib : Feature.supportedAttribs) {
             if((mask & attrib) == attrib) {
                 features.addAll(getFeatures(attrib));
             }
         }
-
-        final List<IStateStack<?>> asList = new ArrayList<>(features);
-
-        maskToFeaturesMap.put(mask, asList);
-        return asList;
+        maskToFeaturesMap.put(mask, features);
+        return features;
     }
 
     private static final Map<Integer, Set<IStateStack<?>>> attribToFeatures = new HashMap<>();

src/main/java/com/gtnewhorizons/angelica/glsm/GLStateManager.java

@@ -26,7 +26,6 @@ import com.gtnewhorizons.angelica.glsm.texture.TextureInfoCache;
 import com.gtnewhorizons.angelica.glsm.texture.TextureTracker;
 import com.gtnewhorizons.angelica.hudcaching.HUDCaching;
 import com.gtnewhorizons.angelica.loading.AngelicaTweaker;
-import cpw.mods.fml.relauncher.ReflectionHelper;
 import it.unimi.dsi.fastutil.ints.Int2ObjectMap;
 import it.unimi.dsi.fastutil.ints.Int2ObjectOpenHashMap;
 import it.unimi.dsi.fastutil.ints.IntArrayList;
@@ -36,7 +35,6 @@ import it.unimi.dsi.fastutil.objects.Object2ObjectArrayMap;
 import it.unimi.dsi.fastutil.objects.ObjectArraySet;
 import lombok.Getter;
 import lombok.Setter;
-import lombok.SneakyThrows;
 import net.coderbot.iris.Iris;
 import net.coderbot.iris.gbuffer_overrides.state.StateTracker;
 import net.coderbot.iris.gl.blending.AlphaTestStorage;
@@ -67,16 +65,11 @@ import org.lwjgl.opengl.GLContext;
 import org.lwjgl.opengl.KHRDebug;
 
 import java.lang.Math;
-import java.lang.invoke.MethodHandle;
-import java.lang.invoke.MethodHandles;
-import java.lang.reflect.Field;
 import java.nio.ByteBuffer;
 import java.nio.DoubleBuffer;
 import java.nio.FloatBuffer;
 import java.nio.IntBuffer;
-import java.nio.ShortBuffer;
 import java.util.AbstractMap;
-import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.function.IntSupplier;
@@ -140,21 +133,11 @@ public class GLStateManager {
     @Getter protected static final MaterialStateStack frontMaterial = new MaterialStateStack(GL11.GL_FRONT);
     @Getter protected static final MaterialStateStack backMaterial = new MaterialStateStack(GL11.GL_BACK);
 
-    private static final MethodHandle MAT4_STACK_CURR_DEPTH;
-
     static {
         for (int i = 0; i < lightStates.length; i ++) {
             lightStates[i] = new BooleanStateStack(GL11.GL_LIGHT0 + i);
             lightDataStates[i] = new LightStateStack(GL11.GL_LIGHT0 + i);
         }
-
-        try {
-            Field curr = ReflectionHelper.findField(Matrix4fStack.class, "curr");
-            curr.setAccessible(true);
-            MAT4_STACK_CURR_DEPTH = MethodHandles.lookup().unreflectGetter(curr);
-        } catch (IllegalAccessException e) {
-            throw new RuntimeException(e);
-        }
     }
 
     @Getter protected static final ViewPortStateStack viewportState = new ViewPortStateStack();
@@ -166,18 +149,11 @@ public class GLStateManager {
         while(!attribs.isEmpty()) {
             attribs.popInt();
         }
-
-        List<IStateStack<?>> stacks = Feature.maskToFeatures(GL11.GL_ALL_ATTRIB_BITS);
-        int size = stacks.size();
-
-        for(int i = 0; i < size; i++) {
-            IStateStack<?> stack = stacks.get(i);
-
+        for(IStateStack<?> stack : Feature.maskToFeatures(GL11.GL_ALL_ATTRIB_BITS)) {
             while(!stack.isEmpty()) {
                 stack.pop();
             }
         }
-
         modelViewMatrix.clear();
         projectionMatrix.clear();
     }
@@ -204,9 +180,6 @@ public class GLStateManager {
 
 
     public static class GLFeatureSet extends IntOpenHashSet {
-
-        private static final long serialVersionUID = 8558779940775721010L;
-
         public GLFeatureSet addFeature(int feature) {
             super.add(feature);
             return this;
@@ -425,11 +398,6 @@ public class GLStateManager {
         }
     }
 
-    @SneakyThrows
-    public static int getMatrixStackDepth(Matrix4fStack stack) {
-        return (int) MAT4_STACK_CURR_DEPTH.invokeExact(stack);
-    }
-
     public static int glGetInteger(int pname) {
         if(shouldBypassCache()) {
             return GL11.glGetInteger(pname);
@@ -449,8 +417,6 @@ public class GLStateManager {
             case GL11.GL_COLOR_MATERIAL_FACE -> colorMaterialFace.getValue();
             case GL11.GL_COLOR_MATERIAL_PARAMETER -> colorMaterialParameter.getValue();
             case GL20.GL_CURRENT_PROGRAM -> activeProgram;
-            case GL11.GL_MODELVIEW_STACK_DEPTH -> getMatrixStackDepth(modelViewMatrix);
-            case GL11.GL_PROJECTION_STACK_DEPTH -> getMatrixStackDepth(projectionMatrix);
 
             default -> GL11.glGetInteger(pname);
         };
@@ -863,16 +829,6 @@ public class GLStateManager {
         GL11.glTexImage2D(target, level, internalformat, width, height, border, format, type, pixels);
     }
 
-    public static void glTexImage2D(int target, int level, int internalformat, int width, int height, int border, int format, int type, FloatBuffer pixels) {
-        TextureInfoCache.INSTANCE.onTexImage2D(target, level, internalformat, width, height, border, format, type, pixels);
-        GL11.glTexImage2D(target, level, internalformat, width, height, border, format, type, pixels);
-    }
-
-    public static void glTexImage2D(int target, int level, int internalformat, int width, int height, int border, int format, int type, DoubleBuffer pixels) {
-        TextureInfoCache.INSTANCE.onTexImage2D(target, level, internalformat, width, height, border, format, type, pixels);
-        GL11.glTexImage2D(target, level, internalformat, width, height, border, format, type, pixels);
-    }
-
     public static void glTexImage2D(int target, int level, int internalformat, int width, int height, int border, int format, int type, ByteBuffer pixels) {
         TextureInfoCache.INSTANCE.onTexImage2D(target, level, internalformat, width, height, border, format, type, pixels != null ? pixels.asIntBuffer() : (IntBuffer) null);
         GL11.glTexImage2D(target, level, internalformat, width, height, border, format, type, pixels);
@@ -1004,50 +960,30 @@ public class GLStateManager {
         glTexParameteri(GL11.GL_TEXTURE_2D, GL11.GL_TEXTURE_MAG_FILTER, j);
     }
 
-    public static void trySyncProgram() {
-        if (AngelicaConfig.enableIris) {
+    public static void glDrawArrays(int mode, int first, int count) {
+        // Iris -- TODO: This doesn't seem to work and is related to matchPass()
+        if(AngelicaConfig.enableIris) {
             Iris.getPipelineManager().getPipeline().ifPresent(WorldRenderingPipeline::syncProgram);
         }
-    }
-
-    public static void glBegin(int mode) {
-        trySyncProgram();
-        GL11.glBegin(mode);
-    }
-
-    public static void glDrawElements(int mode, ByteBuffer indices) {
-        trySyncProgram();
-        GL11.glDrawElements(mode, indices);
-    }
-
-    public static void glDrawElements(int mode, IntBuffer indices) {
-        trySyncProgram();
-        GL11.glDrawElements(mode, indices);
-    }
-
-    public static void glDrawElements(int mode, ShortBuffer indices) {
-        trySyncProgram();
-        GL11.glDrawElements(mode, indices);
-    }
-
-    public static void glDrawElements(int mode, int indices_count, int type, long indices_buffer_offset) {
-        trySyncProgram();
-        GL11.glDrawElements(mode, indices_count, type, indices_buffer_offset);
-    }
-
-    public static void glDrawElements(int mode, int count, int type, ByteBuffer indices) {
-        trySyncProgram();
-        GL11.glDrawElements(mode, count, type, indices);
+        GL11.glDrawArrays(mode, first, count);
     }
 
     public static void glDrawBuffer(int mode) {
-        trySyncProgram();
         GL11.glDrawBuffer(mode);
     }
 
-    public static void glDrawArrays(int mode, int first, int count) {
-        trySyncProgram();
-        GL11.glDrawArrays(mode, first, count);
+    public static void glDrawElements(int mode, IntBuffer indices) {
+        if(AngelicaConfig.enableIris) {
+            Iris.getPipelineManager().getPipeline().ifPresent(WorldRenderingPipeline::syncProgram);
+        }
+        GL11.glDrawElements(mode, indices);
+    }
+
+    public static void glBegin(int mode) {
+        if (AngelicaConfig.enableIris) {
+            Iris.getPipelineManager().getPipeline().ifPresent(WorldRenderingPipeline::syncProgram);
+        }
+        GL11.glBegin(mode);
     }
 
     public static void glLogicOp(int opcode) {
@@ -1255,21 +1191,13 @@ public class GLStateManager {
             glListNesting += 1;
             return;
         }
-
         glListId = list;
         glListMode = mode;
         GL11.glNewList(list, mode);
-
-        List<IStateStack<?>> stacks = Feature.maskToFeatures(GL11.GL_ALL_ATTRIB_BITS);
-        int size = stacks.size();
-
-        for(int i = 0; i < size; i++) {
-            IStateStack<?> stack = stacks.get(i);
-
+        for(IStateStack<?> stack : Feature.maskToFeatures(GL11.GL_ALL_ATTRIB_BITS)) {
             // Feature Stack, copy of current feature state
             glListStates.put(stack, (ISettableState<?>) ((ISettableState<?>)stack).copy());
         }
-
         if(glListMode == GL11.GL_COMPILE) {
             pushState(GL11.GL_ALL_ATTRIB_BITS);
         }
@@ -1312,7 +1240,6 @@ public class GLStateManager {
         if(list < 0) {
             VBOManager.get(list).render();
         } else {
-            trySyncProgram();
             GL11.glCallList(list);
             if(glListChanges.containsKey(list)) {
                 for(Map.Entry<IStateStack<?>, ISettableState<?>> entry : glListChanges.get(list)) {
@@ -1325,23 +1252,15 @@ public class GLStateManager {
 
     public static void pushState(int mask) {
         attribs.push(mask);
-
-        List<IStateStack<?>> stacks = Feature.maskToFeatures(mask);
-        int size = stacks.size();
-
-        for(int i = 0; i < size; i++) {
-            stacks.get(i).push();
+        for(IStateStack<?> stack : Feature.maskToFeatures(mask)) {
+            stack.push();
         }
-    }
 
+    }
     public static void popState() {
         final int mask = attribs.popInt();
-
-        List<IStateStack<?>> stacks = Feature.maskToFeatures(mask);
-        int size = stacks.size();
-
-        for(int i = 0; i < size; i++) {
-            stacks.get(i).pop();
+        for(IStateStack<?> stack : Feature.maskToFeatures(mask)) {
+            stack.pop();
         }
     }
 

src/main/java/com/gtnewhorizons/angelica/glsm/RenderSystem.java

@@ -30,6 +30,7 @@ import java.nio.IntBuffer;
 public class RenderSystem {
     private static final Logger LOGGER = LogManager.getLogger("RenderSystem");
 	private static DSAAccess dsaState;
+	private static boolean hasMultibind;
 	private static boolean supportsCompute;
 
     private RenderSystem() {}
@@ -40,6 +41,7 @@ public class RenderSystem {
                 dsaState = new DSACore();
                 LOGGER.info("OpenGL 4.5 detected, enabling DSA.");
             }
+            hasMultibind = GLStateManager.capabilities.OpenGL45;
 
         } catch (NoSuchFieldError ignored) {}
         try {
@@ -53,6 +55,10 @@ public class RenderSystem {
             LOGGER.info("No DSA support detected, falling back to legacy OpenGL.");
         }
 
+        try {
+            hasMultibind |= GLStateManager.capabilities.GL_ARB_multi_bind;
+        } catch (NoSuchFieldError ignored) {}
+
 		supportsCompute = supportsCompute();
 	}
 
@@ -97,8 +103,6 @@ public class RenderSystem {
         GL20.glUniform3f(location, v0, v1, v2);
 	}
 
-    public static void uniform3i(int location, int v0, int v1, int v2) { GL20.glUniform3i(location, v0, v1, v2); }
-
 	public static void uniform4f(int location, float v0, float v1, float v2, float v3) {
         GL20.glUniform4f(location, v0, v1, v2, v3);
 	}

src/main/java/com/gtnewhorizons/angelica/glsm/states/ISettableState.java

@@ -2,7 +2,6 @@ package com.gtnewhorizons.angelica.glsm.states;
 
 public interface ISettableState<T> extends Cloneable {
     T set(T state);
-    @SuppressWarnings("unchecked")
     default void set(ISettableState<?> value) {
         set((T) value);
     }

src/main/java/com/gtnewhorizons/angelica/glsm/states/LightState.java

@@ -18,6 +18,7 @@ public class LightState implements ISettableState<LightState> {
     private static final Vector4i vector4i = new Vector4i();
     private static final Vector4f vector4f = new Vector4f();
     private static final Vector3i vector3i = new Vector3i();
+    private static final Vector3f vector3f = new Vector3f();
     private static final Matrix3f matrix3f = new Matrix3f();
 
     public int light;

src/main/java/com/gtnewhorizons/angelica/glsm/texture/TextureInfoCache.java

@@ -6,7 +6,7 @@ import it.unimi.dsi.fastutil.ints.Int2ObjectOpenHashMap;
 import org.jetbrains.annotations.Nullable;
 import org.lwjgl.opengl.GL11;
 
-import java.nio.Buffer;
+import java.nio.IntBuffer;
 
 public class TextureInfoCache {
     /**
@@ -25,7 +25,7 @@ public class TextureInfoCache {
 		return cache.computeIfAbsent(id, TextureInfo::new);
 	}
 
-	public void onTexImage2D(int target, int level, int internalformat, int width, int height, int border, int format, int type, @Nullable Buffer pixels) {
+	public void onTexImage2D(int target, int level, int internalformat, int width, int height, int border, int format, int type, @Nullable IntBuffer pixels) {
 		if (target == GL11.GL_TEXTURE_2D && level == 0) {
             final TextureInfo info = getInfo(GLStateManager.getBoundTexture());
             if(info == null) return;

src/main/java/com/gtnewhorizons/angelica/hudcaching/HUDCaching.java

@@ -1,14 +1,17 @@
 package com.gtnewhorizons.angelica.hudcaching;
 
+import java.util.ArrayList;
+import java.util.List;
+
 import com.gtnewhorizon.gtnhlib.client.renderer.TessellatorManager;
 import com.gtnewhorizons.angelica.compat.ModStatus;
-import com.gtnewhorizons.angelica.compat.holoinventory.HoloInventoryReflectionCompat;
 import com.gtnewhorizons.angelica.config.AngelicaConfig;
 import com.gtnewhorizons.angelica.mixins.interfaces.GuiIngameAccessor;
 import com.gtnewhorizons.angelica.mixins.interfaces.GuiIngameForgeAccessor;
 import com.gtnewhorizons.angelica.mixins.interfaces.RenderGameOverlayEventAccessor;
 import com.kentington.thaumichorizons.common.ThaumicHorizons;
 import cpw.mods.fml.common.eventhandler.EventPriority;
+import net.dries007.holoInventory.client.Renderer;
 import net.minecraftforge.client.event.RenderGameOverlayEvent;
 import net.minecraftforge.event.world.WorldEvent;
 import org.lwjgl.opengl.GL11;
@@ -31,7 +34,7 @@ import xaero.common.core.XaeroMinimapCore;
 
 import static com.gtnewhorizons.angelica.loading.AngelicaTweaker.LOGGER;
 
-// See LICENSE-HUDCaching.md for license information.
+// See LICENSE+HUDCaching.md for license information.
 
 public class HUDCaching {
 
@@ -56,10 +59,11 @@ public class HUDCaching {
     public static boolean renderCrosshairsCaptured;
 
     private final static RenderGameOverlayEvent fakeTextEvent = new RenderGameOverlayEvent.Text(new RenderGameOverlayEvent(0, null, 0, 0), null, null);
-    private final static RenderGameOverlayEvent.Post fakePostEvent = new RenderGameOverlayEvent.Post(new RenderGameOverlayEvent(0, null, 0, 0), RenderGameOverlayEvent.ElementType.HELMET);
+    private final static RenderGameOverlayEvent fakePreEvent = new RenderGameOverlayEvent.Pre(new RenderGameOverlayEvent(0, null, 0, 0), RenderGameOverlayEvent.ElementType.HELMET);
 
     public static final HUDCaching INSTANCE = new HUDCaching();
 
+
     private HUDCaching() {}
 
     // highest so it runs before the GLSM load event
@@ -98,7 +102,7 @@ public class HUDCaching {
             renderingCacheOverride = false;
             mc.getFramebuffer().bindFramebuffer(false);
         } else {
-            renderer.setupOverlayRendering();
+        	renderer.setupOverlayRendering();
         }
 
         ScaledResolution resolution = new ScaledResolution(mc, mc.displayWidth, mc.displayHeight);
@@ -111,50 +115,48 @@ public class HUDCaching {
 
         // render bits that were captured when rendering into cache
         GuiIngameAccessor gui = (GuiIngameAccessor) ingame;
-        if (renderVignetteCaptured) {
+        if (renderVignetteCaptured)
+        {
             gui.callRenderVignette(mc.thePlayer.getBrightness(partialTicks), width, height);
         } else {
-            GLStateManager.tryBlendFuncSeparate(GL11.GL_SRC_ALPHA, GL11.GL_ONE_MINUS_SRC_ALPHA, 1, 0);
+        	GLStateManager.tryBlendFuncSeparate(GL11.GL_SRC_ALPHA, GL11.GL_ONE_MINUS_SRC_ALPHA, 1, 0);
         }
-
         if (ingame instanceof GuiIngameForge) {
-            GuiIngameForgeAccessor guiForge = ((GuiIngameForgeAccessor) ingame);
-            if (renderHelmetCaptured) {
-                guiForge.callRenderHelmet(resolution, partialTicks, hasScreen, mouseX, mouseY);
-                if (ModStatus.isHoloInventoryLoaded) {
-                    HoloInventoryReflectionCompat.setAngelicaOverride(false);
+        	GuiIngameForgeAccessor guiForge = ((GuiIngameForgeAccessor) ingame);
+        	if (renderHelmetCaptured) {
+        		guiForge.callRenderHelmet(resolution, partialTicks, hasScreen, mouseX, mouseY);
+                if (ModStatus.isHoloInventoryLoaded){
+                    Renderer.INSTANCE.angelicaOverride = false;
                     // only settings the partial ticks as mouseX and mouseY are not used in renderEvent
-                    ((RenderGameOverlayEventAccessor) fakePostEvent).setPartialTicks(partialTicks);
-                    HoloInventoryReflectionCompat.renderEvent(fakePostEvent);
+                    ((RenderGameOverlayEventAccessor) fakePreEvent).setPartialTicks(partialTicks);
+                    Renderer.INSTANCE.renderEvent(fakePreEvent);
                 }
-            }
-            if (renderPortalCapturedTicks > 0) {
-                guiForge.callRenderPortal(width, height, partialTicks);
-            }
-            if (renderCrosshairsCaptured) {
-                if (ModStatus.isXaerosMinimapLoaded) {
-                    // this fixes the crosshair going invisible when no lines are being drawn under the minimap
-                    GLStateManager.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
-                }
-                guiForge.callRenderCrosshairs(width, height);
-            }
-            if (ModStatus.isThaumcraftLoaded || ModStatus.isThaumicHorizonsLoaded) {
+        	}
+        	if (renderPortalCapturedTicks > 0) {
+        		guiForge.callRenderPortal(width, height, partialTicks);
+        	}
+        	if (renderCrosshairsCaptured) {
+        		guiForge.callRenderCrosshairs(width, height);
+        	}
+            if (ModStatus.isThaumcraftLoaded || ModStatus.isThaumicHorizonsLoaded){
                 ((RenderGameOverlayEventAccessor) fakeTextEvent).setPartialTicks(partialTicks);
                 ((RenderGameOverlayEventAccessor) fakeTextEvent).setResolution(resolution);
                 ((RenderGameOverlayEventAccessor) fakeTextEvent).setMouseX(mouseX);
                 ((RenderGameOverlayEventAccessor) fakeTextEvent).setMouseY(mouseY);
-                if (ModStatus.isThaumcraftLoaded) {
+                if (ModStatus.isThaumcraftLoaded){
                     Thaumcraft.instance.renderEventHandler.renderOverlay(fakeTextEvent);
                 }
-                if (ModStatus.isThaumicHorizonsLoaded) {
+                if (ModStatus.isThaumicHorizonsLoaded){
                     ThaumicHorizons.instance.renderEventHandler.renderOverlay(fakeTextEvent);
                 }
             }
         } else {
-            if (renderHelmetCaptured) {
+            if (renderHelmetCaptured)
+            {
                 gui.callRenderPumpkinBlur(width, height);
             }
-            if (renderPortalCapturedTicks > 0) {
+            if (renderPortalCapturedTicks > 0)
+            {
                 gui.callRenderPortal(renderPortalCapturedTicks, width, height);
             }
         }
@@ -177,19 +179,19 @@ public class HUDCaching {
      * the state before we start rendering
      */
     public static void fixGLStateBeforeRenderingCache() {
-        GLStateManager.glDepthMask(true);
-        GLStateManager.enableDepthTest();
-        GLStateManager.enableAlphaTest();
-        GLStateManager.tryBlendFuncSeparate(GL11.GL_SRC_ALPHA, GL11.GL_ONE_MINUS_SRC_ALPHA, GL11.GL_ONE, GL11.GL_ZERO);
-        GLStateManager.disableBlend();
+    	GLStateManager.glDepthMask(true);
+    	GLStateManager.enableDepthTest();
+    	GLStateManager.enableAlphaTest();
+    	GLStateManager.tryBlendFuncSeparate(GL11.GL_SRC_ALPHA, GL11.GL_ONE_MINUS_SRC_ALPHA, GL11.GL_ONE, GL11.GL_ZERO);
+    	GLStateManager.disableBlend();
     }
 
     private static void resetFramebuffer(int width, int height) {
         if (framebuffer.framebufferWidth != width || framebuffer.framebufferHeight != height) {
-            framebuffer.createBindFramebuffer(width, height);
+        	framebuffer.createBindFramebuffer(width, height);
             framebuffer.setFramebufferFilter(GL11.GL_NEAREST);
         } else {
-            framebuffer.framebufferClear();
+        	framebuffer.framebufferClear();
         }
         // copy depth buffer from MC
         OpenGlHelper.func_153171_g(GL30.GL_READ_FRAMEBUFFER, mc.getFramebuffer().framebufferObject);
@@ -211,9 +213,10 @@ public class HUDCaching {
         GL11.glTexParameteri(GL11.GL_TEXTURE_2D, GL11.GL_TEXTURE_MAG_FILTER, GL11.GL_NEAREST);
     }
 
+    // moved to here due to the method being called from a mixin
     public static void disableHoloInventory() {
         if (ModStatus.isHoloInventoryLoaded) {
-            HoloInventoryReflectionCompat.setAngelicaOverride(true);
+            Renderer.INSTANCE.angelicaOverride = true;
         }
     }
 

src/main/java/com/gtnewhorizons/angelica/loading/AngelicaTweaker.java

@@ -1,5 +1,6 @@
 package com.gtnewhorizons.angelica.loading;
 
+import com.google.common.collect.ImmutableMap;
 import com.gtnewhorizon.gtnhlib.asm.ASMUtil;
 import com.gtnewhorizon.gtnhlib.config.ConfigException;
 import com.gtnewhorizon.gtnhlib.config.ConfigurationManager;
@@ -152,5 +153,18 @@ public class AngelicaTweaker implements IFMLLoadingPlugin, IEarlyMixinLoader {
         }
     }
 
+    private static final ImmutableMap<String, TargetedMod> MODS_BY_CLASS = ImmutableMap.<String, TargetedMod>builder()
+        .put("optifine.OptiFineForgeTweaker", TargetedMod.OPTIFINE)
+        .put("fastcraft.Tweaker", TargetedMod.FASTCRAFT)
+        .put("cofh.asm.LoadingPlugin", TargetedMod.COFHCORE)
+        .build();
     public static final Set<TargetedMod> coreMods = new HashSet<>();
+
+    private static void detectCoreMods(Set<String> loadedCoreMods) {
+        MODS_BY_CLASS.forEach((key, value) -> {
+            if (loadedCoreMods.contains(key))
+                coreMods.add(value);
+        });
+    }
+
 }

src/main/java/com/gtnewhorizons/angelica/mixins/Mixins.java

@@ -80,19 +80,19 @@ public enum Mixins {
 
     ANGELICA_ITEM_DISPLAY_LIST_OPTIMIZATION(new Builder("Optimized item rendering by wrapping them with display lists")
         .setPhase(Phase.EARLY).addMixinClasses("angelica.itemrenderer.MixinItemRenderer").setSide(Side.CLIENT)
-        .setApplyIf(() -> AngelicaConfig.optimizeInWorldItemRendering_WIP)
+        .setApplyIf(() -> AngelicaConfig.optimizeInWorldItemRendering)
         .addTargetedMod(TargetedMod.VANILLA)),
 
     // Not compatible with the lwjgl debug callbacks, so disable if that's enabled
     ARCHAIC_SPLASH(new Builder("ArchaicFix Splash").addTargetedMod(TargetedMod.VANILLA).setSide(Side.CLIENT)
         .setPhase(Phase.EARLY).setApplyIf(() -> AngelicaConfig.showSplashMemoryBar && !AngelicaMod.lwjglDebug).addMixinClasses(
-              "angelica.archaic.MixinSplashProgress$3"
+              "angelica.archaic.MixinSplashProgress"
              ,"angelica.archaic.AccessorSplashProgress"
         )
     ),
 
-    ARCHAIC_CORE(new Builder("Archaic Core").addTargetedMod(TargetedMod.VANILLA).addExcludedMod(TargetedMod.ARCHAICFIX)
-        .setSide(Side.CLIENT).setPhase(Phase.EARLY).addMixinClasses(
+    ARCHAIC_CORE(new Builder("Archaic Core").addTargetedMod(TargetedMod.VANILLA).setSide(Side.CLIENT)
+        .setPhase(Phase.EARLY).addMixinClasses(
              "angelica.archaic.MixinBlockFence"
             ,"angelica.archaic.MixinFMLClientHandler"
             ,"angelica.archaic.MixinGuiIngameForge"
@@ -104,7 +104,7 @@ public enum Mixins {
     IRIS_STARTUP(new Builder("Start Iris").addTargetedMod(TargetedMod.VANILLA).setSide(Side.CLIENT)
         .setPhase(Phase.EARLY).setApplyIf(() -> AngelicaConfig.enableIris).addMixinClasses(
              "shaders.startup.MixinGameSettings"
-            ,"shaders.startup.MixinMinecraft"
+            ,"shaders.startup.MixinGuiMainMenu"
             ,"shaders.startup.MixinInitRenderer"
             ,"shaders.startup.MixinAbstractTexture"
             ,"shaders.startup.MixinTextureAtlasSprite"
@@ -124,17 +124,19 @@ public enum Mixins {
             ,"sodium.MixinEntity_RenderDist"
             ,"sodium.MixinEntityItem_RenderDist"
             ,"sodium.MixinRenderManager"
+            ,"sodium.MixinExtendedBlockStorage"
             ,"sodium.MixinEntityRenderer"
             ,"sodium.MixinFMLClientHandler"
             ,"sodium.MixinForgeHooksClient"
             ,"sodium.MixinGameSettings"
             ,"sodium.MixinFrustrum"
+            ,"sodium.MixinMaterial"
             ,"sodium.MixinMinecraft"
+            ,"sodium.MixinNibbleArray"
             ,"sodium.MixinRenderBlocks"
             ,"sodium.MixinRenderGlobal"
             ,"sodium.MixinWorldClient"
             ,"sodium.MixinTileEntity"
-            ,"sodium.MixinTileEntityMobSpawner"
             ,"sodium.MixinEffectRenderer"
             ,"sodium.MixinTileEntityRendererDispatcher"
             ,"sodium.MixinLongHashMap"
@@ -149,6 +151,7 @@ public enum Mixins {
             ,"shaders.MixinGuiIngameForge"
             ,"shaders.MixinFramebuffer"
             ,"shaders.MixinItem"
+            ,"shaders.MixinItemRenderer"
             ,"shaders.MixinLocale"
             ,"shaders.MixinOpenGlHelper"
             ,"shaders.MixinRender"
@@ -158,21 +161,15 @@ public enum Mixins {
         )
     ),
 
-    IRIS_RENDERING_NOBACKHAND(new Builder("Iris Hand Shaders").addTargetedMod(TargetedMod.VANILLA).setSide(Side.CLIENT)
-        .setPhase(Phase.EARLY).setApplyIf(() -> AngelicaConfig.enableIris).addExcludedMod(TargetedMod.BACKHAND).addMixinClasses(
-            "shaders.MixinItemRenderer"
-        )
-    ),
-
-    IRIS_RENDERING_BACKHAND(new Builder("Iris Hand Shaders (Backhand)").addTargetedMod(TargetedMod.VANILLA)
-        .addTargetedMod(TargetedMod.BACKHAND).setSide(Side.CLIENT)
-        .setPhase(Phase.EARLY).setApplyIf(() -> AngelicaConfig.enableIris).addMixinClasses(
-            "shaders.MixinItemRendererBackhand"
-        )
-    ),
-
     IRIS_ACCESSORS(new Builder("Iris Accessors").addTargetedMod(TargetedMod.VANILLA).setSide(Side.CLIENT)
-        .setPhase(Phase.EARLY).setApplyIf(() -> AngelicaConfig.enableIris).addMixinClasses("shaders.accessors.MixinEntityRenderer")
+        .setPhase(Phase.EARLY).setApplyIf(() -> AngelicaConfig.enableIris).addMixinClasses(
+             "shaders.accessors.MixinMinecraft"
+            ,"shaders.accessors.MixinEntityRenderer"
+            ,"shaders.accessors.MixinSimpleTexture"
+            ,"shaders.accessors.MixinTextureAtlasSprite"
+            ,"shaders.accessors.MixinTextureMap"
+            ,"shaders.accessors.MixinAnimationMetadataSection"
+        )
     ),
 
     ANGELICA_TEXTURE(new Builder("Textures").addTargetedMod(TargetedMod.VANILLA).setSide(Side.CLIENT)
@@ -221,19 +218,6 @@ public enum Mixins {
             "angelica.animation.MixinWorldRenderer",
             "angelica.animation.MixinRenderItem")),
 
-    SCALED_RESOUTION_UNICODE_FIX(new Builder("Removes unicode languages gui scaling being forced to even values").setPhase(Phase.EARLY)
-        .setSide(Side.CLIENT).addTargetedMod(TargetedMod.VANILLA)
-        .setApplyIf(() -> AngelicaConfig.removeUnicodeEvenScaling)
-        .addMixinClasses("angelica.bugfixes.MixinScaledResolution_UnicodeFix")),
-
-    SECURITYCRAFT_COMPAT(new Builder("Fix reflection in SecurityCraft for compat with Angelica").setPhase(Phase.LATE)
-        .addTargetedMod(TargetedMod.SECURITYCRAFT).setSide(Side.CLIENT)
-        .setApplyIf(() -> CompatConfig.fixSecurityCraft)
-        .addMixinClasses(
-            "client.securitycraft.MixinBlockReinforcedFenceGate",
-            "client.securitycraft.MixinBlockReinforcedIronBars"
-        )),
-
     EXTRA_UTILITIES_THREAD_SAFETY(new Builder("Enable thread safety fixes in Extra Utilities").setPhase(Phase.LATE)
         .addTargetedMod(TargetedMod.EXTRAUTILS).setSide(Side.CLIENT)
         .setApplyIf(() -> CompatConfig.fixExtraUtils)
@@ -535,7 +519,7 @@ public enum Mixins {
             && !MCPatcherForgeConfig.ConnectedTextures.enabled
             && MCPatcherForgeConfig.CustomColors.enabled)
         .addTargetedMod(TargetedMod.VANILLA)
-        .addMixinClasses("mcpatcherforge.ctm_cc.MixinRenderBlocksNoCTM")
+        .addMixinClasses("mcpatcherforge.cc_ctm.MixinRenderBlocksNoCTM")
     ),
     MCPATCHER_FORGE_CTM_NO_CC(new Builder("MCP:F Connected Textures, no Custom Colours")
         .setSide(Side.CLIENT).setPhase(Phase.EARLY)

src/main/java/com/gtnewhorizons/angelica/mixins/TargetedMod.java

@@ -8,7 +8,6 @@ public enum TargetedMod {
     , FASTCRAFT("FastCraft", "fastcraft.Tweaker")
     , OPTIFINE("Optifine", "optifine.OptiFineForgeTweaker", "Optifine")
     , ARCHAICFIX("ArchaicFix", "org.embeddedt.archaicfix.ArchaicCore", "archaicfix")
-    , BACKHAND("Backhand","xonin.backhand.coremod.BackhandLoadingPlugin", "backhand")
     , BOTANIA("Botania", null, "Botania")
     , CAMPFIRE_BACKPORT("CampfireBackport", null, "campfirebackport")
     , CHICKENCHUNKS("ChickenChunks", null, "ChickenChunks")
@@ -25,7 +24,6 @@ public enum TargetedMod {
     , MRTJPCORE("MrTJPCore", null, "MrTJPCoreMod")
     , NATURA("Natura", null, "Natura")
     , PROJECTRED_ILLUMINATION("ProjectRed Illumination", null, "ProjRed|Illumination")
-    , SECURITYCRAFT("SecurityCraft", null, "securitycraft")
     , SMART_RENDER("Smart Render", null, "SmartRender")
     , THAUMCRAFT("Thaumcraft", null, "Thaumcraft")
     , TINKERS_CONSTRUCT("Tinker's Construct", null, "TConstruct")

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/AnimationMetadataSectionAccessor.java

@@ -0,0 +1,14 @@
+package com.gtnewhorizons.angelica.mixins.interfaces;
+
+import org.spongepowered.asm.mixin.Mutable;
+import org.spongepowered.asm.mixin.gen.Accessor;
+
+public interface AnimationMetadataSectionAccessor {
+    int getFrameWidth();
+
+    void setFrameWidth(int frameWidth);
+
+    int getFrameHeight();
+
+    void setFrameHeight(int frameHeight);
+}

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/ExtendedBlockStorageAccessor.java

@@ -0,0 +1,9 @@
+package com.gtnewhorizons.angelica.mixins.interfaces;
+
+import org.spongepowered.asm.mixin.gen.Accessor;
+
+public interface ExtendedBlockStorageAccessor {
+    int getYBase();
+    int getBlockRefCount();
+    void setBlockRefCount(int blockRefCount);
+}

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/ExtendedNibbleArray.java

@@ -0,0 +1,7 @@
+package com.gtnewhorizons.angelica.mixins.interfaces;
+
+public interface ExtendedNibbleArray {
+    public byte[] getData();
+    public int getDepthBits();
+    public int getDepthBitsPlusFour();
+}

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/ITexturesCache.java

@@ -8,5 +8,4 @@ public interface ITexturesCache {
 
     Set<IIcon> getRenderedTextures();
     void enableTextureTracking();
-    void track(IPatchedTextureAtlasSprite sprite);
 }

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/MinecraftAccessor.java

@@ -0,0 +1,7 @@
+package com.gtnewhorizons.angelica.mixins.interfaces;
+
+import net.minecraft.util.Timer;
+
+public interface MinecraftAccessor {
+    Timer getTimer();
+}

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/SimpleTextureAccessor.java

@@ -0,0 +1,7 @@
+package com.gtnewhorizons.angelica.mixins.interfaces;
+
+import net.minecraft.util.ResourceLocation;
+
+public interface SimpleTextureAccessor {
+    ResourceLocation getLocation();
+}

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/TextureAtlasSpriteAccessor.java

@@ -0,0 +1,21 @@
+package com.gtnewhorizons.angelica.mixins.interfaces;
+
+import net.minecraft.client.resources.data.AnimationMetadataSection;
+import org.spongepowered.asm.mixin.gen.Accessor;
+
+import java.util.List;
+
+public interface TextureAtlasSpriteAccessor {
+    AnimationMetadataSection getMetadata();
+
+    List<int[][]> getFramesTextureData();
+
+    int getFrame();
+
+    void setFrame(int frame);
+
+    int getSubFrame();
+
+    void setSubFrame(int subFrame);
+
+}

src/main/java/com/gtnewhorizons/angelica/mixins/interfaces/TextureMapAccessor.java

@@ -0,0 +1,16 @@
+package com.gtnewhorizons.angelica.mixins.interfaces;
+
+import net.minecraft.util.ResourceLocation;
+import org.spongepowered.asm.mixin.gen.Accessor;
+
+import java.util.Map;
+
+public interface TextureMapAccessor {
+    Map getMapUploadedSprites();
+
+    int getAnisotropicFiltering();
+
+    int getMipmapLevels();
+
+    ResourceLocation getLocationBlocksTexture();
+}

src/main/java/com/gtnewhorizons/angelica/models/VanillaModels.java

@@ -1,5 +1,6 @@
 package com.gtnewhorizons.angelica.models;
 
+import com.gtnewhorizon.gtnhlib.blockpos.BlockPos;
 import com.gtnewhorizon.gtnhlib.blockpos.IBlockPos;
 import com.gtnewhorizon.gtnhlib.client.model.ModelLoader;
 import com.gtnewhorizon.gtnhlib.client.model.Variant;

src/main/java/com/gtnewhorizons/angelica/proxy/ClientProxy.java

@@ -47,7 +47,6 @@ import net.minecraft.client.gui.Gui;
 import net.minecraft.client.gui.GuiMainMenu;
 import net.minecraft.client.gui.ScaledResolution;
 import net.minecraft.client.network.NetHandlerPlayClient;
-import net.minecraft.client.settings.GameSettings;
 import net.minecraft.client.settings.KeyBinding;
 import net.minecraft.server.integrated.IntegratedServer;
 import net.minecraft.util.Direction;
@@ -153,7 +152,7 @@ public class ClientProxy extends CommonProxy {
 
     @SubscribeEvent
     public void onKeypress(TickEvent.ClientTickEvent event) {
-        final boolean isPressed = glsmKeyBinding.getKeyCode() != 0 && GameSettings.isKeyDown(glsmKeyBinding);
+        final boolean isPressed = glsmKeyBinding.getKeyCode() != 0 && Keyboard.isKeyDown(glsmKeyBinding.getKeyCode());
         if (isPressed && !wasGLSMKeyPressed) {
             OpenGLDebugging.checkGLSM();
         }

src/main/java/com/gtnewhorizons/angelica/rendering/RenderingState.java

@@ -1,10 +1,12 @@
 package com.gtnewhorizons.angelica.rendering;
 
+import com.gtnewhorizons.angelica.glsm.GLStateManager;
 import lombok.Getter;
 import lombok.Setter;
 import org.joml.Matrix4f;
 import org.joml.Vector3d;
 import org.lwjgl.BufferUtils;
+import org.lwjgl.opengl.GL11;
 
 import java.nio.FloatBuffer;
 

src/main/java/com/gtnewhorizons/angelica/transform/RedirectorTransformer.java

@@ -76,6 +76,7 @@ public class RedirectorTransformer implements IClassTransformer {
         "com.gtnewhorizons.angelica.transform",
         "me.eigenraven.lwjgl3ify"
     );
+    private static int remaps = 0;
 
     private static final Set<String> moddedBlockSubclasses = Collections.newSetFromMap(new ConcurrentHashMap<>());
     // Block owners we *shouldn't* redirect because they shadow one of our fields
@@ -347,6 +348,7 @@ public class RedirectorTransformer implements IClassTransformer {
                         }
                         changed = true;
                         redirectInMethod = true;
+                        remaps++;
                     } else if (mNode.owner.startsWith(Drawable) && mNode.name.equals("makeCurrent")) {
                         mNode.setOpcode(Opcodes.INVOKESTATIC);
                         mNode.owner = GLStateManager;
@@ -367,6 +369,7 @@ public class RedirectorTransformer implements IClassTransformer {
                             mNode.name = redirects.get(mNode.name);
                             changed = true;
                             redirectInMethod = true;
+                            remaps++;
                         }
                     }
                 }
@@ -420,9 +423,6 @@ public class RedirectorTransformer implements IClassTransformer {
     }
 
     private static class RedirectMap<K> extends HashMap<K, K> {
-
-        private static final long serialVersionUID = 1712218575345511543L;
-
         public static RedirectMap<String> newMap() {
             return new RedirectMap<>();
         }

src/main/java/com/gtnewhorizons/angelica/transform/compat/handlers/CompatHandlers.java

@@ -1,6 +1,7 @@
 package com.gtnewhorizons.angelica.transform.compat.handlers;
 
 import com.gtnewhorizons.angelica.config.CompatConfig;
+import lombok.Getter;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/com/gtnewhorizons/angelica/utils/AnimationsRenderUtils.java

@@ -1,7 +1,5 @@
 package com.gtnewhorizons.angelica.utils;
 
-import java.util.ArrayDeque;
-
 import com.gtnewhorizons.angelica.mixins.interfaces.IPatchedTextureAtlasSprite;
 import com.gtnewhorizons.angelica.mixins.interfaces.ITexturesCache;
 import net.minecraft.client.Minecraft;
@@ -30,26 +28,4 @@ public class AnimationsRenderUtils {
             }
         }
     }
-
-    private final static ThreadLocal<ArrayDeque<ITexturesCache>> TEXTURE_CACHE_STACK = ThreadLocal.withInitial(ArrayDeque::new);
-
-    public static void onSpriteUsed(IPatchedTextureAtlasSprite sprite) {
-        ArrayDeque<ITexturesCache> stack = TEXTURE_CACHE_STACK.get();
-
-        if (stack == null || stack.isEmpty()) {
-            // icon was used outside of chunk building, it's probably an item in an inventory or something
-            sprite.markNeedsAnimationUpdate();
-            return;
-        }
-
-        stack.peek().track(sprite);
-    }
-
-    public static void pushCache(ITexturesCache cache) {
-        TEXTURE_CACHE_STACK.get().push(cache);
-    }
-
-    public static void popCache() {
-        TEXTURE_CACHE_STACK.get().pop();
-    }
 }

src/main/java/com/prupe/mcpatcher/MCLogger.java

@@ -172,8 +172,6 @@ public class MCLogger {
 
     private static class ErrorLevel extends Level {
 
-        private static final long serialVersionUID = 3987750897854233321L;
-
         protected ErrorLevel() {
             super("ERROR", (Level.WARNING.intValue() + Level.SEVERE.intValue()) / 2);
         }

src/main/java/com/prupe/mcpatcher/cit/CITUtils.java

@@ -221,7 +221,7 @@ public class CITUtils {
         List<T> list = overrides.get(item);
         if (list != null) {
             int[] enchantmentLevels = getEnchantmentLevels(item, itemStack.getTagCompound());
-            boolean hasEffect = itemStack.hasEffect(0);
+            boolean hasEffect = itemStack.hasEffect();
             for (T override : list) {
                 if (override.match(itemStack, enchantmentLevels, hasEffect)) {
                     return override;

src/main/java/com/prupe/mcpatcher/cit/EnchantmentList.java

@@ -49,7 +49,7 @@ final class EnchantmentList {
         Map<Integer, Layer> tmpLayers = new HashMap<>();
         Item item = itemStack.getItem();
         int[] enchantmentLevels = CITUtils.getEnchantmentLevels(item, itemStack.getTagCompound());
-        boolean hasEffect = itemStack.hasEffect(0);
+        boolean hasEffect = itemStack.hasEffect();
         List<Enchantment> list = enchantments.get(item);
         if (list == null) {
             list = allItemEnchantments;

src/main/java/com/prupe/mcpatcher/ctm/TileOverride.java

@@ -32,6 +32,8 @@ abstract class TileOverride implements Comparable<TileOverride> {
 
     private static final MCLogger logger = MCLogger.getLogger(MCLogger.Category.CONNECTED_TEXTURES, "CTM");
 
+    private static final int META_MASK = 0xffff;
+
     private final PropertiesFile properties;
     private final String baseFilename;
     private final TileLoader tileLoader;
@@ -39,6 +41,7 @@ abstract class TileOverride implements Comparable<TileOverride> {
     private final int weight;
     private final List<BlockStateMatcher> matchBlocks;
     private final Set<String> matchTiles;
+    private final int defaultMetaMask;
     private final BlockFaceMatcher faceMatcher;
     private final int connectType;
     private final boolean innerSeams;
@@ -47,6 +50,7 @@ abstract class TileOverride implements Comparable<TileOverride> {
 
     private final List<ResourceLocation> tileNames = new ArrayList<>();
     protected IIcon[] icons;
+    private int matchMetadata = META_MASK;
 
     static TileOverride create(ResourceLocation propertiesFile, TileLoader tileLoader) {
         if (propertiesFile == null) {
@@ -129,6 +133,11 @@ abstract class TileOverride implements Comparable<TileOverride> {
         if (matchBlocks.isEmpty() && matchTiles.isEmpty()) {
             matchTiles.add(baseFilename);
         }
+        int bits = 0;
+        for (int i : properties.getIntList("metadata", 0, 15, "0-15")) {
+            bits |= 1 << i;
+        }
+        defaultMetaMask = bits;
 
         faceMatcher = BlockFaceMatcher.create(properties.getString("faces", ""));
 

src/main/java/com/prupe/mcpatcher/hd/FancyDial.java

@@ -170,7 +170,6 @@ public class FancyDial {
     static void registerAnimations() {
         ITextureObject texture = TexturePackAPI.getTextureObject(TexturePackAPI.ITEMS_PNG);
         if (texture instanceof TextureMap map) {
-            @SuppressWarnings("unchecked")
             List<TextureAtlasSprite> animations = map.listAnimatedSprites;
             for (FancyDial instance : instances.values()) {
                 instance.registerAnimation(animations);

src/main/java/com/prupe/mcpatcher/mal/biome/ColorMap.java

@@ -32,6 +32,7 @@ abstract public class ColorMap implements IColorMap {
     private static final int BIOME_HEIGHT = 2;
 
     private static final int COLORMAP_WIDTH = 256;
+    private static final int COLORMAP_HEIGHT = 256;
 
     public static final String BLOCK_COLORMAP_DIR = TexturePackAPI.MCPATCHER_SUBDIR + "colormap/blocks";
     public static final List<ResourceLocation> unusedPNGs = new ArrayList<>();

src/main/java/com/prupe/mcpatcher/mal/block/BlockAPI.java

@@ -10,6 +10,8 @@ import java.util.Map;
 import java.util.Set;
 
 import net.minecraft.block.Block;
+import net.minecraft.util.IIcon;
+import net.minecraft.world.IBlockAccess;
 
 import com.prupe.mcpatcher.MCPatcherUtils;
 import com.prupe.mcpatcher.mal.resource.PropertiesFile;
@@ -19,7 +21,6 @@ import cpw.mods.fml.common.registry.GameData;
 
 public class BlockAPI {
 
-    @SuppressWarnings("unchecked")
     BlockAPI(FMLControlledNamespacedRegistry<Block> registry) {
         File outputFile = new File("blocks17.txt");
         if (outputFile.isFile()) {

src/main/java/com/prupe/mcpatcher/mal/block/BlockStateMatcher.java

@@ -61,7 +61,6 @@ public class BlockStateMatcher {
         threadLocal.set(data);
     }
 
-    @Override
     final public String toString() {
         return fullString;
     }

src/main/java/com/prupe/mcpatcher/mal/item/ItemAPI.java

@@ -13,7 +13,6 @@ import cpw.mods.fml.common.registry.GameData;
 
 public class ItemAPI {
 
-    @SuppressWarnings("unchecked")
     ItemAPI() {
         File outputFile = new File("items17.txt");
         if (outputFile.isFile()) {

src/main/java/com/prupe/mcpatcher/mal/resource/PropertiesFile.java

@@ -120,7 +120,7 @@ final public class PropertiesFile {
 
     @SuppressWarnings("unchecked")
     public Set<Map.Entry<String, String>> entrySet() {
-        return (Set<Map.Entry<String, String>>) (Set<?>) properties.entrySet();
+        return (Set<Map.Entry<String, String>>) (Set) properties.entrySet();
     }
 
     public ResourceLocation getResource() {

src/main/java/com/prupe/mcpatcher/mal/resource/ResourceList.java

@@ -110,7 +110,6 @@ public class ResourceList {
                 MCPatcherUtils.close(zipFile);
             }
         }
-        @SuppressWarnings("unchecked")
         Map<String, File> map = resourcePack.field_152781_b; // map
         if (map != null) {
             for (Map.Entry<String, File> entry : map.entrySet()) {

src/main/java/com/prupe/mcpatcher/mal/resource/TexturePackAPI.java

@@ -51,7 +51,6 @@ public class TexturePackAPI {
         List<IResourcePack> resourcePacks = new ArrayList<>();
         IResourceManager resourceManager = Minecraft.getMinecraft().getResourceManager();
         if (resourceManager instanceof SimpleReloadableResourceManager) {
-            @SuppressWarnings("unchecked")
             Set<Map.Entry<String, FallbackResourceManager>> entrySet = ((SimpleReloadableResourceManager) resourceManager).domainResourceManagers
                 .entrySet();
             for (Map.Entry<String, FallbackResourceManager> entry : entrySet) {
@@ -67,7 +66,6 @@ public class TexturePackAPI {
         return resourcePacks;
     }
 
-    @SuppressWarnings("unchecked")
     public static Set<String> getNamespaces() {
         Set<String> namespaces = new HashSet<>();
         namespaces.add(DEFAULT_NAMESPACE);
@@ -312,7 +310,6 @@ public class TexturePackAPI {
         TextureManager textureManager = Minecraft.getMinecraft().getTextureManager();
         if (textureManager != null) {
             Set<ResourceLocation> texturesToUnload = new HashSet<>();
-            @SuppressWarnings("unchecked")
             Set<Map.Entry<ResourceLocation, ITextureObject>> entrySet = textureManager.mapTextureObjects.entrySet();
             for (Map.Entry<ResourceLocation, ITextureObject> entry : entrySet) {
                 ResourceLocation resource = entry.getKey();

src/main/java/jss/notfine/asm/mappings/NamerMcp.java

@@ -13,7 +13,6 @@ import jss.notfine.asm.mappings.Names.Meth;
 
 public class NamerMcp extends NamerSrg {
 
-    @Override
     public void setNames() {
         setNamesSrg();
         final String confPath = System.getProperty("net.minecraftforge.gradle.GradleStart.csvDir", "../conf") + "/";

src/main/java/jss/notfine/config/MCPatcherForgeConfig.java

@@ -11,7 +11,7 @@ public class MCPatcherForgeConfig {
     @Config(modid = "mcpatcherforge", category = "custom_colors")
     public static class CustomColors {
         @Config.Comment("Enable the custom colors module")
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean enabled;
 
@@ -32,31 +32,31 @@ public class MCPatcherForgeConfig {
         @Config.RequiresMcRestart
         public static int fogBlendRadius;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean swampColors;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean water;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean tree;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean redstone;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean stem;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean otherBlocks;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean smoothBiomes;
 
@@ -64,43 +64,43 @@ public class MCPatcherForgeConfig {
         @Config.RequiresMcRestart
         public static boolean testColorSmoothing;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean potion;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean particle;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean fog;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean clouds;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean map;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean dye;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean text;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean xporb;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean egg;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean lightmaps;
     }
@@ -108,7 +108,7 @@ public class MCPatcherForgeConfig {
     @Config(modid = "mcpatcherforge", category = "custom_item_textures")
     public static class CustomItemTextures {
         @Config.Comment("Enable the custom item textures module")
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean enabled;
 
@@ -117,15 +117,15 @@ public class MCPatcherForgeConfig {
         @Config.RequiresMcRestart
         public static LogLevel logging;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean items;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean enchantments;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean armor;
     }
@@ -164,7 +164,7 @@ public class MCPatcherForgeConfig {
     @Config(modid = "mcpatcherforge", category = "extended_hd")
     public static class ExtendedHD {
         @Config.Comment("Enable the extended hd module")
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean enabled;
 
@@ -186,27 +186,27 @@ public class MCPatcherForgeConfig {
         @Config.Name("lod bias")
         public static int lodBias;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean animations;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean fancyCompass;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean fancyClock;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean useGL13;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean useScratchTexture;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         @Config.Name("HDFont")
         public static boolean hdFont;
@@ -223,7 +223,7 @@ public class MCPatcherForgeConfig {
     @Config(modid = "mcpatcherforge", category = "random_mobs")
     public static class RandomMobs {
         @Config.Comment("Enable the random mobs module")
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean enabled;
 
@@ -232,7 +232,7 @@ public class MCPatcherForgeConfig {
         @Config.RequiresMcRestart
         public static LogLevel logging;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean leashLine;
     }
@@ -240,7 +240,7 @@ public class MCPatcherForgeConfig {
     @Config(modid = "mcpatcherforge", category = "better_skies")
     public static class BetterSkies {
         @Config.Comment("Enable the better skies module")
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean enabled;
 
@@ -253,15 +253,15 @@ public class MCPatcherForgeConfig {
         @Config.RequiresMcRestart
         public static int horizon;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean brightenFireworks;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean skybox;
 
-        @Config.DefaultBoolean(false)
+        @Config.DefaultBoolean(true)
         @Config.RequiresMcRestart
         public static boolean unloadTextures;
     }

src/main/java/jss/notfine/config/VideoSettings.java

@@ -9,8 +9,6 @@ import java.io.File;
 import java.io.FileReader;
 import java.io.FileWriter;
 import java.io.PrintWriter;
-import java.util.EnumSet;
-import java.util.Set;
 
 public class VideoSettings {
     private final File optionsFile;
@@ -21,12 +19,11 @@ public class VideoSettings {
 
     public void loadSettings() {
         try {
-            if (!optionsFile.exists() && !optionsFile.createNewFile()) {
+            if (!optionsFile.exists()) {
                 return;
             }
             BufferedReader bufferedreader = new BufferedReader(new FileReader(optionsFile));
             String settingString;
-            Set<Settings> loadedSettings = EnumSet.noneOf(Settings.class);
 
             while((settingString = bufferedreader.readLine()) != null) {
                 try {
@@ -34,19 +31,12 @@ public class VideoSettings {
                     Settings setting = Settings.valueOf(fragments[0]);
                     setting.option.deserialize(fragments[1]);
                     setting.applyChanges();
-                    loadedSettings.add(setting);
                 } catch (Exception exception) {
                     NotFine.logger.warn("Skipping bad option: " + settingString);
                 }
 
             }
             bufferedreader.close();
-
-            for (Settings setting : Settings.values()) {
-                if (!loadedSettings.contains(setting)) {
-                    setting.applyChanges();
-                }
-            }
         } catch (Exception exception) {
             NotFine.logger.error("Failed to load options", exception);
         }

src/main/java/jss/notfine/core/Settings.java

@@ -48,7 +48,7 @@ public enum Settings {
         }
     },
     DYNAMIC_FOV(new NotFineOptionTickBox(true, null)),
-    DYNAMIC_LIGHTS(new NotFineOptionCycling<>(DynamicLightsMode.FANCY, OptionImpact.VARIES)){
+    DYNAMIC_LIGHTS(new NotFineOptionCycling(DynamicLightsMode.FANCY, OptionImpact.VARIES)){
         @Override
         public void applyChanges() {
             DynamicLights.Mode = (DynamicLightsMode) this.option.getStore();
@@ -60,7 +60,6 @@ public enum Settings {
             DynamicLights.ShaderForce = (boolean) this.option.getStore();
         }
     },
-    FOG_DISABLE(new NotFineOptionTickBox(false, OptionImpact.LOW)),
     FOG_NEAR_DISTANCE(new NotFineOptionSliderPercentage(75, 1, 100, 1, OptionImpact.LOW)),
     GUI_BACKGROUND(new NotFineOptionCycling<>(BackgroundSelect.DEFAULT, null)) {
         @Override
@@ -133,9 +132,9 @@ public enum Settings {
     },
     VOID_FOG(new NotFineOptionTickBox(false, OptionImpact.LOW));
 
-    public final NotFineOption<?> option;
+    public final NotFineOption option;
 
-    Settings(NotFineOption<?> option) {
+    Settings(NotFineOption option) {
         this.option = option;
     }
 

src/main/java/jss/notfine/gui/GuiCustomOptionsRowList.java

@@ -26,10 +26,10 @@ public class GuiCustomOptionsRowList extends GuiListExtended {
         field_148163_i = false;
 
         for(OptionGroup optionGroup : optionPage.getGroups()) {
-            Iterator<Option<?>> settings = optionGroup.getOptions().stream().iterator();
+            Iterator settings = optionGroup.getOptions().stream().iterator();
             while(settings.hasNext()) {
-                Option<?> optionOne = settings.next();
-                Option<?> optionTwo = settings.hasNext() ? settings.next() : null;
+                Option optionOne = (Option)settings.next();
+                Option optionTwo = settings.hasNext() ? (Option)settings.next() : null;
                 GuiButton buttonOne = (GuiButton)optionOne.getControl().createElement(new Dim2i(width / 2 - 155,0,150,20), factory);
                 GuiButton buttonTwo = optionTwo == null ? null : (GuiButton)optionTwo.getControl().createElement(new Dim2i(width / 2 - 155 + 160,0,150,20), factory);
                 settingsList.add(new GuiCustomOptionsRowList.Row(buttonOne, buttonTwo));

src/main/java/jss/notfine/gui/NotFineGameOptionPages.java

@@ -213,7 +213,6 @@ public class NotFineGameOptionPages {
             .add(Settings.MODE_CLOUD_TRANSLUCENCY.option)
             .add(Settings.MODE_STARS.option)
             .add(Settings.TOTAL_STARS.option)
-            .add(Settings.FOG_DISABLE.option)
             .add(Settings.FOG_NEAR_DISTANCE.option)
         .build());
         return new OptionPage(I18n.format("options.button.sky"), ImmutableList.copyOf(groups));

src/main/java/jss/notfine/gui/options/control/element/CyclingControlElement.java

@@ -22,7 +22,6 @@ public class CyclingControlElement<T extends Enum<T>> extends NotFineControlElem
         }
     }
 
-    @Override
     public String getLabel() {
         Enum<T> value = option.getValue();
         return super.getLabel() + names[value.ordinal()];

src/main/java/jss/notfine/gui/options/control/element/NotFineControlElementFactory.java

@@ -9,17 +9,17 @@ import me.jellysquid.mods.sodium.client.util.Dim2i;
 public class NotFineControlElementFactory implements ControlElementFactory {
 
     @Override
-    public <T extends Enum<T>> ControlElement<T> cyclingControlElement(Option<T> option, Dim2i dim, T[] allowedValues, String[] names) {
-        return new CyclingControlElement<>(option, dim, allowedValues, names);
+    public <T extends Enum<T>> ControlElement cyclingControlElement(Option<T> option, Dim2i dim, T[] allowedValues, String[] names) {
+        return new CyclingControlElement(option, dim, allowedValues, names);
     }
 
     @Override
-    public ControlElement<Integer> sliderControlElement(Option<Integer> option, Dim2i dim, int min, int max, int interval, ControlValueFormatter formatter) {
+    public ControlElement sliderControlElement(Option<Integer> option, Dim2i dim, int min, int max, int interval, ControlValueFormatter formatter) {
         return new SliderControlElement(option, dim, min, max, interval, formatter);
     }
 
     @Override
-    public  ControlElement<Boolean> tickBoxElement(Option<Boolean> option, Dim2i dim) {
+    public  ControlElement tickBoxElement(Option<Boolean> option, Dim2i dim) {
         return new TickBoxControlElement(option, dim);
     }
 

src/main/java/jss/notfine/gui/options/control/element/SliderControlElement.java

@@ -26,7 +26,6 @@ public class SliderControlElement extends NotFineControlElement<Integer> {
         value = MathHelper.clamp_float((value - min) / (max - min), 0f, 1f);
     }
 
-    @Override
     public String getLabel() {
         return super.getLabel() + formatter.format(option.getValue());
     }

src/main/java/jss/notfine/gui/options/control/element/TickBoxControlElement.java

@@ -10,7 +10,6 @@ public class TickBoxControlElement extends NotFineControlElement<Boolean> {
         super(option, dim);
     }
 
-    @Override
     public String getLabel() {
         return super.getLabel() + I18n.format(option.getValue() ? "options.on" : "options.off");
     }

src/main/java/jss/notfine/proxy/ClientProxy.java

@@ -1,23 +1,19 @@
 package jss.notfine.proxy;
 
-import cpw.mods.fml.common.FMLCommonHandler;
+import cpw.mods.fml.common.event.FMLInitializationEvent;
 import cpw.mods.fml.common.event.FMLPostInitializationEvent;
 import cpw.mods.fml.common.event.FMLPreInitializationEvent;
-import cpw.mods.fml.common.eventhandler.SubscribeEvent;
+import cpw.mods.fml.relauncher.Side;
 import jss.notfine.config.NotFineConfig;
 import jss.notfine.core.Settings;
 import jss.notfine.core.SettingsManager;
 import net.minecraft.client.Minecraft;
-import net.minecraftforge.common.MinecraftForge;
-import net.minecraftforge.event.world.WorldEvent;
+import net.minecraft.client.settings.GameSettings;
 
 public class ClientProxy extends CommonProxy {
 
     @Override
     public void preInit(FMLPreInitializationEvent event) {
-        FMLCommonHandler.instance().bus().register(this);
-        MinecraftForge.EVENT_BUS.register(this);
-
         if(!NotFineConfig.allowAdvancedOpenGL) {
             Minecraft.getMinecraft().gameSettings.advancedOpengl = false;
         }
@@ -33,13 +29,12 @@ public class ClientProxy extends CommonProxy {
         }
     }
 
+    @Override
+    public void init(FMLInitializationEvent event) { }
+
     @Override
     public void postInit(FMLPostInitializationEvent event) {
         SettingsManager.settingsFile.loadSettings();
     }
 
-    @SubscribeEvent
-    public void worldLoad(WorldEvent.Load event) {
-        SettingsManager.graphicsUpdated();
-    }
 }

src/main/java/jss/notfine/util/itembreakparticles/IWorldSpawnItemBreakParticle.java

@@ -1,6 +1,7 @@
 package jss.notfine.util.itembreakparticles;
 
 import net.minecraft.item.ItemStack;
+import net.minecraft.util.IIcon;
 
 public interface IWorldSpawnItemBreakParticle {
 

src/main/java/kroppeb/stareval/Util.java

@@ -1,13 +0,0 @@
-package kroppeb.stareval;
-
-import java.util.function.Consumer;
-
-public class Util {
-	public static <T> T make(T item, Consumer<T> init) {
-		init.accept(item);
-		return item;
-	}
-
-}
-
-

src/main/java/kroppeb/stareval/element/tree/AccessExpressionElement.java

@@ -1,12 +1,23 @@
 package kroppeb.stareval.element.tree;
 
-import com.github.bsideup.jabel.Desugar;
 import kroppeb.stareval.element.AccessibleExpressionElement;
 
-@Desugar
-public record AccessExpressionElement(AccessibleExpressionElement base,
-									  String index) implements AccessibleExpressionElement {
+public class AccessExpressionElement implements AccessibleExpressionElement {
+	private final AccessibleExpressionElement base;
+	private final String index;
 
+	public AccessExpressionElement(AccessibleExpressionElement base, String index) {
+		this.base = base;
+		this.index = index;
+	}
+
+	public AccessibleExpressionElement getBase() {
+		return this.base;
+	}
+
+	public String getIndex() {
+		return this.index;
+	}
 
 	@Override
 	public String toString() {

src/main/java/kroppeb/stareval/element/tree/BinaryExpressionElement.java

@@ -1,13 +1,30 @@
 package kroppeb.stareval.element.tree;
 
-import com.github.bsideup.jabel.Desugar;
 import kroppeb.stareval.element.ExpressionElement;
 import kroppeb.stareval.parser.BinaryOp;
 
-@Desugar
-public record BinaryExpressionElement(BinaryOp op, ExpressionElement left,
-									  ExpressionElement right) implements ExpressionElement {
+public class BinaryExpressionElement implements ExpressionElement {
+	private final BinaryOp op;
+	private final ExpressionElement left;
+	private final ExpressionElement right;
 
+	public BinaryExpressionElement(BinaryOp op, ExpressionElement left, ExpressionElement right) {
+		this.op = op;
+		this.left = left;
+		this.right = right;
+	}
+
+	public BinaryOp getOp() {
+		return this.op;
+	}
+
+	public ExpressionElement getLeft() {
+		return this.left;
+	}
+
+	public ExpressionElement getRight() {
+		return this.right;
+	}
 
 	@Override
 	public String toString() {

src/main/java/kroppeb/stareval/element/tree/FunctionCall.java

@@ -1,13 +1,25 @@
 package kroppeb.stareval.element.tree;
 
-import com.github.bsideup.jabel.Desugar;
 import kroppeb.stareval.element.ExpressionElement;
 
 import java.util.List;
 
-@Desugar
-public record FunctionCall(String id, List<? extends ExpressionElement> args) implements ExpressionElement {
+public class FunctionCall implements ExpressionElement {
+	private final String id;
+	private final List<? extends ExpressionElement> args;
 
+	public FunctionCall(String id, List<? extends ExpressionElement> args) {
+		this.id = id;
+		this.args = args;
+	}
+
+	public String getId() {
+		return this.id;
+	}
+
+	public List<? extends ExpressionElement> getArgs() {
+		return this.args;
+	}
 
 	@Override
 	public String toString() {

src/main/java/kroppeb/stareval/element/tree/UnaryExpressionElement.java

@@ -1,12 +1,24 @@
 package kroppeb.stareval.element.tree;
 
-import com.github.bsideup.jabel.Desugar;
 import kroppeb.stareval.element.ExpressionElement;
 import kroppeb.stareval.parser.UnaryOp;
 
-@Desugar
-public record UnaryExpressionElement(UnaryOp op, ExpressionElement inner) implements ExpressionElement {
+public class UnaryExpressionElement implements ExpressionElement {
+	private final UnaryOp op;
+	private final ExpressionElement inner;
 
+	public UnaryExpressionElement(UnaryOp op, ExpressionElement inner) {
+		this.op = op;
+		this.inner = inner;
+	}
+
+	public UnaryOp getOp() {
+		return op;
+	}
+
+	public ExpressionElement getInner() {
+		return inner;
+	}
 
 	@Override
 	public String toString() {

src/main/java/kroppeb/stareval/element/tree/partial/PartialBinaryExpression.java

@@ -21,7 +21,7 @@ public class PartialBinaryExpression extends PartialExpression implements Priori
 
 	@Override
 	public int getPriority() {
-		return this.op.priority();
+		return this.op.getPriority();
 	}
 
 	@Override

src/main/java/kroppeb/stareval/exception/ParseException.java

@@ -1,6 +1,6 @@
 package kroppeb.stareval.exception;
 
-public abstract class ParseException extends Exception {
+public abstract class ParseException extends Exception{
 	public ParseException() {
 	}
 

src/main/java/kroppeb/stareval/expression/BasicVariableExpression.java

@@ -1,31 +0,0 @@
-package kroppeb.stareval.expression;
-
-import kroppeb.stareval.function.FunctionContext;
-import kroppeb.stareval.function.FunctionReturn;
-import kroppeb.stareval.function.Type;
-
-public class BasicVariableExpression implements VariableExpression {
-	final private String name;
-	final private Type type;
-
-	public BasicVariableExpression(String name, Type type) {
-		this.name = name;
-		this.type = type;
-	}
-
-	@Override
-	public void evaluateTo(FunctionContext c, FunctionReturn r) {
-		c.getVariable(name).evaluateTo(c, r);
-	}
-
-	@Override
-	public Expression partialEval(FunctionContext context, FunctionReturn functionReturn) {
-		if (context.hasVariable(this.name)) {
-			context.getVariable(this.name).evaluateTo(context, functionReturn);
-			return type.createConstant(functionReturn);
-		} else {
-			return this;
-		}
-	}
-
-}

src/main/java/kroppeb/stareval/expression/CallExpression.java

@@ -1,84 +0,0 @@
-package kroppeb.stareval.expression;
-
-import kroppeb.stareval.function.FunctionContext;
-import kroppeb.stareval.function.FunctionReturn;
-import kroppeb.stareval.function.TypedFunction;
-
-import java.util.Collection;
-
-public class CallExpression implements Expression {
-	private final TypedFunction function;
-	private final Expression[] arguments;
-
-	public CallExpression(TypedFunction function, Expression[] arguments) {
-		this.function = function;
-		this.arguments = arguments;
-	}
-
-	@Override
-	public void evaluateTo(FunctionContext context, FunctionReturn functionReturn) {
-		this.function.evaluateTo(this.arguments, context, functionReturn);
-	}
-
-	@Override
-	public void listVariables(Collection<? super VariableExpression> variables) {
-		for (Expression argument : this.arguments) {
-			argument.listVariables(variables);
-		}
-	}
-
-	private boolean isConstant() {
-		for (Expression i : arguments) {
-			if (!(i instanceof ConstantExpression)) {
-				return false;
-			}
-		}
-		return true;
-	}
-
-	@Override
-	public Expression partialEval(FunctionContext context, FunctionReturn functionReturn) {
-		// the dirty way would be this
-		/*
-			try{
-				this.evaluateTo(context, functionReturn);
-				return this.function.getReturnType().createConstant(functionReturn);
-			}catch(Exception e){
-				return this;
-			}
-		 */
-
-		if (this.function.isPure() && isConstant()) {
-			this.evaluateTo(context, functionReturn);
-			return function.getReturnType().createConstant(functionReturn);
-		}
-
-		Expression[] partialEvaluatedParams = new Expression[this.arguments.length];
-		boolean allFullySimplified = true;
-		boolean noneSimplified = true;
-		for (int i = 0; i < this.arguments.length; i++) {
-			Expression simplified = this.arguments[i].partialEval(context, functionReturn);
-			if (simplified instanceof ConstantExpression) {
-				noneSimplified = false;
-			} else {
-				allFullySimplified = false;
-				if (simplified != this.arguments[i]) {
-					noneSimplified = false;
-				}
-			}
-			partialEvaluatedParams[i] = simplified;
-		}
-
-		if (this.function.isPure() && allFullySimplified) {
-			this.function.evaluateTo(partialEvaluatedParams, context, functionReturn);
-			return this.function.getReturnType().createConstant(functionReturn);
-		}
-
-		if (noneSimplified) {
-			return this;
-		}
-
-		return new CallExpression(this.function, partialEvaluatedParams);
-
-	}
-}

src/main/java/kroppeb/stareval/expression/ConstantExpression.java

@@ -1,21 +0,0 @@
-package kroppeb.stareval.expression;
-
-import kroppeb.stareval.function.Type;
-
-import java.util.Collection;
-
-public abstract class ConstantExpression implements Expression {
-	private final Type type;
-
-	protected ConstantExpression(Type type) {
-		this.type = type;
-	}
-
-	public Type getType() {
-		return this.type;
-	}
-
-	@Override
-	public void listVariables(Collection<? super VariableExpression> variables) {
-	}
-}

src/main/java/kroppeb/stareval/expression/Expression.java

@@ -1,16 +0,0 @@
-package kroppeb.stareval.expression;
-
-import kroppeb.stareval.function.FunctionContext;
-import kroppeb.stareval.function.FunctionReturn;
-
-import java.util.Collection;
-
-public interface Expression {
-	void evaluateTo(FunctionContext context, FunctionReturn functionReturn);
-
-	default Expression partialEval(FunctionContext context, FunctionReturn functionReturn) {
-		return this;
-	}
-
-	void listVariables(Collection<? super VariableExpression> variables);
-}

src/main/java/kroppeb/stareval/expression/VariableExpression.java

@@ -1,10 +0,0 @@
-package kroppeb.stareval.expression;
-
-import java.util.Collection;
-
-public interface VariableExpression extends Expression {
-	@Override
-	default void listVariables(Collection<? super VariableExpression> variables) {
-		variables.add(this);
-	}
-}

src/main/java/kroppeb/stareval/function/AbstractTypedFunction.java

@@ -1,67 +0,0 @@
-package kroppeb.stareval.function;
-
-import java.util.Arrays;
-import java.util.Objects;
-
-public abstract class AbstractTypedFunction implements TypedFunction {
-	private final Type returnType;
-	private final Parameter[] parameters;
-	private final int priority;
-	private final boolean isPure;
-
-	public AbstractTypedFunction(Type returnType, Parameter[] parameters, int priority, boolean isPure) {
-		this.returnType = returnType;
-		this.parameters = parameters;
-		this.priority = priority;
-		this.isPure = isPure;
-	}
-
-	public AbstractTypedFunction(Type returnType, Type[] parameterType) {
-		this.returnType = returnType;
-		this.parameters = Arrays.stream(parameterType).map(Parameter::new).toArray(Parameter[]::new);
-		this.priority = 0;
-		this.isPure = true;
-	}
-
-	@Override
-	public Type getReturnType() {
-		return this.returnType;
-	}
-
-	@Override
-	public Parameter[] getParameters() {
-		return this.parameters;
-	}
-
-	@Override
-	public boolean isPure() {
-		return this.isPure;
-	}
-
-	@Override
-	public int priority() {
-		return this.priority;
-	}
-
-	@Override
-	public boolean equals(Object obj) {
-		if (obj instanceof AbstractTypedFunction func) {
-			return Objects.equals(returnType, func.returnType) &&
-				Arrays.equals(parameters, func.parameters) &&
-				priority == func.priority &&
-				isPure == func.isPure;
-		}
-
-		return false;
-	}
-
-	@Override
-	public int hashCode() {
-		return Objects.hash(returnType, Arrays.hashCode(parameters), priority, isPure);
-	}
-
-	@Override
-	public String toString() {
-		return TypedFunction.format(this, "unknown");
-	}
-}

src/main/java/kroppeb/stareval/function/B2BFunction.java

@@ -1,26 +0,0 @@
-package kroppeb.stareval.function;
-
-import kroppeb.stareval.expression.Expression;
-
-@FunctionalInterface
-public interface B2BFunction extends TypedFunction {
-	boolean eval(boolean a);
-
-	@Override
-	default void evaluateTo(Expression[] params, FunctionContext context, FunctionReturn functionReturn) {
-		params[0].evaluateTo(context, functionReturn);
-		boolean a = functionReturn.booleanReturn;
-
-		functionReturn.booleanReturn = this.eval(a);
-	}
-
-	@Override
-	default Type getReturnType() {
-		return Type.Boolean;
-	}
-
-	@Override
-	default Parameter[] getParameters() {
-		return new Parameter[]{Type.BooleanParameter};
-	}
-}

src/main/java/kroppeb/stareval/function/BB2BFunction.java

@@ -1,29 +0,0 @@
-package kroppeb.stareval.function;
-
-import kroppeb.stareval.expression.Expression;
-
-@FunctionalInterface
-public interface BB2BFunction extends TypedFunction {
-	boolean eval(boolean a, boolean b);
-
-	@Override
-	default void evaluateTo(Expression[] params, FunctionContext context, FunctionReturn functionReturn) {
-		params[0].evaluateTo(context, functionReturn);
-		boolean a = functionReturn.booleanReturn;
-
-		params[1].evaluateTo(context, functionReturn);
-		boolean b = functionReturn.booleanReturn;
-
-		functionReturn.booleanReturn = this.eval(a, b);
-	}
-
-	@Override
-	default Type getReturnType() {
-		return Type.Boolean;
-	}
-
-	@Override
-	default Parameter[] getParameters() {
-		return new Parameter[]{Type.BooleanParameter, Type.BooleanParameter};
-	}
-}

src/main/java/kroppeb/stareval/function/BasicFunctionContext.java

@@ -1,36 +0,0 @@
-package kroppeb.stareval.function;
-
-import it.unimi.dsi.fastutil.objects.Object2ObjectOpenHashMap;
-import kroppeb.stareval.expression.Expression;
-import kroppeb.stareval.expression.VariableExpression;
-
-import java.util.Map;
-
-public class BasicFunctionContext implements FunctionContext {
-	final private Map<String, Expression> variables = new Object2ObjectOpenHashMap<>();
-
-	public void setVariable(String name, Expression value) {
-		variables.put(name, value);
-	}
-
-	public void setIntVariable(String name, int value) {
-		setVariable(name, (VariableExpression) (c, r) -> r.intReturn = value);
-	}
-
-	public void setFloatVariable(String name, float value) {
-		setVariable(name, (VariableExpression) (c, r) -> r.floatReturn = value);
-	}
-
-	@Override
-	public Expression getVariable(String name) {
-		Expression expression = variables.get(name);
-		if (expression == null)
-			throw new RuntimeException("Variable hasn't been set: " + name);
-		return expression;
-	}
-
-	@Override
-	public boolean hasVariable(String name) {
-		return variables.containsKey(name);
-	}
-}

src/main/java/kroppeb/stareval/function/F2FFunction.java

@@ -1,24 +0,0 @@
-package kroppeb.stareval.function;
-
-import kroppeb.stareval.expression.Expression;
-
-@FunctionalInterface
-public interface F2FFunction extends TypedFunction {
-	float eval(float a);
-
-	@Override
-	default void evaluateTo(Expression[] params, FunctionContext context, FunctionReturn functionReturn) {
-		params[0].evaluateTo(context, functionReturn);
-		functionReturn.floatReturn = this.eval(functionReturn.floatReturn);
-	}
-
-	@Override
-	default Type getReturnType() {
-		return Type.Float;
-	}
-
-	@Override
-	default Parameter[] getParameters() {
-		return new Parameter[]{Type.FloatParameter};
-	}
-}

src/main/java/kroppeb/stareval/function/F2IFunction.java

@@ -1,24 +0,0 @@
-package kroppeb.stareval.function;
-
-import kroppeb.stareval.expression.Expression;
-
-@FunctionalInterface
-public interface F2IFunction extends TypedFunction {
-	int eval(float a);
-
-	@Override
-	default void evaluateTo(Expression[] params, FunctionContext context, FunctionReturn functionReturn) {
-		params[0].evaluateTo(context, functionReturn);
-		functionReturn.intReturn = this.eval(functionReturn.floatReturn);
-	}
-
-	@Override
-	default Type getReturnType() {
-		return Type.Int;
-	}
-
-	@Override
-	default Parameter[] getParameters() {
-		return new Parameter[]{Type.FloatParameter};
-	}
-}

src/main/java/kroppeb/stareval/function/FF2BFunction.java

@@ -1,29 +0,0 @@
-package kroppeb.stareval.function;
-
-import kroppeb.stareval.expression.Expression;
-
-@FunctionalInterface
-public interface FF2BFunction extends TypedFunction {
-	boolean eval(float a, float b);
-
-	@Override
-	default void evaluateTo(Expression[] params, FunctionContext context, FunctionReturn functionReturn) {
-		params[0].evaluateTo(context, functionReturn);
-		float a = functionReturn.floatReturn;
-
-		params[1].evaluateTo(context, functionReturn);
-		float b = functionReturn.floatReturn;
-
-		functionReturn.booleanReturn = this.eval(a, b);
-	}
-
-	@Override
-	default Type getReturnType() {
-		return Type.Boolean;
-	}
-
-	@Override
-	default Parameter[] getParameters() {
-		return new Parameter[]{Type.FloatParameter, Type.FloatParameter};
-	}
-}