Improve logging

src/main/java/com/gtnewhorizons/gravisuiteneo/GraviSuiteNeo.java

@@ -15,6 +15,8 @@ import cpw.mods.fml.common.event.FMLPostInitializationEvent;
 import cpw.mods.fml.common.event.FMLPreInitializationEvent;
 import cpw.mods.fml.common.registry.GameRegistry.Type;
 import net.minecraftforge.common.MinecraftForge;
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 
 @Mod(
         acceptedMinecraftVersions = "[1.7.10]",
@@ -29,6 +31,8 @@ public class GraviSuiteNeo {
     public static final String MODNAME = "Gravitation Suite Neo";
     public static final String VERSION = "GRADLETOKEN_VERSION";
 
+    public static final Logger LOGGER = LogManager.getLogger(MODID);
+
     @SidedProxy(
             clientSide = "com.gtnewhorizons.gravisuiteneo.proxy.ClientProxy",
             serverSide = "com.gtnewhorizons.gravisuiteneo.proxy.CommonProxy")

src/main/java/com/gtnewhorizons/gravisuiteneo/GraviSuiteNeoRegistry.java

@@ -5,7 +5,6 @@ import com.gtnewhorizons.gravisuiteneo.items.ItemEpicLappack;
 import com.gtnewhorizons.gravisuiteneo.items.ItemPlasmaCell;
 import com.gtnewhorizons.gravisuiteneo.items.ItemPlasmaLauncher;
 import com.gtnewhorizons.gravisuiteneo.util.GregTechAPIHelper;
-import cpw.mods.fml.common.FMLLog;
 import cpw.mods.fml.common.Loader;
 import cpw.mods.fml.common.registry.GameRegistry;
 import gravisuite.GraviSuite;
@@ -95,12 +94,12 @@ public class GraviSuiteNeoRegistry {
         // Get the percentage-multiplier for our median value, then loop the fuels again and prepare a static list
         // For all available plasma fuels
         float plasmaMedianMultip = 100.0f / sum * fluids.size();
-        FMLLog.info("_mPlasmaMedianMultip: %f", plasmaMedianMultip);
+        GraviSuiteNeo.LOGGER.debug("Median Plasma Multiplier: {}", plasmaMedianMultip);
         for (Entry<Fluid, Integer> entry : fluids.entrySet()) {
             Fluid fluid = entry.getKey();
             float fuelValue = entry.getValue() * plasmaMedianMultip;
             fuelMap.put(fluid, fuelValue);
-            FMLLog.info("Added Fluid %s with Fuel Value %f", fluid.getLocalizedName(), fuelValue);
+            GraviSuiteNeo.LOGGER.debug("Added Fluid {} with Fuel Value {}", fluid.getLocalizedName(), fuelValue);
         }
     }
 }

src/main/java/com/gtnewhorizons/gravisuiteneo/client/RenderCustomItemBar.java

@@ -1,10 +1,10 @@
 package com.gtnewhorizons.gravisuiteneo.client;
 
+import com.gtnewhorizons.gravisuiteneo.GraviSuiteNeo;
 import com.gtnewhorizons.gravisuiteneo.client.ICustomItemBars.BarAlignment;
 import com.gtnewhorizons.gravisuiteneo.util.ColorUtil;
 import com.gtnewhorizons.gravisuiteneo.util.RenderUtil;
 import com.gtnewhorizons.gravisuiteneo.util.vector.Vector4f;
-import cpw.mods.fml.common.FMLLog;
 import java.awt.Color;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.renderer.entity.RenderItem;
@@ -103,7 +103,7 @@ public class RenderCustomItemBar implements IItemRenderer {
 
             // Failsafe to not draw "outside" our boundaries
             if (currentPosXY <= 0 || currentPosXY >= 15) {
-                FMLLog.warning("[CustomBarItemRenderer] Index violates boundary limits");
+                GraviSuiteNeo.LOGGER.warn("[CustomBarItemRenderer] Index violates boundary limits");
                 return;
             }
         }

src/main/java/com/gtnewhorizons/gravisuiteneo/mixins/MixinGraviSuite.java

@@ -11,7 +11,10 @@ import net.minecraft.item.Item;
 import net.minecraft.item.ItemStack;
 import net.minecraft.nbt.NBTTagCompound;
 import net.minecraftforge.common.config.Configuration;
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Overwrite;
 import org.spongepowered.asm.mixin.injection.At;
 import org.spongepowered.asm.mixin.injection.Constant;
 import org.spongepowered.asm.mixin.injection.Inject;
@@ -28,6 +31,17 @@ import org.spongepowered.libraries.org.objectweb.asm.Opcodes;
 @Mixin(GraviSuite.class)
 public class MixinGraviSuite {
 
+    private static final Logger LOGGER = LogManager.getLogger("GraviSuite");
+
+    /**
+     * @author glowredman
+     * @reason Gravitation Suite Neo
+     */
+    @Overwrite(remap = false)
+    public static void addLog(String logString) {
+        LOGGER.info(logString);
+    }
+
     @Inject(
             at = @At(opcode = Opcodes.PUTSTATIC, remap = false, target = "logWrench:Z", value = "FIELD"),
             locals = LocalCapture.CAPTURE_FAILEXCEPTION,

src/main/java/com/gtnewhorizons/gravisuiteneo/mixins/MixinGuiRelocatorDisplay.java

@@ -2,8 +2,6 @@ package com.gtnewhorizons.gravisuiteneo.mixins;
 
 import com.gtnewhorizons.gravisuiteneo.GraviSuiteNeo;
 import com.gtnewhorizons.gravisuiteneo.client.SelectedItemMKII;
-import cpw.mods.fml.common.FMLLog;
-import gravisuite.GraviSuite;
 import gravisuite.Helpers;
 import gravisuite.ItemRelocator;
 import gravisuite.ItemRelocator.TeleportPoint;
@@ -208,7 +206,7 @@ public abstract class MixinGuiRelocatorDisplay extends GuiContainer {
 
             int realIDX = ((SelectedItemMKII) selectedItem).getRealIDX();
             if (realIDX >= tpList.size()) {
-                FMLLog.bigWarning("WARNING: IndexOutOfBounds catched while using Translocator");
+                GraviSuiteNeo.LOGGER.error("WARNING: Caught IndexOutOfBounds while using Translocator");
                 ServerProxy.sendPlayerMessage(
                         player, EnumChatFormatting.RED + "Used TP Point is invalid. Please remove and add again");
                 player.closeScreen();
@@ -274,7 +272,7 @@ public abstract class MixinGuiRelocatorDisplay extends GuiContainer {
                 }
             }
         } catch (Exception e) {
-            GraviSuite.addLog("Error in draw relocatorGui foregroundLayer:" + e.toString());
+            GraviSuiteNeo.LOGGER.error("GuiRelocatorDisplay#drawGuiContainerForegroundLayer failed!", e);
         }
     }
 
@@ -370,7 +368,7 @@ public abstract class MixinGuiRelocatorDisplay extends GuiContainer {
                     yStart + pageXofYTextY,
                     0xFFFFFF);
         } catch (Exception e) {
-            GraviSuite.addLog("Error in drawGuiContainerBackgroundLayer: " + e.toString());
+            GraviSuiteNeo.LOGGER.error("GuiRelocatorDisplay#drawGuiContainerBackgroundLayer failed!", e);
         }
     }