Get rid of warning on forge

Author: Discusser

common/src/main/java/io/github/discusser/toomanyentities/TooManyEntities.java

@@ -4,14 +4,19 @@
 import dev.architectury.event.events.client.ClientTickEvent;
 import dev.architectury.injectables.annotations.ExpectPlatform;
 import dev.architectury.registry.client.keymappings.KeyMappingRegistry;
+import io.github.discusser.toomanyentities.client.TooManyEntitiesKeys;
 import io.github.discusser.toomanyentities.config.MapGuiProvider;
 import io.github.discusser.toomanyentities.config.TooManyEntitiesConfig;
 import me.shedaniel.autoconfig.AutoConfig;
 import me.shedaniel.autoconfig.gui.registry.GuiRegistry;
 import me.shedaniel.autoconfig.serializer.GsonConfigSerializer;
 import net.minecraft.client.option.KeyBinding;
 import net.minecraft.client.util.InputUtil;
+import net.minecraft.text.Style;
 import net.minecraft.text.Text;
+import net.minecraft.text.TextColor;
+import net.minecraft.util.Colors;
+import net.minecraft.util.Formatting;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -28,32 +33,23 @@ public final class TooManyEntities {
     public static void init() {
     }
 
-    public static void registerKeyBindings() {
-        KeyBinding KEY_TOGGLE_MOD = new KeyBinding(
-                "key.too_many_entities.toggle_mod",
-                InputUtil.Type.KEYSYM,
-                InputUtil.GLFW_KEY_O,
-                "category.too_many_entities.too_many_entities"
-        );
-        KeyMappingRegistry.register(KEY_TOGGLE_MOD);
+    public static void initClient() {
+        AutoConfig.register(TooManyEntitiesConfig.class, GsonConfigSerializer::new);
+        GuiRegistry registry = AutoConfig.getGuiRegistry(TooManyEntitiesConfig.class);
+        registry.registerPredicateProvider(new MapGuiProvider(), field -> Map.class.isAssignableFrom(field.getType()));
+        TooManyEntitiesConfig.instance = AutoConfig.getConfigHolder(TooManyEntitiesConfig.class).getConfig();
+
         ClientTickEvent.CLIENT_POST.register(minecraft -> {
-            while (KEY_TOGGLE_MOD.wasPressed()) {
+            while (TooManyEntitiesKeys.KEY_TOGGLE_MOD.wasPressed()) {
                 modEnabled = !modEnabled;
                 if (minecraft.player != null) {
                     String key = "text.too_many_entities.mod_" + (modEnabled ? "enabled" : "disabled");
-                    minecraft.player.sendMessage(Text.translatable(key));
+                    minecraft.player.sendMessage(Text.translatable(key).setStyle(Style.EMPTY.withColor(modEnabled ? Formatting.GREEN : Formatting.RED)));
                 }
             }
         });
     }
 
-    public static void initClient() {
-        AutoConfig.register(TooManyEntitiesConfig.class, GsonConfigSerializer::new);
-        GuiRegistry registry = AutoConfig.getGuiRegistry(TooManyEntitiesConfig.class);
-        registry.registerPredicateProvider(new MapGuiProvider(), field -> Map.class.isAssignableFrom(field.getType()));
-        TooManyEntitiesConfig.instance = AutoConfig.getConfigHolder(TooManyEntitiesConfig.class).getConfig();
-    }
-
     @ExpectPlatform
     public static boolean isModPresent(String modid) {
         throw new AssertionError();

common/src/main/java/io/github/discusser/toomanyentities/client/TooManyEntitiesKeys.java

@@ -0,0 +1,15 @@
+package io.github.discusser.toomanyentities.client;
+
+import net.minecraft.client.option.KeyBinding;
+import net.minecraft.client.util.InputUtil;
+
+public class TooManyEntitiesKeys {
+    public TooManyEntitiesKeys() {}
+
+    public static final KeyBinding KEY_TOGGLE_MOD = new KeyBinding(
+            "key.too_many_entities.toggle_mod",
+            InputUtil.Type.KEYSYM,
+            InputUtil.GLFW_KEY_O,
+            "category.too_many_entities.too_many_entities"
+    );
+}

fabric/src/main/java/io/github/discusser/toomanyentities/fabric/TooManyEntitiesClientFabric.java

@@ -1,12 +1,15 @@
 package io.github.discusser.toomanyentities.fabric;
 
 import io.github.discusser.toomanyentities.TooManyEntities;
+import io.github.discusser.toomanyentities.client.TooManyEntitiesKeys;
 import net.fabricmc.api.ClientModInitializer;
+import net.fabricmc.fabric.api.client.keybinding.v1.KeyBindingHelper;
 
 public final class TooManyEntitiesClientFabric implements ClientModInitializer {
     @Override
     public void onInitializeClient() {
         TooManyEntities.initClient();
-        TooManyEntities.registerKeyBindings();
+
+        KeyBindingHelper.registerKeyBinding(TooManyEntitiesKeys.KEY_TOGGLE_MOD);
     }
 }

forge/src/main/java/io/github/discusser/toomanyentities/forge/TooManyEntitiesForge.java

@@ -2,6 +2,7 @@
 
 import dev.architectury.platform.forge.EventBuses;
 import io.github.discusser.toomanyentities.TooManyEntities;
+import io.github.discusser.toomanyentities.client.TooManyEntitiesKeys;
 import io.github.discusser.toomanyentities.config.TooManyEntitiesConfig;
 import me.shedaniel.autoconfig.AutoConfig;
 import net.minecraftforge.api.distmarker.Dist;
@@ -32,8 +33,7 @@ private void clientSetup(FMLClientSetupEvent ignoredEvent) {
         TooManyEntities.initClient();
     }
 
-    @SubscribeEvent(priority = EventPriority.HIGHEST)
     public static void registerBindings(RegisterKeyMappingsEvent event) {
-        TooManyEntities.registerKeyBindings();
+        event.register(TooManyEntitiesKeys.KEY_TOGGLE_MOD);
     }
 }