Switch some fast util usages from hash to array implementations when …

…we know we will be dealing with small number of elements

src/api/java/mekanism/api/chemical/ChemicalType.java

@@ -1,6 +1,6 @@
 package mekanism.api.chemical;
 
-import it.unimi.dsi.fastutil.objects.Object2ObjectOpenHashMap;
+import it.unimi.dsi.fastutil.objects.Object2ObjectArrayMap;
 import java.util.Map;
 import java.util.function.Predicate;
 import javax.annotation.Nonnull;
@@ -26,10 +26,12 @@ public enum ChemicalType implements IStringSerializable {
     PIGMENT("pigment", c -> c instanceof Pigment),
     SLURRY("slurry", c -> c instanceof Slurry);
 
-    private static final Map<String, ChemicalType> nameToType = new Object2ObjectOpenHashMap<>();
+    private static final Map<String, ChemicalType> nameToType;
 
     static {
-        for (ChemicalType type : values()) {
+        ChemicalType[] values = values();
+        nameToType = new Object2ObjectArrayMap<>(values.length);
+        for (ChemicalType type : values) {
             nameToType.put(type.getSerializedName(), type);
         }
     }

src/datagen/main/java/mekanism/common/recipe/builder/ExtendedShapedRecipeBuilder.java

@@ -2,7 +2,7 @@
 
 import com.google.gson.JsonArray;
 import com.google.gson.JsonObject;
-import it.unimi.dsi.fastutil.chars.Char2ObjectLinkedOpenHashMap;
+import it.unimi.dsi.fastutil.chars.Char2ObjectArrayMap;
 import it.unimi.dsi.fastutil.chars.Char2ObjectMap;
 import it.unimi.dsi.fastutil.chars.CharOpenHashSet;
 import it.unimi.dsi.fastutil.chars.CharSet;
@@ -23,7 +23,7 @@
 @MethodsReturnNonnullByDefault
 public class ExtendedShapedRecipeBuilder extends BaseRecipeBuilder<ExtendedShapedRecipeBuilder> {
 
-    private final Char2ObjectMap<Ingredient> key = new Char2ObjectLinkedOpenHashMap<>();
+    private final Char2ObjectMap<Ingredient> key = new Char2ObjectArrayMap<>(9);
     private final List<String> pattern = new ArrayList<>();
 
     protected ExtendedShapedRecipeBuilder(IRecipeSerializer<?> serializer, IItemProvider result, int count) {

src/main/java/mekanism/client/ClientTickHandler.java

@@ -2,7 +2,7 @@
 
 import com.mojang.blaze3d.platform.GlStateManager;
 import com.mojang.blaze3d.systems.RenderSystem;
-import it.unimi.dsi.fastutil.objects.Object2ObjectOpenHashMap;
+import it.unimi.dsi.fastutil.objects.Object2ObjectArrayMap;
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Map.Entry;
@@ -73,7 +73,7 @@ public class ClientTickHandler {
 
     public static final Minecraft minecraft = Minecraft.getInstance();
     public static final Random rand = new Random();
-    public static final Map<PlayerEntity, TeleportData> portableTeleports = new Object2ObjectOpenHashMap<>();
+    public static final Map<PlayerEntity, TeleportData> portableTeleports = new Object2ObjectArrayMap<>(1);
     public boolean initHoliday = false;
     public boolean shouldReset = false;
     public static boolean firstTick = true;

src/main/java/mekanism/client/gui/element/tab/GuiEnergyTab.java

@@ -24,8 +24,8 @@
 
 public class GuiEnergyTab extends GuiBiDirectionalTab {
 
+    private static final Map<EnergyType, ResourceLocation> ICONS = new EnumMap<>(EnergyType.class);
     private final IInfoHandler infoHandler;
-    private final Map<EnergyType, ResourceLocation> icons = new EnumMap<>(EnergyType.class);
 
     public GuiEnergyTab(IGuiWrapper gui, IInfoHandler handler) {
         super(MekanismUtils.getResource(ResourceType.GUI_TAB, "energy_info.png"), gui, -26, 137, 26, 26);
@@ -71,7 +71,7 @@ public void renderToolTip(@Nonnull MatrixStack matrix, int mouseX, int mouseY) {
 
     @Override
     protected ResourceLocation getResource() {
-        return icons.computeIfAbsent(MekanismConfig.general.energyUnit.get(), type -> MekanismUtils.getResource(ResourceType.GUI_TAB,
+        return ICONS.computeIfAbsent(MekanismConfig.general.energyUnit.get(), type -> MekanismUtils.getResource(ResourceType.GUI_TAB,
               "energy_info_" + type.name().toLowerCase(Locale.ROOT) + ".png"));
     }
 

src/main/java/mekanism/client/gui/element/tab/GuiHeatTab.java

@@ -1,8 +1,8 @@
 package mekanism.client.gui.element.tab;
 
 import com.mojang.blaze3d.matrix.MatrixStack;
-import it.unimi.dsi.fastutil.objects.Object2ObjectOpenHashMap;
 import java.util.ArrayList;
+import java.util.EnumMap;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -18,8 +18,8 @@
 
 public class GuiHeatTab extends GuiBiDirectionalTab {
 
+    private static final Map<TempType, ResourceLocation> ICONS = new EnumMap<>(TempType.class);
     private final IInfoHandler infoHandler;
-    private final Map<TempType, ResourceLocation> icons = new Object2ObjectOpenHashMap<>();
 
     public GuiHeatTab(IGuiWrapper gui, IInfoHandler handler) {
         super(MekanismUtils.getResource(ResourceType.GUI_TAB, "heat_info.png"), gui, -26, 109, 26, 26);
@@ -43,7 +43,7 @@ public void renderToolTip(@Nonnull MatrixStack matrix, int mouseX, int mouseY) {
 
     @Override
     protected ResourceLocation getResource() {
-        return icons.computeIfAbsent(MekanismConfig.general.tempUnit.get(), type -> MekanismUtils.getResource(ResourceType.GUI_TAB,
+        return ICONS.computeIfAbsent(MekanismConfig.general.tempUnit.get(), type -> MekanismUtils.getResource(ResourceType.GUI_TAB,
               "heat_info_" + type.name().toLowerCase(Locale.ROOT) + ".png"));
     }
 

src/main/java/mekanism/common/base/HolidayManager.java

@@ -1,6 +1,6 @@
 package mekanism.common.base;
 
-import it.unimi.dsi.fastutil.objects.ObjectOpenHashSet;
+import it.unimi.dsi.fastutil.objects.ObjectArraySet;
 import java.util.Calendar;
 import java.util.Set;
 import mekanism.api.math.MathUtils;
@@ -18,7 +18,7 @@ public final class HolidayManager {
 
     private static final Calendar calendar = Calendar.getInstance();
 
-    private static final Set<Holiday> holidays = new ObjectOpenHashSet<>();
+    private static final Set<Holiday> holidays = new ObjectArraySet<>();
 
     public static final Holiday CHRISTMAS = register(new Christmas());
     public static final Holiday NEW_YEAR = register(new NewYear());

src/main/java/mekanism/common/content/qio/SearchQueryParser.java

@@ -1,8 +1,8 @@
 package mekanism.common.content.qio;
 
 import com.google.common.collect.Sets;
+import it.unimi.dsi.fastutil.chars.Char2ObjectArrayMap;
 import it.unimi.dsi.fastutil.chars.Char2ObjectMap;
-import it.unimi.dsi.fastutil.chars.Char2ObjectOpenHashMap;
 import it.unimi.dsi.fastutil.chars.CharSet;
 import java.util.ArrayList;
 import java.util.Collections;
@@ -170,10 +170,12 @@ public enum QueryType {
               .anyMatch(tooltip -> tooltip.contains(key.toLowerCase(Locale.ROOT)))),
         TAG('#', (key, stack) -> TagCache.getItemTags(stack).stream().anyMatch(itemTag -> itemTag.toLowerCase(Locale.ROOT).contains(key.toLowerCase(Locale.ROOT))));
 
-        private static final Char2ObjectMap<QueryType> charLookupMap = new Char2ObjectOpenHashMap<>();
+        private static final Char2ObjectMap<QueryType> charLookupMap;
 
         static {
-            for (QueryType type : QueryType.values()) {
+            QueryType[] values = values();
+            charLookupMap = new Char2ObjectArrayMap<>(values.length);
+            for (QueryType type : values) {
                 charLookupMap.put(type.prefix, type);
             }
         }

src/main/java/mekanism/common/tile/TileEntityTeleporter.java

@@ -1,9 +1,8 @@
 package mekanism.common.tile;
 
+import it.unimi.dsi.fastutil.longs.Long2ObjectArrayMap;
 import it.unimi.dsi.fastutil.longs.Long2ObjectMap;
-import it.unimi.dsi.fastutil.longs.Long2ObjectOpenHashMap;
 import it.unimi.dsi.fastutil.objects.ObjectOpenHashSet;
-import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.HashSet;
@@ -12,6 +11,7 @@
 import java.util.Set;
 import java.util.UUID;
 import java.util.function.Function;
+import java.util.stream.Collectors;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 import mekanism.api.Action;
@@ -191,15 +191,11 @@ private Coord4D getClosest() {
     }
 
     private void cleanTeleportCache() {
-        List<UUID> list = new ArrayList<>();
-        for (Entity e : level.getEntitiesOfClass(Entity.class, teleportBounds)) {
-            list.add(e.getUUID());
-        }
-        Set<UUID> teleportCopy = new ObjectOpenHashSet<>(didTeleport);
-        for (UUID id : teleportCopy) {
-            if (!list.contains(id)) {
-                didTeleport.remove(id);
-            }
+        List<UUID> inTeleporter = level.getEntitiesOfClass(Entity.class, teleportBounds).stream().map(Entity::getUUID).collect(Collectors.toList());
+        if (inTeleporter.isEmpty()) {
+            didTeleport.clear();
+        } else {
+            didTeleport.removeIf(id -> !inTeleporter.contains(id));
         }
     }
 
@@ -494,7 +490,10 @@ private boolean hasFrame(Direction direction, boolean rotated) {
         int yComponent = direction.getStepY();
         int zComponent = direction.getStepZ();
         //Cache the chunks we are looking up to check the frames of
-        Long2ObjectMap<IChunk> chunkMap = new Long2ObjectOpenHashMap<>();
+        // Note: We can use an array based map, because we check suck a small area, that if we do go across chunks
+        // we will be in at most two in general due to the size of our teleporter. But given we need to check multiple
+        // directions we might end up checking two different cross chunk directions which would end up at three
+        Long2ObjectMap<IChunk> chunkMap = new Long2ObjectArrayMap<>(3);
         return isFramePair(chunkMap, 0, alternatingX, 0, alternatingY, 0, alternatingZ) &&
                isFramePair(chunkMap, xComponent, alternatingX, yComponent, alternatingY, zComponent, alternatingZ) &&
                isFramePair(chunkMap, 2 * xComponent, alternatingX, 2 * yComponent, alternatingY, 2 * zComponent, alternatingZ) &&