Fix equals check for certain records

Author: pupnewfster

src/datagen/main/java/mekanism/common/integration/crafttweaker/example/component/CrTExampleRecipeComponentBuilder.java

@@ -17,6 +17,7 @@
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Objects;
 import mekanism.common.integration.crafttweaker.example.BaseCrTExampleProvider;
 import mekanism.common.integration.crafttweaker.example.CrTExampleBuilder;
 import mekanism.common.integration.crafttweaker.recipe.manager.MekanismRecipeManager;
@@ -408,9 +409,7 @@ public boolean equals(Object o) {
 
         @Override
         public int hashCode() {
-            int result = method.hashCode();
-            result = 31 * result + Arrays.hashCode(params);
-            return result;
+            return Objects.hash(method, Arrays.deepHashCode(params));
         }
     }
 }

src/main/java/mekanism/client/model/ModelToVoxelShapeUtil.java

@@ -10,8 +10,10 @@
 import java.nio.file.Paths;
 import java.text.DecimalFormat;
 import java.text.DecimalFormatSymbols;
+import java.util.Arrays;
 import java.util.Locale;
 import java.util.Map;
+import java.util.Objects;
 import java.util.stream.Stream;
 
 //TODO: JavaDoc - contains client side methods to help convert models to VoxelShapes
@@ -120,5 +122,21 @@ private static ChildData from(JsonArray[] childElements) {
             }
             return new ChildData(childElements, elements);
         }
+
+        @Override
+        public boolean equals(Object o) {
+            if (this == o) {
+                return true;
+            } else if (o == null || getClass() != o.getClass()) {
+                return false;
+            }
+            ChildData other = (ChildData) o;
+            return totalElements == other.totalElements && Arrays.equals(childElements, other.childElements);
+        }
+
+        @Override
+        public int hashCode() {
+            return Objects.hash(Arrays.hashCode(childElements), totalElements);
+        }
     }
 }

src/main/java/mekanism/client/render/lib/QuickHash.java

@@ -7,7 +7,7 @@ public record QuickHash(Object... objs) {
     @Override
     public int hashCode() {
         //TODO: Cache the hashcode?
-        return Arrays.hashCode(objs);
+        return Arrays.deepHashCode(objs);
     }
 
     @Override

src/main/java/mekanism/common/base/holiday/HolidayMessage.java

@@ -1,6 +1,7 @@
 package mekanism.common.base.holiday;
 
 import java.util.Arrays;
+import java.util.Objects;
 import net.minecraft.network.chat.Component;
 
 record HolidayMessage(Component themedLines, Component... lines) {
@@ -15,8 +16,6 @@ public boolean equals(Object o) {
 
     @Override
     public int hashCode() {
-        int result = themedLines.hashCode();
-        result = 31 * result + Arrays.hashCode(lines);
-        return result;
+        return Objects.hash(themedLines, Arrays.hashCode(lines));
     }
 }

src/main/java/mekanism/common/integration/computer/MethodData.java

@@ -1,5 +1,7 @@
 package mekanism.common.integration.computer;
 
+import java.util.Arrays;
+import java.util.Objects;
 import mekanism.common.integration.computer.ComputerMethodFactory.ComputerFunctionCaller;
 import net.neoforged.fml.ModList;
 import org.jetbrains.annotations.Nullable;
@@ -27,6 +29,26 @@ private boolean modsLoaded(String[] mods) {
         return true;
     }
 
+    @Override
+    public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        } else if (o == null || getClass() != o.getClass()) {
+            return false;
+        }
+        MethodData<?> other = (MethodData<?>) o;
+        return threadSafe == other.threadSafe && requiresPublicSecurity == other.requiresPublicSecurity && restriction == other.restriction && returnType == other.returnType &&
+               name.equals(other.name) && Arrays.equals(requiredMods, other.requiredMods) && Arrays.equals(argClasses, other.argClasses) &&
+               Arrays.equals(argumentNames, other.argumentNames) && Arrays.equals(returnExtra, other.returnExtra) &&
+               Objects.equals(methodDescription, other.methodDescription) && handler.equals(other.handler);
+    }
+
+    @Override
+    public int hashCode() {
+        return Objects.hash(name, restriction, Arrays.hashCode(requiredMods), threadSafe, Arrays.hashCode(argumentNames), Arrays.hashCode(argClasses), returnType,
+              Arrays.hashCode(returnExtra), handler, methodDescription, requiresPublicSecurity);
+    }
+
     public static <T> Builder<T> builder(String methodName, ComputerFunctionCaller<T> handler) {
         return new Builder<>(methodName, handler);
     }

src/main/java/mekanism/common/integration/computer/MethodHelpData.java

@@ -158,6 +158,22 @@ public static Returns from(MethodData<?> data) {
             }
             return Returns.NOTHING;
         }
+
+        @Override
+        public boolean equals(Object o) {
+            if (this == o) {
+                return true;
+            } else if (o == null || getClass() != o.getClass()) {
+                return false;
+            }
+            Returns other = (Returns) o;
+            return javaType == other.javaType && type.equals(other.type) && Objects.equals(values, other.values) && Arrays.equals(javaExtra, other.javaExtra);
+        }
+
+        @Override
+        public int hashCode() {
+            return Objects.hash(javaType, type, Arrays.hashCode(javaExtra), values);
+        }
     }
 
 }

src/main/java/mekanism/common/integration/computer/TableType.java

@@ -2,9 +2,11 @@
 
 import com.mojang.serialization.Codec;
 import com.mojang.serialization.codecs.RecordCodecBuilder;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.LinkedHashMap;
 import java.util.Map;
+import java.util.Objects;
 import mekanism.api.SerializationConstants;
 import mekanism.common.util.MekCodecs;
 
@@ -37,6 +39,22 @@ public record FieldType(String description, Class<?> javaType, String type, Clas
                     MekCodecs.optionalClassArrayCodec("javaExtra").forGetter(FieldType::javaExtra)
               ).apply(instance, FieldType::new)
         );
+
+        @Override
+        public boolean equals(Object o) {
+            if (this == o) {
+                return true;
+            } else if (o == null || getClass() != o.getClass()) {
+                return false;
+            }
+            FieldType other = (FieldType) o;
+            return javaType == other.javaType && type.equals(other.type) && description.equals(other.description) && Arrays.equals(javaExtra, other.javaExtra);
+        }
+
+        @Override
+        public int hashCode() {
+            return Objects.hash(javaType, description, type, Arrays.hashCode(javaExtra));
+        }
     }
 
     public static class Builder {