Changeset 1777 for branches/eraser6/CodeReview/Eraser.Manager/Task.cs

Timestamp:

2/9/2010 7:23:22 AM (3 years ago)

Author:

lowjoel

Message:

Redesigned the Logger paradigm -- instead of a dealing with a logger object to do any form of logging, we have a static Logger class which code can log to, and define log targets to receive those logs. Also included is a LogSession? class which sets the log target for the current thread or for a few threads. Addresses #320: Logging improvements

File:

• branches/eraser6/CodeReview/Eraser.Manager/Task.cs (modified) (6 diffs)

branches/eraser6/CodeReview/Eraser.Manager/Task.cs

@@ -47 +47 @@
             Targets = (ErasureTargetsCollection)info.GetValue("Targets", typeof(ErasureTargetsCollection));
             Targets.Owner = this;
-            Log = (Logger)info.GetValue("Log", typeof(Logger));
+            Log = (List<LogSink>)info.GetValue("Log", typeof(List<LogSink>));
             Canceled = false;
 
@@ -83 +83 @@
             Schedule = Schedule.RunNow;
             Canceled = false;
-            Log = new Logger();
+            Log = new List<LogSink>();
         }
 
@@ -198 +198 @@
         /// The log entries which this task has accumulated.
         /// </summary>
-        public Logger Log { get; private set; }
+        public List<LogSink> Log { get; private set; }
 
         /// <summary>
@@ -481 +481 @@
                 //The system cannot read the file, assume no ADSes for lack of
                 //more information.
-                Task.Log.LastSessionEntries.Add(new LogEntry(e.Message, LogLevel.Error));
+                Logger.Log(e.Message, LogLevel.Error);
             }
         }
@@ -743 +743 @@
                 catch (UnauthorizedAccessException e)
                 {
-                    Task.Log.LastSessionEntries.Add(new LogEntry(S._("Could not erase files and " +
-                        "subfolders in {0} because {1}", info.FullName, e.Message), LogLevel.Error));
+                    Logger.Log(S._("Could not erase files and subfolders in {0} because {1}",
+                        info.FullName, e.Message), LogLevel.Error);
                 }
             }
@@ -840 +840 @@
             catch (UnauthorizedAccessException e)
             {
-                Task.Log.LastSessionEntries.Add(new LogEntry(e.Message, LogLevel.Error));
+                Logger.Log(e.Message, LogLevel.Error);
             }
         }