ModLogger -> PrefixLogger

Explains what it does better
sarbian · May 1, 2019 · a8bcad2 · a8bcad2
1 parent 0a2c842
commit a8bcad2
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 14 deletions.
diff --git a/ModuleManager/Logging/ModLogger.cs → ModuleManager/Logging/PrefixLogger.cs b/ModuleManager/Logging/ModLogger.cs → ModuleManager/Logging/PrefixLogger.cs
@@ -2,12 +2,12 @@
 
 namespace ModuleManager.Logging
 {
-    public class ModLogger : IBasicLogger
+    public class PrefixLogger : IBasicLogger
     {
         private readonly string prefix;
         private readonly IBasicLogger logger;
 
-        public ModLogger(string prefix, IBasicLogger logger)
+        public PrefixLogger(string prefix, IBasicLogger logger)
         {
             if (string.IsNullOrEmpty(prefix)) throw new ArgumentNullException(nameof(prefix));
             this.prefix = $"[{prefix}] ";

diff --git a/ModuleManager/ModuleManager.cs b/ModuleManager/ModuleManager.cs
@@ -132,7 +132,7 @@ internal void Awake()
                 Log(string.Format("Adding post patch to the loading screen {0}", list.Count));
                 list.Insert(gameDatabaseIndex + 1, aGameObject.AddComponent<PostPatchLoader>());
 
-                patchRunner = new MMPatchRunner(new ModLogger("ModuleManager", new UnityLogger(Debug.unityLogger)));
+                patchRunner = new MMPatchRunner(new PrefixLogger("ModuleManager", new UnityLogger(Debug.unityLogger)));
                 StartCoroutine(patchRunner.Run());
 
                 // Workaround for 1.6.0 Editor bug after a PartDatabase rebuild.
@@ -314,7 +314,7 @@ private IEnumerator DataBaseReloadWithMM(bool dump = false)
             QualitySettings.vSyncCount = 0;
             Application.targetFrameRate = -1;
 
-            patchRunner = new MMPatchRunner(new ModLogger("ModuleManager", new UnityLogger(Debug.unityLogger)));
+            patchRunner = new MMPatchRunner(new PrefixLogger("ModuleManager", new UnityLogger(Debug.unityLogger)));
 
             float totalLoadWeight = GameDatabase.Instance.LoadWeight() + PartLoader.Instance.LoadWeight();
             bool startedReload = false;

diff --git a/ModuleManager/ModuleManager.csproj b/ModuleManager/ModuleManager.csproj
@@ -51,7 +51,7 @@
     <Compile Include="Logging\IBasicLogger.cs" />
     <Compile Include="Logging\ILogMessage.cs" />
     <Compile Include="Logging\LogSplitter.cs" />
-    <Compile Include="Logging\ModLogger.cs" />
+    <Compile Include="Logging\PrefixLogger.cs" />
     <Compile Include="Logging\LogMessage.cs" />
     <Compile Include="Logging\QueueLogger.cs" />
     <Compile Include="Logging\QueueLogRunner.cs" />

diff --git a/ModuleManager/PostPatchLoader.cs b/ModuleManager/PostPatchLoader.cs
@@ -22,7 +22,7 @@ public class PostPatchLoader : LoadingSystem
 
         private static readonly List<ModuleManagerPostPatchCallback> postPatchCallbacks = new List<ModuleManagerPostPatchCallback>();
 
-        private readonly IBasicLogger logger = new ModLogger("ModuleManager", new UnityLogger(UnityEngine.Debug.unityLogger));
+        private readonly IBasicLogger logger = new PrefixLogger("ModuleManager", new UnityLogger(UnityEngine.Debug.unityLogger));
 
         private bool ready = false;
 

diff --git a/ModuleManagerTests/Logging/ModLoggerTest.cs → ...eManagerTests/Logging/PrefixLoggerTest.cs b/ModuleManagerTests/Logging/ModLoggerTest.cs → ...eManagerTests/Logging/PrefixLoggerTest.cs
@@ -6,23 +6,23 @@
 
 namespace ModuleManagerTests.Logging
 {
-    public class ModLoggerTest
+    public class PrefixLoggerTest
     {
         private IBasicLogger innerLogger;
-        private ModLogger logger;
+        private PrefixLogger logger;
 
-        public ModLoggerTest()
+        public PrefixLoggerTest()
         {
             innerLogger = Substitute.For<IBasicLogger>();
-            logger = new ModLogger("MyMod", innerLogger);
+            logger = new PrefixLogger("MyMod", innerLogger);
         }
 
         [Fact]
         public void TestConstructor__PrefixNull()
         {
             ArgumentNullException e = Assert.Throws<ArgumentNullException>(delegate
             {
-                new ModLogger(null, innerLogger);
+                new PrefixLogger(null, innerLogger);
             });
 
             Assert.Equal("prefix", e.ParamName);
@@ -33,7 +33,7 @@ public void TestConstructor__PrefixBlank()
         {
             ArgumentNullException e = Assert.Throws<ArgumentNullException>(delegate
             {
-                new ModLogger("", innerLogger);
+                new PrefixLogger("", innerLogger);
             });
 
             Assert.Equal("prefix", e.ParamName);
@@ -44,7 +44,7 @@ public void TestConstructor__LoggerNull()
         {
             ArgumentNullException e = Assert.Throws<ArgumentNullException>(delegate
             {
-                new ModLogger("blah", null);
+                new PrefixLogger("blah", null);
             });
 
             Assert.Equal("logger", e.ParamName);

diff --git a/ModuleManagerTests/ModuleManagerTests.csproj b/ModuleManagerTests/ModuleManagerTests.csproj
@@ -116,7 +116,7 @@
     <Compile Include="DummyTest.cs" />
     <Compile Include="Extensions\ConfigNodeExtensionsTest.cs" />
     <Compile Include="Extensions\NodeStackExtensionsTest.cs" />
-    <Compile Include="Logging\ModLoggerTest.cs" />
+    <Compile Include="Logging\PrefixLoggerTest.cs" />
     <Compile Include="PatchExtractorTest.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="Progress\PatchProgressTest.cs" />