Merge branch 'main' into feature/firewall

Author: BornToBeRoot

Source/NETworkManager.Profiles/ProfileViewInfo.cs

@@ -16,9 +16,9 @@ public class ProfileViewInfo
     /// <summary>
     /// Initializes a new instance of the ProfileViewInfo class with the specified profile name, icon, and group.
     /// </summary>
-    /// <param name="name">The name of the profile to associate with this view. Cannot be null.</param>
+    /// <param name="name">The name of the profile to associate with this view. Must be a valid ProfileName instance.</param>
     /// <param name="icon">The icon representing the profile. Cannot be null.</param>
-    /// <param name="group">The group to which the profile belongs. Cannot be null.</param>
+    /// <param name="group">The group to which the profile belongs. Must be a valid ProfileGroup instance.</param>
     public ProfileViewInfo(ProfileName name, Canvas icon, ProfileGroup group)
     {
         Name = name;

Source/NETworkManager.Utilities.WPF/BindingProxy.cs

@@ -30,9 +30,11 @@ public object Data
         set => SetValue(DataProperty, value);
     }
 
-    /// Creates a new instance of the BindingProxy class.
+    /// <summary>
+    /// Creates a new instance of the <see cref="BindingProxy"/> class.
+    /// </summary>
     /// <returns>
-    /// A new instance of the BindingProxy class.
+    /// A new instance of the <see cref="BindingProxy"/> class.
     /// </returns>
     protected override Freezable CreateInstanceCore()
     {

Source/NETworkManager.Utilities/ListHelper.cs

@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 
 namespace NETworkManager.Utilities;
 /// <summary>
@@ -17,18 +18,18 @@ public static class ListHelper
     /// <returns>Modified list with the new entry added and oldest entries removed if necessary.</returns>
     public static List<T> Modify<T>(List<T> list, T entry, int length)
     {
-        var removedEntry = false;
-
         int index;
-        
+
         while ((index = list.IndexOf(entry)) != -1)
         {
             list.RemoveAt(index);
-            removedEntry = true;
         }
 
-        if (!removedEntry && list.Count == length)
-            list.RemoveAt(length - 1);
+        if (length <= 0)
+            throw new ArgumentOutOfRangeException(nameof(length), "Length must be greater than zero.");
+
+        while (list.Count >= length)
+            list.RemoveAt(list.Count - 1);
 
         list.Insert(0, entry);