Skip debug logging in release builds, change log limiting method

jeffthiele · Dec 11, 2017 · bb5c722 · bb5c722
1 parent 415ff07
commit bb5c722
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/WindowPositionReset/MainWindow.xaml.cs b/WindowPositionReset/MainWindow.xaml.cs
@@ -10,6 +10,7 @@
 using System.Windows.Threading;
 using System.Drawing;
 using System.IO;
+using System.Text;
 using log4net;
 using log4net.Config;
 using Microsoft.Win32;
@@ -46,13 +47,15 @@ public MainWindow()
 
 		    XmlConfigurator.Configure();
 
+#if DEBUG
 		    foreach (var appender in LogManager.GetRepository().GetAppenders())
 		    {
 		        if (appender is EventTriggerAppender eta)
 		        {
 		            eta.OnLogEvent += Appender_OnLogEvent;
 		        }
 		    }
+#endif
 
 			InitializeComponent();
 			this.WindowState = System.Windows.WindowState.Minimized;
@@ -114,14 +117,15 @@ public MainWindow()
 
         private void Appender_OnLogEvent(object sender, string message)
         {
-            txtLog.Text += message;
+            var value = txtLog.Text + message;
 
-            if (txtLog.Text.Length > MaxLogLength)
+            if (value.Length > MaxLogLength)
             {
-                txtLog.Text = txtLog.Text.Substring(txtLog.Text.Length - MaxLogLength);
-                txtLog.Text = txtLog.Text.Substring(txtLog.Text.IndexOf('\n') + 1);
+                value = value.Substring(value.Length - MaxLogLength);
+                value = value.Substring(value.IndexOf('\n') + 1);
             }
 
+            txtLog.Text = value;
             logViewer.ScrollToBottom();
         }