diff options
| author | Bond-009 <bond.009@outlook.com> | 2024-10-12 10:58:48 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-10-12 10:58:48 +0200 |
| commit | 822d4075843cdd4634faf738c2fb69b6b3201d5a (patch) | |
| tree | a03fb28aeb0e199d507b356999a2489834f4beb4 | |
| parent | 8653722088ac94ae3021608e6dc102035d4b56e5 (diff) | |
| parent | 76ac1e614395a711099ef8d000a9605794053ac3 (diff) | |
Merge pull request #12777 from fice-t/MemberNotNull
NetworkManager: Use MemberNotNull attribute
| -rw-r--r-- | src/Jellyfin.Networking/Manager/NetworkManager.cs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/Jellyfin.Networking/Manager/NetworkManager.cs b/src/Jellyfin.Networking/Manager/NetworkManager.cs index b285b836b..5a13cc417 100644 --- a/src/Jellyfin.Networking/Manager/NetworkManager.cs +++ b/src/Jellyfin.Networking/Manager/NetworkManager.cs @@ -81,7 +81,6 @@ public class NetworkManager : INetworkManager, IDisposable /// <param name="configurationManager">The <see cref="IConfigurationManager"/> instance.</param> /// <param name="startupConfig">The <see cref="IConfiguration"/> instance holding startup parameters.</param> /// <param name="logger">Logger to use for messages.</param> -#pragma warning disable CS8618 // Non-nullable field is uninitialized. : Values are set in UpdateSettings function. Compiler doesn't yet recognise this. public NetworkManager(IConfigurationManager configurationManager, IConfiguration startupConfig, ILogger<NetworkManager> logger) { ArgumentNullException.ThrowIfNull(logger); @@ -109,7 +108,6 @@ public class NetworkManager : INetworkManager, IDisposable _configurationManager.NamedConfigurationUpdated += ConfigurationUpdated; } -#pragma warning restore CS8618 // Non-nullable field is uninitialized. /// <summary> /// Event triggered on network changes. @@ -312,6 +310,7 @@ public class NetworkManager : INetworkManager, IDisposable /// <summary> /// Initializes internal LAN cache. /// </summary> + [MemberNotNull(nameof(_lanSubnets), nameof(_excludedSubnets))] private void InitializeLan(NetworkConfiguration config) { lock (_initLock) @@ -591,6 +590,7 @@ public class NetworkManager : INetworkManager, IDisposable /// Reloads all settings and re-Initializes the instance. /// </summary> /// <param name="configuration">The <see cref="NetworkConfiguration"/> to use.</param> + [MemberNotNull(nameof(_lanSubnets), nameof(_excludedSubnets))] public void UpdateSettings(object configuration) { ArgumentNullException.ThrowIfNull(configuration); |