Use concurrent maps in MIHookTracker to (hopefully) avoid concurrency issues (#158)

Swedz · web-flow · commit 1a7cb8a0885d · 2025-10-19T15:13:10.000-04:00
diff --git a/src/main/java/net/swedz/tesseract/neoforge/compat/mi/hook/MIHookTracker.java b/src/main/java/net/swedz/tesseract/neoforge/compat/mi/hook/MIHookTracker.java
@@ -16,9 +16,9 @@
 @ApiStatus.Internal
 public final class MIHookTracker
 {
-	private static final Map<ResourceLocation, String>                                         REI_CATEGORY_NAMES    = Maps.newHashMap();
-	private static final Map<ResourceLocation, MachineModelProperties>                         MACHINE_MODELS        = Maps.newHashMap();
-	private static final Map<String, List<Consumer<MachineCasingModelsMIHookDatagenProvider>>> MACHINE_CASING_MODELS = Maps.newHashMap();
+	private static final Map<ResourceLocation, String>                                         REI_CATEGORY_NAMES    = Maps.newConcurrentMap();
+	private static final Map<ResourceLocation, MachineModelProperties>                         MACHINE_MODELS        = Maps.newConcurrentMap();
+	private static final Map<String, List<Consumer<MachineCasingModelsMIHookDatagenProvider>>> MACHINE_CASING_MODELS = Maps.newConcurrentMap();
 	
 	public static List<Map.Entry<ResourceLocation, String>> getReiCategoryNames(String modId)
 	{

Original file line number	Diff line number	Diff line change
`@@ -16,9 +16,9 @@`
`16`	`16`	`@ApiStatus.Internal`
`17`	`17`	`public final class MIHookTracker`
`18`	`18`	`{`
`19`		`- private static final Map<ResourceLocation, String> REI_CATEGORY_NAMES = Maps.newHashMap();`
`20`		`- private static final Map<ResourceLocation, MachineModelProperties> MACHINE_MODELS = Maps.newHashMap();`
`21`		`- private static final Map<String, List<Consumer<MachineCasingModelsMIHookDatagenProvider>>> MACHINE_CASING_MODELS = Maps.newHashMap();`
	`19`	`+ private static final Map<ResourceLocation, String> REI_CATEGORY_NAMES = Maps.newConcurrentMap();`
	`20`	`+ private static final Map<ResourceLocation, MachineModelProperties> MACHINE_MODELS = Maps.newConcurrentMap();`
	`21`	`+ private static final Map<String, List<Consumer<MachineCasingModelsMIHookDatagenProvider>>> MACHINE_CASING_MODELS = Maps.newConcurrentMap();`
`22`	`22`
`23`	`23`	`public static List<Map.Entry<ResourceLocation, String>> getReiCategoryNames(String modId)`
`24`	`24`	`{`