From cb824c73fc46c824d497ce54c875c3827d239e63 Mon Sep 17 00:00:00 2001
From: Ellpeck <me@ellpeck.de>
Date: Fri, 1 Dec 2023 10:54:00 +0100
Subject: [PATCH] added WriterSink

---
 ExtremelySimpleLogger/ConsoleSink.cs |  4 +--
 ExtremelySimpleLogger/WriterSink.cs  | 43 ++++++++++++++++++++++++++++
 2 files changed, 45 insertions(+), 2 deletions(-)
 create mode 100644 ExtremelySimpleLogger/WriterSink.cs

diff --git a/ExtremelySimpleLogger/ConsoleSink.cs b/ExtremelySimpleLogger/ConsoleSink.cs
index c3f4e33..8abe01e 100644
--- a/ExtremelySimpleLogger/ConsoleSink.cs
+++ b/ExtremelySimpleLogger/ConsoleSink.cs
@@ -5,6 +5,7 @@ using System.IO;
 namespace ExtremelySimpleLogger {
     /// <summary>
     /// A <see cref="Sink"/> that writes log output to <see cref="Console.Out"/> or <see cref="Console.Error"/>.
+    /// This class is a variation of the <see cref="WriterSink"/>.
     /// </summary>
     public class ConsoleSink : Sink {
 
@@ -17,7 +18,6 @@ namespace ExtremelySimpleLogger {
             {LogLevel.Error, ConsoleColor.DarkRed},
             {LogLevel.Fatal, ConsoleColor.DarkRed}
         };
-        private readonly object locker = new object();
         private readonly TextWriter console;
 
         /// <summary>
@@ -63,7 +63,7 @@ namespace ExtremelySimpleLogger {
         /// <param name="level">The importance level of this message</param>
         /// <param name="s">The message to log</param>
         protected override void Log(Logger logger, LogLevel level, string s) {
-            lock (this.locker) {
+            lock (this.console) {
                 var color = this.GetColor(level);
                 if (color.HasValue)
                     Console.ForegroundColor = color.Value;
diff --git a/ExtremelySimpleLogger/WriterSink.cs b/ExtremelySimpleLogger/WriterSink.cs
new file mode 100644
index 0000000..9f3b5ee
--- /dev/null
+++ b/ExtremelySimpleLogger/WriterSink.cs
@@ -0,0 +1,43 @@
+using System.IO;
+
+namespace ExtremelySimpleLogger {
+    /// <summary>
+    /// A <see cref="Sink"/> that writes log output to an underlying <see cref="TextWriter"/>.
+    /// Note that <see cref="ConsoleSink"/> is a variation of this sink that additionally includes console colors.
+    /// </summary>
+    public class WriterSink : Sink {
+
+        private readonly TextWriter writer;
+        private readonly bool autoClose;
+
+        /// <summary>
+        /// Creates a new writer sink with the given settings.
+        /// </summary>
+        /// <param name="writer">The writer to write to.</param>
+        /// <param name="autoClose">Whether the underlying <paramref name="writer"/> should be closed automatically when this sink is disposed in <see cref="Dispose"/>.</param>
+        public WriterSink(TextWriter writer, bool autoClose = false) {
+            this.writer = writer;
+            this.autoClose = autoClose;
+        }
+
+        /// <summary>
+        /// Logs the given message, which has already been formatted using <see cref="Sink.Formatter"/> or <see cref="Logger.DefaultFormatter"/>.
+        /// </summary>
+        /// <param name="logger">The logger that the message was passed to</param>
+        /// <param name="level">The importance level of this message</param>
+        /// <param name="s">The message to log</param>
+        protected override void Log(Logger logger, LogLevel level, string s) {
+            lock (this.writer)
+                this.writer.WriteLine(s);
+        }
+
+        /// <inheritdoc />
+        public override void Dispose() {
+            if (this.autoClose) {
+                lock (this.writer)
+                    this.writer.Dispose();
+            }
+        }
+
+    }
+}