Code Improvements

src/ColumnizerLib/ColumnizerLib.csproj

@@ -5,7 +5,6 @@
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>LogExpert</RootNamespace>
     <GenerateAssemblyInfo>false</GenerateAssemblyInfo>
-    <ImportWindowsDesktopTargets>true</ImportWindowsDesktopTargets>
     <DocumentationFile>$(SolutionDir)..\bin\Docs\ColumnizerLib.xml</DocumentationFile>
 	<IsTestProject>false</IsTestProject>
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>

src/CsvColumnizer/CsvColumnizer.cs

@@ -6,8 +6,10 @@
 using System.IO;
 using System.Linq;
 using System.Reflection;
+using System.Runtime.Versioning;
 using System.Windows.Forms;
 
+[assembly: SupportedOSPlatform("windows")]
 namespace CsvColumnizer
 {
     /// <summary>

src/CsvColumnizer/CsvColumnizer.csproj

@@ -10,7 +10,6 @@
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)\plugins</OutputPath>
     <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
-    <NoWarn>CA1416;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>

src/DefaultPlugins/DefaultPlugins.csproj

@@ -10,7 +10,6 @@
     <ImportWindowsDesktopTargets>true</ImportWindowsDesktopTargets>
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)\plugins</OutputPath>
-    <NoWarn>CA1416;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>

src/DefaultPlugins/Eminus.cs

@@ -6,7 +6,9 @@
 using System.IO;
 using System.Runtime.Serialization;
 using Newtonsoft.Json;
+using System.Runtime.Versioning;
 
+[assembly: SupportedOSPlatform("windows")]
 namespace LogExpert
 {
     public class Eminus : IContextMenuEntry, ILogExpertPluginConfigurator

src/FlashIconHighlighter/FlashIconHighlighter.csproj

@@ -9,7 +9,6 @@
     <ImportWindowsDesktopTargets>true</ImportWindowsDesktopTargets>
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)\plugins</OutputPath>
-    <NoWarn>CA1416;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>

src/FlashIconHighlighter/FlashIconPlugin.cs

@@ -2,8 +2,10 @@
 
 using System;
 using System.Runtime.InteropServices;
+using System.Runtime.Versioning;
 using System.Windows.Forms;
 
+[assembly: SupportedOSPlatform("windows")]
 namespace FlashIconHighlighter
 {
     internal class FlashIconPlugin : IKeywordAction

src/Log4jXmlColumnizer/Log4jXmlColumnizer.cs

@@ -1,14 +1,14 @@
-using System;
+using Newtonsoft.Json;
+using System;
 using System.Collections.Generic;
 using System.Globalization;
 using System.IO;
 using System.Linq;
-using System.Runtime.Serialization.Formatters.Binary;
 using System.Runtime.Serialization;
+using System.Runtime.Versioning;
 using System.Windows.Forms;
-using Newtonsoft.Json;
-
 
+[assembly: SupportedOSPlatform("windows")]
 namespace LogExpert
 {
     /// <summary>

src/Log4jXmlColumnizer/Log4jXmlColumnizer.csproj

@@ -9,7 +9,6 @@
     <ImportWindowsDesktopTargets>true</ImportWindowsDesktopTargets>
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)\plugins</OutputPath>
-    <NoWarn>CA1416;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>

src/LogExpert/Controls/LogTabWindow/LogTabWindow.cs

@@ -273,8 +273,6 @@ public void ChangeTheme(Control.ControlCollection container)
 
         private delegate void LoadMultiFilesDelegate(string[] fileName, EncodingOptions encodingOptions);
 
-        private delegate void ProgressBarEventFx(ProgressEventArgs e);
-
         private delegate void SetColumnizerFx(ILogLineColumnizer columnizer);
 
         private delegate void SetTabIconDelegate(LogWindow.LogWindow logWindow, Icon icon);

src/LogExpert/Controls/LogTabWindow/LogTabWindowEventHandlers.cs

@@ -353,7 +353,7 @@ private void OnBookmarkRemoved(object sender, EventArgs e)
 
         private void OnProgressBarUpdate(object sender, ProgressEventArgs e)
         {
-            Invoke(new ProgressBarEventFx(ProgressBarUpdateWorker), e);
+            Invoke(ProgressBarUpdateWorker, e);
         }
 
         private void OnStatusLineEvent(object sender, StatusLineEventArgs e)

src/LogExpert/Controls/LogWindow/LogWindow.cs

@@ -614,10 +614,6 @@ private void OnButtonSizeChanged(object sender, EventArgs e)
 
         // used for external wait fx WaitForLoadFinished()
 
-        private delegate void UpdateProgressCallback(LoadFileEventArgs e);
-
-        private delegate void LoadingStartedFx(LoadFileEventArgs e);
-
         private delegate void SelectLineFx(int line, bool triggerSyncCall);
 
         private Action<FilterParams, List<int>, List<int>, List<int>> FilterFxAction;

src/LogExpert/Controls/LogWindow/LogWindowEventHandlers.cs

@@ -122,7 +122,7 @@ private void OnLogWindowDisposed(object sender, EventArgs e)
 
         private void OnLogFileReaderLoadingStarted(object sender, LoadFileEventArgs e)
         {
-            Invoke(new LoadingStartedFx(LoadingStarted), e);
+            Invoke(LoadingStarted, e);
         }
 
         private void OnLogFileReaderFinishedLoading(object sender, EventArgs e)
@@ -211,16 +211,11 @@ private void OnLogFileReaderLoadFile(object sender, LoadFileEventArgs e)
                 //Thread loadThread = new Thread(new ThreadStart(ReloadNewFile));
                 //loadThread.Start();
                 _logger.Debug("Reloading invoked.");
-                return;
             }
-
-            if (!_isLoading)
+            else if (_isLoading)
             {
-                return;
+                BeginInvoke(UpdateProgress, e);
             }
-
-            UpdateProgressCallback callback = UpdateProgress;
-            BeginInvoke(callback, e);
         }
 
         private void OnFileSizeChanged(object sender, LogEventArgs e)

src/LogExpert/Controls/LogWindow/LogWindowPrivate.cs

@@ -1581,7 +1581,6 @@ private void StatusLineFileSize(long size)
 
         private int Search(SearchParams searchParams)
         {
-            UpdateProgressBarFx progressFx = UpdateProgressBar;
             if (searchParams.searchText == null)
             {
                 return -1;
@@ -1679,7 +1678,7 @@ private int Search(SearchParams searchParams)
                     {
                         if (!Disposing)
                         {
-                            Invoke(progressFx, [count]);
+                            Invoke(UpdateProgressBar, [count]);
                         }
                     }
                     catch (ObjectDisposedException ex) // can occur when closing the app while searching

src/LogExpert/Controls/LogWindow/LogWindowsPublic.cs

@@ -81,8 +81,10 @@ public void LoadFile(string fileName, EncodingOptions encodingOptions)
 
                 try
                 {
-                    _logFileReader = new LogfileReader(fileName, EncodingOptions, IsMultiFile, Preferences.bufferCount, Preferences.linesPerBuffer, _multiFileOptions);
-                    _logFileReader.UseNewReader = !Preferences.useLegacyReader;
+                    _logFileReader = new LogfileReader(fileName, EncodingOptions, IsMultiFile, Preferences.bufferCount, Preferences.linesPerBuffer, _multiFileOptions)
+                    {
+                        UseNewReader = !Preferences.useLegacyReader
+                    };
                 }
                 catch (LogFileException lfe)
                 {

src/LogExpert/Entities/EventArgs/LoadFileEventArgs.cs

@@ -1,38 +1,4 @@
-//using System.Linq;
-
-namespace LogExpert.Entities.EventArgs
+namespace LogExpert.Entities.EventArgs
 {
-    public class LoadFileEventArgs
-    {
-        #region Fields
-
-        #endregion
-
-        #region cTor
-
-        public LoadFileEventArgs(string fileName, long pos, bool finished, long fileSize, bool newFile)
-        {
-            this.FileName = fileName;
-            this.ReadPos = pos;
-            this.Finished = finished;
-            this.FileSize = fileSize;
-            this.NewFile = newFile;
-        }
-
-        #endregion
-
-        #region Properties
-
-        public long ReadPos { get; }
-
-        public bool Finished { get; }
-
-        public long FileSize { get; }
-
-        public bool NewFile { get; }
-
-        public string FileName { get; }
-
-        #endregion
-    }
+    public record LoadFileEventArgs(string FileName, long ReadPos, bool Finished, long FileSize, bool NewFile);
 }

src/LogExpert/LogExpert.csproj

@@ -17,8 +17,8 @@
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)</OutputPath>
     <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
     <ApplicationHighDpiMode>PerMonitorV2</ApplicationHighDpiMode>
-	<ForceDesignerDPIUnaware>true</ForceDesignerDPIUnaware>
-    <NoWarn>CS1591;CA1416;</NoWarn>
+    <ForceDesignerDPIUnaware>true</ForceDesignerDPIUnaware>
+    <NoWarn>CS1591;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>

src/RegexColumnizer.UnitTests/RegexColumnizerTests.cs

@@ -1,9 +1,9 @@
 using LogExpert;
-
 using Moq;
-
 using NUnit.Framework;
+using System.Runtime.Versioning;
 
+[assembly: SupportedOSPlatform("windows")]
 namespace RegexColumnizer.UnitTests
 {
     [TestFixture]

src/RegexColumnizer/RegexColumnizer.cs

@@ -2,10 +2,12 @@
 using System;
 using System.IO;
 using System.Linq;
+using System.Runtime.Versioning;
 using System.Text.RegularExpressions;
 using System.Windows.Forms;
 using System.Xml.Serialization;
 
+[assembly: SupportedOSPlatform("windows")]
 namespace RegexColumnizer
 {
     public abstract class BaseRegexColumnizer : ILogLineColumnizer, IColumnizerConfigurator
@@ -122,16 +124,16 @@ public void PushValue(ILogLineColumnizerCallback callback, int column, string va
 
         public void Configure(ILogLineColumnizerCallback callback, string configDir)
         {
-            RegexColumnizerConfigDialog d = new RegexColumnizerConfigDialog {Config = Config};
-            if (d.ShowDialog() == DialogResult.OK)
+            RegexColumnizerConfigDialog dialog = new RegexColumnizerConfigDialog {Config = Config};
+            if (dialog.ShowDialog() == DialogResult.OK)
             {
                 var configFile = GetConfigFile(configDir);
                 using (var w = new FileStream(configFile, FileMode.Create))
                 {
-                    xml.Serialize(w, d.Config);
+                    xml.Serialize(w, dialog.Config);
                 }
 
-                Init(d.Config);
+                Init(dialog.Config);
             }
         }
 

src/RegexColumnizer/RegexColumnizer.csproj

@@ -9,7 +9,6 @@
     <ImportWindowsDesktopTargets>true</ImportWindowsDesktopTargets>
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)\plugins</OutputPath>
-    <NoWarn>CA1416;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>

src/SftpFileSystemx64/ConfigDialog.cs

@@ -1,6 +1,8 @@
 using System;
+using System.Runtime.Versioning;
 using System.Windows.Forms;
 
+[assembly: SupportedOSPlatform("windows")]
 namespace SftpFileSystem
 {
     public partial class ConfigDialog : Form

src/SftpFileSystemx64/SftpFileSystemx64.csproj

@@ -11,7 +11,6 @@
     <ImportWindowsDesktopTargets>true</ImportWindowsDesktopTargets>
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)\plugins</OutputPath>
-    <NoWarn>CA1416;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>

src/SftpFileSystemx86/SftpFileSystemx86.csproj

@@ -12,7 +12,6 @@
     <AssemblyTitle>SftpFileSystemx86</AssemblyTitle>
     <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
     <OutputPath>$(SolutionDir)..\bin\$(Configuration)\pluginsx86</OutputPath>
-    <NoWarn>CA1416;</NoWarn>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
     <Optimize>False</Optimize>