From fc025a7b73d9b3f46ecf2257227657592f5506b5 Mon Sep 17 00:00:00 2001
From: AbrarSyed <sacabrarsyed@gmail.com>
Date: Sat, 11 Jan 2014 16:43:14 +0000
Subject: [PATCH 1/3] Logging Changes

---
 .../cpw/mods/fml/client/FMLClientHandler.java | 14 ------
 src/main/resources/log4j2.xml                 | 46 +++++++++++++++++++
 2 files changed, 46 insertions(+), 14 deletions(-)
 create mode 100644 src/main/resources/log4j2.xml

diff --git a/src/main/java/cpw/mods/fml/client/FMLClientHandler.java b/src/main/java/cpw/mods/fml/client/FMLClientHandler.java
index 1b1688a2ce..4b7fdb07e5 100644
--- a/src/main/java/cpw/mods/fml/client/FMLClientHandler.java
+++ b/src/main/java/cpw/mods/fml/client/FMLClientHandler.java
@@ -18,7 +18,6 @@ import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
-import java.util.logging.Logger;
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.entity.EntityClientPlayerMP;
 import net.minecraft.client.gui.Gui;
@@ -102,10 +101,8 @@ public class FMLClientHandler implements IFMLSidedHandler
 
     private DummyModContainer optifineContainer;
 
-    @SuppressWarnings("unused")
     private boolean guiLoaded;
 
-    @SuppressWarnings("unused")
     private boolean serverIsRunning;
 
     private MissingModsException modsMissing;
@@ -124,7 +121,6 @@ public class FMLClientHandler implements IFMLSidedHandler
 
     private List<IResourcePack> resourcePackList;
 
-    @SuppressWarnings("unused")
     private IReloadableResourceManager resourceManager;
 
     private Map<String, IResourcePack> resourcePackMap;
@@ -277,7 +273,6 @@ public class FMLClientHandler implements IFMLSidedHandler
         loading = false;
     }
 
-    @SuppressWarnings("unused")
     public void extendModList()
     {
         @SuppressWarnings("unchecked")
@@ -338,15 +333,6 @@ public class FMLClientHandler implements IFMLSidedHandler
         return client;
     }
 
-    /**
-     * Get a handle to the client's logger instance
-     * The client actually doesn't have one- so we return null
-     */
-    public Logger getMinecraftLogger()
-    {
-        return null;
-    }
-
     /**
      * @return the instance
      */
diff --git a/src/main/resources/log4j2.xml b/src/main/resources/log4j2.xml
new file mode 100644
index 0000000000..94e1abf947
--- /dev/null
+++ b/src/main/resources/log4j2.xml
@@ -0,0 +1,46 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<Configuration status="WARN" packages="net.minecraft,com.mojang">
+    <Appenders>
+        <Console name="FmlSysOut" target="SYSTEM_OUT">
+            <PatternLayout pattern="[%d{HH:mm:ss}] [%t/%level] [%logger]: %msg%n" />
+        </Console>
+        <Console name="SysOut" target="SYSTEM_OUT">
+            <PatternLayout pattern="[%d{HH:mm:ss}] [%t/%level]: %msg%n" />
+        </Console>
+        <Queue name="ServerGuiConsole">
+            <PatternLayout pattern="[%d{HH:mm:ss} %level]: %msg%n" />
+        </Queue>
+        <Queue name="FmlServerGuiConsole">
+            <PatternLayout pattern="[%d{HH:mm:ss} %level] [%logger]: %msg%n" />
+        </Queue>
+        <RollingRandomAccessFile name="File" fileName="logs/latest.log" filePattern="logs/%d{yyyy-MM-dd}-%i.log.gz">
+            <PatternLayout pattern="[%d{HH:mm:ss}] [%t/%level]: %msg%n" />
+            <Policies>
+                <TimeBasedTriggeringPolicy />
+                <OnStartupTriggeringPolicy />
+            </Policies>
+        </RollingRandomAccessFile>
+        <RollingRandomAccessFile name="FmlFile" fileName="logs/fml-latest.log" filePattern="logs/fml-%d{yyyy-MM-dd}-%i.log.gz">
+            <PatternLayout pattern="[%d{HH:mm:ss}] [%t/%level] [%logger]: %msg%n" />
+            <Policies>
+                <TimeBasedTriggeringPolicy />
+                <OnStartupTriggeringPolicy />
+            </Policies>
+        </RollingRandomAccessFile>
+    </Appenders>
+    <Loggers>
+        <Root level="info" name="net/minecraft" additibity="false">
+            <filters>
+                <MarkerFilter marker="NETWORK_PACKETS" onMatch="DENY" onMismatch="NEUTRAL" />
+            </filters>
+            <AppenderRef ref="SysOut"/>
+            <AppenderRef ref="File"/>
+            <AppenderRef ref="ServerGuiConsole"/>
+        </Root>
+        <Root level="info">
+            <AppenderRef ref="FmlSysOut"/>
+            <AppenderRef ref="FmlFile"/>
+            <AppenderRef ref="FmlServerGuiConsole"/>
+        </Root>
+    </Loggers>
+</Configuration>
\ No newline at end of file

From f0132a6f3b47e746a1a7df3ef84f4be989f140dd Mon Sep 17 00:00:00 2001
From: Abrar Syed <sacabrarsyed@gmail.com>
Date: Sun, 19 Jan 2014 00:49:47 +0000
Subject: [PATCH 2/3] changed fml log level to all

---
 src/main/resources/log4j2.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/resources/log4j2.xml b/src/main/resources/log4j2.xml
index 94e1abf947..4c8d3e54d4 100644
--- a/src/main/resources/log4j2.xml
+++ b/src/main/resources/log4j2.xml
@@ -37,10 +37,10 @@
             <AppenderRef ref="File"/>
             <AppenderRef ref="ServerGuiConsole"/>
         </Root>
-        <Root level="info">
+        <Root level="all">
             <AppenderRef ref="FmlSysOut"/>
             <AppenderRef ref="FmlFile"/>
             <AppenderRef ref="FmlServerGuiConsole"/>
         </Root>
     </Loggers>
-</Configuration>
\ No newline at end of file
+</Configuration>

From f23eba4352c38fd21e04e81f3db72c6cafe65a36 Mon Sep 17 00:00:00 2001
From: Abrar Syed <sacabrarsyed@gmail.com>
Date: Sun, 19 Jan 2014 00:53:58 +0000
Subject: [PATCH 3/3] put max number of FML log files to 3

---
 src/main/resources/log4j2.xml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/resources/log4j2.xml b/src/main/resources/log4j2.xml
index 4c8d3e54d4..5bb9b92ddd 100644
--- a/src/main/resources/log4j2.xml
+++ b/src/main/resources/log4j2.xml
@@ -25,6 +25,7 @@
             <Policies>
                 <TimeBasedTriggeringPolicy />
                 <OnStartupTriggeringPolicy />
+                <DefaultRolloverStrategy max="3"/>
             </Policies>
         </RollingRandomAccessFile>
     </Appenders>