Merge pull request #319 from Hirogen/issue31

MaximumLineLength is no configurable, but beware it can impact the performance to change it to more then 20.000

Author: Hirogen

src/LogExpert.Tests/JSONSaveTest.cs

@@ -1,6 +1,9 @@
 using LogExpert.Config;
+
 using Newtonsoft.Json;
+
 using NUnit.Framework;
+
 using System.IO;
 
 namespace LogExpert.Tests
@@ -17,14 +20,14 @@ public void SaveOptionsAsJSON()
             string settingsFile = configDir + "\\settings.json";
 
             Settings settings = null;
-            
+
             Assert.DoesNotThrow(CastSettings);
             Assert.That(settings, Is.Not.Null);
             Assert.That(settings.alwaysOnTop, Is.True);
 
             ConfigManager.Settings.alwaysOnTop = false;
             ConfigManager.Save(SettingsFlags.All);
-            
+
             settings = null;
             Assert.DoesNotThrow(CastSettings);
 

src/LogExpert/Classes/Log/PositionAwareStreamReaderBase.cs

@@ -1,16 +1,16 @@
-using System;
+using LogExpert.Config;
+using LogExpert.Entities;
+
+using System;
 using System.IO;
 using System.Text;
-using LogExpert.Entities;
 
 namespace LogExpert.Classes.Log
 {
     public abstract class PositionAwareStreamReaderBase : LogStreamReaderBase
     {
         #region Fields
 
-        private const int MAX_LINE_LEN = 20000;
-
         private static readonly Encoding[] _preambleEncodings = { Encoding.UTF8, Encoding.Unicode, Encoding.BigEndianUnicode, Encoding.UTF32 };
 
         private readonly BufferedStream _stream;
@@ -35,7 +35,7 @@ protected PositionAwareStreamReaderBase(Stream stream, EncodingOptions encodingO
             _posIncPrecomputed = GetPosIncPrecomputed(usedEncoding);
 
             _reader = new StreamReader(_stream, usedEncoding, true);
-            
+
             Position = 0;
         }
 
@@ -54,7 +54,7 @@ public sealed override long Position
                 /*
                  * 1: Sometime commented (+Encoding.GetPreamble().Length)
                  * 2: Date 1.1 3207
-                 * 3: Error Message from Piet because of Unicode-Bugs. 
+                 * 3: Error Message from Piet because of Unicode-Bugs.
                  *    No Idea, if this is OK.
                  * 4: 27.07.09: Preamble-Length is now calculated in CT, because Encoding.GetPreamble().Length
                  *    always delivers a fixed length (does not mater what kind of data)
@@ -72,7 +72,7 @@ public sealed override long Position
 
         public sealed override bool IsBufferComplete => true;
 
-        protected static int MaxLineLen => MAX_LINE_LEN;
+        protected static int MaxLineLen => ConfigManager.Settings.preferences.MaxLineLength;
 
         #endregion
 
@@ -149,11 +149,11 @@ protected void MovePosition(int offset)
         private int DetectPreambleLengthAndEncoding(out Encoding detectedEncoding)
         {
             /*
-            UTF-8:                          EF BB BF 
-            UTF-16-Big-Endian-Byteorder:    FE FF 
-            UTF-16-Little-Endian-Byteorder: FF FE 
-            UTF-32-Big-Endian-Byteorder:    00 00 FE FF 
-            UTF-32-Little-Endian-Byteorder: FF FE 00 00 
+            UTF-8:                          EF BB BF
+            UTF-16-Big-Endian-Byteorder:    FE FF
+            UTF-16-Little-Endian-Byteorder: FF FE
+            UTF-32-Big-Endian-Byteorder:    00 00 FE FF
+            UTF-32-Little-Endian-Byteorder: FF FE 00 00
             */
 
             byte[] readPreamble = new byte[4];
@@ -205,17 +205,17 @@ private int GetPosIncPrecomputed(Encoding usedEncoding)
             switch (usedEncoding)
             {
                 case UTF8Encoding _:
-                {
-                    return 0;
-                }
+                    {
+                        return 0;
+                    }
                 case UnicodeEncoding _:
-                {
-                    return 2;
-                }
+                    {
+                        return 2;
+                    }
                 default:
-                {
-                    return 1;
-                }
+                    {
+                        return 1;
+                    }
             }
         }