Fixed CR issues

Author: Randall Flagg

src/LogExpert.Core/Classes/Filter/FilterPipe.cs

@@ -72,6 +72,7 @@ public void CloseFile ()
         }
     }
 
+    //TOOD: check if the callers are checking for null before calling
     public bool WriteToPipe (ILogLine textLine, int orgLineNum)
     {
         ArgumentNullException.ThrowIfNull(textLine, nameof(textLine));

src/LogExpert.Core/Classes/Filter/FilterStarter.cs

@@ -85,7 +85,6 @@ public async void DoFilter (FilterParams filterParams, int startLine, int maxCou
         }
 
         var workStartLine = startLine;
-        List<WaitHandle> handleList = [];
         _progressLineCount = 0;
         _progressCallback = progressCallback;
         while (workStartLine < startLine + maxCount)
@@ -105,13 +104,6 @@ public async void DoFilter (FilterParams filterParams, int startLine, int maxCou
             workStartLine += interval;
         }
 
-        WaitHandle[] handles = [.. handleList];
-        // wait for worker threads completion
-        if (handles.Length > 0)
-        {
-            _ = WaitHandle.WaitAll(handles);
-        }
-
         MergeResults();
     }
 

src/LogExpert.Core/Classes/Util.cs

@@ -61,6 +61,7 @@ public static string GetFileSizeAsText (long size)
                 : string.Empty + $"{size / 1048576.0:0.00}" + " MB";
     }
 
+    //TOOD: check if the callers are checking for null before calling
     public static bool TestFilterCondition (FilterParams filterParams, ILogLine line, ILogLineColumnizerCallback columnizerCallback)
     {
         ArgumentNullException.ThrowIfNull(filterParams, nameof(filterParams));

src/LogExpert.UI/Controls/LogWindow/LogWindowPrivate.cs

@@ -800,7 +800,7 @@ private void CheckFilterAndHighlight (LogEventArgs e)
             for (var i = filterStart; i < e.LineCount; ++i)
             {
                 var line = _logFileReader.GetLogLine(i);
-                //TODO: Why line can equal null here? Prevent this from happening and replace the check with an exception throw
+                //TODO: Why line can be equal null here? Need to understand all the situations and handle them correctlly. Prevent this from happening and replace the check with an exception throw
                 if (line == null)
                 {
                     return;